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