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