Websites/bugs.webkit.org/ChangeLog

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