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