Websites/bugs.webkit.org/ChangeLog

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