62936c57056990a7340d47db7d878cb75feb7574
[WebKit-https.git] / Websites / bugs.webkit.org / ChangeLog
1 2011-02-17  Ojan Vafai  <ojan@chromium.org>
2
3         Reviewed by Adam Barth.
4
5         fix escape handling in code review tool
6         https://bugs.webkit.org/show_bug.cgi?id=54722
7
8         This broke with http://trac.webkit.org/changeset/78948.
9         Apparently jquery events get a wrapped object that
10         does not expose keyIdentifier.
11
12         * code-review.js:
13
14 2011-02-17  Ojan Vafai  <ojan@chromium.org>
15
16         Reviewed by Adam Barth.
17
18         create comments when hitting enter with a focused diff
19         https://bugs.webkit.org/show_bug.cgi?id=54719
20
21         * code-review.js:
22
23 2011-02-17  Ojan Vafai  <ojan@chromium.org>
24
25         Reviewed by Adam Barth.
26
27         use actual browser focus in the code review tool
28         https://bugs.webkit.org/show_bug.cgi?id=54726
29
30         This makes keyboard handling play better with focusable element
31         (i.e. links/textareas/buttons). Also, in theory, this makes
32         the review tool more amenable to screen readers (I think).
33
34         * PrettyPatch/PrettyPatch.rb:
35         * code-review.js:
36
37 2011-02-17  Ojan Vafai  <ojan@chromium.org>
38
39         Reviewed by Antonio Gomes.
40
41         make event handling work in Gecko
42         https://bugs.webkit.org/show_bug.cgi?id=54702
43
44         Gecko doesn't support the global event property. It's an IE-ism.
45
46         * code-review.js:
47
48 2011-02-14  Ojan Vafai  <ojan@chromium.org>
49
50         Reviewed by Adam Barth.
51
52         allow for modifying comments without the mouse
53         https://bugs.webkit.org/show_bug.cgi?id=54433
54
55         Use n/p to navigate the comments.
56         Enter to begin editing a comment.
57         Escape to finish editing a comment.
58
59         * code-review.js:
60
61 2011-02-14  Ojan Vafai  <ojan@chromium.org>
62
63         Reviewed by Adam Barth.
64
65         improve line selection in the code review tool
66         https://bugs.webkit.org/show_bug.cgi?id=54430
67
68         -shift+click now extends the comment context
69         -selecting outside of a diff section no longer clears the selected lines.
70          Instead it just restricts the selected lines to that diff section.
71
72         * code-review.js:
73
74 2011-02-10  Ojan Vafai  <ojan@chromium.org>
75
76         Reviewed by Adam Barth.
77
78         remove extra space from expansion lines
79         https://bugs.webkit.org/show_bug.cgi?id=54266
80
81         * code-review.js:
82
83 2011-02-09  Ojan Vafai  <ojan@chromium.org>
84
85         Reviewed by Adam Barth.
86
87         only erase draft comments after publish is successful
88         https://bugs.webkit.org/show_bug.cgi?id=54163
89
90         If there is a conflict or 500, then draft comments will survive.
91         One drawback here is that the form post is now to the iframe,
92         so to break out of the iframe we redirect to the bug page, which
93         loses the information of who the email was sent to.
94
95         Once WebKit supports seamless iframes we should be able to avoid
96         the redirect.
97
98         * code-review.js:
99
100 2011-02-09  Ojan Vafai  <ojan@chromium.org>
101
102         Reviewed by Adam Barth.
103
104         save overall comments when saving drafts in the review tool
105         https://bugs.webkit.org/show_bug.cgi?id=54165
106
107         -save overall comments in localstorage as well
108         -save all draft comments as you type
109         -give a *subtle* indicator of saved state
110
111         The latter should also make it super easy if someone wanted to do
112         the work to store draft comments in appengine/s3/bugzilla/etc.
113
114         * PrettyPatch/PrettyPatch.rb:
115         * code-review-test.html:
116         * code-review.js:
117
118 2011-02-08  Ojan Vafai  <ojan@chromium.org>
119
120         Reviewed by Adam Barth.
121
122         fix toolbar anchoring in the code review tool
123         https://bugs.webkit.org/show_bug.cgi?id=54058
124
125         Avoid the anchoring cycle of doom when on the cusp
126         of whether the toolbar needs to be anchored and
127         speculatively avoid the Firefox crash when resizing.
128
129         * PrettyPatch/PrettyPatch.rb:
130         * code-review.js:
131
132 2011-02-07  Ojan Vafai  <ojan@chromium.org>
133
134         Reviewed by Adam Barth.
135
136         bring back diff context lines into the review tool
137         https://bugs.webkit.org/show_bug.cgi?id=53974
138
139         * code-review.js:
140
141 2011-02-01  Ojan Vafai  <ojan@chromium.org>
142
143         Reviewed by Adam Roben.
144
145         include svn revisions in git diffs for the code review tool to use
146         https://bugs.webkit.org/show_bug.cgi?id=53569
147
148         * PrettyPatch/PrettyPatch.rb:
149
150 2011-02-01  Ojan Vafai  <ojan@chromium.org>
151
152         Reviewed by Adam Barth.
153
154         make draft comments focusable
155         https://bugs.webkit.org/show_bug.cgi?id=53554
156
157         Makes frozen draft comments focusable. The ones that are currently being edited are not.
158         I'm on the fence whether they should be, but this seems good enough for now.
159
160         * code-review.js:
161
162 2011-02-01  Ojan Vafai  <ojan@chromium.org>
163
164         Reviewed by Adam Barth.
165
166         avoid jitter when loading the comments to a patch
167         https://bugs.webkit.org/show_bug.cgi?id=53570
168
169         * code-review.js:
170
171 2011-02-01  Ojan Vafai  <ojan@chromium.org>
172
173         Reviewed by Mihai Parparita.
174
175         [codereviewtool] extra space at the beginning of lines in side-by-side view
176         https://bugs.webkit.org/show_bug.cgi?id=53550
177
178         The newline between the spans turns into user-visible space in side-by-side mode.
179
180         * PrettyPatch/PrettyPatch.rb:
181
182 2011-02-01  Ojan Vafai  <ojan@chromium.org>
183
184         Increment version number to cache-bust code-review.js.
185
186         * PrettyPatch/PrettyPatch.rb:
187
188 2011-01-31  Ojan Vafai  <ojan@chromium.org>
189
190         Reviewed by Adam Barth.
191
192         Store draft comments in localStorage
193         https://bugs.webkit.org/show_bug.cgi?id=52866
194
195         * code-review.js:
196         * code-review-test.html
197
198 2011-01-20  Ojan Vafai  <ojan@chromium.org>
199
200         Fix the review tool for image diffs. We would get a javascript error
201         because image diffs don't have line numbers.
202
203         * code-review.js:
204
205 2011-01-20  Ojan Vafai  <ojan@chromium.org>
206
207         Reviewed by Oliver Hunt.
208
209         Could review tool include style failure info?
210         https://bugs.webkit.org/show_bug.cgi?id=49049
211
212         This does not handle patches that have old paths (e.g. without the Source prefix).
213         Although that would be fairly straightforward to workaround.
214
215         * code-review.js:
216
217 2011-01-20  Ojan Vafai  <ojan@chromium.org>
218
219         Reviewed by Adam Barth.
220
221         handle cases where PrettyPatch.rb doesn't linkify filenames
222         https://bugs.webkit.org/show_bug.cgi?id=52834
223
224         * code-review.js:
225
226 2011-01-20  Ojan Vafai  <ojan@chromium.org>
227
228         Reviewed by Adam Barth.
229
230         fix goofup in http://trac.webkit.org/changeset/76082
231         https://bugs.webkit.org/show_bug.cgi?id=52830
232
233         We were showing the revision number of the patch at the bottom
234         of the diff and expanding from the bottom of the diff would fail.
235
236         Also, removed an unused variable and scoped queries appropriatly
237         to the file_diff they should have been operating on.
238
239         * PrettyPatch/PrettyPatch.rb:
240         * code-review.js:
241
242 2011-01-19  Ojan Vafai  <ojan@chromium.org>
243
244         Reviewed by Adam Barth.
245
246         [reviewtool] Add a link for annotated trac page on review page
247         https://bugs.webkit.org/show_bug.cgi?id=52747
248
249         -add line numbers to all trac links to the first line in the file diff
250         -add annotate/review log links
251         -make file-diff links other than the file name only show when you mouseover the filediff
252         -fix white-space to only wrap the .text element and it's childen, any of it's siblings/parents
253
254         * PrettyPatch/PrettyPatch.rb:
255         * code-review.js:
256
257 2011-01-14  Ojan Vafai  <ojan@chromium.org>
258
259         Reviewed by Adam Barth.
260
261         make line selection have a extend only from where you start the selection in the code review tool
262         https://bugs.webkit.org/show_bug.cgi?id=52485
263
264         * code-review.js:
265
266 2011-01-18  Ojan Vafai  <ojan@chromium.org>
267
268         Reviewed by Eric Seidel.
269
270         make file expansion use the file at the svn revision the diff was created at
271         https://bugs.webkit.org/show_bug.cgi?id=52650
272
273         This only works for patches created with SVN for now since the git
274         diff does not include SVN revision numbers in it.
275
276         * PrettyPatch/PrettyPatch.rb:
277         * code-review.js:
278
279 2011-01-15  Adam Barth  <abarth@webkit.org>
280
281         Rubber-stamped by Eric Seidel.
282
283         Move WebKit2 into Source
284         https://bugs.webkit.org/show_bug.cgi?id=52438
285
286         Remove WebKit2 from the list of top-level source directories.
287
288         * PrettyPatch/PrettyPatch.rb:
289
290 2011-01-14  Ojan Vafai  <ojan@chromium.org>
291
292         Reviewed by Adam Barth.
293
294         fix expanded lines in the code review tool
295         https://bugs.webkit.org/show_bug.cgi?id=52498
296
297         Fixes them to work now that we have wrapper divs.
298         Also fixes the long-standing bug that the line numbers
299         were incorrect sometimes.
300
301         * code-review.js:
302
303 2011-01-14  Ojan Vafai  <ojan@chromium.org>
304
305         Reviewed by Adam Barth.
306
307         put remove lines to the left of add lines in sidebyside view
308         https://bugs.webkit.org/show_bug.cgi?id=52458
309
310         Also, sneaking in a fix to make long lines with no whitespace wrap.
311
312         * PrettyPatch/PrettyPatch.rb:
313         * code-review.js:
314
315 2011-01-14  Ojan Vafai  <ojan@chromium.org>
316
317         Fix bad merge in r75816.
318
319         * code-review.js:
320
321 2011-01-14  Ojan Vafai  <ojan@chromium.org>
322
323         Reviewed by Adam Barth.
324
325         improve line selection with fast drags
326         https://bugs.webkit.org/show_bug.cgi?id=52477
327
328         * code-review.js:
329
330 2011-01-14  Ojan Vafai  <ojan@chromium.org>
331
332         Reviewed by Adam Barth.
333
334         tweak line selection in code review tool
335         https://bugs.webkit.org/show_bug.cgi?id=52462
336
337         -Improve handling of data-comment-base-line code to deal with
338         lines that have multiple values.
339         -Make it so that if you click on the line immediately above
340         a line that has a comment it will add a new comment instead of
341         adding lines to the following comment. If the last selected line
342         overlaps existing comment lines though they will still get
343         added to the existing comment.
344
345         * code-review.js:
346
347 2011-01-14  Ojan Vafai  <ojan@chromium.org>
348
349         Reviewed by Adam Roben.
350
351         fix bugs going back and forth between unified and sidebyside
352         https://bugs.webkit.org/show_bug.cgi?id=52470
353
354         Now that we're appending DOM nodes directly, we only want to
355         get the first ".text" element in the subtree. In the cases
356         where there are multiple, their contents are the same.
357
358         Also, when making side-by-side expansion lines, we need to
359         clone the contents so we have a second copy to append to the DOM.
360
361         * code-review.js:
362
363 2011-01-13  Ojan Vafai  <ojan@chromium.org>
364
365         Reviewed by Adam Barth.
366
367         simplify keyboard handling in code review tool
368         https://bugs.webkit.org/show_bug.cgi?id=52407
369
370         Now that we have DiffBlock containers, the only things that are
371         focusable are previousComment nodes and DiffBlock containers
372         that contain add/remove lines.
373
374         Also, this means we show the focus border around the entire diff
375         instead of just the first line.
376
377         * code-review.js:
378
379 2011-01-13  Ojan Vafai  <ojan@chromium.org>
380
381         Reviewed by Adam Barth.
382
383         add container divs for diff blocks
384         https://bugs.webkit.org/show_bug.cgi?id=52400
385
386         This will help simplify a lot of code in code-review.js
387         and make side-by-side diffs better (i.e. put removed lines
388         to the left of corresponding added lines).
389
390         Also, allow for running the JS from a local file. Now you can modify code-review.js
391         to point to a local file and then run:
392         ruby prettify.rb < foo.diff > foo.html
393
394         foo.html will load a dummy code review matching foo.diff.
395
396         Before structure:
397         Line
398         Line remove
399         Line add
400         Line add
401         Line
402
403         After structure:
404         DiffBlock
405           DiffBlockPart shared
406             Line shared
407         DiffBlock
408           DiffBlockPart remove
409             Line remove
410           DiffBlockPart add
411             Line add
412             Line add
413         DiffBlock
414           DiffBlockPart shared
415             Line shared
416
417         * PrettyPatch/PrettyPatch.rb:
418         * code-review.js:
419
420 2011-01-12  Ojan Vafai  <ojan@chromium.org>
421
422         Reviewed by Mihai Parparita.
423
424         fix assorted bugs with expansion lines
425         https://bugs.webkit.org/show_bug.cgi?id=52326
426
427         -Clicking "all" would collapse whitespace in the expanded lines.
428         -Converted the remaining line creation code to using the DOM.
429
430         * PrettyPatch/PrettyPatch.rb:
431         * code-review.js:
432
433 2011-01-12  Ojan Vafai  <ojan@chromium.org>
434
435         Reviewed by Adam Barth.
436
437         review tool formatted diff doesn't match the uploaded diff
438         https://bugs.webkit.org/show_bug.cgi?id=51960
439
440         PrettyPatch works just fine if there is no newline at the end of the file.
441
442         * attachment.cgi:
443
444 2011-01-12  Ojan Vafai  <ojan@chromium.org>
445
446         Reviewed by Adam Barth.
447
448         Bugzilla: Add keyboard shortcuts to jump to next change
449         https://bugs.webkit.org/show_bug.cgi?id=52305
450
451         Comments and diff blocks go in the same queue. If you have a 
452         comment focused, then j/k will focus the next/prev diff block
453         with respect to that comment.
454
455         * PrettyPatch/PrettyPatch.rb:
456         * code-review.js:
457
458 2011-01-12  Ojan Vafai  <ojan@chromium.org>
459
460         Reviewed by Adam Barth.
461
462         show shared lines on both sides in code review tool
463         https://bugs.webkit.org/show_bug.cgi?id=52308
464
465         We were appending the same DOM node twice. Instead, make
466         the from column a clone of the original node.
467
468         * code-review.js:
469
470 2011-01-12  Ojan Vafai  <ojan@chromium.org>
471
472         Reviewed by Adam Barth.
473
474         fix wrapping bug in expansion lines in the code review tool
475         https://bugs.webkit.org/show_bug.cgi?id=52270
476
477         * PrettyPatch/PrettyPatch.rb:
478         * code-review.js:
479
480 2011-01-11  Ojan Vafai  <ojan@chromium.org>
481
482         Reviewed by Adam Barth.
483
484         fix difflink centering to be vertical, not horizontal
485         https://bugs.webkit.org/show_bug.cgi?id=52263
486
487         * PrettyPatch/PrettyPatch.rb:
488         * code-review.js:
489
490 2011-01-11  Ojan Vafai  <ojan@chromium.org>
491
492         Reviewed by Adam Barth.
493
494         remember diffstate for review tool
495         https://bugs.webkit.org/show_bug.cgi?id=52253
496
497         If you use the global sidebyside/unified links, store the diff type
498         in localstorage. Then, onload, use that diff type. That way, people
499         who prefer one diff type or the other can always get that by default.
500
501         * code-review.js:
502
503 2011-01-11  Ojan Vafai  <ojan@chromium.org>
504
505         Reviewed by Adam Barth.
506
507         allow sidebysideifying individual files in the code review tool
508         https://bugs.webkit.org/show_bug.cgi?id=52226
509
510         * code-review.js:
511
512 2011-01-11  Ojan Vafai  <ojan@chromium.org>
513
514         Reviewed by Mihai Parparita.
515
516         fix exception when adding a comment to a side-by-side diff
517         https://bugs.webkit.org/show_bug.cgi?id=52240
518
519         prev() and next() won't get the previous and next lines in
520         side-by-side mode. Instead do a query to find them.
521
522         * code-review.js:
523
524 2011-01-11  Ojan Vafai  <ojan@chromium.org>
525
526         Reviewed by Mihai Parparita.
527
528         maintain word diffs when converting to side-by-side and back
529         https://bugs.webkit.org/show_bug.cgi?id=52244
530
531         Using textContent would lose the spans use to make the word-diff highlighting.
532         Just move the span element itself.
533
534         * code-review.js:
535
536 2011-01-11  Ojan Vafai  <ojan@chromium.org>
537
538         Reviewed by Adam Barth.
539
540         fix layout error with comments in code review tool
541         https://bugs.webkit.org/show_bug.cgi?id=52230
542
543         * PrettyPatch/PrettyPatch.rb:
544
545 2011-01-10  Ojan Vafai  <ojan@chromium.org>
546
547         Reviewed by Adam Barth.
548
549         convert back to unified from sidebyside diff
550         https://bugs.webkit.org/show_bug.cgi?id=52180
551
552         Remove url fragment stuff. Having it be per-filediff is too complicated.
553
554         * PrettyPatch/PrettyPatch.rb:
555         * code-review.js:
556
557 2011-01-07  Adam Barth  <abarth@webkit.org>
558
559         Rubber-stamped by Eric Seidel.
560
561         Move WebCore to Source
562         https://bugs.webkit.org/show_bug.cgi?id=52050
563
564         Remove reference to old directory.
565
566         * PrettyPatch/PrettyPatch.rb:
567
568 2011-01-06  Ojan Vafai  <ojan@chromium.org>
569
570         Reviewed by Adam Barth.
571
572         side-by-side diffs in the code review tool
573         https://bugs.webkit.org/show_bug.cgi?id=52019
574
575         Support for conversion from the formatted diff to a side-by-side diff.
576         Maintains comments and new comments can be added.
577
578         The main architectural change is that Line elements are no longer necessarily
579         siblings. Each physical line is now in a LineContainer and LineContainers are
580         siblings. Each Line corresponds to a Line in the unified diff and has an id (e.g. line12).
581         A Line can be a LineContainer or a child of a LineContainer.
582
583         In this way, converting to side-by-side and, in the future, back to unified is non-lossy.
584
585         * PrettyPatch/PrettyPatch.rb:
586         * code-review.js:
587
588 2011-01-06  Ojan Vafai  <ojan@chromium.org>
589
590         Fix line context when replying to comments.
591
592         * code-review.js:
593
594 2011-01-05  Ojan Vafai  <ojan@chromium.org>
595
596         Reviewed by Adam Barth.
597
598         change the way we do comment highlighting in the code review tool
599         https://bugs.webkit.org/show_bug.cgi?id=51971
600
601         Store a space-separated list of base line IDs on each line that has
602         comments associated with that line. This allows for overlapping comments,
603         but more importantly, makes adding side-by-side diff support easier.
604
605         * code-review.js:
606
607 2011-01-05  Ojan Vafai  <ojan@chromium.org>
608
609         Reviewed by Adam Barth.
610
611         minor code cleanup for code review tool
612         https://bugs.webkit.org/show_bug.cgi?id=51962
613
614         Consolidates some queries to using shared functions.
615
616         * code-review.js:
617
618 2011-01-05  Ojan Vafai  <ojan@chromium.org>
619
620         Reviewed by Adam Barth.
621
622         assorted cleanups to prepare for side-by-side diffing
623         https://bugs.webkit.org/show_bug.cgi?id=51961
624
625         Mostly, use selector queries instead of assuming comments are next siblings.
626
627         * code-review.js:
628
629 2011-01-05  Ojan Vafai  <ojan@chromium.org>
630
631         Reviewed by Adam Barth.
632
633         fix message container selector for code review tool
634         https://bugs.webkit.org/show_bug.cgi?id=51959
635
636         * code-review.js:
637
638 2011-01-04  Ojan Vafai  <ojan@chromium.org>
639
640         Increment version number to avoid getting the cached JS file.
641
642         * PrettyPatch/PrettyPatch.rb:
643
644 2010-12-15  Ojan Vafai  <ojan@chromium.org>
645  
646         Reviewed by Adam Barth.
647
648         anchor the toolbar to the bottom of the diff if the diff doesn't take a full screen of height
649         https://bugs.webkit.org/show_bug.cgi?id=51162
650
651         * PrettyPatch/PrettyPatch.rb:
652         * code-review.js:
653
654 2011-01-01  Adam Barth  <abarth@webkit.org>
655
656         Reviewed by Eric Seidel.
657
658         Move JavaScriptCore to Source
659         https://bugs.webkit.org/show_bug.cgi?id=51604
660
661         * PrettyPatch/PrettyPatch.rb:
662           - Remove reference to JavaScriptCore as a source directory.
663
664 2011-01-01  Adam Barth  <abarth@webkit.org>
665
666         Reviewed by Eric Seidel.
667
668         Move Sources to Source
669         https://bugs.webkit.org/show_bug.cgi?id=51794
670
671         * PrettyPatch/PrettyPatch.rb:
672
673 2010-12-31  Adam Barth  <abarth@webkit.org>
674
675         Rubber-stamped by Eric Seidel.
676
677         Move PageLoadTests to PerformanceTests/PageLoad
678         https://bugs.webkit.org/show_bug.cgi?id=51771
679
680         Update list of top-level open-source directories.
681
682         * PrettyPatch/PrettyPatch.rb:
683
684 2010-12-31  Adam Barth  <abarth@webkit.org>
685
686         Rubber-stamped by Eric Seidel.
687
688         Move SunSpider into PerformanceTests
689         https://bugs.webkit.org/show_bug.cgi?id=51769
690
691         Update list of top-level open-source directories.
692
693         * PrettyPatch/PrettyPatch.rb:
694
695 2010-12-26  Adam Barth  <abarth@webkit.org>
696
697         Reviewed by Eric Seidel.
698
699         Move autotools into Sources
700         https://bugs.webkit.org/show_bug.cgi?id=51630
701
702         * PrettyPatch/PrettyPatch.rb:
703           - Remove reference to non-existant autotools directory.
704
705 2010-12-26  Adam Barth  <abarth@webkit.org>
706
707         Reviewed by Eric Seidel.
708
709         Move cmake into Sources
710         https://bugs.webkit.org/show_bug.cgi?id=51631
711
712         * PrettyPatch/PrettyPatch.rb:
713
714 2010-12-23  Adam Barth  <abarth@webkit.org>
715
716         Rubber-stamped by Eric Seidel.
717
718         Move JavaScriptGlue into Sources
719         https://bugs.webkit.org/show_bug.cgi?id=51583
720
721         PrettyPatch keeps a list of the top-level directories that exist in
722         trac.
723
724         * PrettyPatch/PrettyPatch.rb:
725
726 2010-12-20  Adam Barth  <abarth@webkit.org>
727
728         One more tweak caused by moving the web sites around.  I had this
729         change in my patch originally, but I missed it because I had to use
730         server-side svn move commands to move the bulk of the web site files.
731
732         * PrettyPatch/PrettyPatch.rb:
733
734 2010-12-18  Adam Barth  <abarth@webkit.org>
735
736         Reviewed by Sam Weinig.
737
738         Move WebKitExamplePlugins to Examples
739         https://bugs.webkit.org/show_bug.cgi?id=51291
740
741         * PrettyPatch/PrettyPatch.rb:
742
743 2010-12-17  Dan Bernstein  <mitz@apple.com>
744
745         Reviewed by Simon Fraser.
746
747         Rename WebKitTools to Tools
748         https://bugs.webkit.org/show_bug.cgi?id=49861
749
750         * PrettyPatch/PrettyPatch.rb:
751         * committers-autocomplete.js:
752
753 2010-12-17  Ojan Vafai  <ojan@chromium.org>
754
755         Increment psuedo-version number to avoid pulling cached version.
756
757         * PrettyPatch/PrettyPatch.rb:
758
759 2010-12-17  Ojan Vafai  <ojan@chromium.org>
760
761         Speculative fix for the code review tool in Firefox.
762         Fixes a JS error and makes the status bubble mostly work
763         if postMessage is not supported.
764
765         * PrettyPatch/PrettyPatch.rb:
766         * code-review.js:
767
768 2010-12-15  Ojan Vafai  <ojan@chromium.org>
769
770         Reviewed by Adam Barth.
771
772         size status bubble to it's contents on the code review page
773         https://bugs.webkit.org/show_bug.cgi?id=51142
774
775         * PrettyPatch/PrettyPatch.rb:
776         * code-review.js:
777
778 2010-12-15  Ojan Vafai  <ojan@chromium.org>
779
780         Reviewed by Adam Barth.
781
782         fix goofups from r74142 and r74130
783         https://bugs.webkit.org/show_bug.cgi?id=51146
784
785         Accidentally shrank the inline comment boxes and made
786         the toolbar not show up when there were image diffs.
787
788         * PrettyPatch/PrettyPatch.rb:
789         * code-review.js:
790
791 2010-12-14  Ojan Vafai  <ojan@chromium.org>
792
793         Reviewed by Adam Barth.
794
795         add ability to view for file context to the review tool
796         https://bugs.webkit.org/show_bug.cgi?id=51057
797
798         At the beginning/end of each file diff and between each
799         hunk add links to expand the context. For now it grabs the
800         tip of tree version of the file and tries to apply the diff 
801         to that file. If it can't apply, then it gives up as we
802         wouldn't want to show the wrong lines of context.
803
804         In the future, we can consider adding the upload svn revision
805         to the diff itself, then we could fallback to the file at that
806         revision if tip of tree doesn't apply.
807
808         * PrettyPatch/PrettyPatch.rb:
809         * code-review.js:
810
811 2010-12-08  Ojan Vafai  <ojan@chromium.org>
812
813         Reviewed by Adam Barth.
814
815         [reviewtool] should always show overall comments text box
816         https://bugs.webkit.org/show_bug.cgi?id=45870
817
818         * PrettyPatch/PrettyPatch.rb:
819         * code-review.js:
820
821 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
822
823         Reviewed by Adam Barth.
824
825         Increase size of status bubbles
826         https://bugs.webkit.org/show_bug.cgi?id=50496
827
828         The current available space isn't enough to display all
829         pending results including position in the queue.
830
831         * PrettyPatch/PrettyPatch.rb:
832
833 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
834
835         Reviewed by Adam Barth.
836
837         Increase size of status bubbles
838         https://bugs.webkit.org/show_bug.cgi?id=50496
839
840         The current available space isn't enough to display all
841         pending results including position in the queue.
842
843         * template/en/custom/attachment/edit.html.tmpl:
844         * template/en/custom/attachment/list.html.tmpl:
845         * template/en/custom/attachment/reviewform.html.tmpl:
846
847 2010-11-08  Adam Barth  <abarth@webkit.org>
848
849         Update help text to match behavior change.
850
851         * code-review.js:
852
853 2010-11-08  Adam Barth  <abarth@webkit.org>
854
855         Reviewed by Eric Seidel.
856
857         Restore normal meaning of double-click on patch review page?
858         https://bugs.webkit.org/show_bug.cgi?id=47641
859
860         After this patch, you need to click on the line numbers to add a
861         comment.
862
863         * PrettyPatch/PrettyPatch.rb:
864         * code-review.js:
865
866 2010-10-09  Adam Barth  <abarth@webkit.org>
867
868         Reviewed by Alexey Proskuryakov.
869
870         Exception in committers-autocomplete in Firefox
871         https://bugs.webkit.org/show_bug.cgi?id=47456
872
873         Remove dependency on a WebKit-only quirk.  IMHO, this quirk is bad for
874         the long-term health of the web, but that's a discussion for another
875         patch.
876
877         * committers-autocomplete.js:
878
879 2010-10-08  Adam Barth  <abarth@webkit.org>
880
881         Reviewed by Oliver Hunt.
882
883         Bugmail for new attachments should link to the reviewtool
884         https://bugs.webkit.org/show_bug.cgi?id=47440
885
886         As requested by olliej himself.  This patch was constructed by pure
887         reason.  I have no way to actually run this code.
888
889         * Bugzilla/BugMail.pm:
890
891 2010-09-28  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
892
893         Reviewed by Csaba Osztrogonác.
894
895         Convert verbatim unicode values in comitter names to characters
896
897         * committers-autocomplete.js:
898
899 2010-09-24  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
900
901         Reviewed by Csaba Osztrogonác.
902
903         Support Unicode-strings in committers-autocomplete script
904
905         * committers-autocomplete.js:
906
907 2010-09-22  Adam Barth  <abarth@webkit.org>
908
909         Reviewed by John Sullivan.
910
911         New review-page design doesn't include name of reviewer
912         https://bugs.webkit.org/show_bug.cgi?id=46271
913
914         * PrettyPatch/PrettyPatch.rb:
915         * code-review.js:
916
917 2010-09-22  Adam Barth  <abarth@webkit.org>
918
919         Fix typo in CSS.
920
921         * PrettyPatch/PrettyPatch.rb:
922
923 2010-09-22  Adam Barth  <abarth@webkit.org>
924
925         Reviewed by Eric Seidel.
926
927         Add link to bug to review page
928         https://bugs.webkit.org/show_bug.cgi?id=46192
929
930         To make room for the link, I moved the help text to the top of the page
931         and tweaked the language to help folks discover that you can select
932         context using the line numbers.
933
934         Also, move more text to sans-serif since that fits in better with the
935         pretty-patch fonts.
936
937         * PrettyPatch/PrettyPatch.rb:
938         * code-review.js:
939
940 2010-09-21  Adam Barth  <abarth@webkit.org>
941
942         Unreviewed.
943
944         [reviewtool] Publish button doesn't work
945         https://bugs.webkit.org/show_bug.cgi?id=46168
946
947         Darin's recent patch had a runtime error.  Sadly, we don't have any
948         testing for this code.
949
950         * code-review.js:
951
952 2010-09-20  Darin Adler  <darin@apple.com>
953
954         Reviewed by Adam Barth.
955
956         Add Preview button and link to bug to patch review bar
957         https://bugs.webkit.org/show_bug.cgi?id=46153
958
959         * PrettyPatch/PrettyPatch.rb: Bump version number to make caches happy.
960         * code-review.js: Added Preview button and link to bug.
961
962 2010-09-19  Adam Barth  <abarth@webkit.org>
963
964         Reviewed by Eric Seidel.
965
966         [reviewtool] Publish button should actually publish the review comments
967         https://bugs.webkit.org/show_bug.cgi?id=46074
968
969         Instead of showing the confirmation lightbox, the Publish button now
970         publishes the comments directly.  If there's demand for a "Preview"
971         button, we can find a way to add that feature.
972
973         * code-review.js:
974
975 2010-09-19  Adam Barth  <abarth@webkit.org>
976
977         Reviewed by Eric Seidel.
978
979         Move reviewtool to action=review
980         https://bugs.webkit.org/show_bug.cgi?id=46071
981
982         This patch removes the old Review Patch screen and moves the new
983         reviewtool from Pretty Diff to Review Patch.
984
985         * attachment.cgi:
986         * code-review.js:
987
988 2010-09-19  Adam Barth  <abarth@webkit.org>
989
990         Review tool UI tweak.  This patch lengthens the status bubbles so all
991         the bubble fit (even when they have numbers inside).
992
993         * PrettyPatch/PrettyPatch.rb:
994
995 2010-09-19  Dan Bernstein  <mitz@apple.com>
996
997         Reviewed by Anders Carlsson.
998
999         Fixed a typo.
1000
1001         * code-review.js: Changed the label of the OK button from Ok to OK.
1002
1003 2010-09-15  Adam Barth  <abarth@webkit.org>
1004
1005         Unreviewed.
1006
1007         Only show the review-in-context link if there are contextual comments.
1008
1009         * code-review.js:
1010
1011 2010-09-15  Adam Barth  <abarth@webkit.org>
1012
1013         Reviewed by Eric Seidel.
1014
1015         [reviewtool] Show the status bubbles on the toolbar
1016         https://bugs.webkit.org/show_bug.cgi?id=45861
1017
1018         * PrettyPatch/PrettyPatch.rb:
1019         * code-review.js:
1020
1021 2010-09-15  Adam Barth  <abarth@webkit.org>
1022
1023         Reviewed by Eric Seidel.
1024
1025         [reviewtool] Allow setting review and commit-queue flags from main review screen
1026         https://bugs.webkit.org/show_bug.cgi?id=45860
1027
1028         This patch adds drop down menus for adjusting the review and
1029         commit-queue flags from the main review screen.  These controls don't
1030         appear immediately because we need to read the state of the flags off
1031         the details page.
1032
1033         It's a slight sadness that bugzilla doesn't have a clean way of
1034         identifying which select control is associated with a given flag.  The
1035         flags seem to have some sort of id, but it's unclear to me how to map
1036         that id back to a notion of "review" or "commit-queue".  Instead, we
1037         look for some magic strings in the title.  I'm sure this will come back
1038         to bit us at some point, but I'm not sure what to do that's better.
1039         (The webkitpy solution to this problem is to assume that the flags are
1040         in a given order, but this seems better.)
1041
1042         * PrettyPatch/PrettyPatch.rb:
1043             - Bump version number to bust throuh caches.
1044         * code-review.js:
1045
1046 2010-09-15  Adam Barth  <abarth@webkit.org>
1047
1048         Reviewed by Eric Seidel.
1049
1050         [reviewtool] Comments should quote previous comments on the same line
1051         https://bugs.webkit.org/show_bug.cgi?id=45847
1052
1053         Now when you reply to a previous comment, the tool will quote the
1054         previous comment in the bugs.webkit.org post.  This makes it eaiser for
1055         folks following along in email to understand the discussion.
1056
1057         While I was editing this code, I also cleaned up some of the whitespace
1058         handling in comments.
1059
1060         * code-review.js:
1061
1062 2010-09-15  Adam Barth  <abarth@webkit.org>
1063
1064         Minor UI tweak to the review tool.  We want to display newlines in
1065         previous and frozen comments instead of collapsing them.
1066
1067         * PrettyPatch/PrettyPatch.rb:
1068
1069 2010-09-14  Adam Barth  <abarth@webkit.org>
1070
1071         Reviewed by Eric Seidel.
1072
1073         bugs.webkit.org should autocomplete WebKit Committers and Reviewers
1074         https://bugs.webkit.org/show_bug.cgi?id=45799
1075
1076         This patch implements an autocomplete dropdown that helps users type
1077         the names of WebKit committers and reviewers.  The script grabs the
1078         data from SVN now that wms set up CORS for http://svn.webkit.org.
1079         Security technology for the win.
1080
1081         This script is based on (well, essentially copied from) the popular
1082         autocomplete extension:
1083
1084         https://chrome.google.com/extensions/detail/olaabhcgdogcbcoiolomlcodkngnemfb
1085
1086         I asked Ojan, and said he was fine licensing the code under the BSD
1087         license.  I'm not sure how the patch interacts with the extension, but
1088         we can retire the extension if they interact poorly.
1089
1090         * committers-autocomplete.js: Added.
1091         * template/en/default/global/footer.html.tmpl:
1092             - Add the script to every page on bugs.webkit.org
1093
1094 2010-09-14  Adam Barth  <abarth@webkit.org>
1095
1096         Add a license block to the reviewtool.  I should have added this from
1097         the start.  According to the SVN log, I'm the only one to have ever
1098         touched this file.
1099
1100         * code-review.js:
1101
1102 2010-09-07  Adam Barth  <abarth@webkit.org>
1103
1104         Minor tweaks to the reviewtool UI.
1105
1106         Rename Cancel to Discard to make it more clear what the button does.
1107         Also, move the style declaration for the focus ring later in the sheet
1108         so it works.
1109
1110         * PrettyPatch/PrettyPatch.rb:
1111         * code-review.js:
1112
1113 2010-09-06  Adam Barth  <abarth@webkit.org>
1114
1115         Small UI tweak to the reviewtool.  Make the Ok/Cancel buttons a bit
1116         bigger.
1117
1118         * PrettyPatch/PrettyPatch.rb:
1119
1120 2010-09-06  Adam Barth  <abarth@webkit.org>
1121
1122         Reviewed by Eric Seidel.
1123
1124         [reviewtool] Add a quick animation for opening/closing comment boxes
1125         https://bugs.webkit.org/show_bug.cgi?id=45278
1126
1127         The 200ms animation slides in and out the line-by-line comment box.
1128
1129         * code-review.js:
1130
1131 2010-09-06  Adam Barth  <abarth@webkit.org>
1132
1133         Reviewed by Eric Seidel.
1134
1135         [reviewtool] Add a field for overall comments
1136         https://bugs.webkit.org/show_bug.cgi?id=45273
1137
1138         This patch does a couple logically separate things that could be
1139         separated into smaller patches:
1140
1141         1) This patch adds an "overall comments" field where you can enter
1142            overall comments about the patch.  These comments appear at the top
1143            of the bugzilla posting.  Currently, these aren't redisplayed when
1144            viewing the patch, but I plan to add that in a future patch.
1145
1146         2) This patch renames some of the CSS classes to more consistently
1147            follow the camelCase style that PrettyPatch uses.
1148
1149         3) This patch moves the "prepare comments" button to the left of the
1150            toolbar and renames is to "publish comments".  This makes more sense
1151            when you scroll to the bottom of the page and enter in some overall
1152            comments.
1153
1154         4) When you attempt to add a comment to a line that already has a
1155            "frozen" comment, we now unfreeze the comment instead of doing
1156            nothing.  The old behavior was kind of frustrating if you didn't
1157            know that you could unfreeze a comment by clicking on it.
1158
1159         * PrettyPatch/PrettyPatch.rb:
1160             - Update CSS.
1161         * code-review.js:
1162
1163 2010-09-06  Adam Barth  <abarth@webkit.org>
1164
1165         [reviewtool] Tweak the ok button to cancel the comment if the comment
1166         is empty.  Previously we would get into a bad state where a line had a
1167         comment but there was no longer any way to access it.
1168
1169         * code-review.js:
1170
1171 2010-09-06  Adam Barth  <abarth@webkit.org>
1172
1173         Reviewed by Eric Seidel.
1174
1175         [reviewtool] Add an "ok" button that collapses review comments
1176         https://bugs.webkit.org/show_bug.cgi?id=45255
1177
1178         Many folks (including myself) are too trigger-happy with the "delete"
1179         button that we end up deleting perfectly good review comments.  This
1180         patch adds an "ok" button to absorb these clicks (and renames "delete"
1181         to "cancel").  When you click the ok button, the comment becomes
1182         non-editable and the box shrinks to fit the text.  You can click the
1183         text to get back to the editable version.
1184
1185         * code-review.js:
1186
1187 2010-08-31  Adam Barth  <abarth@webkit.org>
1188
1189         Reviewed by Eric Seidel.
1190
1191         [reviewtool] Make it easy to scroll through review comments
1192         https://bugs.webkit.org/show_bug.cgi?id=45002
1193
1194         This patch lets you scroll through review comments using "n" (for next)
1195         and "p" (for previous).  It also attributes comments to their authors.
1196
1197         * PrettyPatch/PrettyPatch.rb:
1198         * code-review.js:
1199
1200 2010-08-31  Adam Barth  <abarth@webkit.org>
1201
1202         Reviewed by Eric Seidel.
1203
1204         [reviewtool] Show previous comments inline in diff
1205         https://bugs.webkit.org/show_bug.cgi?id=44977
1206
1207         This patch adds basic support for showing previous comments inline in
1208         the diff.  We crawl the bugs.webkit.org comments about this attachment
1209         and extract comments related to specific lines.  We then show the
1210         comments inline in the diff.
1211
1212         This part of the tool needs a bunch of polish, but this at least is a
1213         starting point for further work.
1214
1215         * PrettyPatch/PrettyPatch.rb:
1216         * code-review.js:
1217
1218 2010-08-31  Adam Barth  <abarth@webkit.org>
1219
1220         Reviewed by Eric Seidel.
1221
1222         [reviewtool] Allow reviewer to select lines of context by dragging over the line numbers
1223         https://bugs.webkit.org/show_bug.cgi?id=44936
1224
1225         * PrettyPatch/PrettyPatch.rb:
1226             - Add a version number to bust through bugs.webkit.org's aggressive
1227               cacheing.
1228         * code-review.js:
1229
1230 2010-08-30  Adam Barth  <abarth@webkit.org>
1231
1232         Reviewed by Eric Seidel.
1233
1234         [review tool] Let reviewer select how much context to show in snippet
1235         https://bugs.webkit.org/show_bug.cgi?id=44905
1236
1237         We now highlight the context for a comment in yellow on the left (where
1238         the line numbers are).  Clicking a line number expands or contracts the
1239         amount of context, as appropriate.  Informal user testing indicates
1240         that we might want to support drag as well.
1241
1242         This patch also changes the "open a comment box here" action to
1243         double-click to avoid issues with mis-clicks.
1244
1245         * PrettyPatch/PrettyPatch.rb:
1246         * code-review.js:
1247
1248 2010-08-29  Adam Barth  <abarth@webkit.org>
1249
1250         Attempt to make Sam's life easier by not opening a comment text field
1251         if there's a selection.  This should make it easier to copy/paste text
1252         out of the diff without accidentally opening comment boxes.
1253
1254         * code-review.js:
1255
1256 2010-08-29  Adam Barth  <abarth@webkit.org>
1257
1258         Turns out we're supposed to use find instead of children here.
1259
1260         * code-review.js:
1261
1262 2010-08-29  Adam Barth  <abarth@webkit.org>
1263
1264         Reviewed by Eric Seidel.
1265
1266         Awesomify pretty-diff based review tool
1267         https://bugs.webkit.org/show_bug.cgi?id=44835
1268
1269         This patch replaces the existing inline comment feature with a new
1270         implementation.  Hopefully the new implementation is more awesome.
1271         It's not 100% done yet, but I'd like to get this version out there to
1272         see how it works.
1273
1274         Unlike the previous version this version works with the "Formatted
1275         Diff" link instead of the "Review Patch" link.  Hopefully that will
1276         avoid interfering with folks who like the old style review.
1277
1278         * PrettyPatch/PrettyPatch.rb:
1279         * code-review.js: Added.
1280         * prototype.js: Removed.
1281
1282 2010-08-19  Tony Chang  <tony@chromium.org>
1283
1284         Reviewed by Adam Barth.
1285
1286         webkit-patch barfed on upload with a new image test result
1287         https://bugs.webkit.org/show_bug.cgi?id=39097
1288
1289         * PrettyPatch/PrettyPatch.rb: don't use full path to git
1290
1291 2010-08-19  Tony Chang  <tony@chromium.org>
1292
1293         Unreviewed, setting svn:eol-style native on ChangeLog.
1294
1295 2010-08-19  Tony Chang  <tony@chromium.org>
1296
1297         Unreviewed.  Just removing \r's from ChangeLog.
1298
1299 2010-08-02  Adam Roben  <aroben@apple.com>
1300
1301         Tell the patch prettifier about some (not so) new directories
1302
1303         * PrettyPatch/PrettyPatch.rb: Added WebKit2, autotools, and cmake
1304         directories.
1305
1306 2010-07-27  Joseph Pecoraro  <joepeck@webkit.org>
1307
1308         Reviewed by David Kilzer.
1309
1310         Provide a Helpful Link for Web Inspector Bugs in Bugzilla
1311         https://bugs.webkit.org/show_bug.cgi?id=43065
1312
1313         * template/en/custom/global/choose-product.html.tmpl: Added. Explicitly added a link for the Web Inspector.
1314
1315 2010-06-09  Julie Parent  <jparent@chromium.org>
1316
1317         Reviewed by David Kilzer.
1318
1319         Bugzilla: Don't send mail for cancel/deny/granted for in-rietveld.
1320
1321         https://bugs.webkit.org/show_bug.cgi?id=40345
1322
1323         * Bugzilla/Flag.pm:
1324
1325 2010-05-17  Julie Parent  <jparent@chromium.org>
1326
1327         Reviewed by Ojan Vafai.
1328
1329         Rietveld review page should dedicate more space to Rietveld, less to high level comments form.
1330
1331         https://bugs.webkit.org/show_bug.cgi?id=39244
1332
1333         * template/en/custom/attachment/reviewform.html.tmpl:
1334         Delete a random BR amongst the hidden elements that was causing extra whitespace.
1335         * template/en/custom/attachment/rietveldreview.html.tmpl:
1336         Increase size of Rietveld frame to 80%, from 60%.
1337
1338 2010-05-14  Julie Parent  <jparent@chromium.org>
1339
1340         Reviewed by Ojan Vafai.
1341
1342         Bugzilla: Only show "Rietveld Review" link when in-rietveld is set to +.
1343         
1344         https://bugs.webkit.org/show_bug.cgi?id=39141
1345
1346         * template/en/custom/attachment/list.html.tmpl:
1347
1348 2010-05-13  Julie Parent  <jparent@chromium.org>
1349
1350         Reviewed by David Kilzer.
1351
1352         Bugzilla: Don't send mail if the only change is to the in-rietveld flag.
1353         
1354         https://bugs.webkit.org/show_bug.cgi?id=39090
1355
1356         * Bugzilla/BugMail.pm:
1357
1358 2010-05-12  Julie Parent  <jparent@chromium.org>
1359
1360         Reviewed by Ojan Vafai.
1361
1362         Hide the in-rietveld flag in the UI.  It is only used only by tooling and shouldn't clutter the UI.
1363         https://bugs.webkit.org/show_bug.cgi?id=39024
1364
1365         * template/en/custom/attachment/list.html.tmpl:
1366           Removes this flag entirely from the UI, not needed on this page.
1367         * template/en/custom/flag/list.html.tmpl: Added.
1368           Copied from template/en/default/flag/list.html.tmpl with only changes inside the if WEBKIT_CHANGES section.
1369           Hides the flag rather than removing it completely, so tools can still interact with it.  
1370
1371 2010-05-12  Julie Parent  <jparent@chromium.org>
1372
1373         Reviewed by David Kilzer.
1374
1375         Adds support for Rietveld reviews.  When there is an associated
1376         Rietveld id for a patch, adds a link to do a Rietveld review instead.
1377
1378         https://bugs.webkit.org/show_bug.cgi?id=38143
1379
1380         * attachment.cgi:
1381             Added support for handling action=rietveldreview.
1382         * template/en/custom/attachment/list.html.tmpl:
1383             Adds a link to use rietveld review iff the in_rietveld flag is set.
1384         * template/en/custom/attachment/reviewform.html.tmpl:
1385             Detects if we are in rietveld review mode due to query string paramter,
1386             and customizes the review form slightly:
1387                 - Changes text describing comment field.
1388                 - Comment field does not include quoted patch.
1389                 - Submit button fetches from Rietveld instead of just submitting.
1390         * template/en/custom/attachment/rietveldreview.html.tmpl: Added.
1391             New template for rietveldreview mode.  Hosts the rietveld frame in the
1392             top and the regular comments form in the bottom.
1393             Uses postMessage to communicate with Rietveld frame.
1394
1395 2010-05-08  Chris Jerdonek  <cjerdonek@webkit.org>
1396
1397         Reviewed by Maciej Stachowiak.
1398
1399         Allow long comment lines to wrap to the window width instead
1400         of to 80 characters.
1401
1402         https://bugs.webkit.org/show_bug.cgi?id=37792
1403
1404         * Bugzilla/Constants.pm:
1405           - Introduced a new constant COMMENT_COLS_WRAP to replace COMMENT_COLS
1406             inside the wrap_comment() subroutine.  The new constant effectively
1407             disables text wrapping by increasing the value used in wrap_comment()
1408             from 80 characters to a very large value (8000).
1409         * Bugzilla/Util.pm:
1410           - Replaced COMMENT_COLS with COMMENT_COLS_WRAP inside wrap_comment().
1411         * skins/custom/global.css:
1412         (.bz_comment_text):
1413           - Eliminate the width styling on comments to allow long lines
1414             to wrap at the window width.
1415         (.bz_comment_text > span.quote):
1416           - Inherit the "white-space" property.  This prevents long quoted
1417             lines in comments from causing non-quoted lines to extend beyond
1418             the window width.
1419
1420 2010-04-22  Adam Barth  <abarth@webkit.org>
1421
1422         Reviewed by Darin Adler.
1423
1424         The new review tools shouldn't clear the comment box automatically
1425         https://bugs.webkit.org/show_bug.cgi?id=38004
1426
1427         Instead, we now have a button for doing that explicitly.  If people
1428         like this tool, we can look at fancier solutions.
1429
1430         Also, tried to make the tool work on the edit page as well as the review page.
1431
1432         * PrettyPatch/PrettyPatch.rb:
1433
1434 2010-04-22  Adam Barth  <abarth@webkit.org>
1435
1436         Unreviewed.  Turns out we need to do more escaping because Ruby was
1437         explanding our newlines, creating a syntax error.
1438
1439         * PrettyPatch/PrettyPatch.rb:
1440
1441 2010-04-22  Adam Barth  <abarth@webkit.org>
1442
1443         Reviewed by Darin Adler.
1444
1445         Hack up PrettyDiff to allow line-by-line comments
1446         https://bugs.webkit.org/show_bug.cgi?id=37886
1447
1448         Admittedly a bit of a hack, this is a basic line-by-line editor we can
1449         play with thanks to Andrew Scherkus.  It's meant to integrate with the
1450         "review" page at bugs.webkit.org.
1451
1452         I changed a few things from Andrew's original version:
1453         1) Trigger text boxes on single click to make iPhone/iPad reviewing
1454         easier.
1455         2) Clear the main text box on load.
1456         3) Reference a version of prototype.js on bugs.webkit.org.
1457
1458         * PrettyPatch/PrettyPatch.rb:
1459         * prototype.js: Added.
1460
1461 2010-02-22  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
1462
1463         Reviewed by Maciej Stachowiak.
1464
1465         Don't reset the assignee back to default on component change
1466
1467         https://bugs.webkit.org/show_bug.cgi?id=35236
1468
1469         There's a risk of reseting a valid assignee for example when
1470         triaging a bug. All components in WebKit have the default 
1471         assignee set to webkit-unassigned@webkit.org so this should
1472         not cause problems for people relying on this functionality.
1473
1474         * template/en/custom/bug/edit.html.tmpl:
1475
1476 2010-01-22  Janusz Lewandowski  <lew21@xtreeme.org>
1477
1478         Reviewed by Maciej Stachowiak.
1479
1480         run-webkit-tests fails on Ruby 1.9
1481         https://bugs.webkit.org/show_bug.cgi?id=33554
1482         inject does not work with strings any more, so BugsSite/PrettyPatch/PrettyPatch.rb prints lots of errors.
1483
1484         * PrettyPatch/PrettyPatch.rb:
1485
1486 2010-01-20  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
1487
1488         Reviewed by Adam Barth.
1489
1490         Adjust height of status-bubbles to prevent them being cut.
1491
1492         * template/en/custom/attachment/edit.html.tmpl:
1493         * template/en/custom/attachment/list.html.tmpl:
1494         * template/en/custom/attachment/reviewform.html.tmpl:
1495
1496 2010-01-08  David Kilzer  <ddkilzer@apple.com>
1497
1498         <http://webkit.org/b/33410> Fix "View Plain Diff"/"View Formatted Diff" button
1499
1500         Reviewed by Adam Barth.
1501
1502         This button was broken during a previous Bugzilla upgrade after
1503         changes were made to the names of objects exposed on templates.
1504
1505         * template/en/custom/attachment/reviewform.html.tmpl: Changed
1506         attachid to attachment.id and ispatch to attachment.ispatch.
1507         Also tweaked the format to use 4 table columns instead of 5.
1508
1509 2010-01-08  David Kilzer  <ddkilzer@apple.com>
1510
1511         Included UNCONFIRMED bugs by default in advanced search query
1512
1513         Rubber-stamped by Alexey Proskuryakov.
1514
1515         * data/params: Prepended 'bug_status=UNCONFIRMED&' to
1516         'defaultquery' parameter.
1517
1518 2010-01-08  David Kilzer  <ddkilzer@apple.com>
1519
1520         FIX: Make patch status appear on reviewform.html.tmpl
1521
1522         * template/en/custom/attachment/reviewform.html.tmpl: Check
1523         attachment.ispatch (not just ispatch) when deciding whether
1524         to display patch status.  Also update layout a bit.
1525
1526 2010-01-08  David Kilzer  <ddkilzer@apple.com>
1527
1528         Update data/params parameters
1529
1530         * data/params: Updated 'timezone' to turn off DST.  Updated
1531         'usebugaliases' and 'usevotes' to turn them off.
1532
1533 2010-01-08  Adam Barth  <abarth@webkit.org>
1534
1535         Reviewed by Darin Adler.
1536
1537         Show patch status in bugs.webkit.org
1538         https://bugs.webkit.org/show_bug.cgi?id=33368
1539
1540         This patch adds an iframe to the show_bug page and the edit attachment
1541         page that displays whether the attachment passed the various bots.
1542
1543         * template/en/custom/attachment/edit.html.tmpl:
1544         * template/en/custom/attachment/list.html.tmpl:
1545         * template/en/custom/attachment/reviewform.html.tmpl:
1546
1547 2009-12-07  Shinichiro Hamaji  <hamaji@chromium.org>
1548
1549         Reviewed by David Kilzer.
1550
1551         Bugzilla should show images in git patches
1552         https://bugs.webkit.org/show_bug.cgi?id=31395
1553
1554         Attempt to go r51748 again using --directory option of git-apply.
1555
1556         * PrettyPatch/PrettyPatch.rb:
1557
1558 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
1559
1560         Unreviewed. Revert r51748.
1561
1562         Bugzilla should show images in git patches
1563         https://bugs.webkit.org/show_bug.cgi?id=31395
1564
1565         * PrettyPatch/PrettyPatch.rb:
1566
1567 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
1568
1569         Reviewed by David Kilzer.
1570
1571         Bugzilla should show images in git patches
1572         https://bugs.webkit.org/show_bug.cgi?id=31395
1573
1574         Show images in git patches using git-apply.
1575
1576         * PrettyPatch/PrettyPatch.rb:
1577
1578 2009-10-23  Eric Seidel  <eric@webkit.org>
1579
1580         Reviewed by Adam Roben.
1581
1582         PrettyPatch should show images even when they have the wrong mime type
1583         https://bugs.webkit.org/show_bug.cgi?id=29506
1584
1585         * PrettyPatch/PrettyPatch.rb:
1586
1587 2009-10-17  Alexey Proskuryakov  <ap@apple.com>
1588
1589         Reviewed by Eric Carlson.
1590
1591         https://bugs.webkit.org/show_bug.cgi?id=30470
1592         Make marking a bug as duplicate easier
1593
1594         * js/field.js: (setResolutionToDuplicate): Focus the edit field, too.
1595
1596 2009-10-16  Alexey Proskuryakov  <ap@apple.com>
1597
1598         Reviewed by Mark Rowe.
1599
1600         https://bugs.webkit.org/show_bug.cgi?id=30470
1601         Make marking a bug as duplicate easier
1602
1603         * js/field.js: (setResolutionToDuplicate): Show edit field right away, don't make the user
1604         click again.
1605
1606 2009-09-09  David Kilzer  <ddkilzer@apple.com>
1607
1608         Update data/params parameters
1609
1610         * data/params: Updated 'attachment_base' parameter for secure
1611         attachments.  Updated 'mybugstemplate' parameter to search for
1612         bugs in UNCONFIRMED, ASSIGNED and REOPENED status in addition to
1613         NEW status.
1614
1615 2009-08-11  David Kilzer  <ddkilzer@apple.com>
1616
1617         Updated data/params parameter
1618
1619         * data/params: Updated 'loginnetmask' parameter to allow users
1620         to choose whether their cookies are restricted to a single IP
1621         address or not.
1622
1623 2009-07-07  David Kilzer  <ddkilzer@apple.com>
1624
1625         Updated BugsSite with local changes on server
1626
1627         The new files were created by checksetup.pl during the upgrade
1628         to Bugzilla 3.2.3.  The rest are self-explanatory.
1629
1630         * data: Added "bugzilla-update.xml" to svn:ignore.
1631         * data/attachments: Added "groups.*" to svn:ignore.
1632         * data/params: Updated for a few changed parameters.
1633
1634         * lib/.htaccess: Added.
1635         * skins/contrib/Dusk/IE-fixes.css: Added.
1636         * skins/contrib/Dusk/admin.css: Added.
1637         * skins/contrib/Dusk/create_attachment.css: Added.
1638         * skins/contrib/Dusk/dependency-tree.css: Added.
1639         * skins/contrib/Dusk/duplicates.css: Added.
1640         * skins/contrib/Dusk/editusers.css: Added.
1641         * skins/contrib/Dusk/help.css: Added.
1642         * skins/contrib/Dusk/index.css: Added.
1643         * skins/contrib/Dusk/panel.css: Added.
1644         * skins/contrib/Dusk/params.css: Added.
1645         * skins/contrib/Dusk/release-notes.css: Added.
1646         * skins/contrib/Dusk/show_bug.css: Added.
1647         * skins/contrib/Dusk/show_multiple.css: Added.
1648         * skins/contrib/Dusk/summarize-time.css: Added.
1649         * skins/contrib/Dusk/voting.css: Added.
1650         * skins/contrib/Dusk/yui: Added.
1651         * skins/contrib/Dusk/yui/calendar.css: Added.
1652         * skins/custom/IE-fixes.css: Added.
1653         * skins/custom/admin.css: Added.
1654         * skins/custom/buglist.css: Added.
1655         * skins/custom/create_attachment.css: Added.
1656         * skins/custom/dependency-tree.css: Added.
1657         * skins/custom/duplicates.css: Added.
1658         * skins/custom/editusers.css: Added.
1659         * skins/custom/help.css: Added.
1660         * skins/custom/panel.css: Added.
1661         * skins/custom/params.css: Added.
1662         * skins/custom/release-notes.css: Added.
1663         * skins/custom/show_bug.css: Added.
1664         * skins/custom/show_multiple.css: Added.
1665         * skins/custom/summarize-time.css: Added.
1666         * skins/custom/voting.css: Added.
1667         * skins/custom/yui: Added.
1668         * skins/custom/yui/calendar.css: Added.
1669
1670 2009-07-03  David Kilzer  <ddkilzer@apple.com>
1671
1672         Bug 26958: Change edit link to review link in request messages
1673
1674         <https://bugs.webkit.org/show_bug.cgi?id=26958>
1675
1676         Reviewed by Dan Bernstein.
1677
1678         * template/en/custom/request/email.txt.tmpl: Copied from BugsSite/template/en/default/request/email.txt.tmpl.
1679         Changed link from "action=edit" to "action=review".
1680
1681 2009-07-03  David Kilzer  <ddkilzer@apple.com>
1682
1683         Bug 26950: Make the summary and alias fields support click-to-edit
1684         <https://bugs.webkit.org/show_bug.cgi?id=26950>
1685
1686         Reviewed by Maciej Stachowiak.
1687
1688         Original patch by Maciej Stachowiak.
1689
1690         * js/field.js:
1691         (hideEditableField): Updated to add click event listeners to the
1692         alias and short description elements to make them easier to
1693         edit.  Renamed field_id parameter to field2_id (short
1694         description id) and added a field1_id parameter (alias id).
1695         (showEditableField): If a third parameter is passed in the
1696         ContainerInputArray parameter, use it to find the element to
1697         focus.  Otherwise, fall back to the original behavior of
1698         focusing the first input field.
1699         (hideAliasAndSummary): Changed to pass the id for the alias
1700         element to hideEditableField().
1701
1702 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1703
1704         Config file for contrib/recode.pl when upgrading bugs.webkit.org
1705
1706         Current as of midday on Wednesday, July 1, 2009.
1707
1708         * contrib/recode-overrides.txt: Added.
1709
1710 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1711
1712         Workaround for WebKit Bug 9630 when running contrib/recode.pl
1713
1714         A number of WebKit nightly builds included Bug 9630 which caused
1715         non-breaking space characters (0xA0) to be submitted within the
1716         content of textarea elements.
1717
1718         * contrib/recode.pl: To work around these 0xA0 characters, try
1719         an encoding of cp1252 and use it if it works.
1720
1721 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1722
1723         Don't print "Failed to guess" warning if an override is available
1724
1725         * contrib/recode.pl: Check that %overrides does NOT contain a
1726         key matching the current digest before printing out the warning
1727         about a failed charset guess.
1728
1729 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1730
1731         Added --[no-]truncate switch to contrib/recode.pl
1732
1733         * contrib/recode.pl: Added --[no-]truncate switch to make
1734         debugging of failed charset guessing easier.  Often times the
1735         illegal character was truncated in the output.
1736
1737 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1738
1739         Changed Perl scripts to use #!/usr/bin/env perl
1740
1741         perl -pi -e 's@#\!/usr/bin/perl@#\!/usr/bin/env perl@' *.cgi *.pl */*.pl
1742
1743 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1744
1745         Replace Bugzilla favicon with webkit.org favicon
1746
1747         * favicon.ico: Removed.
1748         * images/favicon.ico: Replaced.  Copied from favicon.ico.
1749
1750 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
1751
1752         Merged BugsSite to Bugzilla-3.2.3
1753
1754         Updated to the latest-and-greatest stable version.
1755
1756 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
1757
1758         Merged BugsSite to Bugzilla-3.0.3
1759
1760         Nothing to see here.  Move along.
1761
1762 2009-06-09  Eric Seidel  <eric@webkit.org>
1763
1764         Reviewed by Adam Roben.
1765
1766         Add support for displaying added png files in PrettyPatch diffs
1767         https://bugs.webkit.org/show_bug.cgi?id=26210
1768
1769         Currently this is SVN only (git-send-bugzilla patches exclude binary data)
1770         and only works for PNG files but could easily be made to work for other images as needed.
1771
1772         * PrettyPatch/PrettyPatch.rb:
1773
1774 2009-05-19  Eric Seidel  <eric@webkit.org>
1775
1776         Reviewed by Adam Roben.
1777
1778         Make PrettyPatch understand quoted filenames in git diffs.
1779         https://bugs.webkit.org/show_bug.cgi?id=25881
1780
1781         * PrettyPatch/PrettyPatch.rb:
1782
1783 2009-05-15  Simon Fraser  <simon.fraser@apple.com>
1784
1785         Carrying forwards Darin Adler's rubber stamp
1786         
1787         Make the comment field wider too.
1788
1789         * skins/custom/global.css:
1790
1791 2009-05-14  Simon Fraser  <simon.fraser@apple.com>
1792
1793         Reviewed by Darin Adler
1794         
1795         Make the URL, Summary and Keyword fields wider to mitigate the
1796         effects of https://bugs.webkit.org/show_bug.cgi?id=25566 and make
1797         things generally better.
1798
1799         * skins/custom/global.css:
1800         * template/en/custom/bug/edit.html.tmpl:
1801
1802 2008-11-10  Darin Adler  <darin@apple.com>
1803
1804         - fix more of https://bugs.webkit.org/show_bug.cgi?id=21400
1805           "Edit" links for patches in comments, review queue, review emails
1806           should be replaced by "Review Patch" links
1807
1808         * globals.pl: Fix cases that are automatically generated, such as links
1809         in comments in bugs.
1810
1811         * template/en/custom/attachment/created.html.tmpl: Fix the link on the
1812         "I just created an attachment" page.
1813
1814         * template/en/custom/request/queue.html.tmpl: Fix the link in the queue
1815         page.
1816
1817 2008-11-10  Darin Adler  <darin@apple.com>
1818
1819         * template/en/default/request/queue.html.tmpl: Use review links instead
1820         of edit links in the review queue.
1821
1822 2008-11-10  Darin Adler  <darin@apple.com>
1823
1824         * template/en/custom/request/email.txt.tmpl: Send review links instead
1825         of edit links when flags are set on a bug.
1826
1827 2008-10-27  Darin Adler  <darin@apple.com>
1828
1829         * template/en/custom/attachment/reviewform.html.tmpl: Leave out the
1830         comment if it's untouched.
1831
1832 2008-10-23  Adam Roben  <aroben@apple.com>
1833
1834         Fix Bug 21401: Comments field on "Review Patch" page should be
1835         initially filled with quoted patch
1836
1837         <https://bugs.webkit.org/show_bug.cgi?id=21401>
1838
1839         Reviewed by Dave Hyatt.
1840
1841         * attachment.cgi:
1842         (edit): Retrieve the attachment data from the database instead of just
1843         its length. When the attachment is a patch, create a quotedpatch
1844         template variable that contains the patch data with each line
1845         prepended with "> ".
1846         * template/en/custom/attachment/reviewform.html.tmpl:
1847           - Changed the comments field to have a monospace font
1848           - Added an "Enter comments below:" caption above the comments field
1849             to make it clearer that this is where comments should go, now that
1850             the comments field is not initially empty
1851           - Fill the comments field with the quoted patch
1852
1853 2008-10-14  Adam Roben  <aroben@apple.com>
1854
1855         Fix Bug 21602: Bugzilla times out trying to display formatted diff for
1856         attachment 24345
1857         
1858         https://bugs.webkit.org/show_bug.cgi?id=21602
1859
1860         Reviewed by Dave Kilzer.
1861
1862         * PrettyPatch/PrettyPatch.rb:
1863         (PrettyPatch.BINARY_FILE_MARKER_FORMAT): Added.
1864         (PrettyPatch.FileDiff.initialize): If any of the lines in the diff
1865         are a binary file marker, mark this FileDiff as binary and stop trying
1866         to process the lines.
1867         (PrettyDiff.FileDiff.to_html): If we're binary, just print a string
1868         saying so.
1869
1870 2008-10-02  Adam Roben  <aroben@apple.com>
1871
1872         * template/en/custom/attachment/reviewform.html.tmpl: Fix typo.
1873
1874 2008-10-02  Adam Roben  <aroben@apple.com>
1875
1876         * template/en/custom/attachment/review.html.tmpl: Fix URLs
1877
1878 2008-10-02  Adam Roben  <aroben@apple.com>
1879
1880         Fix Bug 21315: bugs.webkit.org should have a nicer patch review
1881         interface
1882
1883         https://bugs.webkit.org/show_bug.cgi?id=21315
1884
1885         Reviewed by Tim Hatcher.
1886
1887         * attachment.cgi:
1888         (top level): Added support for the "review" and "reviewform" actions.
1889         (sub edit): Accept the template name as a parameter. If no name is
1890         give, use "edit" as before.
1891         * template/en/custom/attachment/list.html.tmpl: Added a "Review Patch"
1892         link for all patch attachments.
1893         * template/en/custom/attachment/review.html.tmpl: Added. Simple
1894         <frameset> page to show the patch on the top and a comment form on the
1895         bottom.
1896         * template/en/custom/attachment/reviewform.html.tmpl: Added. Simple
1897         comment form for reviewing patches.
1898
1899 2008-10-02  Adam Roben  <aroben@apple.com>
1900
1901         Remove references to some backup files I created when implementing
1902         PrettyPatch in Bugzilla
1903
1904         Reviewed by Tim Hatcher.
1905
1906         * attachment-aroben.cgi: Removed.
1907         * template/en/default/attachment/edit-aroben.html.tmpl: Removed.
1908
1909         * template/en/custom/attachment/edit.html.tmpl: Removed an erroneous
1910         reference to attachment-aroben.cgi. attachment.cgi will work just
1911         fine.
1912
1913 2008-06-25  David Kilzer  <ddkilzer@apple.com>
1914
1915         Make PrettyPatch handle diffs with no Index or diff headers
1916
1917         Part of Bug 19290: More patches not handled by PrettyPatch.rb
1918         <https://bugs.webkit.org/show_bug.cgi?id=19290>
1919
1920         Reviewed by Adam.
1921
1922         * PrettyPatch/PrettyPatch.rb:
1923         (PrettyPatch.DIFF_HEADER_FORMATS): Added regular expression to
1924         match on "+++ " lines for patches with no "Index" or "diff" header.
1925         (PrettyPatch.FileDiff.initialize): Look for filename on "+++ " line
1926         when the first line of a patch has no "Index" or "diff" header.
1927         (PrettyPatch.FileDiff.parse): Added haveSeenDiffHeader state
1928         variable to determine when no "Index" or "diff" header has been
1929         found, but a new patch has started with a "--- " line.
1930
1931 2008-06-08  Dan Bernstein  <mitz@apple.com>
1932
1933         Reviewed by Adam Roben.
1934
1935         - update trac URL in PrettyPatch
1936
1937         * PrettyPatch/PrettyPatch.rb:
1938
1939 2008-05-29  Adam Roben  <aroben@apple.com>
1940
1941         Update PrettyPatch directory list
1942
1943         * PrettyPatch/PrettyPatch.rb:
1944
1945 2008-05-28  Adam Roben  <aroben@apple.com>
1946
1947         Make PrettyPatch able to handle diffs taken from outside the WebKit
1948         source tree
1949
1950         Part of Bug 19290: More patches not handled by PrettyPatch.rb
1951         <https://bugs.webkit.org/show_bug.cgi?id=19290>
1952
1953         Reviewed by David Kilzer.
1954
1955         * PrettyPatch/PrettyPatch.rb:
1956         (PrettyPatch.find_url_and_path): Added. Searches the file path from
1957         the bottom up looking for a directory that exists in the source tree.
1958         (PrettyPatch.linkifyFilename): Changed to call find_url_and_path.
1959
1960 2008-05-28  Adam Roben  <aroben@apple.com>
1961
1962         Print exceptions from PrettyPatch to stdout
1963
1964         This will make it much easier to debug problems with PrettyPatch,
1965         since we'll be able to see the exceptions in the browser.
1966
1967         Reviewed by Sam Weinig.
1968
1969         * PrettyPatch/prettify.rb: Added a --html-exceptions option, which
1970         will print exceptions to stdout as HTML.
1971         * attachment.cgi: Changed to pass --html-exceptions to prettify.rb.
1972
1973 2008-04-26  David Kilzer  <ddkilzer@apple.com>
1974
1975         Removed temp files and added appropriate svn:ignore properties.
1976
1977         Rubber-stamped by Mark Rowe.
1978
1979         * data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
1980
1981         * data/template: Added svn:ignore property for "template" subdirectory.
1982         * data/template/template: Removed precompiled template subdirectory.
1983
1984         * data/versioncache: Removed.
1985         * data/versioncache.*: Removed backup versioncache files.
1986
1987         * data/webdot: Added svn:ignore property for "*.dot" files.
1988         * data/webdot/*.dot: Removed cached webdot files.
1989
1990 2008-04-08  Adam Roben  <aroben@apple.com>
1991
1992         Combine :equal operations with the following operation if they are
1993         fewer than 3 characters long
1994
1995         This keeps us from showing lots of small changes on long lines, just
1996         because some letters happened to be the same.
1997
1998         Inspired by
1999         http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/diffviewer/diffutils.py?r=1264#147
2000
2001         * PrettyPatch/PrettyPatch.rb:
2002         (PrettyPatch::DiffSection.initialize): Remove :equal operations fewer
2003         than 3 characters long. The characters from the removed operations
2004         become part of the subsequent operation.
2005         (PrettyPatch::CodeLine.text_as_html): Don't wrap 0-length strings in
2006         <ins>/<del> tags. Also removed the @fromLineNumber.nil? and
2007         @toLineNumber.nil? checks as they are no longer reliable now that
2008         we're removing operations.
2009
2010 2008-04-08  Adam Roben  <aroben@apple.com>
2011
2012         Change PrettyPatch to use DiffBuilder for intra-line diffs
2013
2014         This gives us much prettier intra-line diffs, largely because it can
2015         distinguish multiple changes on the same line. e.g., if a line changes
2016         from:
2017
2018         const int myConstant = 0;
2019
2020         to
2021
2022         static const unsigned myConstant;
2023
2024         You will see that "static " was inserted, "int" changed to "unsigned",
2025         and " = 0" was deleted.
2026
2027         This seems to have also gotten rid of some spurious instances of
2028         "<ins></ins>" and "<del></del>" at the end of a line.
2029
2030         * PrettyPatch/PrettyPatch.rb:
2031         (PrettyPatch::DiffSection.initialize): Use DiffBuilder instead of
2032         setChangeExtentFromLine (which had been copied from Trac's diffing
2033         code).
2034         (PrettyPatch::CodeLine):
2035          - Removed the changeExtent attribute
2036          - Added the operations attribute
2037          - Removed the setChangeExtentFromLine method
2038         (PrettyPatch::CodeLine.text_as_html): Uses @operations to determine
2039         what text to wrap in <ins> and <del> tags.
2040         * PrettyPatch/diff.rb: Deleted a bunch of code we don't use.
2041         (HTMLDiff::DiffBuilder.initialize): Moved code here from the old build
2042         method, but left out the calls to perform_operation, since we build
2043         the diff HTML outside of this class.
2044         (HTMLDiff::DiffBuilder.split_inputs_to_words): Removed calls to
2045         convert_html_to_list_of_words, since we're not diffing HTML.
2046
2047 2008-04-08  Adam Roben  <aroben@apple.com>
2048
2049         Check in diff.rb
2050
2051         This came from
2052         http://instiki.rubyforge.org/svn/instiki/trunk/lib/diff.rb
2053         revision 28521
2054
2055         * PrettyPatch/diff.rb: Added.
2056
2057 2008-03-04  Adam Roben  <aroben@apple.com>
2058
2059         * PrettyPatch/PrettyPatch.rb: Allow the extent of the previous file to
2060         be omitted from the hunk header.
2061