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