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