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