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