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