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