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