Websites/bugs.webkit.org/ChangeLog

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