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