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