Websites/bugs.webkit.org/ChangeLog

   1 2011-06-21  Adam Barth  <abarth@webkit.org>
   2
   3         Reviewed by Darin Adler.
   4
   5         bugs.webkit.org should use Strict-Transport-Security
   6         https://bugs.webkit.org/show_bug.cgi?id=63097
   7
   8         Strict-Transport-Security forces all connections to bugs.webkit.org to
   9         use HTTPS, preventing sslstrip and other attacks.
  10
  11         * .htaccess:
  12
  13 2011-05-04  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
  14
  15         Reviewed by Adam Roben.
  16
  17         REGRESSION (r85657): webkitpy.common.prettypatch_unittest.PrettyPatchTest.test_pretty_diff_encodings failing on Leopard
  18         https://bugs.webkit.org/show_bug.cgi?id=60164
  19
  20         Use Array#length to get the number of elements in an array. Array#count
  21         was added in Ruby 1.8.7, but Leopard have Ruby 1.8.6.
  22
  23         * PrettyPatch/PrettyPatch.rb:
  24
  25 2011-05-03  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
  26
  27         Reviewed by Adam Roben.
  28
  29         Add tests to PrettyPatch
  30         https://bugs.webkit.org/show_bug.cgi?id=57298
  31
  32         Verify that 'prettify' parse the right number of files and parts
  33         (add/remove/shared) by checking a set of patches available in
  34         bugs.webkit.org.
  35
  36         * PrettyPatch/PrettyPatch.rb: exported a couple a globals with
  37         statistics information for our test script.
  38         * PrettyPatch/PrettyPatch_test.rb: Added.
  39
  40 2011-04-22  Alexey Proskuryakov  <ap@apple.com>
  41
  42         Code changes by Adam Barth. Reviewed by Ojan Vafai.
  43
  44         Bugzilla autocomplete should support frequent contributors who are not committers
  45         https://bugs.webkit.org/show_bug.cgi?id=59209
  46
  47         * committers-autocomplete.js: Support Contributor role in committers.py.
  48
  49 2011-04-19  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
  50
  51         Reviewed by Adam Roben.
  52
  53         PrettyPatch tries to show images for deleted files
  54         https://bugs.webkit.org/show_bug.cgi?id=28457
  55
  56         When the image data is empty, instead of generating an <img> tag, generate
  57         a text to indicate the image was deleted.
  58
  59         * PrettyPatch/PrettyPatch.rb:
  60
  61 2011-04-14  Tony Chang  <tony@chromium.org>
  62
  63         Reviewed by Adam Roben.
  64
  65         make PrettyPatch.rb aware of checksums in png files
  66         https://bugs.webkit.org/show_bug.cgi?id=58387
  67
  68         Patch v2 with nil checks in the case of file adds and deletes.
  69
  70         * PrettyPatch/PrettyPatch.rb:
  71
  72 2011-04-13  Simon Fraser  <simon.fraser@apple.com>
  73
  74         Revert http://trac.webkit.org/changeset/83750 because it broke PrettyPatch.
  75         https://bugs.webkit.org/show_bug.cgi?id=58510
  76
  77         * PrettyPatch/PrettyPatch.rb:
  78
  79 2011-04-12  Tony Chang  <tony@chromium.org>
  80
  81         Reviewed by Adam Roben.
  82
  83         make PrettyPatch.rb aware of checksums in png files
  84         https://bugs.webkit.org/show_bug.cgi?id=58387
  85
  86         * PrettyPatch/PrettyPatch.rb:
  87
  88 2011-04-12  Adam Barth  <abarth@webkit.org>
  89
  90         Reviewed by Eric Seidel.
  91
  92         Review tool should work well on iPad
  93         https://bugs.webkit.org/show_bug.cgi?id=58351
  94
  95         On the iPad, the toolbar ends up at some random position in the middle
  96         of the page because of how position:fixed interacts with scrolling on
  97         the iPad.  I feel a bit dirty using a user agent detect, but I'm not
  98         sure if there's a better way to detect whether the user agent has this
  99         behavior.
 100
 101         * code-review.js:
 102
 103 2011-03-15  David Levin  <levin@chromium.org>
 104
 105         Reviewed by Adam Barth.
 106
 107         Links from code reviews to trac are incorrect for files in Source/WebKit.
 108         https://bugs.webkit.org/show_bug.cgi?id=56404
 109
 110         * PrettyPatch/PrettyPatch.rb: Remove WebKit from the list of top level
 111           directories since it is now under Source.
 112
 113 2011-03-10  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
 114
 115         Reviewed by Adam Roben.
 116
 117         Formatted Diff for attachment 23920 is mangled
 118         https://bugs.webkit.org/show_bug.cgi?id=21222
 119
 120         The mangled diff files mentioned in the bug were using Mac line
 121         ending and this was causing problems for String#each_line. Now we
 122         normalize the line endings in patch data before processing it.
 123
 124         * PrettyPatch/PrettyPatch.rb:
 125
 126 2011-03-10  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
 127
 128         Reviewed by Adam Roben.
 129
 130         PrettyPatch displays last two lines of a git-format-patch diff strangely
 131         https://bugs.webkit.org/show_bug.cgi?id=29317
 132
 133         If the diff section has full range information, we parse only enough to cover the
 134         range. This avoids incorrectly showing trailing lines (like git signature) as part
 135         of the patch.
 136
 137         * PrettyPatch/PrettyPatch.rb:
 138
 139 2011-03-10  Adam Roben  <aroben@apple.com>
 140
 141         Don't search for intra-line diffs in really long lines
 142
 143         Doing so can lead to hangs (or at least really slow execution).
 144
 145         Fixes <http://webkit.org/b/56109> run-webkit-tests sometimes times out on Windows XP Debug
 146         (Tests) after fast/text/large-text-composed-char-dos.html fails (due to PrettyPatch
 147         hanging?)
 148
 149         Reviewed by David Kilzer.
 150
 151         * PrettyPatch/PrettyPatch.rb:
 152         (PrettyPatch.MAXIMUM_INTRALINE_DIFF_LINE_LENGTH): Added this new constant.
 153         (PrettyPatch.DiffSection.initialize): Don't bother looking for intra-line diffs in lines
 154         longer than the maximum length.
 155
 156 2011-02-25  Ojan Vafai  <ojan@chromium.org>
 157
 158         Reviewed by Adam Barth.
 159
 160         [codereviewtool] fix expand links on git diffs
 161         https://bugs.webkit.org/show_bug.cgi?id=55208
 162
 163         In git diffs, we don't have a "revision" span at the end
 164         of each FileDiff.
 165         * code-review.js:
 166
 167 2011-02-23  Ojan Vafai  <ojan@chromium.org>
 168
 169         Reviewed by Tony Chang.
 170
 171         autocomplete=off prevents refilling form fields on back/forward navigation
 172         https://bugs.webkit.org/show_bug.cgi?id=47463
 173
 174         I haven't actually run this patch due to the difficulty of getting
 175         a bugzilla instance running. It's simple enough code that I'm not too worried
 176         about it.
 177
 178         * committers-autocomplete.js:
 179
 180 2011-02-21  Ojan Vafai  <ojan@chromium.org>
 181
 182         Reviewed by Adam Barth.
 183
 184         [codereviewtool] focus first visible node if no node is focused
 185         https://bugs.webkit.org/show_bug.cgi?id=54935
 186
 187         Now hitting j/k/n/p will focus the first/last visible node
 188         if no other node is focused. Also tweaked the scroll into view
 189         logic a bit to avoid scrolling in this case.
 190
 191         * code-review.js:
 192
 193 2011-02-21  Ojan Vafai  <ojan@chromium.org>
 194
 195         Reviewed by Adam Barth.
 196
 197         [codereviewtool] remove patch fuzzing
 198         https://bugs.webkit.org/show_bug.cgi?id=54940
 199
 200         This code is buggy and not as necessary now that we
 201         include svn revision numbers in uploaded patches.
 202
 203         * code-review.js:
 204
 205 2011-02-21  Ojan Vafai  <ojan@chromium.org>
 206
 207         Reviewed by Adam Barth.
 208
 209         [codereviewtool] focus comment on accept
 210         https://bugs.webkit.org/show_bug.cgi?id=54930
 211
 212         * code-review.js:
 213
 214 2011-02-20  Ojan Vafai  <ojan@chromium.org>
 215
 216         Reviewed by Adam Barth.
 217
 218         [codereviewtool] avoid overriding ctrl/cmd + r
 219         https://bugs.webkit.org/show_bug.cgi?id=54852
 220
 221         * code-review.js:
 222
 223 2011-02-16  Ojan Vafai  <ojan@chromium.org>
 224
 225         Reviewed by Adam Barth.
 226
 227         keyboard support for extending/shrinking comment context
 228         https://bugs.webkit.org/show_bug.cgi?id=54612
 229
 230         ctrl+shift+up/down will extend/shrink the comment context when
 231         a comment is focused or when one is being edited.
 232
 233         Also, switch over to using keydown instead of keypress events.
 234         This lets us share code for handling escape and other key events.
 235         Also, keypress is evil and should die.
 236
 237         * code-review.js:
 238
 239 2011-02-20  Ojan Vafai  <ojan@chromium.org>
 240
 241         Recommit accidental revert.
 242         r79180 accidentally revereted r79178.
 243         * code-review.js:
 244
 245 2011-02-20  Ojan Vafai  <ojan@chromium.org>
 246
 247         Reviewed by Adam Barth.
 248
 249         [codereviewtool] add more help information
 250         https://bugs.webkit.org/show_bug.cgi?id=54847
 251
 252         * code-review.js:
 253
 254 2011-02-20  Ojan Vafai  <ojan@chromium.org>
 255
 256         Reviewed by Adam Barth.
 257
 258         [codereviewtool] use 'r' to focus the review select element
 259         https://bugs.webkit.org/show_bug.cgi?id=54845
 260
 261         This allows for quick keyboard access to the toolbar items.
 262         Hit r to get to the review select element. Then you can
 263         tab to the cq/preview/publish elements.
 264
 265         * code-review.js:
 266
 267 2011-02-20  Ojan Vafai  <ojan@chromium.org>
 268
 269         Reviewed by Adam Barth.
 270
 271         [codereviewtool] don't let lines/diffblocks be mouse focusable
 272         https://bugs.webkit.org/show_bug.cgi?id=54851
 273
 274         Unfortunately, this will only work once https://bugs.webkit.org/show_bug.cgi?id=54727
 275         is committed. In the interim, it won't break anything.
 276
 277         * code-review.js:
 278
 279 2011-02-20  Ojan Vafai  <ojan@chromium.org>
 280
 281         Reviewed by Adam Barth.
 282
 283         [codereviewtool] use keydown instead of keypress
 284         https://bugs.webkit.org/show_bug.cgi?id=54849
 285
 286         There is no functional change. This is just a cleanup
 287         to make future patches (e.g. https://bugs.webkit.org/show_bug.cgi?id=54612)
 288         cleaner.
 289
 290         * code-review.js:
 291
 292 2011-02-20  Ojan Vafai  <ojan@chromium.org>
 293
 294         Reviewed by Adam Barth.
 295
 296         [codereviewtool] fix layout nit
 297         https://bugs.webkit.org/show_bug.cgi?id=54848
 298
 299         There is occasionally an off-by-one in the 50% width
 300         calculation. This ensures that the add lines always
 301         line up with the shared context lines.
 302
 303         * PrettyPatch/PrettyPatch.rb:
 304
 305 2011-02-20  Ojan Vafai  <ojan@chromium.org>
 306
 307         Reviewed by Adam Barth.
 308
 309         [codereviewtool] make escape hide the preview form
 310         https://bugs.webkit.org/show_bug.cgi?id=54844
 311
 312         * code-review.js:
 313
 314 2011-02-20  Ojan Vafai  <ojan@chromium.org>
 315
 316         Reviewed by Adam Barth.
 317
 318         [codereviewtool] make enter work when an individual line is focused
 319         https://bugs.webkit.org/show_bug.cgi?id=54843
 320
 321         * code-review.js:
 322
 323 2011-02-17  Ojan Vafai  <ojan@chromium.org>
 324
 325         Reviewed by Adam Barth.
 326
 327         use shift+j/k to focus next/previous line
 328         https://bugs.webkit.org/show_bug.cgi?id=54723
 329
 330         Tested on a large Hyatt change to confirm it performed fine.
 331         (https://bugs.webkit.org/attachment.cgi?id=82533&action=review)
 332
 333         * code-review.js:
 334
 335 2011-02-17  Ojan Vafai  <ojan@chromium.org>
 336
 337         Reviewed by Adam Barth.
 338
 339         fix escape handling in code review tool
 340         https://bugs.webkit.org/show_bug.cgi?id=54722
 341
 342         This broke with http://trac.webkit.org/changeset/78948.
 343         Apparently jquery events get a wrapped object that
 344         does not expose keyIdentifier.
 345
 346         * code-review.js:
 347
 348 2011-02-17  Ojan Vafai  <ojan@chromium.org>
 349
 350         Reviewed by Adam Barth.
 351
 352         create comments when hitting enter with a focused diff
 353         https://bugs.webkit.org/show_bug.cgi?id=54719
 354
 355         * code-review.js:
 356
 357 2011-02-17  Ojan Vafai  <ojan@chromium.org>
 358
 359         Reviewed by Adam Barth.
 360
 361         use actual browser focus in the code review tool
 362         https://bugs.webkit.org/show_bug.cgi?id=54726
 363
 364         This makes keyboard handling play better with focusable element
 365         (i.e. links/textareas/buttons). Also, in theory, this makes
 366         the review tool more amenable to screen readers (I think).
 367
 368         * PrettyPatch/PrettyPatch.rb:
 369         * code-review.js:
 370
 371 2011-02-17  Ojan Vafai  <ojan@chromium.org>
 372
 373         Reviewed by Antonio Gomes.
 374
 375         make event handling work in Gecko
 376         https://bugs.webkit.org/show_bug.cgi?id=54702
 377
 378         Gecko doesn't support the global event property. It's an IE-ism.
 379
 380         * code-review.js:
 381
 382 2011-02-14  Ojan Vafai  <ojan@chromium.org>
 383
 384         Reviewed by Adam Barth.
 385
 386         allow for modifying comments without the mouse
 387         https://bugs.webkit.org/show_bug.cgi?id=54433
 388
 389         Use n/p to navigate the comments.
 390         Enter to begin editing a comment.
 391         Escape to finish editing a comment.
 392
 393         * code-review.js:
 394
 395 2011-02-14  Ojan Vafai  <ojan@chromium.org>
 396
 397         Reviewed by Adam Barth.
 398
 399         improve line selection in the code review tool
 400         https://bugs.webkit.org/show_bug.cgi?id=54430
 401
 402         -shift+click now extends the comment context
 403         -selecting outside of a diff section no longer clears the selected lines.
 404          Instead it just restricts the selected lines to that diff section.
 405
 406         * code-review.js:
 407
 408 2011-02-10  Ojan Vafai  <ojan@chromium.org>
 409
 410         Reviewed by Adam Barth.
 411
 412         remove extra space from expansion lines
 413         https://bugs.webkit.org/show_bug.cgi?id=54266
 414
 415         * code-review.js:
 416
 417 2011-02-09  Ojan Vafai  <ojan@chromium.org>
 418
 419         Reviewed by Adam Barth.
 420
 421         only erase draft comments after publish is successful
 422         https://bugs.webkit.org/show_bug.cgi?id=54163
 423
 424         If there is a conflict or 500, then draft comments will survive.
 425         One drawback here is that the form post is now to the iframe,
 426         so to break out of the iframe we redirect to the bug page, which
 427         loses the information of who the email was sent to.
 428
 429         Once WebKit supports seamless iframes we should be able to avoid
 430         the redirect.
 431
 432         * code-review.js:
 433
 434 2011-02-09  Ojan Vafai  <ojan@chromium.org>
 435
 436         Reviewed by Adam Barth.
 437
 438         save overall comments when saving drafts in the review tool
 439         https://bugs.webkit.org/show_bug.cgi?id=54165
 440
 441         -save overall comments in localstorage as well
 442         -save all draft comments as you type
 443         -give a *subtle* indicator of saved state
 444
 445         The latter should also make it super easy if someone wanted to do
 446         the work to store draft comments in appengine/s3/bugzilla/etc.
 447
 448         * PrettyPatch/PrettyPatch.rb:
 449         * code-review-test.html:
 450         * code-review.js:
 451
 452 2011-02-08  Ojan Vafai  <ojan@chromium.org>
 453
 454         Reviewed by Adam Barth.
 455
 456         fix toolbar anchoring in the code review tool
 457         https://bugs.webkit.org/show_bug.cgi?id=54058
 458
 459         Avoid the anchoring cycle of doom when on the cusp
 460         of whether the toolbar needs to be anchored and
 461         speculatively avoid the Firefox crash when resizing.
 462
 463         * PrettyPatch/PrettyPatch.rb:
 464         * code-review.js:
 465
 466 2011-02-07  Ojan Vafai  <ojan@chromium.org>
 467
 468         Reviewed by Adam Barth.
 469
 470         bring back diff context lines into the review tool
 471         https://bugs.webkit.org/show_bug.cgi?id=53974
 472
 473         * code-review.js:
 474
 475 2011-02-01  Ojan Vafai  <ojan@chromium.org>
 476
 477         Reviewed by Adam Roben.
 478
 479         include svn revisions in git diffs for the code review tool to use
 480         https://bugs.webkit.org/show_bug.cgi?id=53569
 481
 482         * PrettyPatch/PrettyPatch.rb:
 483
 484 2011-02-01  Ojan Vafai  <ojan@chromium.org>
 485
 486         Reviewed by Adam Barth.
 487
 488         make draft comments focusable
 489         https://bugs.webkit.org/show_bug.cgi?id=53554
 490
 491         Makes frozen draft comments focusable. The ones that are currently being edited are not.
 492         I'm on the fence whether they should be, but this seems good enough for now.
 493
 494         * code-review.js:
 495
 496 2011-02-01  Ojan Vafai  <ojan@chromium.org>
 497
 498         Reviewed by Adam Barth.
 499
 500         avoid jitter when loading the comments to a patch
 501         https://bugs.webkit.org/show_bug.cgi?id=53570
 502
 503         * code-review.js:
 504
 505 2011-02-01  Ojan Vafai  <ojan@chromium.org>
 506
 507         Reviewed by Mihai Parparita.
 508
 509         [codereviewtool] extra space at the beginning of lines in side-by-side view
 510         https://bugs.webkit.org/show_bug.cgi?id=53550
 511
 512         The newline between the spans turns into user-visible space in side-by-side mode.
 513
 514         * PrettyPatch/PrettyPatch.rb:
 515
 516 2011-02-01  Ojan Vafai  <ojan@chromium.org>
 517
 518         Increment version number to cache-bust code-review.js.
 519
 520         * PrettyPatch/PrettyPatch.rb:
 521
 522 2011-01-31  Ojan Vafai  <ojan@chromium.org>
 523
 524         Reviewed by Adam Barth.
 525
 526         Store draft comments in localStorage
 527         https://bugs.webkit.org/show_bug.cgi?id=52866
 528
 529         * code-review.js:
 530         * code-review-test.html
 531
 532 2011-01-20  Ojan Vafai  <ojan@chromium.org>
 533
 534         Fix the review tool for image diffs. We would get a javascript error
 535         because image diffs don't have line numbers.
 536
 537         * code-review.js:
 538
 539 2011-01-20  Ojan Vafai  <ojan@chromium.org>
 540
 541         Reviewed by Oliver Hunt.
 542
 543         Could review tool include style failure info?
 544         https://bugs.webkit.org/show_bug.cgi?id=49049
 545
 546         This does not handle patches that have old paths (e.g. without the Source prefix).
 547         Although that would be fairly straightforward to workaround.
 548
 549         * code-review.js:
 550
 551 2011-01-20  Ojan Vafai  <ojan@chromium.org>
 552
 553         Reviewed by Adam Barth.
 554
 555         handle cases where PrettyPatch.rb doesn't linkify filenames
 556         https://bugs.webkit.org/show_bug.cgi?id=52834
 557
 558         * code-review.js:
 559
 560 2011-01-20  Ojan Vafai  <ojan@chromium.org>
 561
 562         Reviewed by Adam Barth.
 563
 564         fix goofup in http://trac.webkit.org/changeset/76082
 565         https://bugs.webkit.org/show_bug.cgi?id=52830
 566
 567         We were showing the revision number of the patch at the bottom
 568         of the diff and expanding from the bottom of the diff would fail.
 569
 570         Also, removed an unused variable and scoped queries appropriatly
 571         to the file_diff they should have been operating on.
 572
 573         * PrettyPatch/PrettyPatch.rb:
 574         * code-review.js:
 575
 576 2011-01-19  Ojan Vafai  <ojan@chromium.org>
 577
 578         Reviewed by Adam Barth.
 579
 580         [reviewtool] Add a link for annotated trac page on review page
 581         https://bugs.webkit.org/show_bug.cgi?id=52747
 582
 583         -add line numbers to all trac links to the first line in the file diff
 584         -add annotate/review log links
 585         -make file-diff links other than the file name only show when you mouseover the filediff
 586         -fix white-space to only wrap the .text element and it's childen, any of it's siblings/parents
 587
 588         * PrettyPatch/PrettyPatch.rb:
 589         * code-review.js:
 590
 591 2011-01-14  Ojan Vafai  <ojan@chromium.org>
 592
 593         Reviewed by Adam Barth.
 594
 595         make line selection have a extend only from where you start the selection in the code review tool
 596         https://bugs.webkit.org/show_bug.cgi?id=52485
 597
 598         * code-review.js:
 599
 600 2011-01-18  Ojan Vafai  <ojan@chromium.org>
 601
 602         Reviewed by Eric Seidel.
 603
 604         make file expansion use the file at the svn revision the diff was created at
 605         https://bugs.webkit.org/show_bug.cgi?id=52650
 606
 607         This only works for patches created with SVN for now since the git
 608         diff does not include SVN revision numbers in it.
 609
 610         * PrettyPatch/PrettyPatch.rb:
 611         * code-review.js:
 612
 613 2011-01-15  Adam Barth  <abarth@webkit.org>
 614
 615         Rubber-stamped by Eric Seidel.
 616
 617         Move WebKit2 into Source
 618         https://bugs.webkit.org/show_bug.cgi?id=52438
 619
 620         Remove WebKit2 from the list of top-level source directories.
 621
 622         * PrettyPatch/PrettyPatch.rb:
 623
 624 2011-01-14  Ojan Vafai  <ojan@chromium.org>
 625
 626         Reviewed by Adam Barth.
 627
 628         fix expanded lines in the code review tool
 629         https://bugs.webkit.org/show_bug.cgi?id=52498
 630
 631         Fixes them to work now that we have wrapper divs.
 632         Also fixes the long-standing bug that the line numbers
 633         were incorrect sometimes.
 634
 635         * code-review.js:
 636
 637 2011-01-14  Ojan Vafai  <ojan@chromium.org>
 638
 639         Reviewed by Adam Barth.
 640
 641         put remove lines to the left of add lines in sidebyside view
 642         https://bugs.webkit.org/show_bug.cgi?id=52458
 643
 644         Also, sneaking in a fix to make long lines with no whitespace wrap.
 645
 646         * PrettyPatch/PrettyPatch.rb:
 647         * code-review.js:
 648
 649 2011-01-14  Ojan Vafai  <ojan@chromium.org>
 650
 651         Fix bad merge in r75816.
 652
 653         * code-review.js:
 654
 655 2011-01-14  Ojan Vafai  <ojan@chromium.org>
 656
 657         Reviewed by Adam Barth.
 658
 659         improve line selection with fast drags
 660         https://bugs.webkit.org/show_bug.cgi?id=52477
 661
 662         * code-review.js:
 663
 664 2011-01-14  Ojan Vafai  <ojan@chromium.org>
 665
 666         Reviewed by Adam Barth.
 667
 668         tweak line selection in code review tool
 669         https://bugs.webkit.org/show_bug.cgi?id=52462
 670
 671         -Improve handling of data-comment-base-line code to deal with
 672         lines that have multiple values.
 673         -Make it so that if you click on the line immediately above
 674         a line that has a comment it will add a new comment instead of
 675         adding lines to the following comment. If the last selected line
 676         overlaps existing comment lines though they will still get
 677         added to the existing comment.
 678
 679         * code-review.js:
 680
 681 2011-01-14  Ojan Vafai  <ojan@chromium.org>
 682
 683         Reviewed by Adam Roben.
 684
 685         fix bugs going back and forth between unified and sidebyside
 686         https://bugs.webkit.org/show_bug.cgi?id=52470
 687
 688         Now that we're appending DOM nodes directly, we only want to
 689         get the first ".text" element in the subtree. In the cases
 690         where there are multiple, their contents are the same.
 691
 692         Also, when making side-by-side expansion lines, we need to
 693         clone the contents so we have a second copy to append to the DOM.
 694
 695         * code-review.js:
 696
 697 2011-01-13  Ojan Vafai  <ojan@chromium.org>
 698
 699         Reviewed by Adam Barth.
 700
 701         simplify keyboard handling in code review tool
 702         https://bugs.webkit.org/show_bug.cgi?id=52407
 703
 704         Now that we have DiffBlock containers, the only things that are
 705         focusable are previousComment nodes and DiffBlock containers
 706         that contain add/remove lines.
 707
 708         Also, this means we show the focus border around the entire diff
 709         instead of just the first line.
 710
 711         * code-review.js:
 712
 713 2011-01-13  Ojan Vafai  <ojan@chromium.org>
 714
 715         Reviewed by Adam Barth.
 716
 717         add container divs for diff blocks
 718         https://bugs.webkit.org/show_bug.cgi?id=52400
 719
 720         This will help simplify a lot of code in code-review.js
 721         and make side-by-side diffs better (i.e. put removed lines
 722         to the left of corresponding added lines).
 723
 724         Also, allow for running the JS from a local file. Now you can modify code-review.js
 725         to point to a local file and then run:
 726         ruby prettify.rb < foo.diff > foo.html
 727
 728         foo.html will load a dummy code review matching foo.diff.
 729
 730         Before structure:
 731         Line
 732         Line remove
 733         Line add
 734         Line add
 735         Line
 736
 737         After structure:
 738         DiffBlock
 739           DiffBlockPart shared
 740             Line shared
 741         DiffBlock
 742           DiffBlockPart remove
 743             Line remove
 744           DiffBlockPart add
 745             Line add
 746             Line add
 747         DiffBlock
 748           DiffBlockPart shared
 749             Line shared
 750
 751         * PrettyPatch/PrettyPatch.rb:
 752         * code-review.js:
 753
 754 2011-01-12  Ojan Vafai  <ojan@chromium.org>
 755
 756         Reviewed by Mihai Parparita.
 757
 758         fix assorted bugs with expansion lines
 759         https://bugs.webkit.org/show_bug.cgi?id=52326
 760
 761         -Clicking "all" would collapse whitespace in the expanded lines.
 762         -Converted the remaining line creation code to using the DOM.
 763
 764         * PrettyPatch/PrettyPatch.rb:
 765         * code-review.js:
 766
 767 2011-01-12  Ojan Vafai  <ojan@chromium.org>
 768
 769         Reviewed by Adam Barth.
 770
 771         review tool formatted diff doesn't match the uploaded diff
 772         https://bugs.webkit.org/show_bug.cgi?id=51960
 773
 774         PrettyPatch works just fine if there is no newline at the end of the file.
 775
 776         * attachment.cgi:
 777
 778 2011-01-12  Ojan Vafai  <ojan@chromium.org>
 779
 780         Reviewed by Adam Barth.
 781
 782         Bugzilla: Add keyboard shortcuts to jump to next change
 783         https://bugs.webkit.org/show_bug.cgi?id=52305
 784
 785         Comments and diff blocks go in the same queue. If you have a
 786         comment focused, then j/k will focus the next/prev diff block
 787         with respect to that comment.
 788
 789         * PrettyPatch/PrettyPatch.rb:
 790         * code-review.js:
 791
 792 2011-01-12  Ojan Vafai  <ojan@chromium.org>
 793
 794         Reviewed by Adam Barth.
 795
 796         show shared lines on both sides in code review tool
 797         https://bugs.webkit.org/show_bug.cgi?id=52308
 798
 799         We were appending the same DOM node twice. Instead, make
 800         the from column a clone of the original node.
 801
 802         * code-review.js:
 803
 804 2011-01-12  Ojan Vafai  <ojan@chromium.org>
 805
 806         Reviewed by Adam Barth.
 807
 808         fix wrapping bug in expansion lines in the code review tool
 809         https://bugs.webkit.org/show_bug.cgi?id=52270
 810
 811         * PrettyPatch/PrettyPatch.rb:
 812         * code-review.js:
 813
 814 2011-01-11  Ojan Vafai  <ojan@chromium.org>
 815
 816         Reviewed by Adam Barth.
 817
 818         fix difflink centering to be vertical, not horizontal
 819         https://bugs.webkit.org/show_bug.cgi?id=52263
 820
 821         * PrettyPatch/PrettyPatch.rb:
 822         * code-review.js:
 823
 824 2011-01-11  Ojan Vafai  <ojan@chromium.org>
 825
 826         Reviewed by Adam Barth.
 827
 828         remember diffstate for review tool
 829         https://bugs.webkit.org/show_bug.cgi?id=52253
 830
 831         If you use the global sidebyside/unified links, store the diff type
 832         in localstorage. Then, onload, use that diff type. That way, people
 833         who prefer one diff type or the other can always get that by default.
 834
 835         * code-review.js:
 836
 837 2011-01-11  Ojan Vafai  <ojan@chromium.org>
 838
 839         Reviewed by Adam Barth.
 840
 841         allow sidebysideifying individual files in the code review tool
 842         https://bugs.webkit.org/show_bug.cgi?id=52226
 843
 844         * code-review.js:
 845
 846 2011-01-11  Ojan Vafai  <ojan@chromium.org>
 847
 848         Reviewed by Mihai Parparita.
 849
 850         fix exception when adding a comment to a side-by-side diff
 851         https://bugs.webkit.org/show_bug.cgi?id=52240
 852
 853         prev() and next() won't get the previous and next lines in
 854         side-by-side mode. Instead do a query to find them.
 855
 856         * code-review.js:
 857
 858 2011-01-11  Ojan Vafai  <ojan@chromium.org>
 859
 860         Reviewed by Mihai Parparita.
 861
 862         maintain word diffs when converting to side-by-side and back
 863         https://bugs.webkit.org/show_bug.cgi?id=52244
 864
 865         Using textContent would lose the spans use to make the word-diff highlighting.
 866         Just move the span element itself.
 867
 868         * code-review.js:
 869
 870 2011-01-11  Ojan Vafai  <ojan@chromium.org>
 871
 872         Reviewed by Adam Barth.
 873
 874         fix layout error with comments in code review tool
 875         https://bugs.webkit.org/show_bug.cgi?id=52230
 876
 877         * PrettyPatch/PrettyPatch.rb:
 878
 879 2011-01-10  Ojan Vafai  <ojan@chromium.org>
 880
 881         Reviewed by Adam Barth.
 882
 883         convert back to unified from sidebyside diff
 884         https://bugs.webkit.org/show_bug.cgi?id=52180
 885
 886         Remove url fragment stuff. Having it be per-filediff is too complicated.
 887
 888         * PrettyPatch/PrettyPatch.rb:
 889         * code-review.js:
 890
 891 2011-01-07  Adam Barth  <abarth@webkit.org>
 892
 893         Rubber-stamped by Eric Seidel.
 894
 895         Move WebCore to Source
 896         https://bugs.webkit.org/show_bug.cgi?id=52050
 897
 898         Remove reference to old directory.
 899
 900         * PrettyPatch/PrettyPatch.rb:
 901
 902 2011-01-06  Ojan Vafai  <ojan@chromium.org>
 903
 904         Reviewed by Adam Barth.
 905
 906         side-by-side diffs in the code review tool
 907         https://bugs.webkit.org/show_bug.cgi?id=52019
 908
 909         Support for conversion from the formatted diff to a side-by-side diff.
 910         Maintains comments and new comments can be added.
 911
 912         The main architectural change is that Line elements are no longer necessarily
 913         siblings. Each physical line is now in a LineContainer and LineContainers are
 914         siblings. Each Line corresponds to a Line in the unified diff and has an id (e.g. line12).
 915         A Line can be a LineContainer or a child of a LineContainer.
 916
 917         In this way, converting to side-by-side and, in the future, back to unified is non-lossy.
 918
 919         * PrettyPatch/PrettyPatch.rb:
 920         * code-review.js:
 921
 922 2011-01-06  Ojan Vafai  <ojan@chromium.org>
 923
 924         Fix line context when replying to comments.
 925
 926         * code-review.js:
 927
 928 2011-01-05  Ojan Vafai  <ojan@chromium.org>
 929
 930         Reviewed by Adam Barth.
 931
 932         change the way we do comment highlighting in the code review tool
 933         https://bugs.webkit.org/show_bug.cgi?id=51971
 934
 935         Store a space-separated list of base line IDs on each line that has
 936         comments associated with that line. This allows for overlapping comments,
 937         but more importantly, makes adding side-by-side diff support easier.
 938
 939         * code-review.js:
 940
 941 2011-01-05  Ojan Vafai  <ojan@chromium.org>
 942
 943         Reviewed by Adam Barth.
 944
 945         minor code cleanup for code review tool
 946         https://bugs.webkit.org/show_bug.cgi?id=51962
 947
 948         Consolidates some queries to using shared functions.
 949
 950         * code-review.js:
 951
 952 2011-01-05  Ojan Vafai  <ojan@chromium.org>
 953
 954         Reviewed by Adam Barth.
 955
 956         assorted cleanups to prepare for side-by-side diffing
 957         https://bugs.webkit.org/show_bug.cgi?id=51961
 958
 959         Mostly, use selector queries instead of assuming comments are next siblings.
 960
 961         * code-review.js:
 962
 963 2011-01-05  Ojan Vafai  <ojan@chromium.org>
 964
 965         Reviewed by Adam Barth.
 966
 967         fix message container selector for code review tool
 968         https://bugs.webkit.org/show_bug.cgi?id=51959
 969
 970         * code-review.js:
 971
 972 2011-01-04  Ojan Vafai  <ojan@chromium.org>
 973
 974         Increment version number to avoid getting the cached JS file.
 975
 976         * PrettyPatch/PrettyPatch.rb:
 977
 978 2010-12-15  Ojan Vafai  <ojan@chromium.org>
 979
 980         Reviewed by Adam Barth.
 981
 982         anchor the toolbar to the bottom of the diff if the diff doesn't take a full screen of height
 983         https://bugs.webkit.org/show_bug.cgi?id=51162
 984
 985         * PrettyPatch/PrettyPatch.rb:
 986         * code-review.js:
 987
 988 2011-01-01  Adam Barth  <abarth@webkit.org>
 989
 990         Reviewed by Eric Seidel.
 991
 992         Move JavaScriptCore to Source
 993         https://bugs.webkit.org/show_bug.cgi?id=51604
 994
 995         * PrettyPatch/PrettyPatch.rb:
 996           - Remove reference to JavaScriptCore as a source directory.
 997
 998 2011-01-01  Adam Barth  <abarth@webkit.org>
 999
1000         Reviewed by Eric Seidel.
1001
1002         Move Sources to Source
1003         https://bugs.webkit.org/show_bug.cgi?id=51794
1004
1005         * PrettyPatch/PrettyPatch.rb:
1006
1007 2010-12-31  Adam Barth  <abarth@webkit.org>
1008
1009         Rubber-stamped by Eric Seidel.
1010
1011         Move PageLoadTests to PerformanceTests/PageLoad
1012         https://bugs.webkit.org/show_bug.cgi?id=51771
1013
1014         Update list of top-level open-source directories.
1015
1016         * PrettyPatch/PrettyPatch.rb:
1017
1018 2010-12-31  Adam Barth  <abarth@webkit.org>
1019
1020         Rubber-stamped by Eric Seidel.
1021
1022         Move SunSpider into PerformanceTests
1023         https://bugs.webkit.org/show_bug.cgi?id=51769
1024
1025         Update list of top-level open-source directories.
1026
1027         * PrettyPatch/PrettyPatch.rb:
1028
1029 2010-12-26  Adam Barth  <abarth@webkit.org>
1030
1031         Reviewed by Eric Seidel.
1032
1033         Move autotools into Sources
1034         https://bugs.webkit.org/show_bug.cgi?id=51630
1035
1036         * PrettyPatch/PrettyPatch.rb:
1037           - Remove reference to non-existant autotools directory.
1038
1039 2010-12-26  Adam Barth  <abarth@webkit.org>
1040
1041         Reviewed by Eric Seidel.
1042
1043         Move cmake into Sources
1044         https://bugs.webkit.org/show_bug.cgi?id=51631
1045
1046         * PrettyPatch/PrettyPatch.rb:
1047
1048 2010-12-23  Adam Barth  <abarth@webkit.org>
1049
1050         Rubber-stamped by Eric Seidel.
1051
1052         Move JavaScriptGlue into Sources
1053         https://bugs.webkit.org/show_bug.cgi?id=51583
1054
1055         PrettyPatch keeps a list of the top-level directories that exist in
1056         trac.
1057
1058         * PrettyPatch/PrettyPatch.rb:
1059
1060 2010-12-20  Adam Barth  <abarth@webkit.org>
1061
1062         One more tweak caused by moving the web sites around.  I had this
1063         change in my patch originally, but I missed it because I had to use
1064         server-side svn move commands to move the bulk of the web site files.
1065
1066         * PrettyPatch/PrettyPatch.rb:
1067
1068 2010-12-18  Adam Barth  <abarth@webkit.org>
1069
1070         Reviewed by Sam Weinig.
1071
1072         Move WebKitExamplePlugins to Examples
1073         https://bugs.webkit.org/show_bug.cgi?id=51291
1074
1075         * PrettyPatch/PrettyPatch.rb:
1076
1077 2010-12-17  Dan Bernstein  <mitz@apple.com>
1078
1079         Reviewed by Simon Fraser.
1080
1081         Rename WebKitTools to Tools
1082         https://bugs.webkit.org/show_bug.cgi?id=49861
1083
1084         * PrettyPatch/PrettyPatch.rb:
1085         * committers-autocomplete.js:
1086
1087 2010-12-17  Ojan Vafai  <ojan@chromium.org>
1088
1089         Increment psuedo-version number to avoid pulling cached version.
1090
1091         * PrettyPatch/PrettyPatch.rb:
1092
1093 2010-12-17  Ojan Vafai  <ojan@chromium.org>
1094
1095         Speculative fix for the code review tool in Firefox.
1096         Fixes a JS error and makes the status bubble mostly work
1097         if postMessage is not supported.
1098
1099         * PrettyPatch/PrettyPatch.rb:
1100         * code-review.js:
1101
1102 2010-12-15  Ojan Vafai  <ojan@chromium.org>
1103
1104         Reviewed by Adam Barth.
1105
1106         size status bubble to it's contents on the code review page
1107         https://bugs.webkit.org/show_bug.cgi?id=51142
1108
1109         * PrettyPatch/PrettyPatch.rb:
1110         * code-review.js:
1111
1112 2010-12-15  Ojan Vafai  <ojan@chromium.org>
1113
1114         Reviewed by Adam Barth.
1115
1116         fix goofups from r74142 and r74130
1117         https://bugs.webkit.org/show_bug.cgi?id=51146
1118
1119         Accidentally shrank the inline comment boxes and made
1120         the toolbar not show up when there were image diffs.
1121
1122         * PrettyPatch/PrettyPatch.rb:
1123         * code-review.js:
1124
1125 2010-12-14  Ojan Vafai  <ojan@chromium.org>
1126
1127         Reviewed by Adam Barth.
1128
1129         add ability to view for file context to the review tool
1130         https://bugs.webkit.org/show_bug.cgi?id=51057
1131
1132         At the beginning/end of each file diff and between each
1133         hunk add links to expand the context. For now it grabs the
1134         tip of tree version of the file and tries to apply the diff
1135         to that file. If it can't apply, then it gives up as we
1136         wouldn't want to show the wrong lines of context.
1137
1138         In the future, we can consider adding the upload svn revision
1139         to the diff itself, then we could fallback to the file at that
1140         revision if tip of tree doesn't apply.
1141
1142         * PrettyPatch/PrettyPatch.rb:
1143         * code-review.js:
1144
1145 2010-12-08  Ojan Vafai  <ojan@chromium.org>
1146
1147         Reviewed by Adam Barth.
1148
1149         [reviewtool] should always show overall comments text box
1150         https://bugs.webkit.org/show_bug.cgi?id=45870
1151
1152         * PrettyPatch/PrettyPatch.rb:
1153         * code-review.js:
1154
1155 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
1156
1157         Reviewed by Adam Barth.
1158
1159         Increase size of status bubbles
1160         https://bugs.webkit.org/show_bug.cgi?id=50496
1161
1162         The current available space isn't enough to display all
1163         pending results including position in the queue.
1164
1165         * PrettyPatch/PrettyPatch.rb:
1166
1167 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
1168
1169         Reviewed by Adam Barth.
1170
1171         Increase size of status bubbles
1172         https://bugs.webkit.org/show_bug.cgi?id=50496
1173
1174         The current available space isn't enough to display all
1175         pending results including position in the queue.
1176
1177         * template/en/custom/attachment/edit.html.tmpl:
1178         * template/en/custom/attachment/list.html.tmpl:
1179         * template/en/custom/attachment/reviewform.html.tmpl:
1180
1181 2010-11-08  Adam Barth  <abarth@webkit.org>
1182
1183         Update help text to match behavior change.
1184
1185         * code-review.js:
1186
1187 2010-11-08  Adam Barth  <abarth@webkit.org>
1188
1189         Reviewed by Eric Seidel.
1190
1191         Restore normal meaning of double-click on patch review page?
1192         https://bugs.webkit.org/show_bug.cgi?id=47641
1193
1194         After this patch, you need to click on the line numbers to add a
1195         comment.
1196
1197         * PrettyPatch/PrettyPatch.rb:
1198         * code-review.js:
1199
1200 2010-10-09  Adam Barth  <abarth@webkit.org>
1201
1202         Reviewed by Alexey Proskuryakov.
1203
1204         Exception in committers-autocomplete in Firefox
1205         https://bugs.webkit.org/show_bug.cgi?id=47456
1206
1207         Remove dependency on a WebKit-only quirk.  IMHO, this quirk is bad for
1208         the long-term health of the web, but that's a discussion for another
1209         patch.
1210
1211         * committers-autocomplete.js:
1212
1213 2010-10-08  Adam Barth  <abarth@webkit.org>
1214
1215         Reviewed by Oliver Hunt.
1216
1217         Bugmail for new attachments should link to the reviewtool
1218         https://bugs.webkit.org/show_bug.cgi?id=47440
1219
1220         As requested by olliej himself.  This patch was constructed by pure
1221         reason.  I have no way to actually run this code.
1222
1223         * Bugzilla/BugMail.pm:
1224
1225 2010-09-28  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
1226
1227         Reviewed by Csaba Osztrogonác.
1228
1229         Convert verbatim unicode values in comitter names to characters
1230
1231         * committers-autocomplete.js:
1232
1233 2010-09-24  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
1234
1235         Reviewed by Csaba Osztrogonác.
1236
1237         Support Unicode-strings in committers-autocomplete script
1238
1239         * committers-autocomplete.js:
1240
1241 2010-09-22  Adam Barth  <abarth@webkit.org>
1242
1243         Reviewed by John Sullivan.
1244
1245         New review-page design doesn't include name of reviewer
1246         https://bugs.webkit.org/show_bug.cgi?id=46271
1247
1248         * PrettyPatch/PrettyPatch.rb:
1249         * code-review.js:
1250
1251 2010-09-22  Adam Barth  <abarth@webkit.org>
1252
1253         Fix typo in CSS.
1254
1255         * PrettyPatch/PrettyPatch.rb:
1256
1257 2010-09-22  Adam Barth  <abarth@webkit.org>
1258
1259         Reviewed by Eric Seidel.
1260
1261         Add link to bug to review page
1262         https://bugs.webkit.org/show_bug.cgi?id=46192
1263
1264         To make room for the link, I moved the help text to the top of the page
1265         and tweaked the language to help folks discover that you can select
1266         context using the line numbers.
1267
1268         Also, move more text to sans-serif since that fits in better with the
1269         pretty-patch fonts.
1270
1271         * PrettyPatch/PrettyPatch.rb:
1272         * code-review.js:
1273
1274 2010-09-21  Adam Barth  <abarth@webkit.org>
1275
1276         Unreviewed.
1277
1278         [reviewtool] Publish button doesn't work
1279         https://bugs.webkit.org/show_bug.cgi?id=46168
1280
1281         Darin's recent patch had a runtime error.  Sadly, we don't have any
1282         testing for this code.
1283
1284         * code-review.js:
1285
1286 2010-09-20  Darin Adler  <darin@apple.com>
1287
1288         Reviewed by Adam Barth.
1289
1290         Add Preview button and link to bug to patch review bar
1291         https://bugs.webkit.org/show_bug.cgi?id=46153
1292
1293         * PrettyPatch/PrettyPatch.rb: Bump version number to make caches happy.
1294         * code-review.js: Added Preview button and link to bug.
1295
1296 2010-09-19  Adam Barth  <abarth@webkit.org>
1297
1298         Reviewed by Eric Seidel.
1299
1300         [reviewtool] Publish button should actually publish the review comments
1301         https://bugs.webkit.org/show_bug.cgi?id=46074
1302
1303         Instead of showing the confirmation lightbox, the Publish button now
1304         publishes the comments directly.  If there's demand for a "Preview"
1305         button, we can find a way to add that feature.
1306
1307         * code-review.js:
1308
1309 2010-09-19  Adam Barth  <abarth@webkit.org>
1310
1311         Reviewed by Eric Seidel.
1312
1313         Move reviewtool to action=review
1314         https://bugs.webkit.org/show_bug.cgi?id=46071
1315
1316         This patch removes the old Review Patch screen and moves the new
1317         reviewtool from Pretty Diff to Review Patch.
1318
1319         * attachment.cgi:
1320         * code-review.js:
1321
1322 2010-09-19  Adam Barth  <abarth@webkit.org>
1323
1324         Review tool UI tweak.  This patch lengthens the status bubbles so all
1325         the bubble fit (even when they have numbers inside).
1326
1327         * PrettyPatch/PrettyPatch.rb:
1328
1329 2010-09-19  Dan Bernstein  <mitz@apple.com>
1330
1331         Reviewed by Anders Carlsson.
1332
1333         Fixed a typo.
1334
1335         * code-review.js: Changed the label of the OK button from Ok to OK.
1336
1337 2010-09-15  Adam Barth  <abarth@webkit.org>
1338
1339         Unreviewed.
1340
1341         Only show the review-in-context link if there are contextual comments.
1342
1343         * code-review.js:
1344
1345 2010-09-15  Adam Barth  <abarth@webkit.org>
1346
1347         Reviewed by Eric Seidel.
1348
1349         [reviewtool] Show the status bubbles on the toolbar
1350         https://bugs.webkit.org/show_bug.cgi?id=45861
1351
1352         * PrettyPatch/PrettyPatch.rb:
1353         * code-review.js:
1354
1355 2010-09-15  Adam Barth  <abarth@webkit.org>
1356
1357         Reviewed by Eric Seidel.
1358
1359         [reviewtool] Allow setting review and commit-queue flags from main review screen
1360         https://bugs.webkit.org/show_bug.cgi?id=45860
1361
1362         This patch adds drop down menus for adjusting the review and
1363         commit-queue flags from the main review screen.  These controls don't
1364         appear immediately because we need to read the state of the flags off
1365         the details page.
1366
1367         It's a slight sadness that bugzilla doesn't have a clean way of
1368         identifying which select control is associated with a given flag.  The
1369         flags seem to have some sort of id, but it's unclear to me how to map
1370         that id back to a notion of "review" or "commit-queue".  Instead, we
1371         look for some magic strings in the title.  I'm sure this will come back
1372         to bit us at some point, but I'm not sure what to do that's better.
1373         (The webkitpy solution to this problem is to assume that the flags are
1374         in a given order, but this seems better.)
1375
1376         * PrettyPatch/PrettyPatch.rb:
1377             - Bump version number to bust throuh caches.
1378         * code-review.js:
1379
1380 2010-09-15  Adam Barth  <abarth@webkit.org>
1381
1382         Reviewed by Eric Seidel.
1383
1384         [reviewtool] Comments should quote previous comments on the same line
1385         https://bugs.webkit.org/show_bug.cgi?id=45847
1386
1387         Now when you reply to a previous comment, the tool will quote the
1388         previous comment in the bugs.webkit.org post.  This makes it eaiser for
1389         folks following along in email to understand the discussion.
1390
1391         While I was editing this code, I also cleaned up some of the whitespace
1392         handling in comments.
1393
1394         * code-review.js:
1395
1396 2010-09-15  Adam Barth  <abarth@webkit.org>
1397
1398         Minor UI tweak to the review tool.  We want to display newlines in
1399         previous and frozen comments instead of collapsing them.
1400
1401         * PrettyPatch/PrettyPatch.rb:
1402
1403 2010-09-14  Adam Barth  <abarth@webkit.org>
1404
1405         Reviewed by Eric Seidel.
1406
1407         bugs.webkit.org should autocomplete WebKit Committers and Reviewers
1408         https://bugs.webkit.org/show_bug.cgi?id=45799
1409
1410         This patch implements an autocomplete dropdown that helps users type
1411         the names of WebKit committers and reviewers.  The script grabs the
1412         data from SVN now that wms set up CORS for http://svn.webkit.org.
1413         Security technology for the win.
1414
1415         This script is based on (well, essentially copied from) the popular
1416         autocomplete extension:
1417
1418         https://chrome.google.com/extensions/detail/olaabhcgdogcbcoiolomlcodkngnemfb
1419
1420         I asked Ojan, and said he was fine licensing the code under the BSD
1421         license.  I'm not sure how the patch interacts with the extension, but
1422         we can retire the extension if they interact poorly.
1423
1424         * committers-autocomplete.js: Added.
1425         * template/en/default/global/footer.html.tmpl:
1426             - Add the script to every page on bugs.webkit.org
1427
1428 2010-09-14  Adam Barth  <abarth@webkit.org>
1429
1430         Add a license block to the reviewtool.  I should have added this from
1431         the start.  According to the SVN log, I'm the only one to have ever
1432         touched this file.
1433
1434         * code-review.js:
1435
1436 2010-09-07  Adam Barth  <abarth@webkit.org>
1437
1438         Minor tweaks to the reviewtool UI.
1439
1440         Rename Cancel to Discard to make it more clear what the button does.
1441         Also, move the style declaration for the focus ring later in the sheet
1442         so it works.
1443
1444         * PrettyPatch/PrettyPatch.rb:
1445         * code-review.js:
1446
1447 2010-09-06  Adam Barth  <abarth@webkit.org>
1448
1449         Small UI tweak to the reviewtool.  Make the Ok/Cancel buttons a bit
1450         bigger.
1451
1452         * PrettyPatch/PrettyPatch.rb:
1453
1454 2010-09-06  Adam Barth  <abarth@webkit.org>
1455
1456         Reviewed by Eric Seidel.
1457
1458         [reviewtool] Add a quick animation for opening/closing comment boxes
1459         https://bugs.webkit.org/show_bug.cgi?id=45278
1460
1461         The 200ms animation slides in and out the line-by-line comment box.
1462
1463         * code-review.js:
1464
1465 2010-09-06  Adam Barth  <abarth@webkit.org>
1466
1467         Reviewed by Eric Seidel.
1468
1469         [reviewtool] Add a field for overall comments
1470         https://bugs.webkit.org/show_bug.cgi?id=45273
1471
1472         This patch does a couple logically separate things that could be
1473         separated into smaller patches:
1474
1475         1) This patch adds an "overall comments" field where you can enter
1476            overall comments about the patch.  These comments appear at the top
1477            of the bugzilla posting.  Currently, these aren't redisplayed when
1478            viewing the patch, but I plan to add that in a future patch.
1479
1480         2) This patch renames some of the CSS classes to more consistently
1481            follow the camelCase style that PrettyPatch uses.
1482
1483         3) This patch moves the "prepare comments" button to the left of the
1484            toolbar and renames is to "publish comments".  This makes more sense
1485            when you scroll to the bottom of the page and enter in some overall
1486            comments.
1487
1488         4) When you attempt to add a comment to a line that already has a
1489            "frozen" comment, we now unfreeze the comment instead of doing
1490            nothing.  The old behavior was kind of frustrating if you didn't
1491            know that you could unfreeze a comment by clicking on it.
1492
1493         * PrettyPatch/PrettyPatch.rb:
1494             - Update CSS.
1495         * code-review.js:
1496
1497 2010-09-06  Adam Barth  <abarth@webkit.org>
1498
1499         [reviewtool] Tweak the ok button to cancel the comment if the comment
1500         is empty.  Previously we would get into a bad state where a line had a
1501         comment but there was no longer any way to access it.
1502
1503         * code-review.js:
1504
1505 2010-09-06  Adam Barth  <abarth@webkit.org>
1506
1507         Reviewed by Eric Seidel.
1508
1509         [reviewtool] Add an "ok" button that collapses review comments
1510         https://bugs.webkit.org/show_bug.cgi?id=45255
1511
1512         Many folks (including myself) are too trigger-happy with the "delete"
1513         button that we end up deleting perfectly good review comments.  This
1514         patch adds an "ok" button to absorb these clicks (and renames "delete"
1515         to "cancel").  When you click the ok button, the comment becomes
1516         non-editable and the box shrinks to fit the text.  You can click the
1517         text to get back to the editable version.
1518
1519         * code-review.js:
1520
1521 2010-08-31  Adam Barth  <abarth@webkit.org>
1522
1523         Reviewed by Eric Seidel.
1524
1525         [reviewtool] Make it easy to scroll through review comments
1526         https://bugs.webkit.org/show_bug.cgi?id=45002
1527
1528         This patch lets you scroll through review comments using "n" (for next)
1529         and "p" (for previous).  It also attributes comments to their authors.
1530
1531         * PrettyPatch/PrettyPatch.rb:
1532         * code-review.js:
1533
1534 2010-08-31  Adam Barth  <abarth@webkit.org>
1535
1536         Reviewed by Eric Seidel.
1537
1538         [reviewtool] Show previous comments inline in diff
1539         https://bugs.webkit.org/show_bug.cgi?id=44977
1540
1541         This patch adds basic support for showing previous comments inline in
1542         the diff.  We crawl the bugs.webkit.org comments about this attachment
1543         and extract comments related to specific lines.  We then show the
1544         comments inline in the diff.
1545
1546         This part of the tool needs a bunch of polish, but this at least is a
1547         starting point for further work.
1548
1549         * PrettyPatch/PrettyPatch.rb:
1550         * code-review.js:
1551
1552 2010-08-31  Adam Barth  <abarth@webkit.org>
1553
1554         Reviewed by Eric Seidel.
1555
1556         [reviewtool] Allow reviewer to select lines of context by dragging over the line numbers
1557         https://bugs.webkit.org/show_bug.cgi?id=44936
1558
1559         * PrettyPatch/PrettyPatch.rb:
1560             - Add a version number to bust through bugs.webkit.org's aggressive
1561               cacheing.
1562         * code-review.js:
1563
1564 2010-08-30  Adam Barth  <abarth@webkit.org>
1565
1566         Reviewed by Eric Seidel.
1567
1568         [review tool] Let reviewer select how much context to show in snippet
1569         https://bugs.webkit.org/show_bug.cgi?id=44905
1570
1571         We now highlight the context for a comment in yellow on the left (where
1572         the line numbers are).  Clicking a line number expands or contracts the
1573         amount of context, as appropriate.  Informal user testing indicates
1574         that we might want to support drag as well.
1575
1576         This patch also changes the "open a comment box here" action to
1577         double-click to avoid issues with mis-clicks.
1578
1579         * PrettyPatch/PrettyPatch.rb:
1580         * code-review.js:
1581
1582 2010-08-29  Adam Barth  <abarth@webkit.org>
1583
1584         Attempt to make Sam's life easier by not opening a comment text field
1585         if there's a selection.  This should make it easier to copy/paste text
1586         out of the diff without accidentally opening comment boxes.
1587
1588         * code-review.js:
1589
1590 2010-08-29  Adam Barth  <abarth@webkit.org>
1591
1592         Turns out we're supposed to use find instead of children here.
1593
1594         * code-review.js:
1595
1596 2010-08-29  Adam Barth  <abarth@webkit.org>
1597
1598         Reviewed by Eric Seidel.
1599
1600         Awesomify pretty-diff based review tool
1601         https://bugs.webkit.org/show_bug.cgi?id=44835
1602
1603         This patch replaces the existing inline comment feature with a new
1604         implementation.  Hopefully the new implementation is more awesome.
1605         It's not 100% done yet, but I'd like to get this version out there to
1606         see how it works.
1607
1608         Unlike the previous version this version works with the "Formatted
1609         Diff" link instead of the "Review Patch" link.  Hopefully that will
1610         avoid interfering with folks who like the old style review.
1611
1612         * PrettyPatch/PrettyPatch.rb:
1613         * code-review.js: Added.
1614         * prototype.js: Removed.
1615
1616 2010-08-19  Tony Chang  <tony@chromium.org>
1617
1618         Reviewed by Adam Barth.
1619
1620         webkit-patch barfed on upload with a new image test result
1621         https://bugs.webkit.org/show_bug.cgi?id=39097
1622
1623         * PrettyPatch/PrettyPatch.rb: don't use full path to git
1624
1625 2010-08-19  Tony Chang  <tony@chromium.org>
1626
1627         Unreviewed, setting svn:eol-style native on ChangeLog.
1628
1629 2010-08-19  Tony Chang  <tony@chromium.org>
1630
1631         Unreviewed.  Just removing \r's from ChangeLog.
1632
1633 2010-08-02  Adam Roben  <aroben@apple.com>
1634
1635         Tell the patch prettifier about some (not so) new directories
1636
1637         * PrettyPatch/PrettyPatch.rb: Added WebKit2, autotools, and cmake
1638         directories.
1639
1640 2010-07-27  Joseph Pecoraro  <joepeck@webkit.org>
1641
1642         Reviewed by David Kilzer.
1643
1644         Provide a Helpful Link for Web Inspector Bugs in Bugzilla
1645         https://bugs.webkit.org/show_bug.cgi?id=43065
1646
1647         * template/en/custom/global/choose-product.html.tmpl: Added. Explicitly added a link for the Web Inspector.
1648
1649 2010-06-09  Julie Parent  <jparent@chromium.org>
1650
1651         Reviewed by David Kilzer.
1652
1653         Bugzilla: Don't send mail for cancel/deny/granted for in-rietveld.
1654
1655         https://bugs.webkit.org/show_bug.cgi?id=40345
1656
1657         * Bugzilla/Flag.pm:
1658
1659 2010-05-17  Julie Parent  <jparent@chromium.org>
1660
1661         Reviewed by Ojan Vafai.
1662
1663         Rietveld review page should dedicate more space to Rietveld, less to high level comments form.
1664
1665         https://bugs.webkit.org/show_bug.cgi?id=39244
1666
1667         * template/en/custom/attachment/reviewform.html.tmpl:
1668         Delete a random BR amongst the hidden elements that was causing extra whitespace.
1669         * template/en/custom/attachment/rietveldreview.html.tmpl:
1670         Increase size of Rietveld frame to 80%, from 60%.
1671
1672 2010-05-14  Julie Parent  <jparent@chromium.org>
1673
1674         Reviewed by Ojan Vafai.
1675
1676         Bugzilla: Only show "Rietveld Review" link when in-rietveld is set to +.
1677
1678         https://bugs.webkit.org/show_bug.cgi?id=39141
1679
1680         * template/en/custom/attachment/list.html.tmpl:
1681
1682 2010-05-13  Julie Parent  <jparent@chromium.org>
1683
1684         Reviewed by David Kilzer.
1685
1686         Bugzilla: Don't send mail if the only change is to the in-rietveld flag.
1687
1688         https://bugs.webkit.org/show_bug.cgi?id=39090
1689
1690         * Bugzilla/BugMail.pm:
1691
1692 2010-05-12  Julie Parent  <jparent@chromium.org>
1693
1694         Reviewed by Ojan Vafai.
1695
1696         Hide the in-rietveld flag in the UI.  It is only used only by tooling and shouldn't clutter the UI.
1697         https://bugs.webkit.org/show_bug.cgi?id=39024
1698
1699         * template/en/custom/attachment/list.html.tmpl:
1700           Removes this flag entirely from the UI, not needed on this page.
1701         * template/en/custom/flag/list.html.tmpl: Added.
1702           Copied from template/en/default/flag/list.html.tmpl with only changes inside the if WEBKIT_CHANGES section.
1703           Hides the flag rather than removing it completely, so tools can still interact with it.
1704
1705 2010-05-12  Julie Parent  <jparent@chromium.org>
1706
1707         Reviewed by David Kilzer.
1708
1709         Adds support for Rietveld reviews.  When there is an associated
1710         Rietveld id for a patch, adds a link to do a Rietveld review instead.
1711
1712         https://bugs.webkit.org/show_bug.cgi?id=38143
1713
1714         * attachment.cgi:
1715             Added support for handling action=rietveldreview.
1716         * template/en/custom/attachment/list.html.tmpl:
1717             Adds a link to use rietveld review iff the in_rietveld flag is set.
1718         * template/en/custom/attachment/reviewform.html.tmpl:
1719             Detects if we are in rietveld review mode due to query string paramter,
1720             and customizes the review form slightly:
1721                 - Changes text describing comment field.
1722                 - Comment field does not include quoted patch.
1723                 - Submit button fetches from Rietveld instead of just submitting.
1724         * template/en/custom/attachment/rietveldreview.html.tmpl: Added.
1725             New template for rietveldreview mode.  Hosts the rietveld frame in the
1726             top and the regular comments form in the bottom.
1727             Uses postMessage to communicate with Rietveld frame.
1728
1729 2010-05-08  Chris Jerdonek  <cjerdonek@webkit.org>
1730
1731         Reviewed by Maciej Stachowiak.
1732
1733         Allow long comment lines to wrap to the window width instead
1734         of to 80 characters.
1735
1736         https://bugs.webkit.org/show_bug.cgi?id=37792
1737
1738         * Bugzilla/Constants.pm:
1739           - Introduced a new constant COMMENT_COLS_WRAP to replace COMMENT_COLS
1740             inside the wrap_comment() subroutine.  The new constant effectively
1741             disables text wrapping by increasing the value used in wrap_comment()
1742             from 80 characters to a very large value (8000).
1743         * Bugzilla/Util.pm:
1744           - Replaced COMMENT_COLS with COMMENT_COLS_WRAP inside wrap_comment().
1745         * skins/custom/global.css:
1746         (.bz_comment_text):
1747           - Eliminate the width styling on comments to allow long lines
1748             to wrap at the window width.
1749         (.bz_comment_text > span.quote):
1750           - Inherit the "white-space" property.  This prevents long quoted
1751             lines in comments from causing non-quoted lines to extend beyond
1752             the window width.
1753
1754 2010-04-22  Adam Barth  <abarth@webkit.org>
1755
1756         Reviewed by Darin Adler.
1757
1758         The new review tools shouldn't clear the comment box automatically
1759         https://bugs.webkit.org/show_bug.cgi?id=38004
1760
1761         Instead, we now have a button for doing that explicitly.  If people
1762         like this tool, we can look at fancier solutions.
1763
1764         Also, tried to make the tool work on the edit page as well as the review page.
1765
1766         * PrettyPatch/PrettyPatch.rb:
1767
1768 2010-04-22  Adam Barth  <abarth@webkit.org>
1769
1770         Unreviewed.  Turns out we need to do more escaping because Ruby was
1771         explanding our newlines, creating a syntax error.
1772
1773         * PrettyPatch/PrettyPatch.rb:
1774
1775 2010-04-22  Adam Barth  <abarth@webkit.org>
1776
1777         Reviewed by Darin Adler.
1778
1779         Hack up PrettyDiff to allow line-by-line comments
1780         https://bugs.webkit.org/show_bug.cgi?id=37886
1781
1782         Admittedly a bit of a hack, this is a basic line-by-line editor we can
1783         play with thanks to Andrew Scherkus.  It's meant to integrate with the
1784         "review" page at bugs.webkit.org.
1785
1786         I changed a few things from Andrew's original version:
1787         1) Trigger text boxes on single click to make iPhone/iPad reviewing
1788         easier.
1789         2) Clear the main text box on load.
1790         3) Reference a version of prototype.js on bugs.webkit.org.
1791
1792         * PrettyPatch/PrettyPatch.rb:
1793         * prototype.js: Added.
1794
1795 2010-02-22  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
1796
1797         Reviewed by Maciej Stachowiak.
1798
1799         Don't reset the assignee back to default on component change
1800
1801         https://bugs.webkit.org/show_bug.cgi?id=35236
1802
1803         There's a risk of reseting a valid assignee for example when
1804         triaging a bug. All components in WebKit have the default
1805         assignee set to webkit-unassigned@webkit.org so this should
1806         not cause problems for people relying on this functionality.
1807
1808         * template/en/custom/bug/edit.html.tmpl:
1809
1810 2010-01-22  Janusz Lewandowski  <lew21@xtreeme.org>
1811
1812         Reviewed by Maciej Stachowiak.
1813
1814         run-webkit-tests fails on Ruby 1.9
1815         https://bugs.webkit.org/show_bug.cgi?id=33554
1816         inject does not work with strings any more, so BugsSite/PrettyPatch/PrettyPatch.rb prints lots of errors.
1817
1818         * PrettyPatch/PrettyPatch.rb:
1819
1820 2010-01-20  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
1821
1822         Reviewed by Adam Barth.
1823
1824         Adjust height of status-bubbles to prevent them being cut.
1825
1826         * template/en/custom/attachment/edit.html.tmpl:
1827         * template/en/custom/attachment/list.html.tmpl:
1828         * template/en/custom/attachment/reviewform.html.tmpl:
1829
1830 2010-01-08  David Kilzer  <ddkilzer@apple.com>
1831
1832         <http://webkit.org/b/33410> Fix "View Plain Diff"/"View Formatted Diff" button
1833
1834         Reviewed by Adam Barth.
1835
1836         This button was broken during a previous Bugzilla upgrade after
1837         changes were made to the names of objects exposed on templates.
1838
1839         * template/en/custom/attachment/reviewform.html.tmpl: Changed
1840         attachid to attachment.id and ispatch to attachment.ispatch.
1841         Also tweaked the format to use 4 table columns instead of 5.
1842
1843 2010-01-08  David Kilzer  <ddkilzer@apple.com>
1844
1845         Included UNCONFIRMED bugs by default in advanced search query
1846
1847         Rubber-stamped by Alexey Proskuryakov.
1848
1849         * data/params: Prepended 'bug_status=UNCONFIRMED&' to
1850         'defaultquery' parameter.
1851
1852 2010-01-08  David Kilzer  <ddkilzer@apple.com>
1853
1854         FIX: Make patch status appear on reviewform.html.tmpl
1855
1856         * template/en/custom/attachment/reviewform.html.tmpl: Check
1857         attachment.ispatch (not just ispatch) when deciding whether
1858         to display patch status.  Also update layout a bit.
1859
1860 2010-01-08  David Kilzer  <ddkilzer@apple.com>
1861
1862         Update data/params parameters
1863
1864         * data/params: Updated 'timezone' to turn off DST.  Updated
1865         'usebugaliases' and 'usevotes' to turn them off.
1866
1867 2010-01-08  Adam Barth  <abarth@webkit.org>
1868
1869         Reviewed by Darin Adler.
1870
1871         Show patch status in bugs.webkit.org
1872         https://bugs.webkit.org/show_bug.cgi?id=33368
1873
1874         This patch adds an iframe to the show_bug page and the edit attachment
1875         page that displays whether the attachment passed the various bots.
1876
1877         * template/en/custom/attachment/edit.html.tmpl:
1878         * template/en/custom/attachment/list.html.tmpl:
1879         * template/en/custom/attachment/reviewform.html.tmpl:
1880
1881 2009-12-07  Shinichiro Hamaji  <hamaji@chromium.org>
1882
1883         Reviewed by David Kilzer.
1884
1885         Bugzilla should show images in git patches
1886         https://bugs.webkit.org/show_bug.cgi?id=31395
1887
1888         Attempt to go r51748 again using --directory option of git-apply.
1889
1890         * PrettyPatch/PrettyPatch.rb:
1891
1892 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
1893
1894         Unreviewed. Revert r51748.
1895
1896         Bugzilla should show images in git patches
1897         https://bugs.webkit.org/show_bug.cgi?id=31395
1898
1899         * PrettyPatch/PrettyPatch.rb:
1900
1901 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
1902
1903         Reviewed by David Kilzer.
1904
1905         Bugzilla should show images in git patches
1906         https://bugs.webkit.org/show_bug.cgi?id=31395
1907
1908         Show images in git patches using git-apply.
1909
1910         * PrettyPatch/PrettyPatch.rb:
1911
1912 2009-10-23  Eric Seidel  <eric@webkit.org>
1913
1914         Reviewed by Adam Roben.
1915
1916         PrettyPatch should show images even when they have the wrong mime type
1917         https://bugs.webkit.org/show_bug.cgi?id=29506
1918
1919         * PrettyPatch/PrettyPatch.rb:
1920
1921 2009-10-17  Alexey Proskuryakov  <ap@apple.com>
1922
1923         Reviewed by Eric Carlson.
1924
1925         https://bugs.webkit.org/show_bug.cgi?id=30470
1926         Make marking a bug as duplicate easier
1927
1928         * js/field.js: (setResolutionToDuplicate): Focus the edit field, too.
1929
1930 2009-10-16  Alexey Proskuryakov  <ap@apple.com>
1931
1932         Reviewed by Mark Rowe.
1933
1934         https://bugs.webkit.org/show_bug.cgi?id=30470
1935         Make marking a bug as duplicate easier
1936
1937         * js/field.js: (setResolutionToDuplicate): Show edit field right away, don't make the user
1938         click again.
1939
1940 2009-09-09  David Kilzer  <ddkilzer@apple.com>
1941
1942         Update data/params parameters
1943
1944         * data/params: Updated 'attachment_base' parameter for secure
1945         attachments.  Updated 'mybugstemplate' parameter to search for
1946         bugs in UNCONFIRMED, ASSIGNED and REOPENED status in addition to
1947         NEW status.
1948
1949 2009-08-11  David Kilzer  <ddkilzer@apple.com>
1950
1951         Updated data/params parameter
1952
1953         * data/params: Updated 'loginnetmask' parameter to allow users
1954         to choose whether their cookies are restricted to a single IP
1955         address or not.
1956
1957 2009-07-07  David Kilzer  <ddkilzer@apple.com>
1958
1959         Updated BugsSite with local changes on server
1960
1961         The new files were created by checksetup.pl during the upgrade
1962         to Bugzilla 3.2.3.  The rest are self-explanatory.
1963
1964         * data: Added "bugzilla-update.xml" to svn:ignore.
1965         * data/attachments: Added "groups.*" to svn:ignore.
1966         * data/params: Updated for a few changed parameters.
1967
1968         * lib/.htaccess: Added.
1969         * skins/contrib/Dusk/IE-fixes.css: Added.
1970         * skins/contrib/Dusk/admin.css: Added.
1971         * skins/contrib/Dusk/create_attachment.css: Added.
1972         * skins/contrib/Dusk/dependency-tree.css: Added.
1973         * skins/contrib/Dusk/duplicates.css: Added.
1974         * skins/contrib/Dusk/editusers.css: Added.
1975         * skins/contrib/Dusk/help.css: Added.
1976         * skins/contrib/Dusk/index.css: Added.
1977         * skins/contrib/Dusk/panel.css: Added.
1978         * skins/contrib/Dusk/params.css: Added.
1979         * skins/contrib/Dusk/release-notes.css: Added.
1980         * skins/contrib/Dusk/show_bug.css: Added.
1981         * skins/contrib/Dusk/show_multiple.css: Added.
1982         * skins/contrib/Dusk/summarize-time.css: Added.
1983         * skins/contrib/Dusk/voting.css: Added.
1984         * skins/contrib/Dusk/yui: Added.
1985         * skins/contrib/Dusk/yui/calendar.css: Added.
1986         * skins/custom/IE-fixes.css: Added.
1987         * skins/custom/admin.css: Added.
1988         * skins/custom/buglist.css: Added.
1989         * skins/custom/create_attachment.css: Added.
1990         * skins/custom/dependency-tree.css: Added.
1991         * skins/custom/duplicates.css: Added.
1992         * skins/custom/editusers.css: Added.
1993         * skins/custom/help.css: Added.
1994         * skins/custom/panel.css: Added.
1995         * skins/custom/params.css: Added.
1996         * skins/custom/release-notes.css: Added.
1997         * skins/custom/show_bug.css: Added.
1998         * skins/custom/show_multiple.css: Added.
1999         * skins/custom/summarize-time.css: Added.
2000         * skins/custom/voting.css: Added.
2001         * skins/custom/yui: Added.
2002         * skins/custom/yui/calendar.css: Added.
2003
2004 2009-07-03  David Kilzer  <ddkilzer@apple.com>
2005
2006         Bug 26958: Change edit link to review link in request messages
2007
2008         <https://bugs.webkit.org/show_bug.cgi?id=26958>
2009
2010         Reviewed by Dan Bernstein.
2011
2012         * template/en/custom/request/email.txt.tmpl: Copied from BugsSite/template/en/default/request/email.txt.tmpl.
2013         Changed link from "action=edit" to "action=review".
2014
2015 2009-07-03  David Kilzer  <ddkilzer@apple.com>
2016
2017         Bug 26950: Make the summary and alias fields support click-to-edit
2018         <https://bugs.webkit.org/show_bug.cgi?id=26950>
2019
2020         Reviewed by Maciej Stachowiak.
2021
2022         Original patch by Maciej Stachowiak.
2023
2024         * js/field.js:
2025         (hideEditableField): Updated to add click event listeners to the
2026         alias and short description elements to make them easier to
2027         edit.  Renamed field_id parameter to field2_id (short
2028         description id) and added a field1_id parameter (alias id).
2029         (showEditableField): If a third parameter is passed in the
2030         ContainerInputArray parameter, use it to find the element to
2031         focus.  Otherwise, fall back to the original behavior of
2032         focusing the first input field.
2033         (hideAliasAndSummary): Changed to pass the id for the alias
2034         element to hideEditableField().
2035
2036 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2037
2038         Config file for contrib/recode.pl when upgrading bugs.webkit.org
2039
2040         Current as of midday on Wednesday, July 1, 2009.
2041
2042         * contrib/recode-overrides.txt: Added.
2043
2044 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2045
2046         Workaround for WebKit Bug 9630 when running contrib/recode.pl
2047
2048         A number of WebKit nightly builds included Bug 9630 which caused
2049         non-breaking space characters (0xA0) to be submitted within the
2050         content of textarea elements.
2051
2052         * contrib/recode.pl: To work around these 0xA0 characters, try
2053         an encoding of cp1252 and use it if it works.
2054
2055 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2056
2057         Don't print "Failed to guess" warning if an override is available
2058
2059         * contrib/recode.pl: Check that %overrides does NOT contain a
2060         key matching the current digest before printing out the warning
2061         about a failed charset guess.
2062
2063 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2064
2065         Added --[no-]truncate switch to contrib/recode.pl
2066
2067         * contrib/recode.pl: Added --[no-]truncate switch to make
2068         debugging of failed charset guessing easier.  Often times the
2069         illegal character was truncated in the output.
2070
2071 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2072
2073         Changed Perl scripts to use #!/usr/bin/env perl
2074
2075         perl -pi -e 's@#\!/usr/bin/perl@#\!/usr/bin/env perl@' *.cgi *.pl */*.pl
2076
2077 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2078
2079         Replace Bugzilla favicon with webkit.org favicon
2080
2081         * favicon.ico: Removed.
2082         * images/favicon.ico: Replaced.  Copied from favicon.ico.
2083
2084 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
2085
2086         Merged BugsSite to Bugzilla-3.2.3
2087
2088         Updated to the latest-and-greatest stable version.
2089
2090 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
2091
2092         Merged BugsSite to Bugzilla-3.0.3
2093
2094         Nothing to see here.  Move along.
2095
2096 2009-06-09  Eric Seidel  <eric@webkit.org>
2097
2098         Reviewed by Adam Roben.
2099
2100         Add support for displaying added png files in PrettyPatch diffs
2101         https://bugs.webkit.org/show_bug.cgi?id=26210
2102
2103         Currently this is SVN only (git-send-bugzilla patches exclude binary data)
2104         and only works for PNG files but could easily be made to work for other images as needed.
2105
2106         * PrettyPatch/PrettyPatch.rb:
2107
2108 2009-05-19  Eric Seidel  <eric@webkit.org>
2109
2110         Reviewed by Adam Roben.
2111
2112         Make PrettyPatch understand quoted filenames in git diffs.
2113         https://bugs.webkit.org/show_bug.cgi?id=25881
2114
2115         * PrettyPatch/PrettyPatch.rb:
2116
2117 2009-05-15  Simon Fraser  <simon.fraser@apple.com>
2118
2119         Carrying forwards Darin Adler's rubber stamp
2120
2121         Make the comment field wider too.
2122
2123         * skins/custom/global.css:
2124
2125 2009-05-14  Simon Fraser  <simon.fraser@apple.com>
2126
2127         Reviewed by Darin Adler
2128
2129         Make the URL, Summary and Keyword fields wider to mitigate the
2130         effects of https://bugs.webkit.org/show_bug.cgi?id=25566 and make
2131         things generally better.
2132
2133         * skins/custom/global.css:
2134         * template/en/custom/bug/edit.html.tmpl:
2135
2136 2008-11-10  Darin Adler  <darin@apple.com>
2137
2138         - fix more of https://bugs.webkit.org/show_bug.cgi?id=21400
2139           "Edit" links for patches in comments, review queue, review emails
2140           should be replaced by "Review Patch" links
2141
2142         * globals.pl: Fix cases that are automatically generated, such as links
2143         in comments in bugs.
2144
2145         * template/en/custom/attachment/created.html.tmpl: Fix the link on the
2146         "I just created an attachment" page.
2147
2148         * template/en/custom/request/queue.html.tmpl: Fix the link in the queue
2149         page.
2150
2151 2008-11-10  Darin Adler  <darin@apple.com>
2152
2153         * template/en/default/request/queue.html.tmpl: Use review links instead
2154         of edit links in the review queue.
2155
2156 2008-11-10  Darin Adler  <darin@apple.com>
2157
2158         * template/en/custom/request/email.txt.tmpl: Send review links instead
2159         of edit links when flags are set on a bug.
2160
2161 2008-10-27  Darin Adler  <darin@apple.com>
2162
2163         * template/en/custom/attachment/reviewform.html.tmpl: Leave out the
2164         comment if it's untouched.
2165
2166 2008-10-23  Adam Roben  <aroben@apple.com>
2167
2168         Fix Bug 21401: Comments field on "Review Patch" page should be
2169         initially filled with quoted patch
2170
2171         <https://bugs.webkit.org/show_bug.cgi?id=21401>
2172
2173         Reviewed by Dave Hyatt.
2174
2175         * attachment.cgi:
2176         (edit): Retrieve the attachment data from the database instead of just
2177         its length. When the attachment is a patch, create a quotedpatch
2178         template variable that contains the patch data with each line
2179         prepended with "> ".
2180         * template/en/custom/attachment/reviewform.html.tmpl:
2181           - Changed the comments field to have a monospace font
2182           - Added an "Enter comments below:" caption above the comments field
2183             to make it clearer that this is where comments should go, now that
2184             the comments field is not initially empty
2185           - Fill the comments field with the quoted patch
2186
2187 2008-10-14  Adam Roben  <aroben@apple.com>
2188
2189         Fix Bug 21602: Bugzilla times out trying to display formatted diff for
2190         attachment 24345
2191
2192         https://bugs.webkit.org/show_bug.cgi?id=21602
2193
2194         Reviewed by Dave Kilzer.
2195
2196         * PrettyPatch/PrettyPatch.rb:
2197         (PrettyPatch.BINARY_FILE_MARKER_FORMAT): Added.
2198         (PrettyPatch.FileDiff.initialize): If any of the lines in the diff
2199         are a binary file marker, mark this FileDiff as binary and stop trying
2200         to process the lines.
2201         (PrettyDiff.FileDiff.to_html): If we're binary, just print a string
2202         saying so.
2203
2204 2008-10-02  Adam Roben  <aroben@apple.com>
2205
2206         * template/en/custom/attachment/reviewform.html.tmpl: Fix typo.
2207
2208 2008-10-02  Adam Roben  <aroben@apple.com>
2209
2210         * template/en/custom/attachment/review.html.tmpl: Fix URLs
2211
2212 2008-10-02  Adam Roben  <aroben@apple.com>
2213
2214         Fix Bug 21315: bugs.webkit.org should have a nicer patch review
2215         interface
2216
2217         https://bugs.webkit.org/show_bug.cgi?id=21315
2218
2219         Reviewed by Tim Hatcher.
2220
2221         * attachment.cgi:
2222         (top level): Added support for the "review" and "reviewform" actions.
2223         (sub edit): Accept the template name as a parameter. If no name is
2224         give, use "edit" as before.
2225         * template/en/custom/attachment/list.html.tmpl: Added a "Review Patch"
2226         link for all patch attachments.
2227         * template/en/custom/attachment/review.html.tmpl: Added. Simple
2228         <frameset> page to show the patch on the top and a comment form on the
2229         bottom.
2230         * template/en/custom/attachment/reviewform.html.tmpl: Added. Simple
2231         comment form for reviewing patches.
2232
2233 2008-10-02  Adam Roben  <aroben@apple.com>
2234
2235         Remove references to some backup files I created when implementing
2236         PrettyPatch in Bugzilla
2237
2238         Reviewed by Tim Hatcher.
2239
2240         * attachment-aroben.cgi: Removed.
2241         * template/en/default/attachment/edit-aroben.html.tmpl: Removed.
2242
2243         * template/en/custom/attachment/edit.html.tmpl: Removed an erroneous
2244         reference to attachment-aroben.cgi. attachment.cgi will work just
2245         fine.
2246
2247 2008-06-25  David Kilzer  <ddkilzer@apple.com>
2248
2249         Make PrettyPatch handle diffs with no Index or diff headers
2250
2251         Part of Bug 19290: More patches not handled by PrettyPatch.rb
2252         <https://bugs.webkit.org/show_bug.cgi?id=19290>
2253
2254         Reviewed by Adam.
2255
2256         * PrettyPatch/PrettyPatch.rb:
2257         (PrettyPatch.DIFF_HEADER_FORMATS): Added regular expression to
2258         match on "+++ " lines for patches with no "Index" or "diff" header.
2259         (PrettyPatch.FileDiff.initialize): Look for filename on "+++ " line
2260         when the first line of a patch has no "Index" or "diff" header.
2261         (PrettyPatch.FileDiff.parse): Added haveSeenDiffHeader state
2262         variable to determine when no "Index" or "diff" header has been
2263         found, but a new patch has started with a "--- " line.
2264
2265 2008-06-08  Dan Bernstein  <mitz@apple.com>
2266
2267         Reviewed by Adam Roben.
2268
2269         - update trac URL in PrettyPatch
2270
2271         * PrettyPatch/PrettyPatch.rb:
2272
2273 2008-05-29  Adam Roben  <aroben@apple.com>
2274
2275         Update PrettyPatch directory list
2276
2277         * PrettyPatch/PrettyPatch.rb:
2278
2279 2008-05-28  Adam Roben  <aroben@apple.com>
2280
2281         Make PrettyPatch able to handle diffs taken from outside the WebKit
2282         source tree
2283
2284         Part of Bug 19290: More patches not handled by PrettyPatch.rb
2285         <https://bugs.webkit.org/show_bug.cgi?id=19290>
2286
2287         Reviewed by David Kilzer.
2288
2289         * PrettyPatch/PrettyPatch.rb:
2290         (PrettyPatch.find_url_and_path): Added. Searches the file path from
2291         the bottom up looking for a directory that exists in the source tree.
2292         (PrettyPatch.linkifyFilename): Changed to call find_url_and_path.
2293
2294 2008-05-28  Adam Roben  <aroben@apple.com>
2295
2296         Print exceptions from PrettyPatch to stdout
2297
2298         This will make it much easier to debug problems with PrettyPatch,
2299         since we'll be able to see the exceptions in the browser.
2300
2301         Reviewed by Sam Weinig.
2302
2303         * PrettyPatch/prettify.rb: Added a --html-exceptions option, which
2304         will print exceptions to stdout as HTML.
2305         * attachment.cgi: Changed to pass --html-exceptions to prettify.rb.
2306
2307 2008-04-26  David Kilzer  <ddkilzer@apple.com>
2308
2309         Removed temp files and added appropriate svn:ignore properties.
2310
2311         Rubber-stamped by Mark Rowe.
2312
2313         * data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
2314
2315         * data/template: Added svn:ignore property for "template" subdirectory.
2316         * data/template/template: Removed precompiled template subdirectory.
2317
2318         * data/versioncache: Removed.
2319         * data/versioncache.*: Removed backup versioncache files.
2320
2321         * data/webdot: Added svn:ignore property for "*.dot" files.
2322         * data/webdot/*.dot: Removed cached webdot files.
2323
2324 2008-04-08  Adam Roben  <aroben@apple.com>
2325
2326         Combine :equal operations with the following operation if they are
2327         fewer than 3 characters long
2328
2329         This keeps us from showing lots of small changes on long lines, just
2330         because some letters happened to be the same.
2331
2332         Inspired by
2333         http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/diffviewer/diffutils.py?r=1264#147
2334
2335         * PrettyPatch/PrettyPatch.rb:
2336         (PrettyPatch::DiffSection.initialize): Remove :equal operations fewer
2337         than 3 characters long. The characters from the removed operations
2338         become part of the subsequent operation.
2339         (PrettyPatch::CodeLine.text_as_html): Don't wrap 0-length strings in
2340         <ins>/<del> tags. Also removed the @fromLineNumber.nil? and
2341         @toLineNumber.nil? checks as they are no longer reliable now that
2342         we're removing operations.
2343
2344 2008-04-08  Adam Roben  <aroben@apple.com>
2345
2346         Change PrettyPatch to use DiffBuilder for intra-line diffs
2347
2348         This gives us much prettier intra-line diffs, largely because it can
2349         distinguish multiple changes on the same line. e.g., if a line changes
2350         from:
2351
2352         const int myConstant = 0;
2353
2354         to
2355
2356         static const unsigned myConstant;
2357
2358         You will see that "static " was inserted, "int" changed to "unsigned",
2359         and " = 0" was deleted.
2360
2361         This seems to have also gotten rid of some spurious instances of
2362         "<ins></ins>" and "<del></del>" at the end of a line.
2363
2364         * PrettyPatch/PrettyPatch.rb:
2365         (PrettyPatch::DiffSection.initialize): Use DiffBuilder instead of
2366         setChangeExtentFromLine (which had been copied from Trac's diffing
2367         code).
2368         (PrettyPatch::CodeLine):
2369          - Removed the changeExtent attribute
2370          - Added the operations attribute
2371          - Removed the setChangeExtentFromLine method
2372         (PrettyPatch::CodeLine.text_as_html): Uses @operations to determine
2373         what text to wrap in <ins> and <del> tags.
2374         * PrettyPatch/diff.rb: Deleted a bunch of code we don't use.
2375         (HTMLDiff::DiffBuilder.initialize): Moved code here from the old build
2376         method, but left out the calls to perform_operation, since we build
2377         the diff HTML outside of this class.
2378         (HTMLDiff::DiffBuilder.split_inputs_to_words): Removed calls to
2379         convert_html_to_list_of_words, since we're not diffing HTML.
2380
2381 2008-04-08  Adam Roben  <aroben@apple.com>
2382
2383         Check in diff.rb
2384
2385         This came from
2386         http://instiki.rubyforge.org/svn/instiki/trunk/lib/diff.rb
2387         revision 28521
2388
2389         * PrettyPatch/diff.rb: Added.
2390
2391 2008-03-04  Adam Roben  <aroben@apple.com>
2392
2393         * PrettyPatch/PrettyPatch.rb: Allow the extent of the previous file to
2394         be omitted from the hunk header.
2395