Websites/bugs.webkit.org/ChangeLog

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