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