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