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