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