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