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