0f062eab9dd878c0a5fed10137cbb16ccc33a74b
[WebKit-https.git] / Websites / bugs.webkit.org / ChangeLog
1 2010-12-18  Adam Barth  <abarth@webkit.org>
2
3         Reviewed by Sam Weinig.
4
5         Move WebKitExamplePlugins to Examples
6         https://bugs.webkit.org/show_bug.cgi?id=51291
7
8         * PrettyPatch/PrettyPatch.rb:
9
10 2010-12-17  Dan Bernstein  <mitz@apple.com>
11
12         Reviewed by Simon Fraser.
13
14         Rename WebKitTools to Tools
15         https://bugs.webkit.org/show_bug.cgi?id=49861
16
17         * PrettyPatch/PrettyPatch.rb:
18         * committers-autocomplete.js:
19
20 2010-12-17  Ojan Vafai  <ojan@chromium.org>
21
22         Increment psuedo-version number to avoid pulling cached version.
23
24         * PrettyPatch/PrettyPatch.rb:
25
26 2010-12-17  Ojan Vafai  <ojan@chromium.org>
27
28         Speculative fix for the code review tool in Firefox.
29         Fixes a JS error and makes the status bubble mostly work
30         if postMessage is not supported.
31
32         * PrettyPatch/PrettyPatch.rb:
33         * code-review.js:
34
35 2010-12-15  Ojan Vafai  <ojan@chromium.org>
36
37         Reviewed by Adam Barth.
38
39         size status bubble to it's contents on the code review page
40         https://bugs.webkit.org/show_bug.cgi?id=51142
41
42         * PrettyPatch/PrettyPatch.rb:
43         * code-review.js:
44
45 2010-12-15  Ojan Vafai  <ojan@chromium.org>
46
47         Reviewed by Adam Barth.
48
49         fix goofups from r74142 and r74130
50         https://bugs.webkit.org/show_bug.cgi?id=51146
51
52         Accidentally shrank the inline comment boxes and made
53         the toolbar not show up when there were image diffs.
54
55         * PrettyPatch/PrettyPatch.rb:
56         * code-review.js:
57
58 2010-12-14  Ojan Vafai  <ojan@chromium.org>
59
60         Reviewed by Adam Barth.
61
62         add ability to view for file context to the review tool
63         https://bugs.webkit.org/show_bug.cgi?id=51057
64
65         At the beginning/end of each file diff and between each
66         hunk add links to expand the context. For now it grabs the
67         tip of tree version of the file and tries to apply the diff 
68         to that file. If it can't apply, then it gives up as we
69         wouldn't want to show the wrong lines of context.
70
71         In the future, we can consider adding the upload svn revision
72         to the diff itself, then we could fallback to the file at that
73         revision if tip of tree doesn't apply.
74
75         * PrettyPatch/PrettyPatch.rb:
76         * code-review.js:
77
78 2010-12-08  Ojan Vafai  <ojan@chromium.org>
79
80         Reviewed by Adam Barth.
81
82         [reviewtool] should always show overall comments text box
83         https://bugs.webkit.org/show_bug.cgi?id=45870
84
85         * PrettyPatch/PrettyPatch.rb:
86         * code-review.js:
87
88 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
89
90         Reviewed by Adam Barth.
91
92         Increase size of status bubbles
93         https://bugs.webkit.org/show_bug.cgi?id=50496
94
95         The current available space isn't enough to display all
96         pending results including position in the queue.
97
98         * PrettyPatch/PrettyPatch.rb:
99
100 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
101
102         Reviewed by Adam Barth.
103
104         Increase size of status bubbles
105         https://bugs.webkit.org/show_bug.cgi?id=50496
106
107         The current available space isn't enough to display all
108         pending results including position in the queue.
109
110         * template/en/custom/attachment/edit.html.tmpl:
111         * template/en/custom/attachment/list.html.tmpl:
112         * template/en/custom/attachment/reviewform.html.tmpl:
113
114 2010-11-08  Adam Barth  <abarth@webkit.org>
115
116         Update help text to match behavior change.
117
118         * code-review.js:
119
120 2010-11-08  Adam Barth  <abarth@webkit.org>
121
122         Reviewed by Eric Seidel.
123
124         Restore normal meaning of double-click on patch review page?
125         https://bugs.webkit.org/show_bug.cgi?id=47641
126
127         After this patch, you need to click on the line numbers to add a
128         comment.
129
130         * PrettyPatch/PrettyPatch.rb:
131         * code-review.js:
132
133 2010-10-09  Adam Barth  <abarth@webkit.org>
134
135         Reviewed by Alexey Proskuryakov.
136
137         Exception in committers-autocomplete in Firefox
138         https://bugs.webkit.org/show_bug.cgi?id=47456
139
140         Remove dependency on a WebKit-only quirk.  IMHO, this quirk is bad for
141         the long-term health of the web, but that's a discussion for another
142         patch.
143
144         * committers-autocomplete.js:
145
146 2010-10-08  Adam Barth  <abarth@webkit.org>
147
148         Reviewed by Oliver Hunt.
149
150         Bugmail for new attachments should link to the reviewtool
151         https://bugs.webkit.org/show_bug.cgi?id=47440
152
153         As requested by olliej himself.  This patch was constructed by pure
154         reason.  I have no way to actually run this code.
155
156         * Bugzilla/BugMail.pm:
157
158 2010-09-28  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
159
160         Reviewed by Csaba Osztrogonác.
161
162         Convert verbatim unicode values in comitter names to characters
163
164         * committers-autocomplete.js:
165
166 2010-09-24  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
167
168         Reviewed by Csaba Osztrogonác.
169
170         Support Unicode-strings in committers-autocomplete script
171
172         * committers-autocomplete.js:
173
174 2010-09-22  Adam Barth  <abarth@webkit.org>
175
176         Reviewed by John Sullivan.
177
178         New review-page design doesn't include name of reviewer
179         https://bugs.webkit.org/show_bug.cgi?id=46271
180
181         * PrettyPatch/PrettyPatch.rb:
182         * code-review.js:
183
184 2010-09-22  Adam Barth  <abarth@webkit.org>
185
186         Fix typo in CSS.
187
188         * PrettyPatch/PrettyPatch.rb:
189
190 2010-09-22  Adam Barth  <abarth@webkit.org>
191
192         Reviewed by Eric Seidel.
193
194         Add link to bug to review page
195         https://bugs.webkit.org/show_bug.cgi?id=46192
196
197         To make room for the link, I moved the help text to the top of the page
198         and tweaked the language to help folks discover that you can select
199         context using the line numbers.
200
201         Also, move more text to sans-serif since that fits in better with the
202         pretty-patch fonts.
203
204         * PrettyPatch/PrettyPatch.rb:
205         * code-review.js:
206
207 2010-09-21  Adam Barth  <abarth@webkit.org>
208
209         Unreviewed.
210
211         [reviewtool] Publish button doesn't work
212         https://bugs.webkit.org/show_bug.cgi?id=46168
213
214         Darin's recent patch had a runtime error.  Sadly, we don't have any
215         testing for this code.
216
217         * code-review.js:
218
219 2010-09-20  Darin Adler  <darin@apple.com>
220
221         Reviewed by Adam Barth.
222
223         Add Preview button and link to bug to patch review bar
224         https://bugs.webkit.org/show_bug.cgi?id=46153
225
226         * PrettyPatch/PrettyPatch.rb: Bump version number to make caches happy.
227         * code-review.js: Added Preview button and link to bug.
228
229 2010-09-19  Adam Barth  <abarth@webkit.org>
230
231         Reviewed by Eric Seidel.
232
233         [reviewtool] Publish button should actually publish the review comments
234         https://bugs.webkit.org/show_bug.cgi?id=46074
235
236         Instead of showing the confirmation lightbox, the Publish button now
237         publishes the comments directly.  If there's demand for a "Preview"
238         button, we can find a way to add that feature.
239
240         * code-review.js:
241
242 2010-09-19  Adam Barth  <abarth@webkit.org>
243
244         Reviewed by Eric Seidel.
245
246         Move reviewtool to action=review
247         https://bugs.webkit.org/show_bug.cgi?id=46071
248
249         This patch removes the old Review Patch screen and moves the new
250         reviewtool from Pretty Diff to Review Patch.
251
252         * attachment.cgi:
253         * code-review.js:
254
255 2010-09-19  Adam Barth  <abarth@webkit.org>
256
257         Review tool UI tweak.  This patch lengthens the status bubbles so all
258         the bubble fit (even when they have numbers inside).
259
260         * PrettyPatch/PrettyPatch.rb:
261
262 2010-09-19  Dan Bernstein  <mitz@apple.com>
263
264         Reviewed by Anders Carlsson.
265
266         Fixed a typo.
267
268         * code-review.js: Changed the label of the OK button from Ok to OK.
269
270 2010-09-15  Adam Barth  <abarth@webkit.org>
271
272         Unreviewed.
273
274         Only show the review-in-context link if there are contextual comments.
275
276         * code-review.js:
277
278 2010-09-15  Adam Barth  <abarth@webkit.org>
279
280         Reviewed by Eric Seidel.
281
282         [reviewtool] Show the status bubbles on the toolbar
283         https://bugs.webkit.org/show_bug.cgi?id=45861
284
285         * PrettyPatch/PrettyPatch.rb:
286         * code-review.js:
287
288 2010-09-15  Adam Barth  <abarth@webkit.org>
289
290         Reviewed by Eric Seidel.
291
292         [reviewtool] Allow setting review and commit-queue flags from main review screen
293         https://bugs.webkit.org/show_bug.cgi?id=45860
294
295         This patch adds drop down menus for adjusting the review and
296         commit-queue flags from the main review screen.  These controls don't
297         appear immediately because we need to read the state of the flags off
298         the details page.
299
300         It's a slight sadness that bugzilla doesn't have a clean way of
301         identifying which select control is associated with a given flag.  The
302         flags seem to have some sort of id, but it's unclear to me how to map
303         that id back to a notion of "review" or "commit-queue".  Instead, we
304         look for some magic strings in the title.  I'm sure this will come back
305         to bit us at some point, but I'm not sure what to do that's better.
306         (The webkitpy solution to this problem is to assume that the flags are
307         in a given order, but this seems better.)
308
309         * PrettyPatch/PrettyPatch.rb:
310             - Bump version number to bust throuh caches.
311         * code-review.js:
312
313 2010-09-15  Adam Barth  <abarth@webkit.org>
314
315         Reviewed by Eric Seidel.
316
317         [reviewtool] Comments should quote previous comments on the same line
318         https://bugs.webkit.org/show_bug.cgi?id=45847
319
320         Now when you reply to a previous comment, the tool will quote the
321         previous comment in the bugs.webkit.org post.  This makes it eaiser for
322         folks following along in email to understand the discussion.
323
324         While I was editing this code, I also cleaned up some of the whitespace
325         handling in comments.
326
327         * code-review.js:
328
329 2010-09-15  Adam Barth  <abarth@webkit.org>
330
331         Minor UI tweak to the review tool.  We want to display newlines in
332         previous and frozen comments instead of collapsing them.
333
334         * PrettyPatch/PrettyPatch.rb:
335
336 2010-09-14  Adam Barth  <abarth@webkit.org>
337
338         Reviewed by Eric Seidel.
339
340         bugs.webkit.org should autocomplete WebKit Committers and Reviewers
341         https://bugs.webkit.org/show_bug.cgi?id=45799
342
343         This patch implements an autocomplete dropdown that helps users type
344         the names of WebKit committers and reviewers.  The script grabs the
345         data from SVN now that wms set up CORS for http://svn.webkit.org.
346         Security technology for the win.
347
348         This script is based on (well, essentially copied from) the popular
349         autocomplete extension:
350
351         https://chrome.google.com/extensions/detail/olaabhcgdogcbcoiolomlcodkngnemfb
352
353         I asked Ojan, and said he was fine licensing the code under the BSD
354         license.  I'm not sure how the patch interacts with the extension, but
355         we can retire the extension if they interact poorly.
356
357         * committers-autocomplete.js: Added.
358         * template/en/default/global/footer.html.tmpl:
359             - Add the script to every page on bugs.webkit.org
360
361 2010-09-14  Adam Barth  <abarth@webkit.org>
362
363         Add a license block to the reviewtool.  I should have added this from
364         the start.  According to the SVN log, I'm the only one to have ever
365         touched this file.
366
367         * code-review.js:
368
369 2010-09-07  Adam Barth  <abarth@webkit.org>
370
371         Minor tweaks to the reviewtool UI.
372
373         Rename Cancel to Discard to make it more clear what the button does.
374         Also, move the style declaration for the focus ring later in the sheet
375         so it works.
376
377         * PrettyPatch/PrettyPatch.rb:
378         * code-review.js:
379
380 2010-09-06  Adam Barth  <abarth@webkit.org>
381
382         Small UI tweak to the reviewtool.  Make the Ok/Cancel buttons a bit
383         bigger.
384
385         * PrettyPatch/PrettyPatch.rb:
386
387 2010-09-06  Adam Barth  <abarth@webkit.org>
388
389         Reviewed by Eric Seidel.
390
391         [reviewtool] Add a quick animation for opening/closing comment boxes
392         https://bugs.webkit.org/show_bug.cgi?id=45278
393
394         The 200ms animation slides in and out the line-by-line comment box.
395
396         * code-review.js:
397
398 2010-09-06  Adam Barth  <abarth@webkit.org>
399
400         Reviewed by Eric Seidel.
401
402         [reviewtool] Add a field for overall comments
403         https://bugs.webkit.org/show_bug.cgi?id=45273
404
405         This patch does a couple logically separate things that could be
406         separated into smaller patches:
407
408         1) This patch adds an "overall comments" field where you can enter
409            overall comments about the patch.  These comments appear at the top
410            of the bugzilla posting.  Currently, these aren't redisplayed when
411            viewing the patch, but I plan to add that in a future patch.
412
413         2) This patch renames some of the CSS classes to more consistently
414            follow the camelCase style that PrettyPatch uses.
415
416         3) This patch moves the "prepare comments" button to the left of the
417            toolbar and renames is to "publish comments".  This makes more sense
418            when you scroll to the bottom of the page and enter in some overall
419            comments.
420
421         4) When you attempt to add a comment to a line that already has a
422            "frozen" comment, we now unfreeze the comment instead of doing
423            nothing.  The old behavior was kind of frustrating if you didn't
424            know that you could unfreeze a comment by clicking on it.
425
426         * PrettyPatch/PrettyPatch.rb:
427             - Update CSS.
428         * code-review.js:
429
430 2010-09-06  Adam Barth  <abarth@webkit.org>
431
432         [reviewtool] Tweak the ok button to cancel the comment if the comment
433         is empty.  Previously we would get into a bad state where a line had a
434         comment but there was no longer any way to access it.
435
436         * code-review.js:
437
438 2010-09-06  Adam Barth  <abarth@webkit.org>
439
440         Reviewed by Eric Seidel.
441
442         [reviewtool] Add an "ok" button that collapses review comments
443         https://bugs.webkit.org/show_bug.cgi?id=45255
444
445         Many folks (including myself) are too trigger-happy with the "delete"
446         button that we end up deleting perfectly good review comments.  This
447         patch adds an "ok" button to absorb these clicks (and renames "delete"
448         to "cancel").  When you click the ok button, the comment becomes
449         non-editable and the box shrinks to fit the text.  You can click the
450         text to get back to the editable version.
451
452         * code-review.js:
453
454 2010-08-31  Adam Barth  <abarth@webkit.org>
455
456         Reviewed by Eric Seidel.
457
458         [reviewtool] Make it easy to scroll through review comments
459         https://bugs.webkit.org/show_bug.cgi?id=45002
460
461         This patch lets you scroll through review comments using "n" (for next)
462         and "p" (for previous).  It also attributes comments to their authors.
463
464         * PrettyPatch/PrettyPatch.rb:
465         * code-review.js:
466
467 2010-08-31  Adam Barth  <abarth@webkit.org>
468
469         Reviewed by Eric Seidel.
470
471         [reviewtool] Show previous comments inline in diff
472         https://bugs.webkit.org/show_bug.cgi?id=44977
473
474         This patch adds basic support for showing previous comments inline in
475         the diff.  We crawl the bugs.webkit.org comments about this attachment
476         and extract comments related to specific lines.  We then show the
477         comments inline in the diff.
478
479         This part of the tool needs a bunch of polish, but this at least is a
480         starting point for further work.
481
482         * PrettyPatch/PrettyPatch.rb:
483         * code-review.js:
484
485 2010-08-31  Adam Barth  <abarth@webkit.org>
486
487         Reviewed by Eric Seidel.
488
489         [reviewtool] Allow reviewer to select lines of context by dragging over the line numbers
490         https://bugs.webkit.org/show_bug.cgi?id=44936
491
492         * PrettyPatch/PrettyPatch.rb:
493             - Add a version number to bust through bugs.webkit.org's aggressive
494               cacheing.
495         * code-review.js:
496
497 2010-08-30  Adam Barth  <abarth@webkit.org>
498
499         Reviewed by Eric Seidel.
500
501         [review tool] Let reviewer select how much context to show in snippet
502         https://bugs.webkit.org/show_bug.cgi?id=44905
503
504         We now highlight the context for a comment in yellow on the left (where
505         the line numbers are).  Clicking a line number expands or contracts the
506         amount of context, as appropriate.  Informal user testing indicates
507         that we might want to support drag as well.
508
509         This patch also changes the "open a comment box here" action to
510         double-click to avoid issues with mis-clicks.
511
512         * PrettyPatch/PrettyPatch.rb:
513         * code-review.js:
514
515 2010-08-29  Adam Barth  <abarth@webkit.org>
516
517         Attempt to make Sam's life easier by not opening a comment text field
518         if there's a selection.  This should make it easier to copy/paste text
519         out of the diff without accidentally opening comment boxes.
520
521         * code-review.js:
522
523 2010-08-29  Adam Barth  <abarth@webkit.org>
524
525         Turns out we're supposed to use find instead of children here.
526
527         * code-review.js:
528
529 2010-08-29  Adam Barth  <abarth@webkit.org>
530
531         Reviewed by Eric Seidel.
532
533         Awesomify pretty-diff based review tool
534         https://bugs.webkit.org/show_bug.cgi?id=44835
535
536         This patch replaces the existing inline comment feature with a new
537         implementation.  Hopefully the new implementation is more awesome.
538         It's not 100% done yet, but I'd like to get this version out there to
539         see how it works.
540
541         Unlike the previous version this version works with the "Formatted
542         Diff" link instead of the "Review Patch" link.  Hopefully that will
543         avoid interfering with folks who like the old style review.
544
545         * PrettyPatch/PrettyPatch.rb:
546         * code-review.js: Added.
547         * prototype.js: Removed.
548
549 2010-08-19  Tony Chang  <tony@chromium.org>
550
551         Reviewed by Adam Barth.
552
553         webkit-patch barfed on upload with a new image test result
554         https://bugs.webkit.org/show_bug.cgi?id=39097
555
556         * PrettyPatch/PrettyPatch.rb: don't use full path to git
557
558 2010-08-19  Tony Chang  <tony@chromium.org>
559
560         Unreviewed, setting svn:eol-style native on ChangeLog.
561
562 2010-08-19  Tony Chang  <tony@chromium.org>
563
564         Unreviewed.  Just removing \r's from ChangeLog.
565
566 2010-08-02  Adam Roben  <aroben@apple.com>
567
568         Tell the patch prettifier about some (not so) new directories
569
570         * PrettyPatch/PrettyPatch.rb: Added WebKit2, autotools, and cmake
571         directories.
572
573 2010-07-27  Joseph Pecoraro  <joepeck@webkit.org>
574
575         Reviewed by David Kilzer.
576
577         Provide a Helpful Link for Web Inspector Bugs in Bugzilla
578         https://bugs.webkit.org/show_bug.cgi?id=43065
579
580         * template/en/custom/global/choose-product.html.tmpl: Added. Explicitly added a link for the Web Inspector.
581
582 2010-06-09  Julie Parent  <jparent@chromium.org>
583
584         Reviewed by David Kilzer.
585
586         Bugzilla: Don't send mail for cancel/deny/granted for in-rietveld.
587
588         https://bugs.webkit.org/show_bug.cgi?id=40345
589
590         * Bugzilla/Flag.pm:
591
592 2010-05-17  Julie Parent  <jparent@chromium.org>
593
594         Reviewed by Ojan Vafai.
595
596         Rietveld review page should dedicate more space to Rietveld, less to high level comments form.
597
598         https://bugs.webkit.org/show_bug.cgi?id=39244
599
600         * template/en/custom/attachment/reviewform.html.tmpl:
601         Delete a random BR amongst the hidden elements that was causing extra whitespace.
602         * template/en/custom/attachment/rietveldreview.html.tmpl:
603         Increase size of Rietveld frame to 80%, from 60%.
604
605 2010-05-14  Julie Parent  <jparent@chromium.org>
606
607         Reviewed by Ojan Vafai.
608
609         Bugzilla: Only show "Rietveld Review" link when in-rietveld is set to +.
610         
611         https://bugs.webkit.org/show_bug.cgi?id=39141
612
613         * template/en/custom/attachment/list.html.tmpl:
614
615 2010-05-13  Julie Parent  <jparent@chromium.org>
616
617         Reviewed by David Kilzer.
618
619         Bugzilla: Don't send mail if the only change is to the in-rietveld flag.
620         
621         https://bugs.webkit.org/show_bug.cgi?id=39090
622
623         * Bugzilla/BugMail.pm:
624
625 2010-05-12  Julie Parent  <jparent@chromium.org>
626
627         Reviewed by Ojan Vafai.
628
629         Hide the in-rietveld flag in the UI.  It is only used only by tooling and shouldn't clutter the UI.
630         https://bugs.webkit.org/show_bug.cgi?id=39024
631
632         * template/en/custom/attachment/list.html.tmpl:
633           Removes this flag entirely from the UI, not needed on this page.
634         * template/en/custom/flag/list.html.tmpl: Added.
635           Copied from template/en/default/flag/list.html.tmpl with only changes inside the if WEBKIT_CHANGES section.
636           Hides the flag rather than removing it completely, so tools can still interact with it.  
637
638 2010-05-12  Julie Parent  <jparent@chromium.org>
639
640         Reviewed by David Kilzer.
641
642         Adds support for Rietveld reviews.  When there is an associated
643         Rietveld id for a patch, adds a link to do a Rietveld review instead.
644
645         https://bugs.webkit.org/show_bug.cgi?id=38143
646
647         * attachment.cgi:
648             Added support for handling action=rietveldreview.
649         * template/en/custom/attachment/list.html.tmpl:
650             Adds a link to use rietveld review iff the in_rietveld flag is set.
651         * template/en/custom/attachment/reviewform.html.tmpl:
652             Detects if we are in rietveld review mode due to query string paramter,
653             and customizes the review form slightly:
654                 - Changes text describing comment field.
655                 - Comment field does not include quoted patch.
656                 - Submit button fetches from Rietveld instead of just submitting.
657         * template/en/custom/attachment/rietveldreview.html.tmpl: Added.
658             New template for rietveldreview mode.  Hosts the rietveld frame in the
659             top and the regular comments form in the bottom.
660             Uses postMessage to communicate with Rietveld frame.
661
662 2010-05-08  Chris Jerdonek  <cjerdonek@webkit.org>
663
664         Reviewed by Maciej Stachowiak.
665
666         Allow long comment lines to wrap to the window width instead
667         of to 80 characters.
668
669         https://bugs.webkit.org/show_bug.cgi?id=37792
670
671         * Bugzilla/Constants.pm:
672           - Introduced a new constant COMMENT_COLS_WRAP to replace COMMENT_COLS
673             inside the wrap_comment() subroutine.  The new constant effectively
674             disables text wrapping by increasing the value used in wrap_comment()
675             from 80 characters to a very large value (8000).
676         * Bugzilla/Util.pm:
677           - Replaced COMMENT_COLS with COMMENT_COLS_WRAP inside wrap_comment().
678         * skins/custom/global.css:
679         (.bz_comment_text):
680           - Eliminate the width styling on comments to allow long lines
681             to wrap at the window width.
682         (.bz_comment_text > span.quote):
683           - Inherit the "white-space" property.  This prevents long quoted
684             lines in comments from causing non-quoted lines to extend beyond
685             the window width.
686
687 2010-04-22  Adam Barth  <abarth@webkit.org>
688
689         Reviewed by Darin Adler.
690
691         The new review tools shouldn't clear the comment box automatically
692         https://bugs.webkit.org/show_bug.cgi?id=38004
693
694         Instead, we now have a button for doing that explicitly.  If people
695         like this tool, we can look at fancier solutions.
696
697         Also, tried to make the tool work on the edit page as well as the review page.
698
699         * PrettyPatch/PrettyPatch.rb:
700
701 2010-04-22  Adam Barth  <abarth@webkit.org>
702
703         Unreviewed.  Turns out we need to do more escaping because Ruby was
704         explanding our newlines, creating a syntax error.
705
706         * PrettyPatch/PrettyPatch.rb:
707
708 2010-04-22  Adam Barth  <abarth@webkit.org>
709
710         Reviewed by Darin Adler.
711
712         Hack up PrettyDiff to allow line-by-line comments
713         https://bugs.webkit.org/show_bug.cgi?id=37886
714
715         Admittedly a bit of a hack, this is a basic line-by-line editor we can
716         play with thanks to Andrew Scherkus.  It's meant to integrate with the
717         "review" page at bugs.webkit.org.
718
719         I changed a few things from Andrew's original version:
720         1) Trigger text boxes on single click to make iPhone/iPad reviewing
721         easier.
722         2) Clear the main text box on load.
723         3) Reference a version of prototype.js on bugs.webkit.org.
724
725         * PrettyPatch/PrettyPatch.rb:
726         * prototype.js: Added.
727
728 2010-02-22  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
729
730         Reviewed by Maciej Stachowiak.
731
732         Don't reset the assignee back to default on component change
733
734         https://bugs.webkit.org/show_bug.cgi?id=35236
735
736         There's a risk of reseting a valid assignee for example when
737         triaging a bug. All components in WebKit have the default 
738         assignee set to webkit-unassigned@webkit.org so this should
739         not cause problems for people relying on this functionality.
740
741         * template/en/custom/bug/edit.html.tmpl:
742
743 2010-01-22  Janusz Lewandowski  <lew21@xtreeme.org>
744
745         Reviewed by Maciej Stachowiak.
746
747         run-webkit-tests fails on Ruby 1.9
748         https://bugs.webkit.org/show_bug.cgi?id=33554
749         inject does not work with strings any more, so BugsSite/PrettyPatch/PrettyPatch.rb prints lots of errors.
750
751         * PrettyPatch/PrettyPatch.rb:
752
753 2010-01-20  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
754
755         Reviewed by Adam Barth.
756
757         Adjust height of status-bubbles to prevent them being cut.
758
759         * template/en/custom/attachment/edit.html.tmpl:
760         * template/en/custom/attachment/list.html.tmpl:
761         * template/en/custom/attachment/reviewform.html.tmpl:
762
763 2010-01-08  David Kilzer  <ddkilzer@apple.com>
764
765         <http://webkit.org/b/33410> Fix "View Plain Diff"/"View Formatted Diff" button
766
767         Reviewed by Adam Barth.
768
769         This button was broken during a previous Bugzilla upgrade after
770         changes were made to the names of objects exposed on templates.
771
772         * template/en/custom/attachment/reviewform.html.tmpl: Changed
773         attachid to attachment.id and ispatch to attachment.ispatch.
774         Also tweaked the format to use 4 table columns instead of 5.
775
776 2010-01-08  David Kilzer  <ddkilzer@apple.com>
777
778         Included UNCONFIRMED bugs by default in advanced search query
779
780         Rubber-stamped by Alexey Proskuryakov.
781
782         * data/params: Prepended 'bug_status=UNCONFIRMED&' to
783         'defaultquery' parameter.
784
785 2010-01-08  David Kilzer  <ddkilzer@apple.com>
786
787         FIX: Make patch status appear on reviewform.html.tmpl
788
789         * template/en/custom/attachment/reviewform.html.tmpl: Check
790         attachment.ispatch (not just ispatch) when deciding whether
791         to display patch status.  Also update layout a bit.
792
793 2010-01-08  David Kilzer  <ddkilzer@apple.com>
794
795         Update data/params parameters
796
797         * data/params: Updated 'timezone' to turn off DST.  Updated
798         'usebugaliases' and 'usevotes' to turn them off.
799
800 2010-01-08  Adam Barth  <abarth@webkit.org>
801
802         Reviewed by Darin Adler.
803
804         Show patch status in bugs.webkit.org
805         https://bugs.webkit.org/show_bug.cgi?id=33368
806
807         This patch adds an iframe to the show_bug page and the edit attachment
808         page that displays whether the attachment passed the various bots.
809
810         * template/en/custom/attachment/edit.html.tmpl:
811         * template/en/custom/attachment/list.html.tmpl:
812         * template/en/custom/attachment/reviewform.html.tmpl:
813
814 2009-12-07  Shinichiro Hamaji  <hamaji@chromium.org>
815
816         Reviewed by David Kilzer.
817
818         Bugzilla should show images in git patches
819         https://bugs.webkit.org/show_bug.cgi?id=31395
820
821         Attempt to go r51748 again using --directory option of git-apply.
822
823         * PrettyPatch/PrettyPatch.rb:
824
825 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
826
827         Unreviewed. Revert r51748.
828
829         Bugzilla should show images in git patches
830         https://bugs.webkit.org/show_bug.cgi?id=31395
831
832         * PrettyPatch/PrettyPatch.rb:
833
834 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
835
836         Reviewed by David Kilzer.
837
838         Bugzilla should show images in git patches
839         https://bugs.webkit.org/show_bug.cgi?id=31395
840
841         Show images in git patches using git-apply.
842
843         * PrettyPatch/PrettyPatch.rb:
844
845 2009-10-23  Eric Seidel  <eric@webkit.org>
846
847         Reviewed by Adam Roben.
848
849         PrettyPatch should show images even when they have the wrong mime type
850         https://bugs.webkit.org/show_bug.cgi?id=29506
851
852         * PrettyPatch/PrettyPatch.rb:
853
854 2009-10-17  Alexey Proskuryakov  <ap@apple.com>
855
856         Reviewed by Eric Carlson.
857
858         https://bugs.webkit.org/show_bug.cgi?id=30470
859         Make marking a bug as duplicate easier
860
861         * js/field.js: (setResolutionToDuplicate): Focus the edit field, too.
862
863 2009-10-16  Alexey Proskuryakov  <ap@apple.com>
864
865         Reviewed by Mark Rowe.
866
867         https://bugs.webkit.org/show_bug.cgi?id=30470
868         Make marking a bug as duplicate easier
869
870         * js/field.js: (setResolutionToDuplicate): Show edit field right away, don't make the user
871         click again.
872
873 2009-09-09  David Kilzer  <ddkilzer@apple.com>
874
875         Update data/params parameters
876
877         * data/params: Updated 'attachment_base' parameter for secure
878         attachments.  Updated 'mybugstemplate' parameter to search for
879         bugs in UNCONFIRMED, ASSIGNED and REOPENED status in addition to
880         NEW status.
881
882 2009-08-11  David Kilzer  <ddkilzer@apple.com>
883
884         Updated data/params parameter
885
886         * data/params: Updated 'loginnetmask' parameter to allow users
887         to choose whether their cookies are restricted to a single IP
888         address or not.
889
890 2009-07-07  David Kilzer  <ddkilzer@apple.com>
891
892         Updated BugsSite with local changes on server
893
894         The new files were created by checksetup.pl during the upgrade
895         to Bugzilla 3.2.3.  The rest are self-explanatory.
896
897         * data: Added "bugzilla-update.xml" to svn:ignore.
898         * data/attachments: Added "groups.*" to svn:ignore.
899         * data/params: Updated for a few changed parameters.
900
901         * lib/.htaccess: Added.
902         * skins/contrib/Dusk/IE-fixes.css: Added.
903         * skins/contrib/Dusk/admin.css: Added.
904         * skins/contrib/Dusk/create_attachment.css: Added.
905         * skins/contrib/Dusk/dependency-tree.css: Added.
906         * skins/contrib/Dusk/duplicates.css: Added.
907         * skins/contrib/Dusk/editusers.css: Added.
908         * skins/contrib/Dusk/help.css: Added.
909         * skins/contrib/Dusk/index.css: Added.
910         * skins/contrib/Dusk/panel.css: Added.
911         * skins/contrib/Dusk/params.css: Added.
912         * skins/contrib/Dusk/release-notes.css: Added.
913         * skins/contrib/Dusk/show_bug.css: Added.
914         * skins/contrib/Dusk/show_multiple.css: Added.
915         * skins/contrib/Dusk/summarize-time.css: Added.
916         * skins/contrib/Dusk/voting.css: Added.
917         * skins/contrib/Dusk/yui: Added.
918         * skins/contrib/Dusk/yui/calendar.css: Added.
919         * skins/custom/IE-fixes.css: Added.
920         * skins/custom/admin.css: Added.
921         * skins/custom/buglist.css: Added.
922         * skins/custom/create_attachment.css: Added.
923         * skins/custom/dependency-tree.css: Added.
924         * skins/custom/duplicates.css: Added.
925         * skins/custom/editusers.css: Added.
926         * skins/custom/help.css: Added.
927         * skins/custom/panel.css: Added.
928         * skins/custom/params.css: Added.
929         * skins/custom/release-notes.css: Added.
930         * skins/custom/show_bug.css: Added.
931         * skins/custom/show_multiple.css: Added.
932         * skins/custom/summarize-time.css: Added.
933         * skins/custom/voting.css: Added.
934         * skins/custom/yui: Added.
935         * skins/custom/yui/calendar.css: Added.
936
937 2009-07-03  David Kilzer  <ddkilzer@apple.com>
938
939         Bug 26958: Change edit link to review link in request messages
940
941         <https://bugs.webkit.org/show_bug.cgi?id=26958>
942
943         Reviewed by Dan Bernstein.
944
945         * template/en/custom/request/email.txt.tmpl: Copied from BugsSite/template/en/default/request/email.txt.tmpl.
946         Changed link from "action=edit" to "action=review".
947
948 2009-07-03  David Kilzer  <ddkilzer@apple.com>
949
950         Bug 26950: Make the summary and alias fields support click-to-edit
951         <https://bugs.webkit.org/show_bug.cgi?id=26950>
952
953         Reviewed by Maciej Stachowiak.
954
955         Original patch by Maciej Stachowiak.
956
957         * js/field.js:
958         (hideEditableField): Updated to add click event listeners to the
959         alias and short description elements to make them easier to
960         edit.  Renamed field_id parameter to field2_id (short
961         description id) and added a field1_id parameter (alias id).
962         (showEditableField): If a third parameter is passed in the
963         ContainerInputArray parameter, use it to find the element to
964         focus.  Otherwise, fall back to the original behavior of
965         focusing the first input field.
966         (hideAliasAndSummary): Changed to pass the id for the alias
967         element to hideEditableField().
968
969 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
970
971         Config file for contrib/recode.pl when upgrading bugs.webkit.org
972
973         Current as of midday on Wednesday, July 1, 2009.
974
975         * contrib/recode-overrides.txt: Added.
976
977 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
978
979         Workaround for WebKit Bug 9630 when running contrib/recode.pl
980
981         A number of WebKit nightly builds included Bug 9630 which caused
982         non-breaking space characters (0xA0) to be submitted within the
983         content of textarea elements.
984
985         * contrib/recode.pl: To work around these 0xA0 characters, try
986         an encoding of cp1252 and use it if it works.
987
988 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
989
990         Don't print "Failed to guess" warning if an override is available
991
992         * contrib/recode.pl: Check that %overrides does NOT contain a
993         key matching the current digest before printing out the warning
994         about a failed charset guess.
995
996 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
997
998         Added --[no-]truncate switch to contrib/recode.pl
999
1000         * contrib/recode.pl: Added --[no-]truncate switch to make
1001         debugging of failed charset guessing easier.  Often times the
1002         illegal character was truncated in the output.
1003
1004 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1005
1006         Changed Perl scripts to use #!/usr/bin/env perl
1007
1008         perl -pi -e 's@#\!/usr/bin/perl@#\!/usr/bin/env perl@' *.cgi *.pl */*.pl
1009
1010 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1011
1012         Replace Bugzilla favicon with webkit.org favicon
1013
1014         * favicon.ico: Removed.
1015         * images/favicon.ico: Replaced.  Copied from favicon.ico.
1016
1017 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
1018
1019         Merged BugsSite to Bugzilla-3.2.3
1020
1021         Updated to the latest-and-greatest stable version.
1022
1023 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
1024
1025         Merged BugsSite to Bugzilla-3.0.3
1026
1027         Nothing to see here.  Move along.
1028
1029 2009-06-09  Eric Seidel  <eric@webkit.org>
1030
1031         Reviewed by Adam Roben.
1032
1033         Add support for displaying added png files in PrettyPatch diffs
1034         https://bugs.webkit.org/show_bug.cgi?id=26210
1035
1036         Currently this is SVN only (git-send-bugzilla patches exclude binary data)
1037         and only works for PNG files but could easily be made to work for other images as needed.
1038
1039         * PrettyPatch/PrettyPatch.rb:
1040
1041 2009-05-19  Eric Seidel  <eric@webkit.org>
1042
1043         Reviewed by Adam Roben.
1044
1045         Make PrettyPatch understand quoted filenames in git diffs.
1046         https://bugs.webkit.org/show_bug.cgi?id=25881
1047
1048         * PrettyPatch/PrettyPatch.rb:
1049
1050 2009-05-15  Simon Fraser  <simon.fraser@apple.com>
1051
1052         Carrying forwards Darin Adler's rubber stamp
1053         
1054         Make the comment field wider too.
1055
1056         * skins/custom/global.css:
1057
1058 2009-05-14  Simon Fraser  <simon.fraser@apple.com>
1059
1060         Reviewed by Darin Adler
1061         
1062         Make the URL, Summary and Keyword fields wider to mitigate the
1063         effects of https://bugs.webkit.org/show_bug.cgi?id=25566 and make
1064         things generally better.
1065
1066         * skins/custom/global.css:
1067         * template/en/custom/bug/edit.html.tmpl:
1068
1069 2008-11-10  Darin Adler  <darin@apple.com>
1070
1071         - fix more of https://bugs.webkit.org/show_bug.cgi?id=21400
1072           "Edit" links for patches in comments, review queue, review emails
1073           should be replaced by "Review Patch" links
1074
1075         * globals.pl: Fix cases that are automatically generated, such as links
1076         in comments in bugs.
1077
1078         * template/en/custom/attachment/created.html.tmpl: Fix the link on the
1079         "I just created an attachment" page.
1080
1081         * template/en/custom/request/queue.html.tmpl: Fix the link in the queue
1082         page.
1083
1084 2008-11-10  Darin Adler  <darin@apple.com>
1085
1086         * template/en/default/request/queue.html.tmpl: Use review links instead
1087         of edit links in the review queue.
1088
1089 2008-11-10  Darin Adler  <darin@apple.com>
1090
1091         * template/en/custom/request/email.txt.tmpl: Send review links instead
1092         of edit links when flags are set on a bug.
1093
1094 2008-10-27  Darin Adler  <darin@apple.com>
1095
1096         * template/en/custom/attachment/reviewform.html.tmpl: Leave out the
1097         comment if it's untouched.
1098
1099 2008-10-23  Adam Roben  <aroben@apple.com>
1100
1101         Fix Bug 21401: Comments field on "Review Patch" page should be
1102         initially filled with quoted patch
1103
1104         <https://bugs.webkit.org/show_bug.cgi?id=21401>
1105
1106         Reviewed by Dave Hyatt.
1107
1108         * attachment.cgi:
1109         (edit): Retrieve the attachment data from the database instead of just
1110         its length. When the attachment is a patch, create a quotedpatch
1111         template variable that contains the patch data with each line
1112         prepended with "> ".
1113         * template/en/custom/attachment/reviewform.html.tmpl:
1114           - Changed the comments field to have a monospace font
1115           - Added an "Enter comments below:" caption above the comments field
1116             to make it clearer that this is where comments should go, now that
1117             the comments field is not initially empty
1118           - Fill the comments field with the quoted patch
1119
1120 2008-10-14  Adam Roben  <aroben@apple.com>
1121
1122         Fix Bug 21602: Bugzilla times out trying to display formatted diff for
1123         attachment 24345
1124         
1125         https://bugs.webkit.org/show_bug.cgi?id=21602
1126
1127         Reviewed by Dave Kilzer.
1128
1129         * PrettyPatch/PrettyPatch.rb:
1130         (PrettyPatch.BINARY_FILE_MARKER_FORMAT): Added.
1131         (PrettyPatch.FileDiff.initialize): If any of the lines in the diff
1132         are a binary file marker, mark this FileDiff as binary and stop trying
1133         to process the lines.
1134         (PrettyDiff.FileDiff.to_html): If we're binary, just print a string
1135         saying so.
1136
1137 2008-10-02  Adam Roben  <aroben@apple.com>
1138
1139         * template/en/custom/attachment/reviewform.html.tmpl: Fix typo.
1140
1141 2008-10-02  Adam Roben  <aroben@apple.com>
1142
1143         * template/en/custom/attachment/review.html.tmpl: Fix URLs
1144
1145 2008-10-02  Adam Roben  <aroben@apple.com>
1146
1147         Fix Bug 21315: bugs.webkit.org should have a nicer patch review
1148         interface
1149
1150         https://bugs.webkit.org/show_bug.cgi?id=21315
1151
1152         Reviewed by Tim Hatcher.
1153
1154         * attachment.cgi:
1155         (top level): Added support for the "review" and "reviewform" actions.
1156         (sub edit): Accept the template name as a parameter. If no name is
1157         give, use "edit" as before.
1158         * template/en/custom/attachment/list.html.tmpl: Added a "Review Patch"
1159         link for all patch attachments.
1160         * template/en/custom/attachment/review.html.tmpl: Added. Simple
1161         <frameset> page to show the patch on the top and a comment form on the
1162         bottom.
1163         * template/en/custom/attachment/reviewform.html.tmpl: Added. Simple
1164         comment form for reviewing patches.
1165
1166 2008-10-02  Adam Roben  <aroben@apple.com>
1167
1168         Remove references to some backup files I created when implementing
1169         PrettyPatch in Bugzilla
1170
1171         Reviewed by Tim Hatcher.
1172
1173         * attachment-aroben.cgi: Removed.
1174         * template/en/default/attachment/edit-aroben.html.tmpl: Removed.
1175
1176         * template/en/custom/attachment/edit.html.tmpl: Removed an erroneous
1177         reference to attachment-aroben.cgi. attachment.cgi will work just
1178         fine.
1179
1180 2008-06-25  David Kilzer  <ddkilzer@apple.com>
1181
1182         Make PrettyPatch handle diffs with no Index or diff headers
1183
1184         Part of Bug 19290: More patches not handled by PrettyPatch.rb
1185         <https://bugs.webkit.org/show_bug.cgi?id=19290>
1186
1187         Reviewed by Adam.
1188
1189         * PrettyPatch/PrettyPatch.rb:
1190         (PrettyPatch.DIFF_HEADER_FORMATS): Added regular expression to
1191         match on "+++ " lines for patches with no "Index" or "diff" header.
1192         (PrettyPatch.FileDiff.initialize): Look for filename on "+++ " line
1193         when the first line of a patch has no "Index" or "diff" header.
1194         (PrettyPatch.FileDiff.parse): Added haveSeenDiffHeader state
1195         variable to determine when no "Index" or "diff" header has been
1196         found, but a new patch has started with a "--- " line.
1197
1198 2008-06-08  Dan Bernstein  <mitz@apple.com>
1199
1200         Reviewed by Adam Roben.
1201
1202         - update trac URL in PrettyPatch
1203
1204         * PrettyPatch/PrettyPatch.rb:
1205
1206 2008-05-29  Adam Roben  <aroben@apple.com>
1207
1208         Update PrettyPatch directory list
1209
1210         * PrettyPatch/PrettyPatch.rb:
1211
1212 2008-05-28  Adam Roben  <aroben@apple.com>
1213
1214         Make PrettyPatch able to handle diffs taken from outside the WebKit
1215         source tree
1216
1217         Part of Bug 19290: More patches not handled by PrettyPatch.rb
1218         <https://bugs.webkit.org/show_bug.cgi?id=19290>
1219
1220         Reviewed by David Kilzer.
1221
1222         * PrettyPatch/PrettyPatch.rb:
1223         (PrettyPatch.find_url_and_path): Added. Searches the file path from
1224         the bottom up looking for a directory that exists in the source tree.
1225         (PrettyPatch.linkifyFilename): Changed to call find_url_and_path.
1226
1227 2008-05-28  Adam Roben  <aroben@apple.com>
1228
1229         Print exceptions from PrettyPatch to stdout
1230
1231         This will make it much easier to debug problems with PrettyPatch,
1232         since we'll be able to see the exceptions in the browser.
1233
1234         Reviewed by Sam Weinig.
1235
1236         * PrettyPatch/prettify.rb: Added a --html-exceptions option, which
1237         will print exceptions to stdout as HTML.
1238         * attachment.cgi: Changed to pass --html-exceptions to prettify.rb.
1239
1240 2008-04-26  David Kilzer  <ddkilzer@apple.com>
1241
1242         Removed temp files and added appropriate svn:ignore properties.
1243
1244         Rubber-stamped by Mark Rowe.
1245
1246         * data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
1247
1248         * data/template: Added svn:ignore property for "template" subdirectory.
1249         * data/template/template: Removed precompiled template subdirectory.
1250
1251         * data/versioncache: Removed.
1252         * data/versioncache.*: Removed backup versioncache files.
1253
1254         * data/webdot: Added svn:ignore property for "*.dot" files.
1255         * data/webdot/*.dot: Removed cached webdot files.
1256
1257 2008-04-08  Adam Roben  <aroben@apple.com>
1258
1259         Combine :equal operations with the following operation if they are
1260         fewer than 3 characters long
1261
1262         This keeps us from showing lots of small changes on long lines, just
1263         because some letters happened to be the same.
1264
1265         Inspired by
1266         http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/diffviewer/diffutils.py?r=1264#147
1267
1268         * PrettyPatch/PrettyPatch.rb:
1269         (PrettyPatch::DiffSection.initialize): Remove :equal operations fewer
1270         than 3 characters long. The characters from the removed operations
1271         become part of the subsequent operation.
1272         (PrettyPatch::CodeLine.text_as_html): Don't wrap 0-length strings in
1273         <ins>/<del> tags. Also removed the @fromLineNumber.nil? and
1274         @toLineNumber.nil? checks as they are no longer reliable now that
1275         we're removing operations.
1276
1277 2008-04-08  Adam Roben  <aroben@apple.com>
1278
1279         Change PrettyPatch to use DiffBuilder for intra-line diffs
1280
1281         This gives us much prettier intra-line diffs, largely because it can
1282         distinguish multiple changes on the same line. e.g., if a line changes
1283         from:
1284
1285         const int myConstant = 0;
1286
1287         to
1288
1289         static const unsigned myConstant;
1290
1291         You will see that "static " was inserted, "int" changed to "unsigned",
1292         and " = 0" was deleted.
1293
1294         This seems to have also gotten rid of some spurious instances of
1295         "<ins></ins>" and "<del></del>" at the end of a line.
1296
1297         * PrettyPatch/PrettyPatch.rb:
1298         (PrettyPatch::DiffSection.initialize): Use DiffBuilder instead of
1299         setChangeExtentFromLine (which had been copied from Trac's diffing
1300         code).
1301         (PrettyPatch::CodeLine):
1302          - Removed the changeExtent attribute
1303          - Added the operations attribute
1304          - Removed the setChangeExtentFromLine method
1305         (PrettyPatch::CodeLine.text_as_html): Uses @operations to determine
1306         what text to wrap in <ins> and <del> tags.
1307         * PrettyPatch/diff.rb: Deleted a bunch of code we don't use.
1308         (HTMLDiff::DiffBuilder.initialize): Moved code here from the old build
1309         method, but left out the calls to perform_operation, since we build
1310         the diff HTML outside of this class.
1311         (HTMLDiff::DiffBuilder.split_inputs_to_words): Removed calls to
1312         convert_html_to_list_of_words, since we're not diffing HTML.
1313
1314 2008-04-08  Adam Roben  <aroben@apple.com>
1315
1316         Check in diff.rb
1317
1318         This came from
1319         http://instiki.rubyforge.org/svn/instiki/trunk/lib/diff.rb
1320         revision 28521
1321
1322         * PrettyPatch/diff.rb: Added.
1323
1324 2008-03-04  Adam Roben  <aroben@apple.com>
1325
1326         * PrettyPatch/PrettyPatch.rb: Allow the extent of the previous file to
1327         be omitted from the hunk header.
1328