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