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