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