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