Websites/bugs.webkit.org/ChangeLog

   1 2011-01-06  Ojan Vafai  <ojan@chromium.org>
   2
   3         Fix line context when replying to comments.
   4
   5         * code-review.js:
   6
   7 2011-01-05  Ojan Vafai  <ojan@chromium.org>
   8
   9         Reviewed by Adam Barth.
  10
  11         change the way we do comment highlighting in the code review tool
  12         https://bugs.webkit.org/show_bug.cgi?id=51971
  13
  14         Store a space-separated list of base line IDs on each line that has
  15         comments associated with that line. This allows for overlapping comments,
  16         but more importantly, makes adding side-by-side diff support easier.
  17
  18         * code-review.js:
  19
  20 2011-01-05  Ojan Vafai  <ojan@chromium.org>
  21
  22         Reviewed by Adam Barth.
  23
  24         minor code cleanup for code review tool
  25         https://bugs.webkit.org/show_bug.cgi?id=51962
  26
  27         Consolidates some queries to using shared functions.
  28
  29         * code-review.js:
  30
  31 2011-01-05  Ojan Vafai  <ojan@chromium.org>
  32
  33         Reviewed by Adam Barth.
  34
  35         assorted cleanups to prepare for side-by-side diffing
  36         https://bugs.webkit.org/show_bug.cgi?id=51961
  37
  38         Mostly, use selector queries instead of assuming comments are next siblings.
  39
  40         * code-review.js:
  41
  42 2011-01-05  Ojan Vafai  <ojan@chromium.org>
  43
  44         Reviewed by Adam Barth.
  45
  46         fix message container selector for code review tool
  47         https://bugs.webkit.org/show_bug.cgi?id=51959
  48
  49         * code-review.js:
  50
  51 2011-01-04  Ojan Vafai  <ojan@chromium.org>
  52
  53         Increment version number to avoid getting the cached JS file.
  54
  55         * PrettyPatch/PrettyPatch.rb:
  56
  57 2010-12-15  Ojan Vafai  <ojan@chromium.org>
  58
  59         Reviewed by Adam Barth.
  60
  61         anchor the toolbar to the bottom of the diff if the diff doesn't take a full screen of height
  62         https://bugs.webkit.org/show_bug.cgi?id=51162
  63
  64         * PrettyPatch/PrettyPatch.rb:
  65         * code-review.js:
  66
  67 2011-01-01  Adam Barth  <abarth@webkit.org>
  68
  69         Reviewed by Eric Seidel.
  70
  71         Move JavaScriptCore to Source
  72         https://bugs.webkit.org/show_bug.cgi?id=51604
  73
  74         * PrettyPatch/PrettyPatch.rb:
  75           - Remove reference to JavaScriptCore as a source directory.
  76
  77 2011-01-01  Adam Barth  <abarth@webkit.org>
  78
  79         Reviewed by Eric Seidel.
  80
  81         Move Sources to Source
  82         https://bugs.webkit.org/show_bug.cgi?id=51794
  83
  84         * PrettyPatch/PrettyPatch.rb:
  85
  86 2010-12-31  Adam Barth  <abarth@webkit.org>
  87
  88         Rubber-stamped by Eric Seidel.
  89
  90         Move PageLoadTests to PerformanceTests/PageLoad
  91         https://bugs.webkit.org/show_bug.cgi?id=51771
  92
  93         Update list of top-level open-source directories.
  94
  95         * PrettyPatch/PrettyPatch.rb:
  96
  97 2010-12-31  Adam Barth  <abarth@webkit.org>
  98
  99         Rubber-stamped by Eric Seidel.
 100
 101         Move SunSpider into PerformanceTests
 102         https://bugs.webkit.org/show_bug.cgi?id=51769
 103
 104         Update list of top-level open-source directories.
 105
 106         * PrettyPatch/PrettyPatch.rb:
 107
 108 2010-12-26  Adam Barth  <abarth@webkit.org>
 109
 110         Reviewed by Eric Seidel.
 111
 112         Move autotools into Sources
 113         https://bugs.webkit.org/show_bug.cgi?id=51630
 114
 115         * PrettyPatch/PrettyPatch.rb:
 116           - Remove reference to non-existant autotools directory.
 117
 118 2010-12-26  Adam Barth  <abarth@webkit.org>
 119
 120         Reviewed by Eric Seidel.
 121
 122         Move cmake into Sources
 123         https://bugs.webkit.org/show_bug.cgi?id=51631
 124
 125         * PrettyPatch/PrettyPatch.rb:
 126
 127 2010-12-23  Adam Barth  <abarth@webkit.org>
 128
 129         Rubber-stamped by Eric Seidel.
 130
 131         Move JavaScriptGlue into Sources
 132         https://bugs.webkit.org/show_bug.cgi?id=51583
 133
 134         PrettyPatch keeps a list of the top-level directories that exist in
 135         trac.
 136
 137         * PrettyPatch/PrettyPatch.rb:
 138
 139 2010-12-20  Adam Barth  <abarth@webkit.org>
 140
 141         One more tweak caused by moving the web sites around.  I had this
 142         change in my patch originally, but I missed it because I had to use
 143         server-side svn move commands to move the bulk of the web site files.
 144
 145         * PrettyPatch/PrettyPatch.rb:
 146
 147 2010-12-18  Adam Barth  <abarth@webkit.org>
 148
 149         Reviewed by Sam Weinig.
 150
 151         Move WebKitExamplePlugins to Examples
 152         https://bugs.webkit.org/show_bug.cgi?id=51291
 153
 154         * PrettyPatch/PrettyPatch.rb:
 155
 156 2010-12-17  Dan Bernstein  <mitz@apple.com>
 157
 158         Reviewed by Simon Fraser.
 159
 160         Rename WebKitTools to Tools
 161         https://bugs.webkit.org/show_bug.cgi?id=49861
 162
 163         * PrettyPatch/PrettyPatch.rb:
 164         * committers-autocomplete.js:
 165
 166 2010-12-17  Ojan Vafai  <ojan@chromium.org>
 167
 168         Increment psuedo-version number to avoid pulling cached version.
 169
 170         * PrettyPatch/PrettyPatch.rb:
 171
 172 2010-12-17  Ojan Vafai  <ojan@chromium.org>
 173
 174         Speculative fix for the code review tool in Firefox.
 175         Fixes a JS error and makes the status bubble mostly work
 176         if postMessage is not supported.
 177
 178         * PrettyPatch/PrettyPatch.rb:
 179         * code-review.js:
 180
 181 2010-12-15  Ojan Vafai  <ojan@chromium.org>
 182
 183         Reviewed by Adam Barth.
 184
 185         size status bubble to it's contents on the code review page
 186         https://bugs.webkit.org/show_bug.cgi?id=51142
 187
 188         * PrettyPatch/PrettyPatch.rb:
 189         * code-review.js:
 190
 191 2010-12-15  Ojan Vafai  <ojan@chromium.org>
 192
 193         Reviewed by Adam Barth.
 194
 195         fix goofups from r74142 and r74130
 196         https://bugs.webkit.org/show_bug.cgi?id=51146
 197
 198         Accidentally shrank the inline comment boxes and made
 199         the toolbar not show up when there were image diffs.
 200
 201         * PrettyPatch/PrettyPatch.rb:
 202         * code-review.js:
 203
 204 2010-12-14  Ojan Vafai  <ojan@chromium.org>
 205
 206         Reviewed by Adam Barth.
 207
 208         add ability to view for file context to the review tool
 209         https://bugs.webkit.org/show_bug.cgi?id=51057
 210
 211         At the beginning/end of each file diff and between each
 212         hunk add links to expand the context. For now it grabs the
 213         tip of tree version of the file and tries to apply the diff
 214         to that file. If it can't apply, then it gives up as we
 215         wouldn't want to show the wrong lines of context.
 216
 217         In the future, we can consider adding the upload svn revision
 218         to the diff itself, then we could fallback to the file at that
 219         revision if tip of tree doesn't apply.
 220
 221         * PrettyPatch/PrettyPatch.rb:
 222         * code-review.js:
 223
 224 2010-12-08  Ojan Vafai  <ojan@chromium.org>
 225
 226         Reviewed by Adam Barth.
 227
 228         [reviewtool] should always show overall comments text box
 229         https://bugs.webkit.org/show_bug.cgi?id=45870
 230
 231         * PrettyPatch/PrettyPatch.rb:
 232         * code-review.js:
 233
 234 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
 235
 236         Reviewed by Adam Barth.
 237
 238         Increase size of status bubbles
 239         https://bugs.webkit.org/show_bug.cgi?id=50496
 240
 241         The current available space isn't enough to display all
 242         pending results including position in the queue.
 243
 244         * PrettyPatch/PrettyPatch.rb:
 245
 246 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
 247
 248         Reviewed by Adam Barth.
 249
 250         Increase size of status bubbles
 251         https://bugs.webkit.org/show_bug.cgi?id=50496
 252
 253         The current available space isn't enough to display all
 254         pending results including position in the queue.
 255
 256         * template/en/custom/attachment/edit.html.tmpl:
 257         * template/en/custom/attachment/list.html.tmpl:
 258         * template/en/custom/attachment/reviewform.html.tmpl:
 259
 260 2010-11-08  Adam Barth  <abarth@webkit.org>
 261
 262         Update help text to match behavior change.
 263
 264         * code-review.js:
 265
 266 2010-11-08  Adam Barth  <abarth@webkit.org>
 267
 268         Reviewed by Eric Seidel.
 269
 270         Restore normal meaning of double-click on patch review page?
 271         https://bugs.webkit.org/show_bug.cgi?id=47641
 272
 273         After this patch, you need to click on the line numbers to add a
 274         comment.
 275
 276         * PrettyPatch/PrettyPatch.rb:
 277         * code-review.js:
 278
 279 2010-10-09  Adam Barth  <abarth@webkit.org>
 280
 281         Reviewed by Alexey Proskuryakov.
 282
 283         Exception in committers-autocomplete in Firefox
 284         https://bugs.webkit.org/show_bug.cgi?id=47456
 285
 286         Remove dependency on a WebKit-only quirk.  IMHO, this quirk is bad for
 287         the long-term health of the web, but that's a discussion for another
 288         patch.
 289
 290         * committers-autocomplete.js:
 291
 292 2010-10-08  Adam Barth  <abarth@webkit.org>
 293
 294         Reviewed by Oliver Hunt.
 295
 296         Bugmail for new attachments should link to the reviewtool
 297         https://bugs.webkit.org/show_bug.cgi?id=47440
 298
 299         As requested by olliej himself.  This patch was constructed by pure
 300         reason.  I have no way to actually run this code.
 301
 302         * Bugzilla/BugMail.pm:
 303
 304 2010-09-28  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
 305
 306         Reviewed by Csaba Osztrogonác.
 307
 308         Convert verbatim unicode values in comitter names to characters
 309
 310         * committers-autocomplete.js:
 311
 312 2010-09-24  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
 313
 314         Reviewed by Csaba Osztrogonác.
 315
 316         Support Unicode-strings in committers-autocomplete script
 317
 318         * committers-autocomplete.js:
 319
 320 2010-09-22  Adam Barth  <abarth@webkit.org>
 321
 322         Reviewed by John Sullivan.
 323
 324         New review-page design doesn't include name of reviewer
 325         https://bugs.webkit.org/show_bug.cgi?id=46271
 326
 327         * PrettyPatch/PrettyPatch.rb:
 328         * code-review.js:
 329
 330 2010-09-22  Adam Barth  <abarth@webkit.org>
 331
 332         Fix typo in CSS.
 333
 334         * PrettyPatch/PrettyPatch.rb:
 335
 336 2010-09-22  Adam Barth  <abarth@webkit.org>
 337
 338         Reviewed by Eric Seidel.
 339
 340         Add link to bug to review page
 341         https://bugs.webkit.org/show_bug.cgi?id=46192
 342
 343         To make room for the link, I moved the help text to the top of the page
 344         and tweaked the language to help folks discover that you can select
 345         context using the line numbers.
 346
 347         Also, move more text to sans-serif since that fits in better with the
 348         pretty-patch fonts.
 349
 350         * PrettyPatch/PrettyPatch.rb:
 351         * code-review.js:
 352
 353 2010-09-21  Adam Barth  <abarth@webkit.org>
 354
 355         Unreviewed.
 356
 357         [reviewtool] Publish button doesn't work
 358         https://bugs.webkit.org/show_bug.cgi?id=46168
 359
 360         Darin's recent patch had a runtime error.  Sadly, we don't have any
 361         testing for this code.
 362
 363         * code-review.js:
 364
 365 2010-09-20  Darin Adler  <darin@apple.com>
 366
 367         Reviewed by Adam Barth.
 368
 369         Add Preview button and link to bug to patch review bar
 370         https://bugs.webkit.org/show_bug.cgi?id=46153
 371
 372         * PrettyPatch/PrettyPatch.rb: Bump version number to make caches happy.
 373         * code-review.js: Added Preview button and link to bug.
 374
 375 2010-09-19  Adam Barth  <abarth@webkit.org>
 376
 377         Reviewed by Eric Seidel.
 378
 379         [reviewtool] Publish button should actually publish the review comments
 380         https://bugs.webkit.org/show_bug.cgi?id=46074
 381
 382         Instead of showing the confirmation lightbox, the Publish button now
 383         publishes the comments directly.  If there's demand for a "Preview"
 384         button, we can find a way to add that feature.
 385
 386         * code-review.js:
 387
 388 2010-09-19  Adam Barth  <abarth@webkit.org>
 389
 390         Reviewed by Eric Seidel.
 391
 392         Move reviewtool to action=review
 393         https://bugs.webkit.org/show_bug.cgi?id=46071
 394
 395         This patch removes the old Review Patch screen and moves the new
 396         reviewtool from Pretty Diff to Review Patch.
 397
 398         * attachment.cgi:
 399         * code-review.js:
 400
 401 2010-09-19  Adam Barth  <abarth@webkit.org>
 402
 403         Review tool UI tweak.  This patch lengthens the status bubbles so all
 404         the bubble fit (even when they have numbers inside).
 405
 406         * PrettyPatch/PrettyPatch.rb:
 407
 408 2010-09-19  Dan Bernstein  <mitz@apple.com>
 409
 410         Reviewed by Anders Carlsson.
 411
 412         Fixed a typo.
 413
 414         * code-review.js: Changed the label of the OK button from Ok to OK.
 415
 416 2010-09-15  Adam Barth  <abarth@webkit.org>
 417
 418         Unreviewed.
 419
 420         Only show the review-in-context link if there are contextual comments.
 421
 422         * code-review.js:
 423
 424 2010-09-15  Adam Barth  <abarth@webkit.org>
 425
 426         Reviewed by Eric Seidel.
 427
 428         [reviewtool] Show the status bubbles on the toolbar
 429         https://bugs.webkit.org/show_bug.cgi?id=45861
 430
 431         * PrettyPatch/PrettyPatch.rb:
 432         * code-review.js:
 433
 434 2010-09-15  Adam Barth  <abarth@webkit.org>
 435
 436         Reviewed by Eric Seidel.
 437
 438         [reviewtool] Allow setting review and commit-queue flags from main review screen
 439         https://bugs.webkit.org/show_bug.cgi?id=45860
 440
 441         This patch adds drop down menus for adjusting the review and
 442         commit-queue flags from the main review screen.  These controls don't
 443         appear immediately because we need to read the state of the flags off
 444         the details page.
 445
 446         It's a slight sadness that bugzilla doesn't have a clean way of
 447         identifying which select control is associated with a given flag.  The
 448         flags seem to have some sort of id, but it's unclear to me how to map
 449         that id back to a notion of "review" or "commit-queue".  Instead, we
 450         look for some magic strings in the title.  I'm sure this will come back
 451         to bit us at some point, but I'm not sure what to do that's better.
 452         (The webkitpy solution to this problem is to assume that the flags are
 453         in a given order, but this seems better.)
 454
 455         * PrettyPatch/PrettyPatch.rb:
 456             - Bump version number to bust throuh caches.
 457         * code-review.js:
 458
 459 2010-09-15  Adam Barth  <abarth@webkit.org>
 460
 461         Reviewed by Eric Seidel.
 462
 463         [reviewtool] Comments should quote previous comments on the same line
 464         https://bugs.webkit.org/show_bug.cgi?id=45847
 465
 466         Now when you reply to a previous comment, the tool will quote the
 467         previous comment in the bugs.webkit.org post.  This makes it eaiser for
 468         folks following along in email to understand the discussion.
 469
 470         While I was editing this code, I also cleaned up some of the whitespace
 471         handling in comments.
 472
 473         * code-review.js:
 474
 475 2010-09-15  Adam Barth  <abarth@webkit.org>
 476
 477         Minor UI tweak to the review tool.  We want to display newlines in
 478         previous and frozen comments instead of collapsing them.
 479
 480         * PrettyPatch/PrettyPatch.rb:
 481
 482 2010-09-14  Adam Barth  <abarth@webkit.org>
 483
 484         Reviewed by Eric Seidel.
 485
 486         bugs.webkit.org should autocomplete WebKit Committers and Reviewers
 487         https://bugs.webkit.org/show_bug.cgi?id=45799
 488
 489         This patch implements an autocomplete dropdown that helps users type
 490         the names of WebKit committers and reviewers.  The script grabs the
 491         data from SVN now that wms set up CORS for http://svn.webkit.org.
 492         Security technology for the win.
 493
 494         This script is based on (well, essentially copied from) the popular
 495         autocomplete extension:
 496
 497         https://chrome.google.com/extensions/detail/olaabhcgdogcbcoiolomlcodkngnemfb
 498
 499         I asked Ojan, and said he was fine licensing the code under the BSD
 500         license.  I'm not sure how the patch interacts with the extension, but
 501         we can retire the extension if they interact poorly.
 502
 503         * committers-autocomplete.js: Added.
 504         * template/en/default/global/footer.html.tmpl:
 505             - Add the script to every page on bugs.webkit.org
 506
 507 2010-09-14  Adam Barth  <abarth@webkit.org>
 508
 509         Add a license block to the reviewtool.  I should have added this from
 510         the start.  According to the SVN log, I'm the only one to have ever
 511         touched this file.
 512
 513         * code-review.js:
 514
 515 2010-09-07  Adam Barth  <abarth@webkit.org>
 516
 517         Minor tweaks to the reviewtool UI.
 518
 519         Rename Cancel to Discard to make it more clear what the button does.
 520         Also, move the style declaration for the focus ring later in the sheet
 521         so it works.
 522
 523         * PrettyPatch/PrettyPatch.rb:
 524         * code-review.js:
 525
 526 2010-09-06  Adam Barth  <abarth@webkit.org>
 527
 528         Small UI tweak to the reviewtool.  Make the Ok/Cancel buttons a bit
 529         bigger.
 530
 531         * PrettyPatch/PrettyPatch.rb:
 532
 533 2010-09-06  Adam Barth  <abarth@webkit.org>
 534
 535         Reviewed by Eric Seidel.
 536
 537         [reviewtool] Add a quick animation for opening/closing comment boxes
 538         https://bugs.webkit.org/show_bug.cgi?id=45278
 539
 540         The 200ms animation slides in and out the line-by-line comment box.
 541
 542         * code-review.js:
 543
 544 2010-09-06  Adam Barth  <abarth@webkit.org>
 545
 546         Reviewed by Eric Seidel.
 547
 548         [reviewtool] Add a field for overall comments
 549         https://bugs.webkit.org/show_bug.cgi?id=45273
 550
 551         This patch does a couple logically separate things that could be
 552         separated into smaller patches:
 553
 554         1) This patch adds an "overall comments" field where you can enter
 555            overall comments about the patch.  These comments appear at the top
 556            of the bugzilla posting.  Currently, these aren't redisplayed when
 557            viewing the patch, but I plan to add that in a future patch.
 558
 559         2) This patch renames some of the CSS classes to more consistently
 560            follow the camelCase style that PrettyPatch uses.
 561
 562         3) This patch moves the "prepare comments" button to the left of the
 563            toolbar and renames is to "publish comments".  This makes more sense
 564            when you scroll to the bottom of the page and enter in some overall
 565            comments.
 566
 567         4) When you attempt to add a comment to a line that already has a
 568            "frozen" comment, we now unfreeze the comment instead of doing
 569            nothing.  The old behavior was kind of frustrating if you didn't
 570            know that you could unfreeze a comment by clicking on it.
 571
 572         * PrettyPatch/PrettyPatch.rb:
 573             - Update CSS.
 574         * code-review.js:
 575
 576 2010-09-06  Adam Barth  <abarth@webkit.org>
 577
 578         [reviewtool] Tweak the ok button to cancel the comment if the comment
 579         is empty.  Previously we would get into a bad state where a line had a
 580         comment but there was no longer any way to access it.
 581
 582         * code-review.js:
 583
 584 2010-09-06  Adam Barth  <abarth@webkit.org>
 585
 586         Reviewed by Eric Seidel.
 587
 588         [reviewtool] Add an "ok" button that collapses review comments
 589         https://bugs.webkit.org/show_bug.cgi?id=45255
 590
 591         Many folks (including myself) are too trigger-happy with the "delete"
 592         button that we end up deleting perfectly good review comments.  This
 593         patch adds an "ok" button to absorb these clicks (and renames "delete"
 594         to "cancel").  When you click the ok button, the comment becomes
 595         non-editable and the box shrinks to fit the text.  You can click the
 596         text to get back to the editable version.
 597
 598         * code-review.js:
 599
 600 2010-08-31  Adam Barth  <abarth@webkit.org>
 601
 602         Reviewed by Eric Seidel.
 603
 604         [reviewtool] Make it easy to scroll through review comments
 605         https://bugs.webkit.org/show_bug.cgi?id=45002
 606
 607         This patch lets you scroll through review comments using "n" (for next)
 608         and "p" (for previous).  It also attributes comments to their authors.
 609
 610         * PrettyPatch/PrettyPatch.rb:
 611         * code-review.js:
 612
 613 2010-08-31  Adam Barth  <abarth@webkit.org>
 614
 615         Reviewed by Eric Seidel.
 616
 617         [reviewtool] Show previous comments inline in diff
 618         https://bugs.webkit.org/show_bug.cgi?id=44977
 619
 620         This patch adds basic support for showing previous comments inline in
 621         the diff.  We crawl the bugs.webkit.org comments about this attachment
 622         and extract comments related to specific lines.  We then show the
 623         comments inline in the diff.
 624
 625         This part of the tool needs a bunch of polish, but this at least is a
 626         starting point for further work.
 627
 628         * PrettyPatch/PrettyPatch.rb:
 629         * code-review.js:
 630
 631 2010-08-31  Adam Barth  <abarth@webkit.org>
 632
 633         Reviewed by Eric Seidel.
 634
 635         [reviewtool] Allow reviewer to select lines of context by dragging over the line numbers
 636         https://bugs.webkit.org/show_bug.cgi?id=44936
 637
 638         * PrettyPatch/PrettyPatch.rb:
 639             - Add a version number to bust through bugs.webkit.org's aggressive
 640               cacheing.
 641         * code-review.js:
 642
 643 2010-08-30  Adam Barth  <abarth@webkit.org>
 644
 645         Reviewed by Eric Seidel.
 646
 647         [review tool] Let reviewer select how much context to show in snippet
 648         https://bugs.webkit.org/show_bug.cgi?id=44905
 649
 650         We now highlight the context for a comment in yellow on the left (where
 651         the line numbers are).  Clicking a line number expands or contracts the
 652         amount of context, as appropriate.  Informal user testing indicates
 653         that we might want to support drag as well.
 654
 655         This patch also changes the "open a comment box here" action to
 656         double-click to avoid issues with mis-clicks.
 657
 658         * PrettyPatch/PrettyPatch.rb:
 659         * code-review.js:
 660
 661 2010-08-29  Adam Barth  <abarth@webkit.org>
 662
 663         Attempt to make Sam's life easier by not opening a comment text field
 664         if there's a selection.  This should make it easier to copy/paste text
 665         out of the diff without accidentally opening comment boxes.
 666
 667         * code-review.js:
 668
 669 2010-08-29  Adam Barth  <abarth@webkit.org>
 670
 671         Turns out we're supposed to use find instead of children here.
 672
 673         * code-review.js:
 674
 675 2010-08-29  Adam Barth  <abarth@webkit.org>
 676
 677         Reviewed by Eric Seidel.
 678
 679         Awesomify pretty-diff based review tool
 680         https://bugs.webkit.org/show_bug.cgi?id=44835
 681
 682         This patch replaces the existing inline comment feature with a new
 683         implementation.  Hopefully the new implementation is more awesome.
 684         It's not 100% done yet, but I'd like to get this version out there to
 685         see how it works.
 686
 687         Unlike the previous version this version works with the "Formatted
 688         Diff" link instead of the "Review Patch" link.  Hopefully that will
 689         avoid interfering with folks who like the old style review.
 690
 691         * PrettyPatch/PrettyPatch.rb:
 692         * code-review.js: Added.
 693         * prototype.js: Removed.
 694
 695 2010-08-19  Tony Chang  <tony@chromium.org>
 696
 697         Reviewed by Adam Barth.
 698
 699         webkit-patch barfed on upload with a new image test result
 700         https://bugs.webkit.org/show_bug.cgi?id=39097
 701
 702         * PrettyPatch/PrettyPatch.rb: don't use full path to git
 703
 704 2010-08-19  Tony Chang  <tony@chromium.org>
 705
 706         Unreviewed, setting svn:eol-style native on ChangeLog.
 707
 708 2010-08-19  Tony Chang  <tony@chromium.org>
 709
 710         Unreviewed.  Just removing \r's from ChangeLog.
 711
 712 2010-08-02  Adam Roben  <aroben@apple.com>
 713
 714         Tell the patch prettifier about some (not so) new directories
 715
 716         * PrettyPatch/PrettyPatch.rb: Added WebKit2, autotools, and cmake
 717         directories.
 718
 719 2010-07-27  Joseph Pecoraro  <joepeck@webkit.org>
 720
 721         Reviewed by David Kilzer.
 722
 723         Provide a Helpful Link for Web Inspector Bugs in Bugzilla
 724         https://bugs.webkit.org/show_bug.cgi?id=43065
 725
 726         * template/en/custom/global/choose-product.html.tmpl: Added. Explicitly added a link for the Web Inspector.
 727
 728 2010-06-09  Julie Parent  <jparent@chromium.org>
 729
 730         Reviewed by David Kilzer.
 731
 732         Bugzilla: Don't send mail for cancel/deny/granted for in-rietveld.
 733
 734         https://bugs.webkit.org/show_bug.cgi?id=40345
 735
 736         * Bugzilla/Flag.pm:
 737
 738 2010-05-17  Julie Parent  <jparent@chromium.org>
 739
 740         Reviewed by Ojan Vafai.
 741
 742         Rietveld review page should dedicate more space to Rietveld, less to high level comments form.
 743
 744         https://bugs.webkit.org/show_bug.cgi?id=39244
 745
 746         * template/en/custom/attachment/reviewform.html.tmpl:
 747         Delete a random BR amongst the hidden elements that was causing extra whitespace.
 748         * template/en/custom/attachment/rietveldreview.html.tmpl:
 749         Increase size of Rietveld frame to 80%, from 60%.
 750
 751 2010-05-14  Julie Parent  <jparent@chromium.org>
 752
 753         Reviewed by Ojan Vafai.
 754
 755         Bugzilla: Only show "Rietveld Review" link when in-rietveld is set to +.
 756
 757         https://bugs.webkit.org/show_bug.cgi?id=39141
 758
 759         * template/en/custom/attachment/list.html.tmpl:
 760
 761 2010-05-13  Julie Parent  <jparent@chromium.org>
 762
 763         Reviewed by David Kilzer.
 764
 765         Bugzilla: Don't send mail if the only change is to the in-rietveld flag.
 766
 767         https://bugs.webkit.org/show_bug.cgi?id=39090
 768
 769         * Bugzilla/BugMail.pm:
 770
 771 2010-05-12  Julie Parent  <jparent@chromium.org>
 772
 773         Reviewed by Ojan Vafai.
 774
 775         Hide the in-rietveld flag in the UI.  It is only used only by tooling and shouldn't clutter the UI.
 776         https://bugs.webkit.org/show_bug.cgi?id=39024
 777
 778         * template/en/custom/attachment/list.html.tmpl:
 779           Removes this flag entirely from the UI, not needed on this page.
 780         * template/en/custom/flag/list.html.tmpl: Added.
 781           Copied from template/en/default/flag/list.html.tmpl with only changes inside the if WEBKIT_CHANGES section.
 782           Hides the flag rather than removing it completely, so tools can still interact with it.
 783
 784 2010-05-12  Julie Parent  <jparent@chromium.org>
 785
 786         Reviewed by David Kilzer.
 787
 788         Adds support for Rietveld reviews.  When there is an associated
 789         Rietveld id for a patch, adds a link to do a Rietveld review instead.
 790
 791         https://bugs.webkit.org/show_bug.cgi?id=38143
 792
 793         * attachment.cgi:
 794             Added support for handling action=rietveldreview.
 795         * template/en/custom/attachment/list.html.tmpl:
 796             Adds a link to use rietveld review iff the in_rietveld flag is set.
 797         * template/en/custom/attachment/reviewform.html.tmpl:
 798             Detects if we are in rietveld review mode due to query string paramter,
 799             and customizes the review form slightly:
 800                 - Changes text describing comment field.
 801                 - Comment field does not include quoted patch.
 802                 - Submit button fetches from Rietveld instead of just submitting.
 803         * template/en/custom/attachment/rietveldreview.html.tmpl: Added.
 804             New template for rietveldreview mode.  Hosts the rietveld frame in the
 805             top and the regular comments form in the bottom.
 806             Uses postMessage to communicate with Rietveld frame.
 807
 808 2010-05-08  Chris Jerdonek  <cjerdonek@webkit.org>
 809
 810         Reviewed by Maciej Stachowiak.
 811
 812         Allow long comment lines to wrap to the window width instead
 813         of to 80 characters.
 814
 815         https://bugs.webkit.org/show_bug.cgi?id=37792
 816
 817         * Bugzilla/Constants.pm:
 818           - Introduced a new constant COMMENT_COLS_WRAP to replace COMMENT_COLS
 819             inside the wrap_comment() subroutine.  The new constant effectively
 820             disables text wrapping by increasing the value used in wrap_comment()
 821             from 80 characters to a very large value (8000).
 822         * Bugzilla/Util.pm:
 823           - Replaced COMMENT_COLS with COMMENT_COLS_WRAP inside wrap_comment().
 824         * skins/custom/global.css:
 825         (.bz_comment_text):
 826           - Eliminate the width styling on comments to allow long lines
 827             to wrap at the window width.
 828         (.bz_comment_text > span.quote):
 829           - Inherit the "white-space" property.  This prevents long quoted
 830             lines in comments from causing non-quoted lines to extend beyond
 831             the window width.
 832
 833 2010-04-22  Adam Barth  <abarth@webkit.org>
 834
 835         Reviewed by Darin Adler.
 836
 837         The new review tools shouldn't clear the comment box automatically
 838         https://bugs.webkit.org/show_bug.cgi?id=38004
 839
 840         Instead, we now have a button for doing that explicitly.  If people
 841         like this tool, we can look at fancier solutions.
 842
 843         Also, tried to make the tool work on the edit page as well as the review page.
 844
 845         * PrettyPatch/PrettyPatch.rb:
 846
 847 2010-04-22  Adam Barth  <abarth@webkit.org>
 848
 849         Unreviewed.  Turns out we need to do more escaping because Ruby was
 850         explanding our newlines, creating a syntax error.
 851
 852         * PrettyPatch/PrettyPatch.rb:
 853
 854 2010-04-22  Adam Barth  <abarth@webkit.org>
 855
 856         Reviewed by Darin Adler.
 857
 858         Hack up PrettyDiff to allow line-by-line comments
 859         https://bugs.webkit.org/show_bug.cgi?id=37886
 860
 861         Admittedly a bit of a hack, this is a basic line-by-line editor we can
 862         play with thanks to Andrew Scherkus.  It's meant to integrate with the
 863         "review" page at bugs.webkit.org.
 864
 865         I changed a few things from Andrew's original version:
 866         1) Trigger text boxes on single click to make iPhone/iPad reviewing
 867         easier.
 868         2) Clear the main text box on load.
 869         3) Reference a version of prototype.js on bugs.webkit.org.
 870
 871         * PrettyPatch/PrettyPatch.rb:
 872         * prototype.js: Added.
 873
 874 2010-02-22  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
 875
 876         Reviewed by Maciej Stachowiak.
 877
 878         Don't reset the assignee back to default on component change
 879
 880         https://bugs.webkit.org/show_bug.cgi?id=35236
 881
 882         There's a risk of reseting a valid assignee for example when
 883         triaging a bug. All components in WebKit have the default
 884         assignee set to webkit-unassigned@webkit.org so this should
 885         not cause problems for people relying on this functionality.
 886
 887         * template/en/custom/bug/edit.html.tmpl:
 888
 889 2010-01-22  Janusz Lewandowski  <lew21@xtreeme.org>
 890
 891         Reviewed by Maciej Stachowiak.
 892
 893         run-webkit-tests fails on Ruby 1.9
 894         https://bugs.webkit.org/show_bug.cgi?id=33554
 895         inject does not work with strings any more, so BugsSite/PrettyPatch/PrettyPatch.rb prints lots of errors.
 896
 897         * PrettyPatch/PrettyPatch.rb:
 898
 899 2010-01-20  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
 900
 901         Reviewed by Adam Barth.
 902
 903         Adjust height of status-bubbles to prevent them being cut.
 904
 905         * template/en/custom/attachment/edit.html.tmpl:
 906         * template/en/custom/attachment/list.html.tmpl:
 907         * template/en/custom/attachment/reviewform.html.tmpl:
 908
 909 2010-01-08  David Kilzer  <ddkilzer@apple.com>
 910
 911         <http://webkit.org/b/33410> Fix "View Plain Diff"/"View Formatted Diff" button
 912
 913         Reviewed by Adam Barth.
 914
 915         This button was broken during a previous Bugzilla upgrade after
 916         changes were made to the names of objects exposed on templates.
 917
 918         * template/en/custom/attachment/reviewform.html.tmpl: Changed
 919         attachid to attachment.id and ispatch to attachment.ispatch.
 920         Also tweaked the format to use 4 table columns instead of 5.
 921
 922 2010-01-08  David Kilzer  <ddkilzer@apple.com>
 923
 924         Included UNCONFIRMED bugs by default in advanced search query
 925
 926         Rubber-stamped by Alexey Proskuryakov.
 927
 928         * data/params: Prepended 'bug_status=UNCONFIRMED&' to
 929         'defaultquery' parameter.
 930
 931 2010-01-08  David Kilzer  <ddkilzer@apple.com>
 932
 933         FIX: Make patch status appear on reviewform.html.tmpl
 934
 935         * template/en/custom/attachment/reviewform.html.tmpl: Check
 936         attachment.ispatch (not just ispatch) when deciding whether
 937         to display patch status.  Also update layout a bit.
 938
 939 2010-01-08  David Kilzer  <ddkilzer@apple.com>
 940
 941         Update data/params parameters
 942
 943         * data/params: Updated 'timezone' to turn off DST.  Updated
 944         'usebugaliases' and 'usevotes' to turn them off.
 945
 946 2010-01-08  Adam Barth  <abarth@webkit.org>
 947
 948         Reviewed by Darin Adler.
 949
 950         Show patch status in bugs.webkit.org
 951         https://bugs.webkit.org/show_bug.cgi?id=33368
 952
 953         This patch adds an iframe to the show_bug page and the edit attachment
 954         page that displays whether the attachment passed the various bots.
 955
 956         * template/en/custom/attachment/edit.html.tmpl:
 957         * template/en/custom/attachment/list.html.tmpl:
 958         * template/en/custom/attachment/reviewform.html.tmpl:
 959
 960 2009-12-07  Shinichiro Hamaji  <hamaji@chromium.org>
 961
 962         Reviewed by David Kilzer.
 963
 964         Bugzilla should show images in git patches
 965         https://bugs.webkit.org/show_bug.cgi?id=31395
 966
 967         Attempt to go r51748 again using --directory option of git-apply.
 968
 969         * PrettyPatch/PrettyPatch.rb:
 970
 971 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
 972
 973         Unreviewed. Revert r51748.
 974
 975         Bugzilla should show images in git patches
 976         https://bugs.webkit.org/show_bug.cgi?id=31395
 977
 978         * PrettyPatch/PrettyPatch.rb:
 979
 980 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
 981
 982         Reviewed by David Kilzer.
 983
 984         Bugzilla should show images in git patches
 985         https://bugs.webkit.org/show_bug.cgi?id=31395
 986
 987         Show images in git patches using git-apply.
 988
 989         * PrettyPatch/PrettyPatch.rb:
 990
 991 2009-10-23  Eric Seidel  <eric@webkit.org>
 992
 993         Reviewed by Adam Roben.
 994
 995         PrettyPatch should show images even when they have the wrong mime type
 996         https://bugs.webkit.org/show_bug.cgi?id=29506
 997
 998         * PrettyPatch/PrettyPatch.rb:
 999
