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