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