Hide the in-rietveld flag in the UI. It is only used only by tooling and shouldn...
[WebKit-https.git] / 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