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