2011-01-12 Ojan Vafai <ojan@chromium.org>
[WebKit-https.git] / Websites / bugs.webkit.org / ChangeLog
1 2011-01-12  Ojan Vafai  <ojan@chromium.org>
2
3         Reviewed by Adam Barth.
4
5         review tool formatted diff doesn't match the uploaded diff
6         https://bugs.webkit.org/show_bug.cgi?id=51960
7
8         PrettyPatch works just fine if there is no newline at the end of the file.
9
10         * attachment.cgi:
11
12 2011-01-12  Ojan Vafai  <ojan@chromium.org>
13
14         Reviewed by Adam Barth.
15
16         Bugzilla: Add keyboard shortcuts to jump to next change
17         https://bugs.webkit.org/show_bug.cgi?id=52305
18
19         Comments and diff blocks go in the same queue. If you have a 
20         comment focused, then j/k will focus the next/prev diff block
21         with respect to that comment.
22
23         * PrettyPatch/PrettyPatch.rb:
24         * code-review.js:
25
26 2011-01-12  Ojan Vafai  <ojan@chromium.org>
27
28         Reviewed by Adam Barth.
29
30         show shared lines on both sides in code review tool
31         https://bugs.webkit.org/show_bug.cgi?id=52308
32
33         We were appending the same DOM node twice. Instead, make
34         the from column a clone of the original node.
35
36         * code-review.js:
37
38 2011-01-12  Ojan Vafai  <ojan@chromium.org>
39
40         Reviewed by Adam Barth.
41
42         fix wrapping bug in expansion lines in the code review tool
43         https://bugs.webkit.org/show_bug.cgi?id=52270
44
45         * PrettyPatch/PrettyPatch.rb:
46         * code-review.js:
47
48 2011-01-11  Ojan Vafai  <ojan@chromium.org>
49
50         Reviewed by Adam Barth.
51
52         fix difflink centering to be vertical, not horizontal
53         https://bugs.webkit.org/show_bug.cgi?id=52263
54
55         * PrettyPatch/PrettyPatch.rb:
56         * code-review.js:
57
58 2011-01-11  Ojan Vafai  <ojan@chromium.org>
59
60         Reviewed by Adam Barth.
61
62         remember diffstate for review tool
63         https://bugs.webkit.org/show_bug.cgi?id=52253
64
65         If you use the global sidebyside/unified links, store the diff type
66         in localstorage. Then, onload, use that diff type. That way, people
67         who prefer one diff type or the other can always get that by default.
68
69         * code-review.js:
70
71 2011-01-11  Ojan Vafai  <ojan@chromium.org>
72
73         Reviewed by Adam Barth.
74
75         allow sidebysideifying individual files in the code review tool
76         https://bugs.webkit.org/show_bug.cgi?id=52226
77
78         * code-review.js:
79
80 2011-01-11  Ojan Vafai  <ojan@chromium.org>
81
82         Reviewed by Mihai Parparita.
83
84         fix exception when adding a comment to a side-by-side diff
85         https://bugs.webkit.org/show_bug.cgi?id=52240
86
87         prev() and next() won't get the previous and next lines in
88         side-by-side mode. Instead do a query to find them.
89
90         * code-review.js:
91
92 2011-01-11  Ojan Vafai  <ojan@chromium.org>
93
94         Reviewed by Mihai Parparita.
95
96         maintain word diffs when converting to side-by-side and back
97         https://bugs.webkit.org/show_bug.cgi?id=52244
98
99         Using textContent would lose the spans use to make the word-diff highlighting.
100         Just move the span element itself.
101
102         * code-review.js:
103
104 2011-01-11  Ojan Vafai  <ojan@chromium.org>
105
106         Reviewed by Adam Barth.
107
108         fix layout error with comments in code review tool
109         https://bugs.webkit.org/show_bug.cgi?id=52230
110
111         * PrettyPatch/PrettyPatch.rb:
112
113 2011-01-10  Ojan Vafai  <ojan@chromium.org>
114
115         Reviewed by Adam Barth.
116
117         convert back to unified from sidebyside diff
118         https://bugs.webkit.org/show_bug.cgi?id=52180
119
120         Remove url fragment stuff. Having it be per-filediff is too complicated.
121
122         * PrettyPatch/PrettyPatch.rb:
123         * code-review.js:
124
125 2011-01-07  Adam Barth  <abarth@webkit.org>
126
127         Rubber-stamped by Eric Seidel.
128
129         Move WebCore to Source
130         https://bugs.webkit.org/show_bug.cgi?id=52050
131
132         Remove reference to old directory.
133
134         * PrettyPatch/PrettyPatch.rb:
135
136 2011-01-06  Ojan Vafai  <ojan@chromium.org>
137
138         Reviewed by Adam Barth.
139
140         side-by-side diffs in the code review tool
141         https://bugs.webkit.org/show_bug.cgi?id=52019
142
143         Support for conversion from the formatted diff to a side-by-side diff.
144         Maintains comments and new comments can be added.
145
146         The main architectural change is that Line elements are no longer necessarily
147         siblings. Each physical line is now in a LineContainer and LineContainers are
148         siblings. Each Line corresponds to a Line in the unified diff and has an id (e.g. line12).
149         A Line can be a LineContainer or a child of a LineContainer.
150
151         In this way, converting to side-by-side and, in the future, back to unified is non-lossy.
152
153         * PrettyPatch/PrettyPatch.rb:
154         * code-review.js:
155
156 2011-01-06  Ojan Vafai  <ojan@chromium.org>
157
158         Fix line context when replying to comments.
159
160         * code-review.js:
161
162 2011-01-05  Ojan Vafai  <ojan@chromium.org>
163
164         Reviewed by Adam Barth.
165
166         change the way we do comment highlighting in the code review tool
167         https://bugs.webkit.org/show_bug.cgi?id=51971
168
169         Store a space-separated list of base line IDs on each line that has
170         comments associated with that line. This allows for overlapping comments,
171         but more importantly, makes adding side-by-side diff support easier.
172
173         * code-review.js:
174
175 2011-01-05  Ojan Vafai  <ojan@chromium.org>
176
177         Reviewed by Adam Barth.
178
179         minor code cleanup for code review tool
180         https://bugs.webkit.org/show_bug.cgi?id=51962
181
182         Consolidates some queries to using shared functions.
183
184         * code-review.js:
185
186 2011-01-05  Ojan Vafai  <ojan@chromium.org>
187
188         Reviewed by Adam Barth.
189
190         assorted cleanups to prepare for side-by-side diffing
191         https://bugs.webkit.org/show_bug.cgi?id=51961
192
193         Mostly, use selector queries instead of assuming comments are next siblings.
194
195         * code-review.js:
196
197 2011-01-05  Ojan Vafai  <ojan@chromium.org>
198
199         Reviewed by Adam Barth.
200
201         fix message container selector for code review tool
202         https://bugs.webkit.org/show_bug.cgi?id=51959
203
204         * code-review.js:
205
206 2011-01-04  Ojan Vafai  <ojan@chromium.org>
207
208         Increment version number to avoid getting the cached JS file.
209
210         * PrettyPatch/PrettyPatch.rb:
211
212 2010-12-15  Ojan Vafai  <ojan@chromium.org>
213  
214         Reviewed by Adam Barth.
215
216         anchor the toolbar to the bottom of the diff if the diff doesn't take a full screen of height
217         https://bugs.webkit.org/show_bug.cgi?id=51162
218
219         * PrettyPatch/PrettyPatch.rb:
220         * code-review.js:
221
222 2011-01-01  Adam Barth  <abarth@webkit.org>
223
224         Reviewed by Eric Seidel.
225
226         Move JavaScriptCore to Source
227         https://bugs.webkit.org/show_bug.cgi?id=51604
228
229         * PrettyPatch/PrettyPatch.rb:
230           - Remove reference to JavaScriptCore as a source directory.
231
232 2011-01-01  Adam Barth  <abarth@webkit.org>
233
234         Reviewed by Eric Seidel.
235
236         Move Sources to Source
237         https://bugs.webkit.org/show_bug.cgi?id=51794
238
239         * PrettyPatch/PrettyPatch.rb:
240
241 2010-12-31  Adam Barth  <abarth@webkit.org>
242
243         Rubber-stamped by Eric Seidel.
244
245         Move PageLoadTests to PerformanceTests/PageLoad
246         https://bugs.webkit.org/show_bug.cgi?id=51771
247
248         Update list of top-level open-source directories.
249
250         * PrettyPatch/PrettyPatch.rb:
251
252 2010-12-31  Adam Barth  <abarth@webkit.org>
253
254         Rubber-stamped by Eric Seidel.
255
256         Move SunSpider into PerformanceTests
257         https://bugs.webkit.org/show_bug.cgi?id=51769
258
259         Update list of top-level open-source directories.
260
261         * PrettyPatch/PrettyPatch.rb:
262
263 2010-12-26  Adam Barth  <abarth@webkit.org>
264
265         Reviewed by Eric Seidel.
266
267         Move autotools into Sources
268         https://bugs.webkit.org/show_bug.cgi?id=51630
269
270         * PrettyPatch/PrettyPatch.rb:
271           - Remove reference to non-existant autotools directory.
272
273 2010-12-26  Adam Barth  <abarth@webkit.org>
274
275         Reviewed by Eric Seidel.
276
277         Move cmake into Sources
278         https://bugs.webkit.org/show_bug.cgi?id=51631
279
280         * PrettyPatch/PrettyPatch.rb:
281
282 2010-12-23  Adam Barth  <abarth@webkit.org>
283
284         Rubber-stamped by Eric Seidel.
285
286         Move JavaScriptGlue into Sources
287         https://bugs.webkit.org/show_bug.cgi?id=51583
288
289         PrettyPatch keeps a list of the top-level directories that exist in
290         trac.
291
292         * PrettyPatch/PrettyPatch.rb:
293
294 2010-12-20  Adam Barth  <abarth@webkit.org>
295
296         One more tweak caused by moving the web sites around.  I had this
297         change in my patch originally, but I missed it because I had to use
298         server-side svn move commands to move the bulk of the web site files.
299
300         * PrettyPatch/PrettyPatch.rb:
301
302 2010-12-18  Adam Barth  <abarth@webkit.org>
303
304         Reviewed by Sam Weinig.
305
306         Move WebKitExamplePlugins to Examples
307         https://bugs.webkit.org/show_bug.cgi?id=51291
308
309         * PrettyPatch/PrettyPatch.rb:
310
311 2010-12-17  Dan Bernstein  <mitz@apple.com>
312
313         Reviewed by Simon Fraser.
314
315         Rename WebKitTools to Tools
316         https://bugs.webkit.org/show_bug.cgi?id=49861
317
318         * PrettyPatch/PrettyPatch.rb:
319         * committers-autocomplete.js:
320
321 2010-12-17  Ojan Vafai  <ojan@chromium.org>
322
323         Increment psuedo-version number to avoid pulling cached version.
324
325         * PrettyPatch/PrettyPatch.rb:
326
327 2010-12-17  Ojan Vafai  <ojan@chromium.org>
328
329         Speculative fix for the code review tool in Firefox.
330         Fixes a JS error and makes the status bubble mostly work
331         if postMessage is not supported.
332
333         * PrettyPatch/PrettyPatch.rb:
334         * code-review.js:
335
336 2010-12-15  Ojan Vafai  <ojan@chromium.org>
337
338         Reviewed by Adam Barth.
339
340         size status bubble to it's contents on the code review page
341         https://bugs.webkit.org/show_bug.cgi?id=51142
342
343         * PrettyPatch/PrettyPatch.rb:
344         * code-review.js:
345
346 2010-12-15  Ojan Vafai  <ojan@chromium.org>
347
348         Reviewed by Adam Barth.
349
350         fix goofups from r74142 and r74130
351         https://bugs.webkit.org/show_bug.cgi?id=51146
352
353         Accidentally shrank the inline comment boxes and made
354         the toolbar not show up when there were image diffs.
355
356         * PrettyPatch/PrettyPatch.rb:
357         * code-review.js:
358
359 2010-12-14  Ojan Vafai  <ojan@chromium.org>
360
361         Reviewed by Adam Barth.
362
363         add ability to view for file context to the review tool
364         https://bugs.webkit.org/show_bug.cgi?id=51057
365
366         At the beginning/end of each file diff and between each
367         hunk add links to expand the context. For now it grabs the
368         tip of tree version of the file and tries to apply the diff 
369         to that file. If it can't apply, then it gives up as we
370         wouldn't want to show the wrong lines of context.
371
372         In the future, we can consider adding the upload svn revision
373         to the diff itself, then we could fallback to the file at that
374         revision if tip of tree doesn't apply.
375
376         * PrettyPatch/PrettyPatch.rb:
377         * code-review.js:
378
379 2010-12-08  Ojan Vafai  <ojan@chromium.org>
380
381         Reviewed by Adam Barth.
382
383         [reviewtool] should always show overall comments text box
384         https://bugs.webkit.org/show_bug.cgi?id=45870
385
386         * PrettyPatch/PrettyPatch.rb:
387         * code-review.js:
388
389 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
390
391         Reviewed by Adam Barth.
392
393         Increase size of status bubbles
394         https://bugs.webkit.org/show_bug.cgi?id=50496
395
396         The current available space isn't enough to display all
397         pending results including position in the queue.
398
399         * PrettyPatch/PrettyPatch.rb:
400
401 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
402
403         Reviewed by Adam Barth.
404
405         Increase size of status bubbles
406         https://bugs.webkit.org/show_bug.cgi?id=50496
407
408         The current available space isn't enough to display all
409         pending results including position in the queue.
410
411         * template/en/custom/attachment/edit.html.tmpl:
412         * template/en/custom/attachment/list.html.tmpl:
413         * template/en/custom/attachment/reviewform.html.tmpl:
414
415 2010-11-08  Adam Barth  <abarth@webkit.org>
416
417         Update help text to match behavior change.
418
419         * code-review.js:
420
421 2010-11-08  Adam Barth  <abarth@webkit.org>
422
423         Reviewed by Eric Seidel.
424
425         Restore normal meaning of double-click on patch review page?
426         https://bugs.webkit.org/show_bug.cgi?id=47641
427
428         After this patch, you need to click on the line numbers to add a
429         comment.
430
431         * PrettyPatch/PrettyPatch.rb:
432         * code-review.js:
433
434 2010-10-09  Adam Barth  <abarth@webkit.org>
435
436         Reviewed by Alexey Proskuryakov.
437
438         Exception in committers-autocomplete in Firefox
439         https://bugs.webkit.org/show_bug.cgi?id=47456
440
441         Remove dependency on a WebKit-only quirk.  IMHO, this quirk is bad for
442         the long-term health of the web, but that's a discussion for another
443         patch.
444
445         * committers-autocomplete.js:
446
447 2010-10-08  Adam Barth  <abarth@webkit.org>
448
449         Reviewed by Oliver Hunt.
450
451         Bugmail for new attachments should link to the reviewtool
452         https://bugs.webkit.org/show_bug.cgi?id=47440
453
454         As requested by olliej himself.  This patch was constructed by pure
455         reason.  I have no way to actually run this code.
456
457         * Bugzilla/BugMail.pm:
458
459 2010-09-28  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
460
461         Reviewed by Csaba Osztrogonác.
462
463         Convert verbatim unicode values in comitter names to characters
464
465         * committers-autocomplete.js:
466
467 2010-09-24  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
468
469         Reviewed by Csaba Osztrogonác.
470
471         Support Unicode-strings in committers-autocomplete script
472
473         * committers-autocomplete.js:
474
475 2010-09-22  Adam Barth  <abarth@webkit.org>
476
477         Reviewed by John Sullivan.
478
479         New review-page design doesn't include name of reviewer
480         https://bugs.webkit.org/show_bug.cgi?id=46271
481
482         * PrettyPatch/PrettyPatch.rb:
483         * code-review.js:
484
485 2010-09-22  Adam Barth  <abarth@webkit.org>
486
487         Fix typo in CSS.
488
489         * PrettyPatch/PrettyPatch.rb:
490
491 2010-09-22  Adam Barth  <abarth@webkit.org>
492
493         Reviewed by Eric Seidel.
494
495         Add link to bug to review page
496         https://bugs.webkit.org/show_bug.cgi?id=46192
497
498         To make room for the link, I moved the help text to the top of the page
499         and tweaked the language to help folks discover that you can select
500         context using the line numbers.
501
502         Also, move more text to sans-serif since that fits in better with the
503         pretty-patch fonts.
504
505         * PrettyPatch/PrettyPatch.rb:
506         * code-review.js:
507
508 2010-09-21  Adam Barth  <abarth@webkit.org>
509
510         Unreviewed.
511
512         [reviewtool] Publish button doesn't work
513         https://bugs.webkit.org/show_bug.cgi?id=46168
514
515         Darin's recent patch had a runtime error.  Sadly, we don't have any
516         testing for this code.
517
518         * code-review.js:
519
520 2010-09-20  Darin Adler  <darin@apple.com>
521
522         Reviewed by Adam Barth.
523
524         Add Preview button and link to bug to patch review bar
525         https://bugs.webkit.org/show_bug.cgi?id=46153
526
527         * PrettyPatch/PrettyPatch.rb: Bump version number to make caches happy.
528         * code-review.js: Added Preview button and link to bug.
529
530 2010-09-19  Adam Barth  <abarth@webkit.org>
531
532         Reviewed by Eric Seidel.
533
534         [reviewtool] Publish button should actually publish the review comments
535         https://bugs.webkit.org/show_bug.cgi?id=46074
536
537         Instead of showing the confirmation lightbox, the Publish button now
538         publishes the comments directly.  If there's demand for a "Preview"
539         button, we can find a way to add that feature.
540
541         * code-review.js:
542
543 2010-09-19  Adam Barth  <abarth@webkit.org>
544
545         Reviewed by Eric Seidel.
546
547         Move reviewtool to action=review
548         https://bugs.webkit.org/show_bug.cgi?id=46071
549
550         This patch removes the old Review Patch screen and moves the new
551         reviewtool from Pretty Diff to Review Patch.
552
553         * attachment.cgi:
554         * code-review.js:
555
556 2010-09-19  Adam Barth  <abarth@webkit.org>
557
558         Review tool UI tweak.  This patch lengthens the status bubbles so all
559         the bubble fit (even when they have numbers inside).
560
561         * PrettyPatch/PrettyPatch.rb:
562
563 2010-09-19  Dan Bernstein  <mitz@apple.com>
564
565         Reviewed by Anders Carlsson.
566
567         Fixed a typo.
568
569         * code-review.js: Changed the label of the OK button from Ok to OK.
570
571 2010-09-15  Adam Barth  <abarth@webkit.org>
572
573         Unreviewed.
574
575         Only show the review-in-context link if there are contextual comments.
576
577         * code-review.js:
578
579 2010-09-15  Adam Barth  <abarth@webkit.org>
580
581         Reviewed by Eric Seidel.
582
583         [reviewtool] Show the status bubbles on the toolbar
584         https://bugs.webkit.org/show_bug.cgi?id=45861
585
586         * PrettyPatch/PrettyPatch.rb:
587         * code-review.js:
588
589 2010-09-15  Adam Barth  <abarth@webkit.org>
590
591         Reviewed by Eric Seidel.
592
593         [reviewtool] Allow setting review and commit-queue flags from main review screen
594         https://bugs.webkit.org/show_bug.cgi?id=45860
595
596         This patch adds drop down menus for adjusting the review and
597         commit-queue flags from the main review screen.  These controls don't
598         appear immediately because we need to read the state of the flags off
599         the details page.
600
601         It's a slight sadness that bugzilla doesn't have a clean way of
602         identifying which select control is associated with a given flag.  The
603         flags seem to have some sort of id, but it's unclear to me how to map
604         that id back to a notion of "review" or "commit-queue".  Instead, we
605         look for some magic strings in the title.  I'm sure this will come back
606         to bit us at some point, but I'm not sure what to do that's better.
607         (The webkitpy solution to this problem is to assume that the flags are
608         in a given order, but this seems better.)
609
610         * PrettyPatch/PrettyPatch.rb:
611             - Bump version number to bust throuh caches.
612         * code-review.js:
613
614 2010-09-15  Adam Barth  <abarth@webkit.org>
615
616         Reviewed by Eric Seidel.
617
618         [reviewtool] Comments should quote previous comments on the same line
619         https://bugs.webkit.org/show_bug.cgi?id=45847
620
621         Now when you reply to a previous comment, the tool will quote the
622         previous comment in the bugs.webkit.org post.  This makes it eaiser for
623         folks following along in email to understand the discussion.
624
625         While I was editing this code, I also cleaned up some of the whitespace
626         handling in comments.
627
628         * code-review.js:
629
630 2010-09-15  Adam Barth  <abarth@webkit.org>
631
632         Minor UI tweak to the review tool.  We want to display newlines in
633         previous and frozen comments instead of collapsing them.
634
635         * PrettyPatch/PrettyPatch.rb:
636
637 2010-09-14  Adam Barth  <abarth@webkit.org>
638
639         Reviewed by Eric Seidel.
640
641         bugs.webkit.org should autocomplete WebKit Committers and Reviewers
642         https://bugs.webkit.org/show_bug.cgi?id=45799
643
644         This patch implements an autocomplete dropdown that helps users type
645         the names of WebKit committers and reviewers.  The script grabs the
646         data from SVN now that wms set up CORS for http://svn.webkit.org.
647         Security technology for the win.
648
649         This script is based on (well, essentially copied from) the popular
650         autocomplete extension:
651
652         https://chrome.google.com/extensions/detail/olaabhcgdogcbcoiolomlcodkngnemfb
653
654         I asked Ojan, and said he was fine licensing the code under the BSD
655         license.  I'm not sure how the patch interacts with the extension, but
656         we can retire the extension if they interact poorly.
657
658         * committers-autocomplete.js: Added.
659         * template/en/default/global/footer.html.tmpl:
660             - Add the script to every page on bugs.webkit.org
661
662 2010-09-14  Adam Barth  <abarth@webkit.org>
663
664         Add a license block to the reviewtool.  I should have added this from
665         the start.  According to the SVN log, I'm the only one to have ever
666         touched this file.
667
668         * code-review.js:
669
670 2010-09-07  Adam Barth  <abarth@webkit.org>
671
672         Minor tweaks to the reviewtool UI.
673
674         Rename Cancel to Discard to make it more clear what the button does.
675         Also, move the style declaration for the focus ring later in the sheet
676         so it works.
677
678         * PrettyPatch/PrettyPatch.rb:
679         * code-review.js:
680
681 2010-09-06  Adam Barth  <abarth@webkit.org>
682
683         Small UI tweak to the reviewtool.  Make the Ok/Cancel buttons a bit
684         bigger.
685
686         * PrettyPatch/PrettyPatch.rb:
687
688 2010-09-06  Adam Barth  <abarth@webkit.org>
689
690         Reviewed by Eric Seidel.
691
692         [reviewtool] Add a quick animation for opening/closing comment boxes
693         https://bugs.webkit.org/show_bug.cgi?id=45278
694
695         The 200ms animation slides in and out the line-by-line comment box.
696
697         * code-review.js:
698
699 2010-09-06  Adam Barth  <abarth@webkit.org>
700
701         Reviewed by Eric Seidel.
702
703         [reviewtool] Add a field for overall comments
704         https://bugs.webkit.org/show_bug.cgi?id=45273
705
706         This patch does a couple logically separate things that could be
707         separated into smaller patches:
708
709         1) This patch adds an "overall comments" field where you can enter
710            overall comments about the patch.  These comments appear at the top
711            of the bugzilla posting.  Currently, these aren't redisplayed when
712            viewing the patch, but I plan to add that in a future patch.
713
714         2) This patch renames some of the CSS classes to more consistently
715            follow the camelCase style that PrettyPatch uses.
716
717         3) This patch moves the "prepare comments" button to the left of the
718            toolbar and renames is to "publish comments".  This makes more sense
719            when you scroll to the bottom of the page and enter in some overall
720            comments.
721
722         4) When you attempt to add a comment to a line that already has a
723            "frozen" comment, we now unfreeze the comment instead of doing
724            nothing.  The old behavior was kind of frustrating if you didn't
725            know that you could unfreeze a comment by clicking on it.
726
727         * PrettyPatch/PrettyPatch.rb:
728             - Update CSS.
729         * code-review.js:
730
731 2010-09-06  Adam Barth  <abarth@webkit.org>
732
733         [reviewtool] Tweak the ok button to cancel the comment if the comment
734         is empty.  Previously we would get into a bad state where a line had a
735         comment but there was no longer any way to access it.
736
737         * code-review.js:
738
739 2010-09-06  Adam Barth  <abarth@webkit.org>
740
741         Reviewed by Eric Seidel.
742
743         [reviewtool] Add an "ok" button that collapses review comments
744         https://bugs.webkit.org/show_bug.cgi?id=45255
745
746         Many folks (including myself) are too trigger-happy with the "delete"
747         button that we end up deleting perfectly good review comments.  This
748         patch adds an "ok" button to absorb these clicks (and renames "delete"
749         to "cancel").  When you click the ok button, the comment becomes
750         non-editable and the box shrinks to fit the text.  You can click the
751         text to get back to the editable version.
752
753         * code-review.js:
754
755 2010-08-31  Adam Barth  <abarth@webkit.org>
756
757         Reviewed by Eric Seidel.
758
759         [reviewtool] Make it easy to scroll through review comments
760         https://bugs.webkit.org/show_bug.cgi?id=45002
761
762         This patch lets you scroll through review comments using "n" (for next)
763         and "p" (for previous).  It also attributes comments to their authors.
764
765         * PrettyPatch/PrettyPatch.rb:
766         * code-review.js:
767
768 2010-08-31  Adam Barth  <abarth@webkit.org>
769
770         Reviewed by Eric Seidel.
771
772         [reviewtool] Show previous comments inline in diff
773         https://bugs.webkit.org/show_bug.cgi?id=44977
774
775         This patch adds basic support for showing previous comments inline in
776         the diff.  We crawl the bugs.webkit.org comments about this attachment
777         and extract comments related to specific lines.  We then show the
778         comments inline in the diff.
779
780         This part of the tool needs a bunch of polish, but this at least is a
781         starting point for further work.
782
783         * PrettyPatch/PrettyPatch.rb:
784         * code-review.js:
785
786 2010-08-31  Adam Barth  <abarth@webkit.org>
787
788         Reviewed by Eric Seidel.
789
790         [reviewtool] Allow reviewer to select lines of context by dragging over the line numbers
791         https://bugs.webkit.org/show_bug.cgi?id=44936
792
793         * PrettyPatch/PrettyPatch.rb:
794             - Add a version number to bust through bugs.webkit.org's aggressive
795               cacheing.
796         * code-review.js:
797
798 2010-08-30  Adam Barth  <abarth@webkit.org>
799
800         Reviewed by Eric Seidel.
801
802         [review tool] Let reviewer select how much context to show in snippet
803         https://bugs.webkit.org/show_bug.cgi?id=44905
804
805         We now highlight the context for a comment in yellow on the left (where
806         the line numbers are).  Clicking a line number expands or contracts the
807         amount of context, as appropriate.  Informal user testing indicates
808         that we might want to support drag as well.
809
810         This patch also changes the "open a comment box here" action to
811         double-click to avoid issues with mis-clicks.
812
813         * PrettyPatch/PrettyPatch.rb:
814         * code-review.js:
815
816 2010-08-29  Adam Barth  <abarth@webkit.org>
817
818         Attempt to make Sam's life easier by not opening a comment text field
819         if there's a selection.  This should make it easier to copy/paste text
820         out of the diff without accidentally opening comment boxes.
821
822         * code-review.js:
823
824 2010-08-29  Adam Barth  <abarth@webkit.org>
825
826         Turns out we're supposed to use find instead of children here.
827
828         * code-review.js:
829
830 2010-08-29  Adam Barth  <abarth@webkit.org>
831
832         Reviewed by Eric Seidel.
833
834         Awesomify pretty-diff based review tool
835         https://bugs.webkit.org/show_bug.cgi?id=44835
836
837         This patch replaces the existing inline comment feature with a new
838         implementation.  Hopefully the new implementation is more awesome.
839         It's not 100% done yet, but I'd like to get this version out there to
840         see how it works.
841
842         Unlike the previous version this version works with the "Formatted
843         Diff" link instead of the "Review Patch" link.  Hopefully that will
844         avoid interfering with folks who like the old style review.
845
846         * PrettyPatch/PrettyPatch.rb:
847         * code-review.js: Added.
848         * prototype.js: Removed.
849
850 2010-08-19  Tony Chang  <tony@chromium.org>
851
852         Reviewed by Adam Barth.
853
854         webkit-patch barfed on upload with a new image test result
855         https://bugs.webkit.org/show_bug.cgi?id=39097
856
857         * PrettyPatch/PrettyPatch.rb: don't use full path to git
858
859 2010-08-19  Tony Chang  <tony@chromium.org>
860
861         Unreviewed, setting svn:eol-style native on ChangeLog.
862
863 2010-08-19  Tony Chang  <tony@chromium.org>
864
865         Unreviewed.  Just removing \r's from ChangeLog.
866
867 2010-08-02  Adam Roben  <aroben@apple.com>
868
869         Tell the patch prettifier about some (not so) new directories
870
871         * PrettyPatch/PrettyPatch.rb: Added WebKit2, autotools, and cmake
872         directories.
873
874 2010-07-27  Joseph Pecoraro  <joepeck@webkit.org>
875
876         Reviewed by David Kilzer.
877
878         Provide a Helpful Link for Web Inspector Bugs in Bugzilla
879         https://bugs.webkit.org/show_bug.cgi?id=43065
880
881         * template/en/custom/global/choose-product.html.tmpl: Added. Explicitly added a link for the Web Inspector.
882
883 2010-06-09  Julie Parent  <jparent@chromium.org>
884
885         Reviewed by David Kilzer.
886
887         Bugzilla: Don't send mail for cancel/deny/granted for in-rietveld.
888
889         https://bugs.webkit.org/show_bug.cgi?id=40345
890
891         * Bugzilla/Flag.pm:
892
893 2010-05-17  Julie Parent  <jparent@chromium.org>
894
895         Reviewed by Ojan Vafai.
896
897         Rietveld review page should dedicate more space to Rietveld, less to high level comments form.
898
899         https://bugs.webkit.org/show_bug.cgi?id=39244
900
901         * template/en/custom/attachment/reviewform.html.tmpl:
902         Delete a random BR amongst the hidden elements that was causing extra whitespace.
903         * template/en/custom/attachment/rietveldreview.html.tmpl:
904         Increase size of Rietveld frame to 80%, from 60%.
905
906 2010-05-14  Julie Parent  <jparent@chromium.org>
907
908         Reviewed by Ojan Vafai.
909
910         Bugzilla: Only show "Rietveld Review" link when in-rietveld is set to +.
911         
912         https://bugs.webkit.org/show_bug.cgi?id=39141
913
914         * template/en/custom/attachment/list.html.tmpl:
915
916 2010-05-13  Julie Parent  <jparent@chromium.org>
917
918         Reviewed by David Kilzer.
919
920         Bugzilla: Don't send mail if the only change is to the in-rietveld flag.
921         
922         https://bugs.webkit.org/show_bug.cgi?id=39090
923
924         * Bugzilla/BugMail.pm:
925
926 2010-05-12  Julie Parent  <jparent@chromium.org>
927
928         Reviewed by Ojan Vafai.
929
930         Hide the in-rietveld flag in the UI.  It is only used only by tooling and shouldn't clutter the UI.
931         https://bugs.webkit.org/show_bug.cgi?id=39024
932
933         * template/en/custom/attachment/list.html.tmpl:
934           Removes this flag entirely from the UI, not needed on this page.
935         * template/en/custom/flag/list.html.tmpl: Added.
936           Copied from template/en/default/flag/list.html.tmpl with only changes inside the if WEBKIT_CHANGES section.
937           Hides the flag rather than removing it completely, so tools can still interact with it.  
938
939 2010-05-12  Julie Parent  <jparent@chromium.org>
940
941         Reviewed by David Kilzer.
942
943         Adds support for Rietveld reviews.  When there is an associated
944         Rietveld id for a patch, adds a link to do a Rietveld review instead.
945
946         https://bugs.webkit.org/show_bug.cgi?id=38143
947
948         * attachment.cgi:
949             Added support for handling action=rietveldreview.
950         * template/en/custom/attachment/list.html.tmpl:
951             Adds a link to use rietveld review iff the in_rietveld flag is set.
952         * template/en/custom/attachment/reviewform.html.tmpl:
953             Detects if we are in rietveld review mode due to query string paramter,
954             and customizes the review form slightly:
955                 - Changes text describing comment field.
956                 - Comment field does not include quoted patch.
957                 - Submit button fetches from Rietveld instead of just submitting.
958         * template/en/custom/attachment/rietveldreview.html.tmpl: Added.
959             New template for rietveldreview mode.  Hosts the rietveld frame in the
960             top and the regular comments form in the bottom.
961             Uses postMessage to communicate with Rietveld frame.
962
963 2010-05-08  Chris Jerdonek  <cjerdonek@webkit.org>
964
965         Reviewed by Maciej Stachowiak.
966
967         Allow long comment lines to wrap to the window width instead
968         of to 80 characters.
969
970         https://bugs.webkit.org/show_bug.cgi?id=37792
971
972         * Bugzilla/Constants.pm:
973           - Introduced a new constant COMMENT_COLS_WRAP to replace COMMENT_COLS
974             inside the wrap_comment() subroutine.  The new constant effectively
975             disables text wrapping by increasing the value used in wrap_comment()
976             from 80 characters to a very large value (8000).
977         * Bugzilla/Util.pm:
978           - Replaced COMMENT_COLS with COMMENT_COLS_WRAP inside wrap_comment().
979         * skins/custom/global.css:
980         (.bz_comment_text):
981           - Eliminate the width styling on comments to allow long lines
982             to wrap at the window width.
983         (.bz_comment_text > span.quote):
984           - Inherit the "white-space" property.  This prevents long quoted
985             lines in comments from causing non-quoted lines to extend beyond
986             the window width.
987
988 2010-04-22  Adam Barth  <abarth@webkit.org>
989
990         Reviewed by Darin Adler.
991
992         The new review tools shouldn't clear the comment box automatically
993         https://bugs.webkit.org/show_bug.cgi?id=38004
994
995         Instead, we now have a button for doing that explicitly.  If people
996         like this tool, we can look at fancier solutions.
997
998         Also, tried to make the tool work on the edit page as well as the review page.
999
1000         * PrettyPatch/PrettyPatch.rb:
1001
1002 2010-04-22  Adam Barth  <abarth@webkit.org>
1003
1004         Unreviewed.  Turns out we need to do more escaping because Ruby was
1005         explanding our newlines, creating a syntax error.
1006
1007         * PrettyPatch/PrettyPatch.rb:
1008
1009 2010-04-22  Adam Barth  <abarth@webkit.org>
1010
1011         Reviewed by Darin Adler.
1012
1013         Hack up PrettyDiff to allow line-by-line comments
1014         https://bugs.webkit.org/show_bug.cgi?id=37886
1015
1016         Admittedly a bit of a hack, this is a basic line-by-line editor we can
1017         play with thanks to Andrew Scherkus.  It's meant to integrate with the
1018         "review" page at bugs.webkit.org.
1019
1020         I changed a few things from Andrew's original version:
1021         1) Trigger text boxes on single click to make iPhone/iPad reviewing
1022         easier.
1023         2) Clear the main text box on load.
1024         3) Reference a version of prototype.js on bugs.webkit.org.
1025
1026         * PrettyPatch/PrettyPatch.rb:
1027         * prototype.js: Added.
1028
1029 2010-02-22  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
1030
1031         Reviewed by Maciej Stachowiak.
1032
1033         Don't reset the assignee back to default on component change
1034
1035         https://bugs.webkit.org/show_bug.cgi?id=35236
1036
1037         There's a risk of reseting a valid assignee for example when
1038         triaging a bug. All components in WebKit have the default 
1039         assignee set to webkit-unassigned@webkit.org so this should
1040         not cause problems for people relying on this functionality.
1041
1042         * template/en/custom/bug/edit.html.tmpl:
1043
1044 2010-01-22  Janusz Lewandowski  <lew21@xtreeme.org>
1045
1046         Reviewed by Maciej Stachowiak.
1047
1048         run-webkit-tests fails on Ruby 1.9
1049         https://bugs.webkit.org/show_bug.cgi?id=33554
1050         inject does not work with strings any more, so BugsSite/PrettyPatch/PrettyPatch.rb prints lots of errors.
1051
1052         * PrettyPatch/PrettyPatch.rb:
1053
1054 2010-01-20  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
1055
1056         Reviewed by Adam Barth.
1057
1058         Adjust height of status-bubbles to prevent them being cut.
1059
1060         * template/en/custom/attachment/edit.html.tmpl:
1061         * template/en/custom/attachment/list.html.tmpl:
1062         * template/en/custom/attachment/reviewform.html.tmpl:
1063
1064 2010-01-08  David Kilzer  <ddkilzer@apple.com>
1065
1066         <http://webkit.org/b/33410> Fix "View Plain Diff"/"View Formatted Diff" button
1067
1068         Reviewed by Adam Barth.
1069
1070         This button was broken during a previous Bugzilla upgrade after
1071         changes were made to the names of objects exposed on templates.
1072
1073         * template/en/custom/attachment/reviewform.html.tmpl: Changed
1074         attachid to attachment.id and ispatch to attachment.ispatch.
1075         Also tweaked the format to use 4 table columns instead of 5.
1076
1077 2010-01-08  David Kilzer  <ddkilzer@apple.com>
1078
1079         Included UNCONFIRMED bugs by default in advanced search query
1080
1081         Rubber-stamped by Alexey Proskuryakov.
1082
1083         * data/params: Prepended 'bug_status=UNCONFIRMED&' to
1084         'defaultquery' parameter.
1085
1086 2010-01-08  David Kilzer  <ddkilzer@apple.com>
1087
1088         FIX: Make patch status appear on reviewform.html.tmpl
1089
1090         * template/en/custom/attachment/reviewform.html.tmpl: Check
1091         attachment.ispatch (not just ispatch) when deciding whether
1092         to display patch status.  Also update layout a bit.
1093
1094 2010-01-08  David Kilzer  <ddkilzer@apple.com>
1095
1096         Update data/params parameters
1097
1098         * data/params: Updated 'timezone' to turn off DST.  Updated
1099         'usebugaliases' and 'usevotes' to turn them off.
1100
1101 2010-01-08  Adam Barth  <abarth@webkit.org>
1102
1103         Reviewed by Darin Adler.
1104
1105         Show patch status in bugs.webkit.org
1106         https://bugs.webkit.org/show_bug.cgi?id=33368
1107
1108         This patch adds an iframe to the show_bug page and the edit attachment
1109         page that displays whether the attachment passed the various bots.
1110
1111         * template/en/custom/attachment/edit.html.tmpl:
1112         * template/en/custom/attachment/list.html.tmpl:
1113         * template/en/custom/attachment/reviewform.html.tmpl:
1114
1115 2009-12-07  Shinichiro Hamaji  <hamaji@chromium.org>
1116
1117         Reviewed by David Kilzer.
1118
1119         Bugzilla should show images in git patches
1120         https://bugs.webkit.org/show_bug.cgi?id=31395
1121
1122         Attempt to go r51748 again using --directory option of git-apply.
1123
1124         * PrettyPatch/PrettyPatch.rb:
1125
1126 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
1127
1128         Unreviewed. Revert r51748.
1129
1130         Bugzilla should show images in git patches
1131         https://bugs.webkit.org/show_bug.cgi?id=31395
1132
1133         * PrettyPatch/PrettyPatch.rb:
1134
1135 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
1136
1137         Reviewed by David Kilzer.
1138
1139         Bugzilla should show images in git patches
1140         https://bugs.webkit.org/show_bug.cgi?id=31395
1141
1142         Show images in git patches using git-apply.
1143
1144         * PrettyPatch/PrettyPatch.rb:
1145
1146 2009-10-23  Eric Seidel  <eric@webkit.org>
1147
1148         Reviewed by Adam Roben.
1149
1150         PrettyPatch should show images even when they have the wrong mime type
1151         https://bugs.webkit.org/show_bug.cgi?id=29506
1152
1153         * PrettyPatch/PrettyPatch.rb:
1154
1155 2009-10-17  Alexey Proskuryakov  <ap@apple.com>
1156
1157         Reviewed by Eric Carlson.
1158
1159         https://bugs.webkit.org/show_bug.cgi?id=30470
1160         Make marking a bug as duplicate easier
1161
1162         * js/field.js: (setResolutionToDuplicate): Focus the edit field, too.
1163
1164 2009-10-16  Alexey Proskuryakov  <ap@apple.com>
1165
1166         Reviewed by Mark Rowe.
1167
1168         https://bugs.webkit.org/show_bug.cgi?id=30470
1169         Make marking a bug as duplicate easier
1170
1171         * js/field.js: (setResolutionToDuplicate): Show edit field right away, don't make the user
1172         click again.
1173
1174 2009-09-09  David Kilzer  <ddkilzer@apple.com>
1175
1176         Update data/params parameters
1177
1178         * data/params: Updated 'attachment_base' parameter for secure
1179         attachments.  Updated 'mybugstemplate' parameter to search for
1180         bugs in UNCONFIRMED, ASSIGNED and REOPENED status in addition to
1181         NEW status.
1182
1183 2009-08-11  David Kilzer  <ddkilzer@apple.com>
1184
1185         Updated data/params parameter
1186
1187         * data/params: Updated 'loginnetmask' parameter to allow users
1188         to choose whether their cookies are restricted to a single IP
1189         address or not.
1190
1191 2009-07-07  David Kilzer  <ddkilzer@apple.com>
1192
1193         Updated BugsSite with local changes on server
1194
1195         The new files were created by checksetup.pl during the upgrade
1196         to Bugzilla 3.2.3.  The rest are self-explanatory.
1197
1198         * data: Added "bugzilla-update.xml" to svn:ignore.
1199         * data/attachments: Added "groups.*" to svn:ignore.
1200         * data/params: Updated for a few changed parameters.
1201
1202         * lib/.htaccess: Added.
1203         * skins/contrib/Dusk/IE-fixes.css: Added.
1204         * skins/contrib/Dusk/admin.css: Added.
1205         * skins/contrib/Dusk/create_attachment.css: Added.
1206         * skins/contrib/Dusk/dependency-tree.css: Added.
1207         * skins/contrib/Dusk/duplicates.css: Added.
1208         * skins/contrib/Dusk/editusers.css: Added.
1209         * skins/contrib/Dusk/help.css: Added.
1210         * skins/contrib/Dusk/index.css: Added.
1211         * skins/contrib/Dusk/panel.css: Added.
1212         * skins/contrib/Dusk/params.css: Added.
1213         * skins/contrib/Dusk/release-notes.css: Added.
1214         * skins/contrib/Dusk/show_bug.css: Added.
1215         * skins/contrib/Dusk/show_multiple.css: Added.
1216         * skins/contrib/Dusk/summarize-time.css: Added.
1217         * skins/contrib/Dusk/voting.css: Added.
1218         * skins/contrib/Dusk/yui: Added.
1219         * skins/contrib/Dusk/yui/calendar.css: Added.
1220         * skins/custom/IE-fixes.css: Added.
1221         * skins/custom/admin.css: Added.
1222         * skins/custom/buglist.css: Added.
1223         * skins/custom/create_attachment.css: Added.
1224         * skins/custom/dependency-tree.css: Added.
1225         * skins/custom/duplicates.css: Added.
1226         * skins/custom/editusers.css: Added.
1227         * skins/custom/help.css: Added.
1228         * skins/custom/panel.css: Added.
1229         * skins/custom/params.css: Added.
1230         * skins/custom/release-notes.css: Added.
1231         * skins/custom/show_bug.css: Added.
1232         * skins/custom/show_multiple.css: Added.
1233         * skins/custom/summarize-time.css: Added.
1234         * skins/custom/voting.css: Added.
1235         * skins/custom/yui: Added.
1236         * skins/custom/yui/calendar.css: Added.
1237
1238 2009-07-03  David Kilzer  <ddkilzer@apple.com>
1239
1240         Bug 26958: Change edit link to review link in request messages
1241
1242         <https://bugs.webkit.org/show_bug.cgi?id=26958>
1243
1244         Reviewed by Dan Bernstein.
1245
1246         * template/en/custom/request/email.txt.tmpl: Copied from BugsSite/template/en/default/request/email.txt.tmpl.
1247         Changed link from "action=edit" to "action=review".
1248
1249 2009-07-03  David Kilzer  <ddkilzer@apple.com>
1250
1251         Bug 26950: Make the summary and alias fields support click-to-edit
1252         <https://bugs.webkit.org/show_bug.cgi?id=26950>
1253
1254         Reviewed by Maciej Stachowiak.
1255
1256         Original patch by Maciej Stachowiak.
1257
1258         * js/field.js:
1259         (hideEditableField): Updated to add click event listeners to the
1260         alias and short description elements to make them easier to
1261         edit.  Renamed field_id parameter to field2_id (short
1262         description id) and added a field1_id parameter (alias id).
1263         (showEditableField): If a third parameter is passed in the
1264         ContainerInputArray parameter, use it to find the element to
1265         focus.  Otherwise, fall back to the original behavior of
1266         focusing the first input field.
1267         (hideAliasAndSummary): Changed to pass the id for the alias
1268         element to hideEditableField().
1269
1270 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1271
1272         Config file for contrib/recode.pl when upgrading bugs.webkit.org
1273
1274         Current as of midday on Wednesday, July 1, 2009.
1275
1276         * contrib/recode-overrides.txt: Added.
1277
1278 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1279
1280         Workaround for WebKit Bug 9630 when running contrib/recode.pl
1281
1282         A number of WebKit nightly builds included Bug 9630 which caused
1283         non-breaking space characters (0xA0) to be submitted within the
1284         content of textarea elements.
1285
1286         * contrib/recode.pl: To work around these 0xA0 characters, try
1287         an encoding of cp1252 and use it if it works.
1288
1289 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1290
1291         Don't print "Failed to guess" warning if an override is available
1292
1293         * contrib/recode.pl: Check that %overrides does NOT contain a
1294         key matching the current digest before printing out the warning
1295         about a failed charset guess.
1296
1297 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1298
1299         Added --[no-]truncate switch to contrib/recode.pl
1300
1301         * contrib/recode.pl: Added --[no-]truncate switch to make
1302         debugging of failed charset guessing easier.  Often times the
1303         illegal character was truncated in the output.
1304
1305 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1306
1307         Changed Perl scripts to use #!/usr/bin/env perl
1308
1309         perl -pi -e 's@#\!/usr/bin/perl@#\!/usr/bin/env perl@' *.cgi *.pl */*.pl
1310
1311 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1312
1313         Replace Bugzilla favicon with webkit.org favicon
1314
1315         * favicon.ico: Removed.
1316         * images/favicon.ico: Replaced.  Copied from favicon.ico.
1317
1318 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
1319
1320         Merged BugsSite to Bugzilla-3.2.3
1321
1322         Updated to the latest-and-greatest stable version.
1323
1324 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
1325
1326         Merged BugsSite to Bugzilla-3.0.3
1327
1328         Nothing to see here.  Move along.
1329
1330 2009-06-09  Eric Seidel  <eric@webkit.org>
1331
1332         Reviewed by Adam Roben.
1333
1334         Add support for displaying added png files in PrettyPatch diffs
1335         https://bugs.webkit.org/show_bug.cgi?id=26210
1336
1337         Currently this is SVN only (git-send-bugzilla patches exclude binary data)
1338         and only works for PNG files but could easily be made to work for other images as needed.
1339
1340         * PrettyPatch/PrettyPatch.rb:
1341
1342 2009-05-19  Eric Seidel  <eric@webkit.org>
1343
1344         Reviewed by Adam Roben.
1345
1346         Make PrettyPatch understand quoted filenames in git diffs.
1347         https://bugs.webkit.org/show_bug.cgi?id=25881
1348
1349         * PrettyPatch/PrettyPatch.rb:
1350
1351 2009-05-15  Simon Fraser  <simon.fraser@apple.com>
1352
1353         Carrying forwards Darin Adler's rubber stamp
1354         
1355         Make the comment field wider too.
1356
1357         * skins/custom/global.css:
1358
1359 2009-05-14  Simon Fraser  <simon.fraser@apple.com>
1360
1361         Reviewed by Darin Adler
1362         
1363         Make the URL, Summary and Keyword fields wider to mitigate the
1364         effects of https://bugs.webkit.org/show_bug.cgi?id=25566 and make
1365         things generally better.
1366
1367         * skins/custom/global.css:
1368         * template/en/custom/bug/edit.html.tmpl:
1369
1370 2008-11-10  Darin Adler  <darin@apple.com>
1371
1372         - fix more of https://bugs.webkit.org/show_bug.cgi?id=21400
1373           "Edit" links for patches in comments, review queue, review emails
1374           should be replaced by "Review Patch" links
1375
1376         * globals.pl: Fix cases that are automatically generated, such as links
1377         in comments in bugs.
1378
1379         * template/en/custom/attachment/created.html.tmpl: Fix the link on the
1380         "I just created an attachment" page.
1381
1382         * template/en/custom/request/queue.html.tmpl: Fix the link in the queue
1383         page.
1384
1385 2008-11-10  Darin Adler  <darin@apple.com>
1386
1387         * template/en/default/request/queue.html.tmpl: Use review links instead
1388         of edit links in the review queue.
1389
1390 2008-11-10  Darin Adler  <darin@apple.com>
1391
1392         * template/en/custom/request/email.txt.tmpl: Send review links instead
1393         of edit links when flags are set on a bug.
1394
1395 2008-10-27  Darin Adler  <darin@apple.com>
1396
1397         * template/en/custom/attachment/reviewform.html.tmpl: Leave out the
1398         comment if it's untouched.
1399
1400 2008-10-23  Adam Roben  <aroben@apple.com>
1401
1402         Fix Bug 21401: Comments field on "Review Patch" page should be
1403         initially filled with quoted patch
1404
1405         <https://bugs.webkit.org/show_bug.cgi?id=21401>
1406
1407         Reviewed by Dave Hyatt.
1408
1409         * attachment.cgi:
1410         (edit): Retrieve the attachment data from the database instead of just
1411         its length. When the attachment is a patch, create a quotedpatch
1412         template variable that contains the patch data with each line
1413         prepended with "> ".
1414         * template/en/custom/attachment/reviewform.html.tmpl:
1415           - Changed the comments field to have a monospace font
1416           - Added an "Enter comments below:" caption above the comments field
1417             to make it clearer that this is where comments should go, now that
1418             the comments field is not initially empty
1419           - Fill the comments field with the quoted patch
1420
1421 2008-10-14  Adam Roben  <aroben@apple.com>
1422
1423         Fix Bug 21602: Bugzilla times out trying to display formatted diff for
1424         attachment 24345
1425         
1426         https://bugs.webkit.org/show_bug.cgi?id=21602
1427
1428         Reviewed by Dave Kilzer.
1429
1430         * PrettyPatch/PrettyPatch.rb:
1431         (PrettyPatch.BINARY_FILE_MARKER_FORMAT): Added.
1432         (PrettyPatch.FileDiff.initialize): If any of the lines in the diff
1433         are a binary file marker, mark this FileDiff as binary and stop trying
1434         to process the lines.
1435         (PrettyDiff.FileDiff.to_html): If we're binary, just print a string
1436         saying so.
1437
1438 2008-10-02  Adam Roben  <aroben@apple.com>
1439
1440         * template/en/custom/attachment/reviewform.html.tmpl: Fix typo.
1441
1442 2008-10-02  Adam Roben  <aroben@apple.com>
1443
1444         * template/en/custom/attachment/review.html.tmpl: Fix URLs
1445
1446 2008-10-02  Adam Roben  <aroben@apple.com>
1447
1448         Fix Bug 21315: bugs.webkit.org should have a nicer patch review
1449         interface
1450
1451         https://bugs.webkit.org/show_bug.cgi?id=21315
1452
1453         Reviewed by Tim Hatcher.
1454
1455         * attachment.cgi:
1456         (top level): Added support for the "review" and "reviewform" actions.
1457         (sub edit): Accept the template name as a parameter. If no name is
1458         give, use "edit" as before.
1459         * template/en/custom/attachment/list.html.tmpl: Added a "Review Patch"
1460         link for all patch attachments.
1461         * template/en/custom/attachment/review.html.tmpl: Added. Simple
1462         <frameset> page to show the patch on the top and a comment form on the
1463         bottom.
1464         * template/en/custom/attachment/reviewform.html.tmpl: Added. Simple
1465         comment form for reviewing patches.
1466
1467 2008-10-02  Adam Roben  <aroben@apple.com>
1468
1469         Remove references to some backup files I created when implementing
1470         PrettyPatch in Bugzilla
1471
1472         Reviewed by Tim Hatcher.
1473
1474         * attachment-aroben.cgi: Removed.
1475         * template/en/default/attachment/edit-aroben.html.tmpl: Removed.
1476
1477         * template/en/custom/attachment/edit.html.tmpl: Removed an erroneous
1478         reference to attachment-aroben.cgi. attachment.cgi will work just
1479         fine.
1480
1481 2008-06-25  David Kilzer  <ddkilzer@apple.com>
1482
1483         Make PrettyPatch handle diffs with no Index or diff headers
1484
1485         Part of Bug 19290: More patches not handled by PrettyPatch.rb
1486         <https://bugs.webkit.org/show_bug.cgi?id=19290>
1487
1488         Reviewed by Adam.
1489
1490         * PrettyPatch/PrettyPatch.rb:
1491         (PrettyPatch.DIFF_HEADER_FORMATS): Added regular expression to
1492         match on "+++ " lines for patches with no "Index" or "diff" header.
1493         (PrettyPatch.FileDiff.initialize): Look for filename on "+++ " line
1494         when the first line of a patch has no "Index" or "diff" header.
1495         (PrettyPatch.FileDiff.parse): Added haveSeenDiffHeader state
1496         variable to determine when no "Index" or "diff" header has been
1497         found, but a new patch has started with a "--- " line.
1498
1499 2008-06-08  Dan Bernstein  <mitz@apple.com>
1500
1501         Reviewed by Adam Roben.
1502
1503         - update trac URL in PrettyPatch
1504
1505         * PrettyPatch/PrettyPatch.rb:
1506
1507 2008-05-29  Adam Roben  <aroben@apple.com>
1508
1509         Update PrettyPatch directory list
1510
1511         * PrettyPatch/PrettyPatch.rb:
1512
1513 2008-05-28  Adam Roben  <aroben@apple.com>
1514
1515         Make PrettyPatch able to handle diffs taken from outside the WebKit
1516         source tree
1517
1518         Part of Bug 19290: More patches not handled by PrettyPatch.rb
1519         <https://bugs.webkit.org/show_bug.cgi?id=19290>
1520
1521         Reviewed by David Kilzer.
1522
1523         * PrettyPatch/PrettyPatch.rb:
1524         (PrettyPatch.find_url_and_path): Added. Searches the file path from
1525         the bottom up looking for a directory that exists in the source tree.
1526         (PrettyPatch.linkifyFilename): Changed to call find_url_and_path.
1527
1528 2008-05-28  Adam Roben  <aroben@apple.com>
1529
1530         Print exceptions from PrettyPatch to stdout
1531
1532         This will make it much easier to debug problems with PrettyPatch,
1533         since we'll be able to see the exceptions in the browser.
1534
1535         Reviewed by Sam Weinig.
1536
1537         * PrettyPatch/prettify.rb: Added a --html-exceptions option, which
1538         will print exceptions to stdout as HTML.
1539         * attachment.cgi: Changed to pass --html-exceptions to prettify.rb.
1540
1541 2008-04-26  David Kilzer  <ddkilzer@apple.com>
1542
1543         Removed temp files and added appropriate svn:ignore properties.
1544
1545         Rubber-stamped by Mark Rowe.
1546
1547         * data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
1548
1549         * data/template: Added svn:ignore property for "template" subdirectory.
1550         * data/template/template: Removed precompiled template subdirectory.
1551
1552         * data/versioncache: Removed.
1553         * data/versioncache.*: Removed backup versioncache files.
1554
1555         * data/webdot: Added svn:ignore property for "*.dot" files.
1556         * data/webdot/*.dot: Removed cached webdot files.
1557
1558 2008-04-08  Adam Roben  <aroben@apple.com>
1559
1560         Combine :equal operations with the following operation if they are
1561         fewer than 3 characters long
1562
1563         This keeps us from showing lots of small changes on long lines, just
1564         because some letters happened to be the same.
1565
1566         Inspired by
1567         http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/diffviewer/diffutils.py?r=1264#147
1568
1569         * PrettyPatch/PrettyPatch.rb:
1570         (PrettyPatch::DiffSection.initialize): Remove :equal operations fewer
1571         than 3 characters long. The characters from the removed operations
1572         become part of the subsequent operation.
1573         (PrettyPatch::CodeLine.text_as_html): Don't wrap 0-length strings in
1574         <ins>/<del> tags. Also removed the @fromLineNumber.nil? and
1575         @toLineNumber.nil? checks as they are no longer reliable now that
1576         we're removing operations.
1577
1578 2008-04-08  Adam Roben  <aroben@apple.com>
1579
1580         Change PrettyPatch to use DiffBuilder for intra-line diffs
1581
1582         This gives us much prettier intra-line diffs, largely because it can
1583         distinguish multiple changes on the same line. e.g., if a line changes
1584         from:
1585
1586         const int myConstant = 0;
1587
1588         to
1589
1590         static const unsigned myConstant;
1591
1592         You will see that "static " was inserted, "int" changed to "unsigned",
1593         and " = 0" was deleted.
1594
1595         This seems to have also gotten rid of some spurious instances of
1596         "<ins></ins>" and "<del></del>" at the end of a line.
1597
1598         * PrettyPatch/PrettyPatch.rb:
1599         (PrettyPatch::DiffSection.initialize): Use DiffBuilder instead of
1600         setChangeExtentFromLine (which had been copied from Trac's diffing
1601         code).
1602         (PrettyPatch::CodeLine):
1603          - Removed the changeExtent attribute
1604          - Added the operations attribute
1605          - Removed the setChangeExtentFromLine method
1606         (PrettyPatch::CodeLine.text_as_html): Uses @operations to determine
1607         what text to wrap in <ins> and <del> tags.
1608         * PrettyPatch/diff.rb: Deleted a bunch of code we don't use.
1609         (HTMLDiff::DiffBuilder.initialize): Moved code here from the old build
1610         method, but left out the calls to perform_operation, since we build
1611         the diff HTML outside of this class.
1612         (HTMLDiff::DiffBuilder.split_inputs_to_words): Removed calls to
1613         convert_html_to_list_of_words, since we're not diffing HTML.
1614
1615 2008-04-08  Adam Roben  <aroben@apple.com>
1616
1617         Check in diff.rb
1618
1619         This came from
1620         http://instiki.rubyforge.org/svn/instiki/trunk/lib/diff.rb
1621         revision 28521
1622
1623         * PrettyPatch/diff.rb: Added.
1624
1625 2008-03-04  Adam Roben  <aroben@apple.com>
1626
1627         * PrettyPatch/PrettyPatch.rb: Allow the extent of the previous file to
1628         be omitted from the hunk header.
1629