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