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