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