Websites/bugs.webkit.org/ChangeLog

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