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