6a5cf391004a5e06b205a7b8bdec01a324a739c9
[WebKit-https.git] / 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