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