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