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