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