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