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