1000 2009-10-17  Alexey Proskuryakov  <ap@apple.com>
1001
1002         Reviewed by Eric Carlson.
1003
1004         https://bugs.webkit.org/show_bug.cgi?id=30470
1005         Make marking a bug as duplicate easier
1006
1007         * js/field.js: (setResolutionToDuplicate): Focus the edit field, too.
1008
1009 2009-10-16  Alexey Proskuryakov  <ap@apple.com>
1010
1011         Reviewed by Mark Rowe.
1012
1013         https://bugs.webkit.org/show_bug.cgi?id=30470
1014         Make marking a bug as duplicate easier
1015
1016         * js/field.js: (setResolutionToDuplicate): Show edit field right away, don't make the user
1017         click again.
1018
1019 2009-09-09  David Kilzer  <ddkilzer@apple.com>
1020
1021         Update data/params parameters
1022
1023         * data/params: Updated 'attachment_base' parameter for secure
1024         attachments.  Updated 'mybugstemplate' parameter to search for
1025         bugs in UNCONFIRMED, ASSIGNED and REOPENED status in addition to
1026         NEW status.
1027
1028 2009-08-11  David Kilzer  <ddkilzer@apple.com>
1029
1030         Updated data/params parameter
1031
1032         * data/params: Updated 'loginnetmask' parameter to allow users
1033         to choose whether their cookies are restricted to a single IP
1034         address or not.
1035
1036 2009-07-07  David Kilzer  <ddkilzer@apple.com>
1037
1038         Updated BugsSite with local changes on server
1039
1040         The new files were created by checksetup.pl during the upgrade
1041         to Bugzilla 3.2.3.  The rest are self-explanatory.
1042
1043         * data: Added "bugzilla-update.xml" to svn:ignore.
1044         * data/attachments: Added "groups.*" to svn:ignore.
1045         * data/params: Updated for a few changed parameters.
1046
1047         * lib/.htaccess: Added.
1048         * skins/contrib/Dusk/IE-fixes.css: Added.
1049         * skins/contrib/Dusk/admin.css: Added.
1050         * skins/contrib/Dusk/create_attachment.css: Added.
1051         * skins/contrib/Dusk/dependency-tree.css: Added.
1052         * skins/contrib/Dusk/duplicates.css: Added.
1053         * skins/contrib/Dusk/editusers.css: Added.
1054         * skins/contrib/Dusk/help.css: Added.
1055         * skins/contrib/Dusk/index.css: Added.
1056         * skins/contrib/Dusk/panel.css: Added.
1057         * skins/contrib/Dusk/params.css: Added.
1058         * skins/contrib/Dusk/release-notes.css: Added.
1059         * skins/contrib/Dusk/show_bug.css: Added.
1060         * skins/contrib/Dusk/show_multiple.css: Added.
1061         * skins/contrib/Dusk/summarize-time.css: Added.
1062         * skins/contrib/Dusk/voting.css: Added.
1063         * skins/contrib/Dusk/yui: Added.
1064         * skins/contrib/Dusk/yui/calendar.css: Added.
1065         * skins/custom/IE-fixes.css: Added.
1066         * skins/custom/admin.css: Added.
1067         * skins/custom/buglist.css: Added.
1068         * skins/custom/create_attachment.css: Added.
1069         * skins/custom/dependency-tree.css: Added.
1070         * skins/custom/duplicates.css: Added.
1071         * skins/custom/editusers.css: Added.
1072         * skins/custom/help.css: Added.
1073         * skins/custom/panel.css: Added.
1074         * skins/custom/params.css: Added.
1075         * skins/custom/release-notes.css: Added.
1076         * skins/custom/show_bug.css: Added.
1077         * skins/custom/show_multiple.css: Added.
1078         * skins/custom/summarize-time.css: Added.
1079         * skins/custom/voting.css: Added.
1080         * skins/custom/yui: Added.
1081         * skins/custom/yui/calendar.css: Added.
1082
1083 2009-07-03  David Kilzer  <ddkilzer@apple.com>
1084
1085         Bug 26958: Change edit link to review link in request messages
1086
1087         <https://bugs.webkit.org/show_bug.cgi?id=26958>
1088
1089         Reviewed by Dan Bernstein.
1090
1091         * template/en/custom/request/email.txt.tmpl: Copied from BugsSite/template/en/default/request/email.txt.tmpl.
1092         Changed link from "action=edit" to "action=review".
1093
1094 2009-07-03  David Kilzer  <ddkilzer@apple.com>
1095
1096         Bug 26950: Make the summary and alias fields support click-to-edit
1097         <https://bugs.webkit.org/show_bug.cgi?id=26950>
1098
1099         Reviewed by Maciej Stachowiak.
1100
1101         Original patch by Maciej Stachowiak.
1102
1103         * js/field.js:
1104         (hideEditableField): Updated to add click event listeners to the
1105         alias and short description elements to make them easier to
1106         edit.  Renamed field_id parameter to field2_id (short
1107         description id) and added a field1_id parameter (alias id).
1108         (showEditableField): If a third parameter is passed in the
1109         ContainerInputArray parameter, use it to find the element to
1110         focus.  Otherwise, fall back to the original behavior of
1111         focusing the first input field.
1112         (hideAliasAndSummary): Changed to pass the id for the alias
1113         element to hideEditableField().
1114
1115 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1116
1117         Config file for contrib/recode.pl when upgrading bugs.webkit.org
1118
1119         Current as of midday on Wednesday, July 1, 2009.
1120
1121         * contrib/recode-overrides.txt: Added.
1122
1123 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1124
1125         Workaround for WebKit Bug 9630 when running contrib/recode.pl
1126
1127         A number of WebKit nightly builds included Bug 9630 which caused
1128         non-breaking space characters (0xA0) to be submitted within the
1129         content of textarea elements.
1130
1131         * contrib/recode.pl: To work around these 0xA0 characters, try
1132         an encoding of cp1252 and use it if it works.
1133
1134 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1135
1136         Don't print "Failed to guess" warning if an override is available
1137
1138         * contrib/recode.pl: Check that %overrides does NOT contain a
1139         key matching the current digest before printing out the warning
1140         about a failed charset guess.
1141
1142 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1143
1144         Added --[no-]truncate switch to contrib/recode.pl
1145
1146         * contrib/recode.pl: Added --[no-]truncate switch to make
1147         debugging of failed charset guessing easier.  Often times the
1148         illegal character was truncated in the output.
1149
1150 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1151
1152         Changed Perl scripts to use #!/usr/bin/env perl
1153
1154         perl -pi -e 's@#\!/usr/bin/perl@#\!/usr/bin/env perl@' *.cgi *.pl */*.pl
1155
1156 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1157
1158         Replace Bugzilla favicon with webkit.org favicon
1159
1160         * favicon.ico: Removed.
1161         * images/favicon.ico: Replaced.  Copied from favicon.ico.
1162
1163 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
1164
1165         Merged BugsSite to Bugzilla-3.2.3
1166
1167         Updated to the latest-and-greatest stable version.
1168
1169 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
1170
1171         Merged BugsSite to Bugzilla-3.0.3
1172
1173         Nothing to see here.  Move along.
1174
1175 2009-06-09  Eric Seidel  <eric@webkit.org>
1176
1177         Reviewed by Adam Roben.
1178
1179         Add support for displaying added png files in PrettyPatch diffs
1180         https://bugs.webkit.org/show_bug.cgi?id=26210
1181
1182         Currently this is SVN only (git-send-bugzilla patches exclude binary data)
1183         and only works for PNG files but could easily be made to work for other images as needed.
1184
1185         * PrettyPatch/PrettyPatch.rb:
1186
1187 2009-05-19  Eric Seidel  <eric@webkit.org>
1188
1189         Reviewed by Adam Roben.
1190
1191         Make PrettyPatch understand quoted filenames in git diffs.
1192         https://bugs.webkit.org/show_bug.cgi?id=25881
1193
1194         * PrettyPatch/PrettyPatch.rb:
1195
1196 2009-05-15  Simon Fraser  <simon.fraser@apple.com>
1197
1198         Carrying forwards Darin Adler's rubber stamp
1199
1200         Make the comment field wider too.
1201
1202         * skins/custom/global.css:
1203
1204 2009-05-14  Simon Fraser  <simon.fraser@apple.com>
1205
1206         Reviewed by Darin Adler
1207
1208         Make the URL, Summary and Keyword fields wider to mitigate the
1209         effects of https://bugs.webkit.org/show_bug.cgi?id=25566 and make
1210         things generally better.
1211
1212         * skins/custom/global.css:
1213         * template/en/custom/bug/edit.html.tmpl:
1214
1215 2008-11-10  Darin Adler  <darin@apple.com>
1216
1217         - fix more of https://bugs.webkit.org/show_bug.cgi?id=21400
1218           "Edit" links for patches in comments, review queue, review emails
1219           should be replaced by "Review Patch" links
1220
1221         * globals.pl: Fix cases that are automatically generated, such as links
1222         in comments in bugs.
1223
1224         * template/en/custom/attachment/created.html.tmpl: Fix the link on the
1225         "I just created an attachment" page.
1226
1227         * template/en/custom/request/queue.html.tmpl: Fix the link in the queue
1228         page.
1229
1230 2008-11-10  Darin Adler  <darin@apple.com>
1231
1232         * template/en/default/request/queue.html.tmpl: Use review links instead
1233         of edit links in the review queue.
1234
1235 2008-11-10  Darin Adler  <darin@apple.com>
1236
1237         * template/en/custom/request/email.txt.tmpl: Send review links instead
1238         of edit links when flags are set on a bug.
1239
1240 2008-10-27  Darin Adler  <darin@apple.com>
1241
1242         * template/en/custom/attachment/reviewform.html.tmpl: Leave out the
1243         comment if it's untouched.
1244
1245 2008-10-23  Adam Roben  <aroben@apple.com>
1246
1247         Fix Bug 21401: Comments field on "Review Patch" page should be
1248         initially filled with quoted patch
1249
1250         <https://bugs.webkit.org/show_bug.cgi?id=21401>
1251
1252         Reviewed by Dave Hyatt.
1253
1254         * attachment.cgi:
1255         (edit): Retrieve the attachment data from the database instead of just
1256         its length. When the attachment is a patch, create a quotedpatch
1257         template variable that contains the patch data with each line
1258         prepended with "> ".
1259         * template/en/custom/attachment/reviewform.html.tmpl:
1260           - Changed the comments field to have a monospace font
1261           - Added an "Enter comments below:" caption above the comments field
1262             to make it clearer that this is where comments should go, now that
1263             the comments field is not initially empty
1264           - Fill the comments field with the quoted patch
1265
1266 2008-10-14  Adam Roben  <aroben@apple.com>
1267
1268         Fix Bug 21602: Bugzilla times out trying to display formatted diff for
1269         attachment 24345
1270
1271         https://bugs.webkit.org/show_bug.cgi?id=21602
1272
1273         Reviewed by Dave Kilzer.
1274
1275         * PrettyPatch/PrettyPatch.rb:
1276         (PrettyPatch.BINARY_FILE_MARKER_FORMAT): Added.
1277         (PrettyPatch.FileDiff.initialize): If any of the lines in the diff
1278         are a binary file marker, mark this FileDiff as binary and stop trying
1279         to process the lines.
1280         (PrettyDiff.FileDiff.to_html): If we're binary, just print a string
1281         saying so.
1282
1283 2008-10-02  Adam Roben  <aroben@apple.com>
1284
1285         * template/en/custom/attachment/reviewform.html.tmpl: Fix typo.
1286
1287 2008-10-02  Adam Roben  <aroben@apple.com>
1288
1289         * template/en/custom/attachment/review.html.tmpl: Fix URLs
1290
1291 2008-10-02  Adam Roben  <aroben@apple.com>
1292
1293         Fix Bug 21315: bugs.webkit.org should have a nicer patch review
1294         interface
1295
1296         https://bugs.webkit.org/show_bug.cgi?id=21315
1297
1298         Reviewed by Tim Hatcher.
1299
1300         * attachment.cgi:
1301         (top level): Added support for the "review" and "reviewform" actions.
1302         (sub edit): Accept the template name as a parameter. If no name is
1303         give, use "edit" as before.
1304         * template/en/custom/attachment/list.html.tmpl: Added a "Review Patch"
1305         link for all patch attachments.
1306         * template/en/custom/attachment/review.html.tmpl: Added. Simple
1307         <frameset> page to show the patch on the top and a comment form on the
1308         bottom.
1309         * template/en/custom/attachment/reviewform.html.tmpl: Added. Simple
1310         comment form for reviewing patches.
1311
1312 2008-10-02  Adam Roben  <aroben@apple.com>
1313
1314         Remove references to some backup files I created when implementing
1315         PrettyPatch in Bugzilla
1316
1317         Reviewed by Tim Hatcher.
1318
1319         * attachment-aroben.cgi: Removed.
1320         * template/en/default/attachment/edit-aroben.html.tmpl: Removed.
1321
1322         * template/en/custom/attachment/edit.html.tmpl: Removed an erroneous
1323         reference to attachment-aroben.cgi. attachment.cgi will work just
1324         fine.
1325
1326 2008-06-25  David Kilzer  <ddkilzer@apple.com>
1327
1328         Make PrettyPatch handle diffs with no Index or diff headers
1329
1330         Part of Bug 19290: More patches not handled by PrettyPatch.rb
1331         <https://bugs.webkit.org/show_bug.cgi?id=19290>
1332
1333         Reviewed by Adam.
1334
1335         * PrettyPatch/PrettyPatch.rb:
1336         (PrettyPatch.DIFF_HEADER_FORMATS): Added regular expression to
1337         match on "+++ " lines for patches with no "Index" or "diff" header.
1338         (PrettyPatch.FileDiff.initialize): Look for filename on "+++ " line
1339         when the first line of a patch has no "Index" or "diff" header.
1340         (PrettyPatch.FileDiff.parse): Added haveSeenDiffHeader state
1341         variable to determine when no "Index" or "diff" header has been
1342         found, but a new patch has started with a "--- " line.
1343
1344 2008-06-08  Dan Bernstein  <mitz@apple.com>
1345
1346         Reviewed by Adam Roben.
1347
1348         - update trac URL in PrettyPatch
1349
1350         * PrettyPatch/PrettyPatch.rb:
1351
1352 2008-05-29  Adam Roben  <aroben@apple.com>
1353
1354         Update PrettyPatch directory list
1355
1356         * PrettyPatch/PrettyPatch.rb:
1357
1358 2008-05-28  Adam Roben  <aroben@apple.com>
1359
1360         Make PrettyPatch able to handle diffs taken from outside the WebKit
1361         source tree
1362
1363         Part of Bug 19290: More patches not handled by PrettyPatch.rb
1364         <https://bugs.webkit.org/show_bug.cgi?id=19290>
1365
1366         Reviewed by David Kilzer.
1367
1368         * PrettyPatch/PrettyPatch.rb:
1369         (PrettyPatch.find_url_and_path): Added. Searches the file path from
1370         the bottom up looking for a directory that exists in the source tree.
1371         (PrettyPatch.linkifyFilename): Changed to call find_url_and_path.
1372
1373 2008-05-28  Adam Roben  <aroben@apple.com>
1374
1375         Print exceptions from PrettyPatch to stdout
1376
1377         This will make it much easier to debug problems with PrettyPatch,
1378         since we'll be able to see the exceptions in the browser.
1379
1380         Reviewed by Sam Weinig.
1381
1382         * PrettyPatch/prettify.rb: Added a --html-exceptions option, which
1383         will print exceptions to stdout as HTML.
1384         * attachment.cgi: Changed to pass --html-exceptions to prettify.rb.
1385
1386 2008-04-26  David Kilzer  <ddkilzer@apple.com>
1387
1388         Removed temp files and added appropriate svn:ignore properties.
1389
1390         Rubber-stamped by Mark Rowe.
1391
1392         * data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
1393
1394         * data/template: Added svn:ignore property for "template" subdirectory.
1395         * data/template/template: Removed precompiled template subdirectory.
1396
1397         * data/versioncache: Removed.
1398         * data/versioncache.*: Removed backup versioncache files.
1399
1400         * data/webdot: Added svn:ignore property for "*.dot" files.
1401         * data/webdot/*.dot: Removed cached webdot files.
1402
1403 2008-04-08  Adam Roben  <aroben@apple.com>
1404
1405         Combine :equal operations with the following operation if they are
1406         fewer than 3 characters long
1407
1408         This keeps us from showing lots of small changes on long lines, just
1409         because some letters happened to be the same.
1410
1411         Inspired by
1412         http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/diffviewer/diffutils.py?r=1264#147
1413
1414         * PrettyPatch/PrettyPatch.rb:
1415         (PrettyPatch::DiffSection.initialize): Remove :equal operations fewer
1416         than 3 characters long. The characters from the removed operations
1417         become part of the subsequent operation.
1418         (PrettyPatch::CodeLine.text_as_html): Don't wrap 0-length strings in
1419         <ins>/<del> tags. Also removed the @fromLineNumber.nil? and
1420         @toLineNumber.nil? checks as they are no longer reliable now that
1421         we're removing operations.
1422
1423 2008-04-08  Adam Roben  <aroben@apple.com>
1424
1425         Change PrettyPatch to use DiffBuilder for intra-line diffs
1426
1427         This gives us much prettier intra-line diffs, largely because it can
1428         distinguish multiple changes on the same line. e.g., if a line changes
1429         from:
1430
1431         const int myConstant = 0;
1432
1433         to
1434
1435         static const unsigned myConstant;
1436
1437         You will see that "static " was inserted, "int" changed to "unsigned",
1438         and " = 0" was deleted.
1439
1440         This seems to have also gotten rid of some spurious instances of
1441         "<ins></ins>" and "<del></del>" at the end of a line.
1442
1443         * PrettyPatch/PrettyPatch.rb:
1444         (PrettyPatch::DiffSection.initialize): Use DiffBuilder instead of
1445         setChangeExtentFromLine (which had been copied from Trac's diffing
1446         code).
1447         (PrettyPatch::CodeLine):
1448          - Removed the changeExtent attribute
1449          - Added the operations attribute
1450          - Removed the setChangeExtentFromLine method
1451         (PrettyPatch::CodeLine.text_as_html): Uses @operations to determine
1452         what text to wrap in <ins> and <del> tags.
1453         * PrettyPatch/diff.rb: Deleted a bunch of code we don't use.
1454         (HTMLDiff::DiffBuilder.initialize): Moved code here from the old build
1455         method, but left out the calls to perform_operation, since we build
1456         the diff HTML outside of this class.
1457         (HTMLDiff::DiffBuilder.split_inputs_to_words): Removed calls to
1458         convert_html_to_list_of_words, since we're not diffing HTML.
1459
1460 2008-04-08  Adam Roben  <aroben@apple.com>
1461
1462         Check in diff.rb
1463
1464         This came from
1465         http://instiki.rubyforge.org/svn/instiki/trunk/lib/diff.rb
1466         revision 28521
1467
1468         * PrettyPatch/diff.rb: Added.
1469
1470 2008-03-04  Adam Roben  <aroben@apple.com>
1471
1472         * PrettyPatch/PrettyPatch.rb: Allow the extent of the previous file to
1473         be omitted from the hunk header.
1474