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