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