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