BugsSite/ChangeLog

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