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