BugsSite/ChangeLog

   1 2010-05-13  Julie Parent  <jparent@chromium.org>
   2
   3         Reviewed by David Kilzer.
   4
   5         Bugzilla: Don't send mail if the only change is to the in-rietveld flag.
   6
   7         https://bugs.webkit.org/show_bug.cgi?id=39090
   8
   9         * Bugzilla/BugMail.pm:
  10
  11 2010-05-12  Julie Parent  <jparent@chromium.org>
  12
  13         Reviewed by Ojan Vafai.
  14
  15         Hide the in-rietveld flag in the UI.  It is only used only by tooling and shouldn't clutter the UI.
  16         https://bugs.webkit.org/show_bug.cgi?id=39024
  17
  18         * template/en/custom/attachment/list.html.tmpl:
  19           Removes this flag entirely from the UI, not needed on this page.
  20         * template/en/custom/flag/list.html.tmpl: Added.
  21           Copied from template/en/default/flag/list.html.tmpl with only changes inside the if WEBKIT_CHANGES section.
  22           Hides the flag rather than removing it completely, so tools can still interact with it.
  23
  24 2010-05-12  Julie Parent  <jparent@chromium.org>
  25
  26         Reviewed by David Kilzer.
  27
  28         Adds support for Rietveld reviews.  When there is an associated
  29         Rietveld id for a patch, adds a link to do a Rietveld review instead.
  30
  31         https://bugs.webkit.org/show_bug.cgi?id=38143
  32
  33         * attachment.cgi:
  34             Added support for handling action=rietveldreview.
  35         * template/en/custom/attachment/list.html.tmpl:
  36             Adds a link to use rietveld review iff the in_rietveld flag is set.
  37         * template/en/custom/attachment/reviewform.html.tmpl:
  38             Detects if we are in rietveld review mode due to query string paramter,
  39             and customizes the review form slightly:
  40                 - Changes text describing comment field.
  41                 - Comment field does not include quoted patch.
  42                 - Submit button fetches from Rietveld instead of just submitting.
  43         * template/en/custom/attachment/rietveldreview.html.tmpl: Added.
  44             New template for rietveldreview mode.  Hosts the rietveld frame in the
  45             top and the regular comments form in the bottom.
  46             Uses postMessage to communicate with Rietveld frame.
  47
  48 2010-05-08  Chris Jerdonek  <cjerdonek@webkit.org>
  49
  50         Reviewed by Maciej Stachowiak.
  51
  52         Allow long comment lines to wrap to the window width instead
  53         of to 80 characters.
  54
  55         https://bugs.webkit.org/show_bug.cgi?id=37792
  56
  57         * Bugzilla/Constants.pm:
  58           - Introduced a new constant COMMENT_COLS_WRAP to replace COMMENT_COLS
  59             inside the wrap_comment() subroutine.  The new constant effectively
  60             disables text wrapping by increasing the value used in wrap_comment()
  61             from 80 characters to a very large value (8000).
  62         * Bugzilla/Util.pm:
  63           - Replaced COMMENT_COLS with COMMENT_COLS_WRAP inside wrap_comment().
  64         * skins/custom/global.css:
  65         (.bz_comment_text):
  66           - Eliminate the width styling on comments to allow long lines
  67             to wrap at the window width.
  68         (.bz_comment_text > span.quote):
  69           - Inherit the "white-space" property.  This prevents long quoted
  70             lines in comments from causing non-quoted lines to extend beyond
  71             the window width.
  72
  73 2010-04-22  Adam Barth  <abarth@webkit.org>
  74
  75         Reviewed by Darin Adler.
  76
  77         The new review tools shouldn't clear the comment box automatically
  78         https://bugs.webkit.org/show_bug.cgi?id=38004
  79
  80         Instead, we now have a button for doing that explicitly.  If people
  81         like this tool, we can look at fancier solutions.
  82
  83         Also, tried to make the tool work on the edit page as well as the review page.
  84
  85         * PrettyPatch/PrettyPatch.rb:
  86
  87 2010-04-22  Adam Barth  <abarth@webkit.org>
  88
  89         Unreviewed.  Turns out we need to do more escaping because Ruby was
  90         explanding our newlines, creating a syntax error.
  91
  92         * PrettyPatch/PrettyPatch.rb:
  93
  94 2010-04-22  Adam Barth  <abarth@webkit.org>
  95
  96         Reviewed by Darin Adler.
  97
  98         Hack up PrettyDiff to allow line-by-line comments
  99         https://bugs.webkit.org/show_bug.cgi?id=37886
 100
 101         Admittedly a bit of a hack, this is a basic line-by-line editor we can
 102         play with thanks to Andrew Scherkus.  It's meant to integrate with the
 103         "review" page at bugs.webkit.org.
 104
 105         I changed a few things from Andrew's original version:
 106         1) Trigger text boxes on single click to make iPhone/iPad reviewing
 107         easier.
 108         2) Clear the main text box on load.
 109         3) Reference a version of prototype.js on bugs.webkit.org.
 110
 111         * PrettyPatch/PrettyPatch.rb:
 112         * prototype.js: Added.
 113
 114 2010-02-22  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
 115
 116         Reviewed by Maciej Stachowiak.
 117
 118         Don't reset the assignee back to default on component change
 119
 120         https://bugs.webkit.org/show_bug.cgi?id=35236
 121
 122         There's a risk of reseting a valid assignee for example when
 123         triaging a bug. All components in WebKit have the default
 124         assignee set to webkit-unassigned@webkit.org so this should
 125         not cause problems for people relying on this functionality.
 126
 127         * template/en/custom/bug/edit.html.tmpl:
 128
 129 2010-01-22  Janusz Lewandowski  <lew21@xtreeme.org>
 130
 131         Reviewed by Maciej Stachowiak.
 132
 133         run-webkit-tests fails on Ruby 1.9
 134         https://bugs.webkit.org/show_bug.cgi?id=33554
 135         inject does not work with strings any more, so BugsSite/PrettyPatch/PrettyPatch.rb prints lots of errors.
 136
 137         * PrettyPatch/PrettyPatch.rb:
 138
 139 2010-01-20  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
 140
 141         Reviewed by Adam Barth.
 142
 143         Adjust height of status-bubbles to prevent them being cut.
 144
 145         * template/en/custom/attachment/edit.html.tmpl:
 146         * template/en/custom/attachment/list.html.tmpl:
 147         * template/en/custom/attachment/reviewform.html.tmpl:
 148
 149 2010-01-08  David Kilzer  <ddkilzer@apple.com>
 150
 151         <http://webkit.org/b/33410> Fix "View Plain Diff"/"View Formatted Diff" button
 152
 153         Reviewed by Adam Barth.
 154
 155         This button was broken during a previous Bugzilla upgrade after
 156         changes were made to the names of objects exposed on templates.
 157
 158         * template/en/custom/attachment/reviewform.html.tmpl: Changed
 159         attachid to attachment.id and ispatch to attachment.ispatch.
 160         Also tweaked the format to use 4 table columns instead of 5.
 161
 162 2010-01-08  David Kilzer  <ddkilzer@apple.com>
 163
 164         Included UNCONFIRMED bugs by default in advanced search query
 165
 166         Rubber-stamped by Alexey Proskuryakov.
 167
 168         * data/params: Prepended 'bug_status=UNCONFIRMED&' to
 169         'defaultquery' parameter.
 170
 171 2010-01-08  David Kilzer  <ddkilzer@apple.com>
 172
 173         FIX: Make patch status appear on reviewform.html.tmpl
 174
 175         * template/en/custom/attachment/reviewform.html.tmpl: Check
 176         attachment.ispatch (not just ispatch) when deciding whether
 177         to display patch status.  Also update layout a bit.
 178
 179 2010-01-08  David Kilzer  <ddkilzer@apple.com>
 180
 181         Update data/params parameters
 182
 183         * data/params: Updated 'timezone' to turn off DST.  Updated
 184         'usebugaliases' and 'usevotes' to turn them off.
 185
 186 2010-01-08  Adam Barth  <abarth@webkit.org>
 187
 188         Reviewed by Darin Adler.
 189
 190         Show patch status in bugs.webkit.org
 191         https://bugs.webkit.org/show_bug.cgi?id=33368
 192
 193         This patch adds an iframe to the show_bug page and the edit attachment
 194         page that displays whether the attachment passed the various bots.
 195
 196         * template/en/custom/attachment/edit.html.tmpl:
 197         * template/en/custom/attachment/list.html.tmpl:
 198         * template/en/custom/attachment/reviewform.html.tmpl:
 199
 200 2009-12-07  Shinichiro Hamaji  <hamaji@chromium.org>
 201
 202         Reviewed by David Kilzer.
 203
 204         Bugzilla should show images in git patches
 205         https://bugs.webkit.org/show_bug.cgi?id=31395
 206
 207         Attempt to go r51748 again using --directory option of git-apply.
 208
 209         * PrettyPatch/PrettyPatch.rb:
 210
 211 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
 212
 213         Unreviewed. Revert r51748.
 214
 215         Bugzilla should show images in git patches
 216         https://bugs.webkit.org/show_bug.cgi?id=31395
 217
 218         * PrettyPatch/PrettyPatch.rb:
 219
 220 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
 221
 222         Reviewed by David Kilzer.
 223
 224         Bugzilla should show images in git patches
 225         https://bugs.webkit.org/show_bug.cgi?id=31395
 226
 227         Show images in git patches using git-apply.
 228
 229         * PrettyPatch/PrettyPatch.rb:
 230
 231 2009-10-23  Eric Seidel  <eric@webkit.org>
 232
 233         Reviewed by Adam Roben.
 234
 235         PrettyPatch should show images even when they have the wrong mime type
 236         https://bugs.webkit.org/show_bug.cgi?id=29506
 237
 238         * PrettyPatch/PrettyPatch.rb:
 239
 240 2009-10-17  Alexey Proskuryakov  <ap@apple.com>
 241
 242         Reviewed by Eric Carlson.
 243
 244         https://bugs.webkit.org/show_bug.cgi?id=30470
 245         Make marking a bug as duplicate easier
 246
 247         * js/field.js: (setResolutionToDuplicate): Focus the edit field, too.
 248
 249 2009-10-16  Alexey Proskuryakov  <ap@apple.com>
 250
 251         Reviewed by Mark Rowe.
 252
 253         https://bugs.webkit.org/show_bug.cgi?id=30470
 254         Make marking a bug as duplicate easier
 255
 256         * js/field.js: (setResolutionToDuplicate): Show edit field right away, don't make the user
 257         click again.
 258
 259 2009-09-09  David Kilzer  <ddkilzer@apple.com>
 260
 261         Update data/params parameters
 262
 263         * data/params: Updated 'attachment_base' parameter for secure
 264         attachments.  Updated 'mybugstemplate' parameter to search for
 265         bugs in UNCONFIRMED, ASSIGNED and REOPENED status in addition to
 266         NEW status.
 267
 268 2009-08-11  David Kilzer  <ddkilzer@apple.com>
 269
 270         Updated data/params parameter
 271
 272         * data/params: Updated 'loginnetmask' parameter to allow users
 273         to choose whether their cookies are restricted to a single IP
 274         address or not.
 275
 276 2009-07-07  David Kilzer  <ddkilzer@apple.com>
 277
 278         Updated BugsSite with local changes on server
 279
 280         The new files were created by checksetup.pl during the upgrade
 281         to Bugzilla 3.2.3.  The rest are self-explanatory.
 282
 283         * data: Added "bugzilla-update.xml" to svn:ignore.
 284         * data/attachments: Added "groups.*" to svn:ignore.
 285         * data/params: Updated for a few changed parameters.
 286
 287         * lib/.htaccess: Added.
 288         * skins/contrib/Dusk/IE-fixes.css: Added.
 289         * skins/contrib/Dusk/admin.css: Added.
 290         * skins/contrib/Dusk/create_attachment.css: Added.
 291         * skins/contrib/Dusk/dependency-tree.css: Added.
 292         * skins/contrib/Dusk/duplicates.css: Added.
 293         * skins/contrib/Dusk/editusers.css: Added.
 294         * skins/contrib/Dusk/help.css: Added.
 295         * skins/contrib/Dusk/index.css: Added.
 296         * skins/contrib/Dusk/panel.css: Added.
 297         * skins/contrib/Dusk/params.css: Added.
 298         * skins/contrib/Dusk/release-notes.css: Added.
 299         * skins/contrib/Dusk/show_bug.css: Added.
 300         * skins/contrib/Dusk/show_multiple.css: Added.
 301         * skins/contrib/Dusk/summarize-time.css: Added.
 302         * skins/contrib/Dusk/voting.css: Added.
 303         * skins/contrib/Dusk/yui: Added.
 304         * skins/contrib/Dusk/yui/calendar.css: Added.
 305         * skins/custom/IE-fixes.css: Added.
 306         * skins/custom/admin.css: Added.
 307         * skins/custom/buglist.css: Added.
 308         * skins/custom/create_attachment.css: Added.
 309         * skins/custom/dependency-tree.css: Added.
 310         * skins/custom/duplicates.css: Added.
 311         * skins/custom/editusers.css: Added.
 312         * skins/custom/help.css: Added.
 313         * skins/custom/panel.css: Added.
 314         * skins/custom/params.css: Added.
 315         * skins/custom/release-notes.css: Added.
 316         * skins/custom/show_bug.css: Added.
 317         * skins/custom/show_multiple.css: Added.
 318         * skins/custom/summarize-time.css: Added.
 319         * skins/custom/voting.css: Added.
 320         * skins/custom/yui: Added.
 321         * skins/custom/yui/calendar.css: Added.
 322
 323 2009-07-03  David Kilzer  <ddkilzer@apple.com>
 324
 325         Bug 26958: Change edit link to review link in request messages
 326
 327         <https://bugs.webkit.org/show_bug.cgi?id=26958>
 328
 329         Reviewed by Dan Bernstein.
 330
 331         * template/en/custom/request/email.txt.tmpl: Copied from BugsSite/template/en/default/request/email.txt.tmpl.
 332         Changed link from "action=edit" to "action=review".
 333
 334 2009-07-03  David Kilzer  <ddkilzer@apple.com>
 335
 336         Bug 26950: Make the summary and alias fields support click-to-edit
 337         <https://bugs.webkit.org/show_bug.cgi?id=26950>
 338
 339         Reviewed by Maciej Stachowiak.
 340
 341         Original patch by Maciej Stachowiak.
 342
 343         * js/field.js:
 344         (hideEditableField): Updated to add click event listeners to the
 345         alias and short description elements to make them easier to
 346         edit.  Renamed field_id parameter to field2_id (short
 347         description id) and added a field1_id parameter (alias id).
 348         (showEditableField): If a third parameter is passed in the
 349         ContainerInputArray parameter, use it to find the element to
 350         focus.  Otherwise, fall back to the original behavior of
 351         focusing the first input field.
 352         (hideAliasAndSummary): Changed to pass the id for the alias
 353         element to hideEditableField().
 354
 355 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
 356
 357         Config file for contrib/recode.pl when upgrading bugs.webkit.org
 358
 359         Current as of midday on Wednesday, July 1, 2009.
 360
 361         * contrib/recode-overrides.txt: Added.
 362
 363 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
 364
 365         Workaround for WebKit Bug 9630 when running contrib/recode.pl
 366
 367         A number of WebKit nightly builds included Bug 9630 which caused
 368         non-breaking space characters (0xA0) to be submitted within the
 369         content of textarea elements.
 370
 371         * contrib/recode.pl: To work around these 0xA0 characters, try
 372         an encoding of cp1252 and use it if it works.
 373
 374 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
 375
 376         Don't print "Failed to guess" warning if an override is available
 377
 378         * contrib/recode.pl: Check that %overrides does NOT contain a
 379         key matching the current digest before printing out the warning
 380         about a failed charset guess.
 381
 382 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
 383
 384         Added --[no-]truncate switch to contrib/recode.pl
 385
 386         * contrib/recode.pl: Added --[no-]truncate switch to make
 387         debugging of failed charset guessing easier.  Often times the
 388         illegal character was truncated in the output.
 389
 390 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
 391
 392         Changed Perl scripts to use #!/usr/bin/env perl
 393
 394         perl -pi -e 's@#\!/usr/bin/perl@#\!/usr/bin/env perl@' *.cgi *.pl */*.pl
 395
 396 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
 397
 398         Replace Bugzilla favicon with webkit.org favicon
 399
 400         * favicon.ico: Removed.
 401         * images/favicon.ico: Replaced.  Copied from favicon.ico.
 402
 403 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
 404
 405         Merged BugsSite to Bugzilla-3.2.3
 406
 407         Updated to the latest-and-greatest stable version.
 408
 409 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
 410
 411         Merged BugsSite to Bugzilla-3.0.3
 412
 413         Nothing to see here.  Move along.
 414
 415 2009-06-09  Eric Seidel  <eric@webkit.org>
 416
 417         Reviewed by Adam Roben.
 418
 419         Add support for displaying added png files in PrettyPatch diffs
 420         https://bugs.webkit.org/show_bug.cgi?id=26210
 421
 422         Currently this is SVN only (git-send-bugzilla patches exclude binary data)
 423         and only works for PNG files but could easily be made to work for other images as needed.
 424
 425         * PrettyPatch/PrettyPatch.rb:
 426
 427 2009-05-19  Eric Seidel  <eric@webkit.org>
 428
 429         Reviewed by Adam Roben.
 430
 431         Make PrettyPatch understand quoted filenames in git diffs.
 432         https://bugs.webkit.org/show_bug.cgi?id=25881
 433
 434         * PrettyPatch/PrettyPatch.rb:
 435
 436 2009-05-15  Simon Fraser  <simon.fraser@apple.com>
 437
 438         Carrying forwards Darin Adler's rubber stamp
 439
 440         Make the comment field wider too.
 441
 442         * skins/custom/global.css:
 443
 444 2009-05-14  Simon Fraser  <simon.fraser@apple.com>
 445
 446         Reviewed by Darin Adler
 447
 448         Make the URL, Summary and Keyword fields wider to mitigate the
 449         effects of https://bugs.webkit.org/show_bug.cgi?id=25566 and make
 450         things generally better.
 451
 452         * skins/custom/global.css:
 453         * template/en/custom/bug/edit.html.tmpl:
 454
 455 2008-11-10  Darin Adler  <darin@apple.com>
 456
 457         - fix more of https://bugs.webkit.org/show_bug.cgi?id=21400
 458           "Edit" links for patches in comments, review queue, review emails
 459           should be replaced by "Review Patch" links
 460
 461         * globals.pl: Fix cases that are automatically generated, such as links
 462         in comments in bugs.
 463
 464         * template/en/custom/attachment/created.html.tmpl: Fix the link on the
 465         "I just created an attachment" page.
 466
 467         * template/en/custom/request/queue.html.tmpl: Fix the link in the queue
 468         page.
 469
 470 2008-11-10  Darin Adler  <darin@apple.com>
 471
 472         * template/en/default/request/queue.html.tmpl: Use review links instead
 473         of edit links in the review queue.
 474
 475 2008-11-10  Darin Adler  <darin@apple.com>
 476
 477         * template/en/custom/request/email.txt.tmpl: Send review links instead
 478         of edit links when flags are set on a bug.
 479
 480 2008-10-27  Darin Adler  <darin@apple.com>
 481
 482         * template/en/custom/attachment/reviewform.html.tmpl: Leave out the
 483         comment if it's untouched.
 484
 485 2008-10-23  Adam Roben  <aroben@apple.com>
 486
 487         Fix Bug 21401: Comments field on "Review Patch" page should be
 488         initially filled with quoted patch
 489
 490         <https://bugs.webkit.org/show_bug.cgi?id=21401>
 491
 492         Reviewed by Dave Hyatt.
 493
 494         * attachment.cgi:
 495         (edit): Retrieve the attachment data from the database instead of just
 496         its length. When the attachment is a patch, create a quotedpatch
 497         template variable that contains the patch data with each line
 498         prepended with "> ".
 499         * template/en/custom/attachment/reviewform.html.tmpl:
 500           - Changed the comments field to have a monospace font
 501           - Added an "Enter comments below:" caption above the comments field
 502             to make it clearer that this is where comments should go, now that
 503             the comments field is not initially empty
 504           - Fill the comments field with the quoted patch
 505
 506 2008-10-14  Adam Roben  <aroben@apple.com>
 507
 508         Fix Bug 21602: Bugzilla times out trying to display formatted diff for
 509         attachment 24345
 510
 511         https://bugs.webkit.org/show_bug.cgi?id=21602
 512
 513         Reviewed by Dave Kilzer.
 514
 515         * PrettyPatch/PrettyPatch.rb:
 516         (PrettyPatch.BINARY_FILE_MARKER_FORMAT): Added.
 517         (PrettyPatch.FileDiff.initialize): If any of the lines in the diff
 518         are a binary file marker, mark this FileDiff as binary and stop trying
 519         to process the lines.
 520         (PrettyDiff.FileDiff.to_html): If we're binary, just print a string
 521         saying so.
 522
 523 2008-10-02  Adam Roben  <aroben@apple.com>
 524
 525         * template/en/custom/attachment/reviewform.html.tmpl: Fix typo.
 526
 527 2008-10-02  Adam Roben  <aroben@apple.com>
 528
 529         * template/en/custom/attachment/review.html.tmpl: Fix URLs
 530
 531 2008-10-02  Adam Roben  <aroben@apple.com>
 532
 533         Fix Bug 21315: bugs.webkit.org should have a nicer patch review
 534         interface
 535
 536         https://bugs.webkit.org/show_bug.cgi?id=21315
 537
 538         Reviewed by Tim Hatcher.
 539
 540         * attachment.cgi:
 541         (top level): Added support for the "review" and "reviewform" actions.
 542         (sub edit): Accept the template name as a parameter. If no name is
 543         give, use "edit" as before.
 544         * template/en/custom/attachment/list.html.tmpl: Added a "Review Patch"
 545         link for all patch attachments.
 546         * template/en/custom/attachment/review.html.tmpl: Added. Simple
 547         <frameset> page to show the patch on the top and a comment form on the
 548         bottom.
 549         * template/en/custom/attachment/reviewform.html.tmpl: Added. Simple
 550         comment form for reviewing patches.
 551
 552 2008-10-02  Adam Roben  <aroben@apple.com>
 553
 554         Remove references to some backup files I created when implementing
 555         PrettyPatch in Bugzilla
 556
 557         Reviewed by Tim Hatcher.
 558
 559         * attachment-aroben.cgi: Removed.
 560         * template/en/default/attachment/edit-aroben.html.tmpl: Removed.
 561
 562         * template/en/custom/attachment/edit.html.tmpl: Removed an erroneous
 563         reference to attachment-aroben.cgi. attachment.cgi will work just
 564         fine.
 565
 566 2008-06-25  David Kilzer  <ddkilzer@apple.com>
 567
 568         Make PrettyPatch handle diffs with no Index or diff headers
 569
 570         Part of Bug 19290: More patches not handled by PrettyPatch.rb
 571         <https://bugs.webkit.org/show_bug.cgi?id=19290>
 572
 573         Reviewed by Adam.
 574
 575         * PrettyPatch/PrettyPatch.rb:
 576         (PrettyPatch.DIFF_HEADER_FORMATS): Added regular expression to
 577         match on "+++ " lines for patches with no "Index" or "diff" header.
 578         (PrettyPatch.FileDiff.initialize): Look for filename on "+++ " line
 579         when the first line of a patch has no "Index" or "diff" header.
 580         (PrettyPatch.FileDiff.parse): Added haveSeenDiffHeader state
 581         variable to determine when no "Index" or "diff" header has been
 582         found, but a new patch has started with a "--- " line.
 583
 584 2008-06-08  Dan Bernstein  <mitz@apple.com>
 585
 586         Reviewed by Adam Roben.
 587
 588         - update trac URL in PrettyPatch
 589
 590         * PrettyPatch/PrettyPatch.rb:
 591
 592 2008-05-29  Adam Roben  <aroben@apple.com>
 593
 594         Update PrettyPatch directory list
 595
 596         * PrettyPatch/PrettyPatch.rb:
 597
 598 2008-05-28  Adam Roben  <aroben@apple.com>
 599
 600         Make PrettyPatch able to handle diffs taken from outside the WebKit
 601         source tree
 602
 603         Part of Bug 19290: More patches not handled by PrettyPatch.rb
 604         <https://bugs.webkit.org/show_bug.cgi?id=19290>
 605
 606         Reviewed by David Kilzer.
 607
 608         * PrettyPatch/PrettyPatch.rb:
 609         (PrettyPatch.find_url_and_path): Added. Searches the file path from
 610         the bottom up looking for a directory that exists in the source tree.
 611         (PrettyPatch.linkifyFilename): Changed to call find_url_and_path.
 612
 613 2008-05-28  Adam Roben  <aroben@apple.com>
 614
 615         Print exceptions from PrettyPatch to stdout
 616
 617         This will make it much easier to debug problems with PrettyPatch,
 618         since we'll be able to see the exceptions in the browser.
 619
 620         Reviewed by Sam Weinig.
 621
 622         * PrettyPatch/prettify.rb: Added a --html-exceptions option, which
 623         will print exceptions to stdout as HTML.
 624         * attachment.cgi: Changed to pass --html-exceptions to prettify.rb.
 625
 626 2008-04-26  David Kilzer  <ddkilzer@apple.com>
 627
 628         Removed temp files and added appropriate svn:ignore properties.
 629
 630         Rubber-stamped by Mark Rowe.
 631
 632         * data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
 633
 634         * data/template: Added svn:ignore property for "template" subdirectory.
 635         * data/template/template: Removed precompiled template subdirectory.
 636
 637         * data/versioncache: Removed.
 638         * data/versioncache.*: Removed backup versioncache files.
 639
 640         * data/webdot: Added svn:ignore property for "*.dot" files.
 641         * data/webdot/*.dot: Removed cached webdot files.
 642
 643 2008-04-08  Adam Roben  <aroben@apple.com>
 644
 645         Combine :equal operations with the following operation if they are
 646         fewer than 3 characters long
 647
 648         This keeps us from showing lots of small changes on long lines, just
 649         because some letters happened to be the same.
 650
 651         Inspired by
 652         http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/diffviewer/diffutils.py?r=1264#147
 653
 654         * PrettyPatch/PrettyPatch.rb:
 655         (PrettyPatch::DiffSection.initialize): Remove :equal operations fewer
 656         than 3 characters long. The characters from the removed operations
 657         become part of the subsequent operation.
 658         (PrettyPatch::CodeLine.text_as_html): Don't wrap 0-length strings in
 659         <ins>/<del> tags. Also removed the @fromLineNumber.nil? and
 660         @toLineNumber.nil? checks as they are no longer reliable now that
 661         we're removing operations.
 662
 663 2008-04-08  Adam Roben  <aroben@apple.com>
 664
 665         Change PrettyPatch to use DiffBuilder for intra-line diffs
 666
 667         This gives us much prettier intra-line diffs, largely because it can
 668         distinguish multiple changes on the same line. e.g., if a line changes
 669         from:
 670
 671         const int myConstant = 0;
 672
 673         to
 674
 675         static const unsigned myConstant;
 676
 677         You will see that "static " was inserted, "int" changed to "unsigned",
 678         and " = 0" was deleted.
 679
 680         This seems to have also gotten rid of some spurious instances of
 681         "<ins></ins>" and "<del></del>" at the end of a line.
 682
 683         * PrettyPatch/PrettyPatch.rb:
 684         (PrettyPatch::DiffSection.initialize): Use DiffBuilder instead of
 685         setChangeExtentFromLine (which had been copied from Trac's diffing
 686         code).
 687         (PrettyPatch::CodeLine):
 688          - Removed the changeExtent attribute
 689          - Added the operations attribute
 690          - Removed the setChangeExtentFromLine method
 691         (PrettyPatch::CodeLine.text_as_html): Uses @operations to determine
 692         what text to wrap in <ins> and <del> tags.
 693         * PrettyPatch/diff.rb: Deleted a bunch of code we don't use.
 694         (HTMLDiff::DiffBuilder.initialize): Moved code here from the old build
 695         method, but left out the calls to perform_operation, since we build
 696         the diff HTML outside of this class.
 697         (HTMLDiff::DiffBuilder.split_inputs_to_words): Removed calls to
 698         convert_html_to_list_of_words, since we're not diffing HTML.
 699
 700 2008-04-08  Adam Roben  <aroben@apple.com>
 701
 702         Check in diff.rb
 703
 704         This came from
 705         http://instiki.rubyforge.org/svn/instiki/trunk/lib/diff.rb
 706         revision 28521
 707
 708         * PrettyPatch/diff.rb: Added.
 709
 710 2008-03-04  Adam Roben  <aroben@apple.com>
 711
 712         * PrettyPatch/PrettyPatch.rb: Allow the extent of the previous file to
 713         be omitted from the hunk header.
 714