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