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