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