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