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