Websites/bugs.webkit.org/ChangeLog

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