Websites/bugs.webkit.org/ChangeLog

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