fc635071124ed373715620e3064e4fa6e1052e2a
[WebKit-https.git] / Websites / bugs.webkit.org / ChangeLog
1 2016-10-09  Simon Fraser  <simon.fraser@apple.com>
2
3         Convert contributors.json to a flat list
4         https://bugs.webkit.org/show_bug.cgi?id=163183
5
6         Reviewed by Darin Adler.
7         
8         Make contributors.json easier to maintain by eliminating the big groupings ("Committers", "Reviewers" etc)
9         and moving that data to a "status" property on each contributor.
10         
11         Fix script that does bugzilla autocomplete.
12
13         * committers-autocomplete.js:
14         (statusToType):
15         (parseCommittersPy):
16         (parseType): Deleted.
17
18 2016-07-16  David Kilzer  <ddkilzer@apple.com>
19
20         Use secure (https) links on enter_bug.cgi
21         <https://webkit.org/b/159853>
22
23         Reviewed by Alexey Proskuryakov.
24
25         * template/en/default/global/choose-product.html.tmpl:
26         Use https instead of http for links to webkit.org.
27
28 2016-05-18  David Kilzer  <ddkilzer@apple.com>
29
30         CVE-2016-2803: [SECURITY] XSS vulnerability in dependency graphs via bug summary
31         <https://bugzilla.mozilla.org/show_bug.cgi?id=1253263>
32
33         Merge Bugzilla upstream master dd61903154fd363fb4e763d60aa155a507c2c3fc.
34
35         * showdependencygraph.cgi:
36         (CreateImagemap): Fix XSS vulnerability.
37
38 2016-05-05  David Kilzer  <ddkilzer@apple.com>
39
40         bugs.webkit.org: "See Also" field should support Chromium bugs
41
42         Merge Bugzilla upstream master 3af55bfe0bd10a85b7cd69e26a19034a6d2e78f5.
43         <https://bugzilla.mozilla.org/show_bug.cgi?id=1252782>
44
45         * Bugzilla/BugUrl.pm: Add Bugzilla::BugUrl::Chromium class to
46         list of classes that parse supported URLs.
47         * Bugzilla/BugUrl/Chromium.pm: Added.
48         (should_handle):
49         (_check_value):
50         * template/en/default/global/user-error.html.tmpl: Update error
51         message.
52
53 2015-12-09  Daniel Bates  <dabates@apple.com>
54
55         REGRESSION: code-review-tests.html fails with error "Received an error at line 214"
56         https://bugs.webkit.org/show_bug.cgi?id=152103
57
58         Reviewed by Darin Adler.
59
60         Fixes an issue where opening the file code-review-test.html in Safari shows the error message
61         "FAIL: Received an error at line 214" and in the console there is a JavaScript TypeError:
62             undefined is not an object (evaluating '$('.overallComments textarea').val().trim')
63
64         Notice that test testReaddDiscardedCommentWithPreviousComment() is the only test that appends
65         the toolbar, which inserts the overall comments textarea. The test testSaveCommentsWithMissingLineIds()
66         assumes that the own properties of the window object are enumerated in the same order as they
67         were defined in the file code-review-test.html such that the test testReaddDiscardedCommentWithPreviousComment()
68         is executed before test testSaveCommentsWithMissingLineIds(). The behavior of JavaScriptCore with
69         respect to the enumeration order of properties on the window object have changed since the
70         test testSaveCommentsWithMissingLineIds() was written such that the order of the own properties
71         on the window object do not reflect the order in which they were defined in the file. We should
72         append the toolbar before running the tests and explicitly execute the tests in chosen order so
73         as to be able to reason of the test results regardless of the enumeration ordering of the
74         properties in the DOM window object.
75
76         Also, removed duplicate 'd' in the name of function "testReaddDiscardedCommentWithPreviousComment"
77         such that it reads "testReadDiscardedCommentWithPreviousComment".
78
79         * code-review-test.html:
80
81 2015-11-17  Lucas Forschler  <lforschler@apple.com>
82
83         bugs.webkit.org is running with changes that are not checked in.
84         Check them in.
85
86         Reviewed by Matt Hanson.
87  
88 2015-07-30  Aakash Jain  <aakash_jain@apple.com>
89
90         Replace Google App Engine with AppScale
91         https://bugs.webkit.org/show_bug.cgi?id=147178
92
93         Reviewed by Daniel Bates.
94
95         * code-review.js:
96         * template/en/default/attachment/edit.html.tmpl:
97         * template/en/default/attachment/list.html.tmpl:
98         * template/en/default/attachment/reviewform.html.tmpl:
99
100 2014-09-03  David Kilzer  <ddkilzer@apple.com>
101
102         Make images work with patches created using svn 1.7
103         <http://webkit.org/b/136507>
104
105         Reviewed by Darin Adler.
106
107         * PrettyPatch/PrettyPatch.rb:
108         (PrettyPatch.prettify): Delete redundant patches that claim
109         newly added images are actually removed.
110         (PrettyPatch.SVN_BINARY_FILE_MARKER_FORMAT): Rename from
111         BINARY_FILE_MARKER_FORMAT.
112         (PrettyPatch.SVN_IMAGE_FILE_MARKER_FORMAT): Rename from
113         IMAGE_FILE_MARKER_FORMAT.
114         (PrettyPatch.SVN_PROPERTY_CHANGES_FORMAT): Add. Used to find
115         and ignore property changes in svn 1.7 patches.
116         (PrettyPatch.SVN_START_OF_BINARY_DATA_FORMAT): Rename from
117         START_OF_BINARY_DATA_FORMAT.
118         (FileDiff.filename): Add read-only accessor.
119         (FileDiff.image): Add read-only accessor.
120         (FileDiff.image_url): Add read-only accessor.
121         (FileDiff.initialize): Add special case for svn-1.7 image
122         patches that add a file.  Update various regex constants per
123         above.
124
125 2014-02-25  Jozsef Berta  <jberta.u-szeged@partner.samsung.com>
126
127         PrettyPatch.rb should be more descriptive for "git diff -M" styled patches
128         https://bugs.webkit.org/show_bug.cgi?id=125514
129
130         Reviewed by Csaba Osztrogonác.
131
132         * PrettyPatch/PrettyPatch.rb: PrettyPatch.rb is modified to show which file was renamed or moved.
133
134 2013-12-16  Dániel Bátyai  <Batyai.Daniel@stud.u-szeged.hu>
135
136         PrettyPatch.rb doesn't handle diffs where lines matching /^--/ are missing properly
137         https://bugs.webkit.org/show_bug.cgi?id=54928
138
139         Reviewed by Adam Roben.
140
141         PrettyPatch will now check if the next line after /^--- / matches /^+++ /, and if not,
142         then it's just a normal line
143
144         * PrettyPatch/PrettyPatch.rb:
145
146 2013-08-16  David Farler  <dfarler@apple.com>
147
148         PrettyPatch: Wrap pre blocks
149         https://bugs.webkit.org/show_bug.cgi?id=119906
150
151         Reviewed by Darin Adler.
152
153         * PrettyPatch/PrettyPatch.rb:
154         <pre> will use the same wrap settings as the text class:
155         white-space: pre-wrap and word-wrap: break-word.
156
157 2013-07-16  James Craig  <james@cookiecrook.com>
158
159         AX: Add a 'choose product' link for the new-ax-bug alias.
160         https://bugs.webkit.org/show_bug.cgi?id=118697
161
162         Reviewed by Chris Fleizach.
163
164         * template/en/custom/global/choose-product.html.tmpl: Added new link for accessibility bugs.
165
166 2013-07-09  Martin Robinson  <mrobinson@igalia.com>
167
168         PrettyPatch doesn't handle UTF-8 characters correctly
169         https://bugs.webkit.org/show_bug.cgi?id=45760
170
171         Reviewed by Adam Roben.
172
173         * PrettyPatch/PrettyPatch.rb: Since the diff lines are typically in UTF-8 and the results
174         of PrettyPatch may be served by a web server or not (for instance when accessed via
175         webkit-patch pretty-diff), we should use a meta tag to force the page encoding to UTF-8.
176
177 2013-05-23  Patrick Gansterer  <paroga@webkit.org>
178
179         EWS bubbles need more space
180         https://bugs.webkit.org/show_bug.cgi?id=116675
181
182         Reviewed by Andreas Kling.
183
184         * template/en/custom/attachment/list.html.tmpl:
185
186 2013-04-16  Ryosuke Niwa  <rniwa@webkit.org>
187
188         Build fix after r148527.
189
190         * committers-autocomplete.js:
191         (WebKitCommitters):
192
193 2013-04-16  Ryosuke Niwa  <rniwa@webkit.org>
194
195         The list of contributors in committers.py should be a separate JSON
196         https://bugs.webkit.org/show_bug.cgi?id=114673
197
198         Reviewed by Anders Carlsson.
199
200         Updated the Bugzilla autocompletion code to use contributors.json.
201         Basically, this removes a large chunk of this JavaScript file.
202
203         * committers-autocomplete.js:
204         (WebKitCommitters):
205
206 2013-04-13  Ryosuke Niwa  <rniwa@webkit.org>
207
208         Delete bubbles from webkit-commit-queue.appspot.com now that we've completed the transition
209         https://bugs.webkit.org/show_bug.cgi?id=114476
210
211         Reviewed by Gyuyoung Kim.
212
213         * code-review.js:
214         * template/en/custom/attachment/edit.html.tmpl:
215         * template/en/custom/attachment/list.html.tmpl:
216         * template/en/custom/attachment/reviewform.html.tmpl:
217
218 2013-04-10  Ryosuke Niwa  <rniwa@webkit.org>
219
220         Make old EWS bubbles less opaque to disambiguate two set of bubbles.
221
222         Rubber-stamped by Benjamin Poulain.
223
224         * code-review.js:
225         * template/en/custom/attachment/edit.html.tmpl:
226         * template/en/custom/attachment/list.html.tmpl:
227         * template/en/custom/attachment/reviewform.html.tmpl:
228
229 2013-04-10  Ryosuke Niwa  <rniwa@webkit.org>
230
231         Bugzilla should show bubbles from webkit-queues.appspot.com
232         https://bugs.webkit.org/show_bug.cgi?id=114391
233
234         Reviewed by Benjamin Poulain.
235
236         Include iframe from webkit-queues.appspot.com as well as one for webkit-commit-queue.appspot.com.
237
238         * code-review.js:
239         * template/en/custom/attachment/edit.html.tmpl:
240         * template/en/custom/attachment/list.html.tmpl:
241         * template/en/custom/attachment/reviewform.html.tmpl:
242
243 2013-03-21  Ryosuke Niwa  <rniwa@webkit.org>
244
245         EWS bubbles need more space
246         https://bugs.webkit.org/show_bug.cgi?id=112964
247
248         Reviewed by Philip Rogers.
249
250         Increased the width from 500px to 600px so that all bubbles fit.
251
252         * template/en/custom/attachment/list.html.tmpl:
253
254 2013-03-08  David Kilzer  <ddkilzer@apple.com>
255
256         (CVE-2013-0786) [SECURITY] build_subselect() leaks the existence of products and components you cannot access
257         <https://bugzilla.mozilla.org/show_bug.cgi?id=824399>
258         <exp2://Ticket/14465628>
259
260         Applied "v5 patch, 3.6" to bugs.webkit.org.
261
262         * Bugzilla/Config/GroupSecurity.pm:
263         (get_param_list):
264         * buglist.cgi:
265         * report.cgi:
266         * template/en/default/admin/params/groupsecurity.html.tmpl:
267
268 2013-03-08  David Kilzer  <ddkilzer@apple.com>
269
270         (CVE-2013-0785) [SECURITY] XSS in show_bug.cgi when using an invalid page format
271         <https://bugzilla.mozilla.org/show_bug.cgi?id=842038>
272         <exp2://Ticket/14465628>
273
274         Applied "patch for 3.6 and 4.0, v1" to bugs.webkit.org.
275
276         * Bugzilla/Template.pm:
277         (get_format):
278         * show_bug.cgi:
279
280 2013-02-01  Nathan de Vries  <ndevries@apple.com>
281
282         Allow prettify.rb to be run from any directory, and don't hard-code the system ruby path
283         https://bugs.webkit.org/show_bug.cgi?id=108569
284
285         Reviewed by Joseph Pecoraro.
286
287         * PrettyPatch/prettify.rb:
288
289 2013-01-03  James Robinson  <jamesr@chromium.org>
290
291         Avoid mixed content when expanding context in code review tool
292         https://bugs.webkit.org/show_bug.cgi?id=106060
293
294         Reviewed by Tony Chang.
295
296         Example failure: "The page at https://bugs.webkit.org/attachment.cgi?id=181222&action=review displayed insecure
297         content from http://svn.webkit.org/repository/webkit/trunk/Source/WebCore/page/EventHandler.cpp?p=138617." This
298         loads from "//svn.webkit.org/..." so the protocol matches.
299
300         * code-review.js:
301
302 2013-01-02  Ojan Vafai  <ojan@chromium.org>
303
304         REGRESSION: Review tool sometimes doesn't include some comments in preview & posts
305         https://bugs.webkit.org/show_bug.cgi?id=105252
306
307         Reviewed by Tony Chang.
308
309         When adding context, the LineContainer for the context line can get removed.
310         In that case, forEachLine needs to know to keep looping past that line number.
311
312         Also, make it so that you can't leave comments on context lines.
313
314         * code-review-test.html:
315         * code-review.js:
316
317 2012-12-30  Martin Robinson  <mrobinson@igalia.com>
318
319         PrettyDiff.rb fails to render image diffs with Ruby 1.9.3p194
320         https://bugs.webkit.org/show_bug.cgi?id=104932
321
322         Reviewed by Eric Seidel.
323
324         Ruby 1.9 added the concept of string encodings, so the use of `match' with binary data
325         treats it as UTF-8. Forcing the png_bytes string to have the "binary" encoding avoids
326         this error.
327
328         * PrettyPatch/PrettyPatch.rb: Use force_encoding.
329
330 2012-12-06  Ojan Vafai  <ojan@chromium.org>
331
332         Review page is super flakey now
333         https://bugs.webkit.org/show_bug.cgi?id=104331
334
335         Committing without review since this is breaking the code review tool.
336
337         remove is only shipping in Chromium. Use jQuery's remove instead.
338
339         * PrettyPatch/PrettyPatch.rb:
340         * code-review.js:
341
342 2012-12-05  Ojan Vafai  <ojan@chromium.org>
343
344         Make the strip line numbers link into a checkbox
345         https://bugs.webkit.org/show_bug.cgi?id=104187
346
347         Reviewed by Tony Chang.
348
349         The link text was confusing.
350
351         * PrettyPatch/PrettyPatch.rb:
352         * code-review.js:
353
354 2012-12-05  Ojan Vafai  <ojan@chromium.org>
355
356         Sanitize content on copy in the code review tool
357         https://bugs.webkit.org/show_bug.cgi?id=104155
358
359         Reviewed by Tony Chang.
360
361         Always remove expand/header/annotate links. Provide an option
362         to remove line numbers as well. Store the option in localStorage so
363         people can always get whichever behavior they want.
364
365         A better solution would be to restructure the DOM, but that would require gutting
366         the whole code review tool and would make it difficult to include line numbers if
367         you wanted them.
368
369         * PrettyPatch/PrettyPatch.rb:
370         * code-review-test.html:
371         * code-review.js:
372
373 2012-12-04  Ojan Vafai  <ojan@chromium.org>
374
375         Use sticky positioning for the code review toolbar
376         https://bugs.webkit.org/show_bug.cgi?id=104056
377
378         Reviewed by Adam Barth.
379
380         This simplifies the code and gives a nicer user-experience.
381         Also, while here, I fixed up the CSS to not have toolbar items
382         overlap when you make the window too small.
383
384         * PrettyPatch/PrettyPatch.rb:
385         * code-review-test.html:
386         * code-review.js:
387
388 2012-12-04  Ojan Vafai  <ojan@chromium.org>
389
390         Properly create the header links in the code review tool
391         https://bugs.webkit.org/show_bug.cgi?id=104037
392
393         Reviewed by Adam Barth.
394
395         Right now, we just prepend "header" to the two existing links and change
396         the first link to point to the header. The problem is that jQuery's prepend
397         method just prepends to each item in the jQuery object instead of adding
398         the item to the beginning of the list as this code assumed.
399
400         * code-review-test.html:
401         * code-review.js:
402
403 2012-12-04  Ojan Vafai  <ojan@chromium.org>
404
405         Can't add followup comment to a previous comment
406         https://bugs.webkit.org/show_bug.cgi?id=104025
407
408         Reviewed by Adam Barth.
409
410         If we side-by-sidify a shared diff line, and then apply
411         a previous comment, we would incorrectly put the comment
412         on the Line instead of the LineContainer.
413
414         Also, get rid of global next_line_id to simplify testing.
415
416         * code-review-test.html:
417         * code-review.js:
418
419 2012-11-06  Ryosuke Niwa  <rniwa@webkit.org>
420
421         committers-autocomplete.js works only with WebKit based browsers
422         https://bugs.webkit.org/show_bug.cgi?id=66752
423
424         Reviewed by Csaba Osztrogonác.
425
426         Based on the patch posted by Peter Gal. Attach event listeners
427         to all input elements we want to enable auto-completions on.
428
429         * committers-autocomplete.js:
430
431 2012-10-03  Simon Fraser  <simon.fraser@apple.com>
432
433         Make the status bubble iframe big enough to show all the bubbles
434         https://bugs.webkit.org/show_bug.cgi?id=98329
435
436         Reviewed by Eric Seidel.
437
438         Make the status bubble iframe big enough.
439
440         * template/en/custom/attachment/edit.html.tmpl:
441         * template/en/custom/attachment/list.html.tmpl:
442
443 2012-10-03  Simon Pena  <spena@igalia.com>
444
445         ruby1.9 fails in PrettyPatch.rb with invalid byte sequence in UTF-8
446         https://bugs.webkit.org/show_bug.cgi?id=91212
447
448         Reviewed by Hajime Morita.
449
450         ruby1.9 changed the way it handles encoding, so the gsub method used
451         when normalizing line ending complains of invalid byte sequence in UTF-8.
452         This patch uses the "encode" string method for versions of Ruby >= 1.9
453         to provide the encoding conversion (keeping the current implementation in
454         Ruby 1.8).
455
456         * PrettyPatch/PrettyPatch.rb: Use the "encode" string method for encoding
457         conversion when the Ruby version is >= 1.9.
458
459 2012-09-26  Sheriff Bot  <webkit.review.bot@gmail.com>
460
461         Unreviewed, rolling out r129521.
462         http://trac.webkit.org/changeset/129521
463         https://bugs.webkit.org/show_bug.cgi?id=97710
464
465         Broke autocomplete on enter_bug.cgi (Requested by abarth on
466         #webkit).
467
468         * committers-autocomplete.js:
469
470 2012-09-25  Peter Gal  <galpeter@inf.u-szeged.hu>
471
472         committers-autocomplete.js works only with WebKit based browsers
473         https://bugs.webkit.org/show_bug.cgi?id=66752
474
475         Reviewed by Darin Adler.
476
477         Modified the code do only add event listeners to the given inputs,
478         and not to the full document.
479
480         * committers-autocomplete.js:
481
482 2012-09-24  Peter Beverloo  <peter@chromium.org>
483
484         Allow robots to index bugs on Bugzilla
485         https://bugs.webkit.org/show_bug.cgi?id=95688
486
487         Reviewed by Eric Seidel.
488
489         This allows robots to access show_bug.cgi, thus granting them access to
490         index contents and discussion on WebKit bugs. All other pages, except
491         for the site index itself, still have indexing disabled.
492
493         * robots.txt:
494
495 2012-07-13  Ojan Vafai  <ojan@chromium.org>
496
497         PrettyPatch.rb complains about missing checksum for new pixel results
498         https://bugs.webkit.org/show_bug.cgi?id=88368
499
500         Reviewed by Tony Chang.
501
502         When adding or removing a file, we incorrectly iterpreted not having an image
503         as not having a checksum.
504
505         * PrettyPatch/PrettyPatch.rb:
506         * PrettyPatch/PrettyPatch_test.rb:
507         I tried to fix the TempFile issue in these tests, but after a couple hours
508         of banging my head against this, I have no idea what's breaking.
509
510 2012-07-11  Alice Cheng  <alice_cheng@apple.com>
511
512         Deleting content at the top of prettypatch emails destroys HTML formatting
513         https://bugs.webkit.org/show_bug.cgi?id=90700
514         <rdar://problem/7488232>
515
516         Reviewed by David Kilzer.
517
518         * PrettyPatch/PrettyPatch.rb:
519
520 2012-07-10  Adam Barth  <abarth@webkit.org>
521
522         bugs.webkit.org has mixed content
523         https://bugs.webkit.org/show_bug.cgi?id=90907
524
525         Reviewed by Eric Seidel.
526
527         Now that we detect http XMLHttpRequests as mixed content, I've noticed
528         that we're loading committers.py over http instead of https.  For
529         better security, we should use https.
530
531         * committers-autocomplete.js:
532
533 2012-05-30  Ojan Vafai  <ojan@chromium.org>
534
535         Only include the image checksum error in pretty-diffs for -expected.png files.
536         https://bugs.webkit.org/show_bug.cgi?id=87881
537
538         Reviewed by Tony Chang.
539
540         * PrettyPatch/PrettyPatch.rb:
541         * PrettyPatch/PrettyPatch_test.rb:
542
543 2012-05-29  Ojan Vafai  <ojan@chromium.org>
544
545         Show an error in the pretty diff when an image lacks a checksum
546         https://bugs.webkit.org/show_bug.cgi?id=87791
547
548         Reviewed by Dirk Pranke.
549
550         * PrettyPatch/PrettyPatch.rb:
551         * PrettyPatch/PrettyPatch_test.rb:
552
553 2012-04-01  Adam Barth  <abarth@webkit.org>
554
555         Code review tool no longer needs to work around position:fixed handling on iPad
556         https://bugs.webkit.org/show_bug.cgi?id=82850
557
558         Reviewed by Eric Seidel.
559
560         We no longer need to avoid position:fixed on iPad because Mobile Safari
561         now does something reasonable with position:fixed elements.
562
563         * code-review.js:
564
565 2012-03-09  Ashod Nakashian  <ashodnakashian@yahoo.com>
566
567         Bash scripts should support LF endings only
568         https://bugs.webkit.org/show_bug.cgi?id=79509
569
570         Reviewed by David Kilzer.
571
572         * contrib/yp_nomail.sh: Added property svn:executable.
573
574 2012-03-05  Alexander Færøy  <alexander.faeroy@nokia.com>
575
576         The Bugzilla autocompleter doesn't support multiple Unicode characters per name
577         https://bugs.webkit.org/show_bug.cgi?id=80319
578
579         Reviewed by Adam Barth.
580
581         The current regular expression for matching Unicode characters uses
582         the global modifier, which means that it will never enter its second
583         iteration of the match-loop. This patch fixes this by removing the
584         global modifier from the regex.
585
586         * committers-autocomplete.js:
587         (WebKitCommitters):
588
589 2012-01-03  Ojan Vafai  <ojan@chromium.org>
590
591         Increment the code-review.js version number to bust browser
592         caches.
593
594         * PrettyPatch/PrettyPatch.rb:
595
596 2011-12-13  Ojan Vafai  <ojan@chromium.org>
597
598         Give a resize handle for sidebyside diffs.
599         https://bugs.webkit.org/show_bug.cgi?id=74465
600
601         Reviewed by Ryosuke Niwa.
602
603         This gives a drag handle so you can make the side-by-side
604         diff not be 50/50 on a per-file-diff basis. This is useful
605         for cases like new files where there's no value on the
606         left side.
607
608         * PrettyPatch/PrettyPatch.rb:
609         * code-review.js:
610
611 2011-12-13  Ojan Vafai  <ojan@chromium.org>
612
613         Expanding context is broken for prepare-ChangeLog in the code review tool.
614         https://bugs.webkit.org/show_bug.cgi?id=74458
615
616         Reviewed by Adam Barth.
617
618         * code-review-test.html:
619         -Moved all the tests into test* functions.
620         -Automated calling all test* functions.
621         -Added testIsChangeLog.
622         * code-review.js:
623         Made the check for whether it's a ChangeLog file more robust.
624
625 2011-12-13  Ojan Vafai  <ojan@chromium.org>
626
627         Fix bug in the code review tool when readding a discarded comment
628         https://bugs.webkit.org/show_bug.cgi?id=74450
629
630         Reviewed by Adam Barth.
631
632         If you discard a comment that has a corresponding previousComment,
633         then we would incorrectly remove the comment baseline. So, the next
634         time you added a comment by clicking on the previousComment, we
635         would get undefined as the start line for the new comment.
636
637         All of this works fine until you try to restore the comment from
638         localStorage, at which point we throw an error because the start
639         line is undefined.
640
641         Also added some failsafes to better handle the case of corrupted comments.
642
643         * code-review-test.html:
644         * code-review.js:
645
646 2011-11-15  Tony Chang  <tony@chromium.org>
647
648         set a max-width on the codereview overall comments textarea
649         https://bugs.webkit.org/show_bug.cgi?id=72415
650
651         Reviewed by Ojan Vafai.
652
653         Otherwise, when resizing the textarea you can end up making wider, but
654         are no longer able to click the resize gripper to make it smaller
655         (it's under another div).
656
657         * PrettyPatch/PrettyPatch.rb:
658
659 2011-10-13  Adam Barth  <abarth@webkit.org>
660
661         Would be nice if the review-tool offered to show the corresponding header
662         https://bugs.webkit.org/show_bug.cgi?id=63963
663
664         Reviewed by Ojan Vafai.
665
666         This is a first cut at adding links to the header file in the review
667         tool.  If folks like this feature, we can cover more cases in the
668         future.
669
670         * code-review.js:
671
672 2011-09-14  Ben Wells  <benwells@chromium.org>
673
674         PrettyPatch should handle "delta" patch mechanism in git binary patches
675         https://bugs.webkit.org/show_bug.cgi?id=67628
676
677         Git patches are encoded using two mechanisms - "literal" and "delta".
678         For details of these mechanisms, see the function emit_binary_diff_body
679         in the git source file diff.c (https://github.com/git/git/blob/master/diff.c).
680
681         When determining if a binary file patch is an image or not we should accept
682         both literal and delta patch encodings.
683
684         When reconstructing the images from the patches, if we have a delta patch
685         we may download the previous revision from svn.webkit.org to get the image data.
686
687         Reviewed by Adam Roben.
688
689         * PrettyPatch/PrettyPatch.rb:
690         * PrettyPatch/PrettyPatch_test.rb:
691
692 2011-09-06  Sheriff Bot  <webkit.review.bot@gmail.com>
693
694         Unreviewed, rolling out r94554.
695         http://trac.webkit.org/changeset/94554
696         https://bugs.webkit.org/show_bug.cgi?id=67631
697
698         This patch did not quite fix the problem (Requested by
699         benwells on #webkit).
700
701         * PrettyPatch/PrettyPatch.rb:
702         * PrettyPatch/PrettyPatch_test.rb:
703
704 2011-09-06  Ben Wells  <benwells@chromium.org>
705
706         PrettyPatch should handle "delta" patch mechanism in git binary patches
707         https://bugs.webkit.org/show_bug.cgi?id=67628
708
709         Git patches are encoded using two mechanisms - "literal" and "delta".
710         See this email from the git mailing list archive for info
711         http://marc.info/?l=git&m=114682417113315&w=2
712
713         When determining if a binary file patch is an image or not we should accept
714         both literal and delta patch encodings.
715
716         Reviewed by Shinichiro Hamaji.
717
718         * PrettyPatch/PrettyPatch.rb:
719         * PrettyPatch/PrettyPatch_test.rb:
720
721 2011-06-30  Adam Barth  <abarth@webkit.org>
722
723         Reviewed by Eric Seidel.
724
725         [reviewtool] Expanding lines "below" does not remove purple context line
726         https://bugs.webkit.org/show_bug.cgi?id=63695
727
728         Honestly, this is the last bug in the review tool that bothers me.
729
730         * PrettyPatch/PrettyPatch.rb:
731             - Jump straight to version 42, because, well, you know.
732         * code-review.js:
733             - In addition to removing the context bar when we expand all, we
734               now remove it when we expand below (but not when we expand
735               above).
736
737 2011-06-21  Adam Barth  <abarth@webkit.org>
738
739         Reviewed by Darin Adler.
740
741         bugs.webkit.org should use Strict-Transport-Security
742         https://bugs.webkit.org/show_bug.cgi?id=63097
743
744         Strict-Transport-Security forces all connections to bugs.webkit.org to
745         use HTTPS, preventing sslstrip and other attacks.
746
747         * .htaccess:
748
749 2011-05-04  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
750
751         Reviewed by Adam Roben.
752
753         REGRESSION (r85657): webkitpy.common.prettypatch_unittest.PrettyPatchTest.test_pretty_diff_encodings failing on Leopard
754         https://bugs.webkit.org/show_bug.cgi?id=60164
755
756         Use Array#length to get the number of elements in an array. Array#count
757         was added in Ruby 1.8.7, but Leopard have Ruby 1.8.6.
758
759         * PrettyPatch/PrettyPatch.rb:
760
761 2011-05-03  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
762
763         Reviewed by Adam Roben.
764
765         Add tests to PrettyPatch
766         https://bugs.webkit.org/show_bug.cgi?id=57298
767
768         Verify that 'prettify' parse the right number of files and parts
769         (add/remove/shared) by checking a set of patches available in
770         bugs.webkit.org.
771
772         * PrettyPatch/PrettyPatch.rb: exported a couple a globals with
773         statistics information for our test script.
774         * PrettyPatch/PrettyPatch_test.rb: Added.
775
776 2011-04-22  Alexey Proskuryakov  <ap@apple.com>
777
778         Code changes by Adam Barth. Reviewed by Ojan Vafai.
779
780         Bugzilla autocomplete should support frequent contributors who are not committers
781         https://bugs.webkit.org/show_bug.cgi?id=59209
782
783         * committers-autocomplete.js: Support Contributor role in committers.py.
784
785 2011-04-19  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
786
787         Reviewed by Adam Roben.
788
789         PrettyPatch tries to show images for deleted files
790         https://bugs.webkit.org/show_bug.cgi?id=28457
791
792         When the image data is empty, instead of generating an <img> tag, generate
793         a text to indicate the image was deleted.
794
795         * PrettyPatch/PrettyPatch.rb:
796
797 2011-04-14  Tony Chang  <tony@chromium.org>
798
799         Reviewed by Adam Roben.
800
801         make PrettyPatch.rb aware of checksums in png files
802         https://bugs.webkit.org/show_bug.cgi?id=58387
803
804         Patch v2 with nil checks in the case of file adds and deletes.
805
806         * PrettyPatch/PrettyPatch.rb:
807
808 2011-04-13  Simon Fraser  <simon.fraser@apple.com>
809
810         Revert http://trac.webkit.org/changeset/83750 because it broke PrettyPatch.
811         https://bugs.webkit.org/show_bug.cgi?id=58510        
812
813         * PrettyPatch/PrettyPatch.rb:
814
815 2011-04-12  Tony Chang  <tony@chromium.org>
816
817         Reviewed by Adam Roben.
818
819         make PrettyPatch.rb aware of checksums in png files
820         https://bugs.webkit.org/show_bug.cgi?id=58387
821
822         * PrettyPatch/PrettyPatch.rb:
823
824 2011-04-12  Adam Barth  <abarth@webkit.org>
825
826         Reviewed by Eric Seidel.
827
828         Review tool should work well on iPad
829         https://bugs.webkit.org/show_bug.cgi?id=58351
830
831         On the iPad, the toolbar ends up at some random position in the middle
832         of the page because of how position:fixed interacts with scrolling on
833         the iPad.  I feel a bit dirty using a user agent detect, but I'm not
834         sure if there's a better way to detect whether the user agent has this
835         behavior.
836
837         * code-review.js:
838
839 2011-03-15  David Levin  <levin@chromium.org>
840
841         Reviewed by Adam Barth.
842
843         Links from code reviews to trac are incorrect for files in Source/WebKit.
844         https://bugs.webkit.org/show_bug.cgi?id=56404
845
846         * PrettyPatch/PrettyPatch.rb: Remove WebKit from the list of top level
847           directories since it is now under Source.
848
849 2011-03-10  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
850
851         Reviewed by Adam Roben.
852
853         Formatted Diff for attachment 23920 is mangled
854         https://bugs.webkit.org/show_bug.cgi?id=21222
855
856         The mangled diff files mentioned in the bug were using Mac line
857         ending and this was causing problems for String#each_line. Now we
858         normalize the line endings in patch data before processing it.
859
860         * PrettyPatch/PrettyPatch.rb:
861
862 2011-03-10  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
863
864         Reviewed by Adam Roben.
865
866         PrettyPatch displays last two lines of a git-format-patch diff strangely
867         https://bugs.webkit.org/show_bug.cgi?id=29317
868
869         If the diff section has full range information, we parse only enough to cover the
870         range. This avoids incorrectly showing trailing lines (like git signature) as part
871         of the patch.
872
873         * PrettyPatch/PrettyPatch.rb:
874
875 2011-03-10  Adam Roben  <aroben@apple.com>
876
877         Don't search for intra-line diffs in really long lines
878
879         Doing so can lead to hangs (or at least really slow execution).
880
881         Fixes <http://webkit.org/b/56109> run-webkit-tests sometimes times out on Windows XP Debug
882         (Tests) after fast/text/large-text-composed-char-dos.html fails (due to PrettyPatch
883         hanging?)
884
885         Reviewed by David Kilzer.
886
887         * PrettyPatch/PrettyPatch.rb:
888         (PrettyPatch.MAXIMUM_INTRALINE_DIFF_LINE_LENGTH): Added this new constant.
889         (PrettyPatch.DiffSection.initialize): Don't bother looking for intra-line diffs in lines
890         longer than the maximum length.
891
892 2011-02-25  Ojan Vafai  <ojan@chromium.org>
893
894         Reviewed by Adam Barth.
895
896         [codereviewtool] fix expand links on git diffs
897         https://bugs.webkit.org/show_bug.cgi?id=55208
898
899         In git diffs, we don't have a "revision" span at the end
900         of each FileDiff.
901         * code-review.js:
902
903 2011-02-23  Ojan Vafai  <ojan@chromium.org>
904
905         Reviewed by Tony Chang.
906
907         autocomplete=off prevents refilling form fields on back/forward navigation
908         https://bugs.webkit.org/show_bug.cgi?id=47463
909
910         I haven't actually run this patch due to the difficulty of getting
911         a bugzilla instance running. It's simple enough code that I'm not too worried
912         about it.
913
914         * committers-autocomplete.js:
915
916 2011-02-21  Ojan Vafai  <ojan@chromium.org>
917
918         Reviewed by Adam Barth.
919
920         [codereviewtool] focus first visible node if no node is focused
921         https://bugs.webkit.org/show_bug.cgi?id=54935
922
923         Now hitting j/k/n/p will focus the first/last visible node
924         if no other node is focused. Also tweaked the scroll into view
925         logic a bit to avoid scrolling in this case.
926
927         * code-review.js:
928
929 2011-02-21  Ojan Vafai  <ojan@chromium.org>
930
931         Reviewed by Adam Barth.
932
933         [codereviewtool] remove patch fuzzing
934         https://bugs.webkit.org/show_bug.cgi?id=54940
935
936         This code is buggy and not as necessary now that we
937         include svn revision numbers in uploaded patches.
938
939         * code-review.js:
940
941 2011-02-21  Ojan Vafai  <ojan@chromium.org>
942
943         Reviewed by Adam Barth.
944
945         [codereviewtool] focus comment on accept
946         https://bugs.webkit.org/show_bug.cgi?id=54930
947
948         * code-review.js:
949
950 2011-02-20  Ojan Vafai  <ojan@chromium.org>
951
952         Reviewed by Adam Barth.
953
954         [codereviewtool] avoid overriding ctrl/cmd + r
955         https://bugs.webkit.org/show_bug.cgi?id=54852
956
957         * code-review.js:
958
959 2011-02-16  Ojan Vafai  <ojan@chromium.org>
960
961         Reviewed by Adam Barth.
962
963         keyboard support for extending/shrinking comment context
964         https://bugs.webkit.org/show_bug.cgi?id=54612
965
966         ctrl+shift+up/down will extend/shrink the comment context when
967         a comment is focused or when one is being edited.
968
969         Also, switch over to using keydown instead of keypress events.
970         This lets us share code for handling escape and other key events.
971         Also, keypress is evil and should die.
972
973         * code-review.js:
974
975 2011-02-20  Ojan Vafai  <ojan@chromium.org>
976
977         Recommit accidental revert.
978         r79180 accidentally revereted r79178.
979         * code-review.js:
980
981 2011-02-20  Ojan Vafai  <ojan@chromium.org>
982
983         Reviewed by Adam Barth.
984
985         [codereviewtool] add more help information
986         https://bugs.webkit.org/show_bug.cgi?id=54847
987
988         * code-review.js:
989
990 2011-02-20  Ojan Vafai  <ojan@chromium.org>
991
992         Reviewed by Adam Barth.
993
994         [codereviewtool] use 'r' to focus the review select element
995         https://bugs.webkit.org/show_bug.cgi?id=54845
996
997         This allows for quick keyboard access to the toolbar items.
998         Hit r to get to the review select element. Then you can
999         tab to the cq/preview/publish elements.
1000
1001         * code-review.js:
1002
1003 2011-02-20  Ojan Vafai  <ojan@chromium.org>
1004
1005         Reviewed by Adam Barth.
1006
1007         [codereviewtool] don't let lines/diffblocks be mouse focusable
1008         https://bugs.webkit.org/show_bug.cgi?id=54851
1009
1010         Unfortunately, this will only work once https://bugs.webkit.org/show_bug.cgi?id=54727
1011         is committed. In the interim, it won't break anything.
1012
1013         * code-review.js:
1014
1015 2011-02-20  Ojan Vafai  <ojan@chromium.org>
1016
1017         Reviewed by Adam Barth.
1018
1019         [codereviewtool] use keydown instead of keypress
1020         https://bugs.webkit.org/show_bug.cgi?id=54849
1021
1022         There is no functional change. This is just a cleanup
1023         to make future patches (e.g. https://bugs.webkit.org/show_bug.cgi?id=54612)
1024         cleaner.
1025
1026         * code-review.js:
1027
1028 2011-02-20  Ojan Vafai  <ojan@chromium.org>
1029
1030         Reviewed by Adam Barth.
1031
1032         [codereviewtool] fix layout nit
1033         https://bugs.webkit.org/show_bug.cgi?id=54848
1034
1035         There is occasionally an off-by-one in the 50% width
1036         calculation. This ensures that the add lines always
1037         line up with the shared context lines.
1038
1039         * PrettyPatch/PrettyPatch.rb:
1040
1041 2011-02-20  Ojan Vafai  <ojan@chromium.org>
1042
1043         Reviewed by Adam Barth.
1044
1045         [codereviewtool] make escape hide the preview form
1046         https://bugs.webkit.org/show_bug.cgi?id=54844
1047
1048         * code-review.js:
1049
1050 2011-02-20  Ojan Vafai  <ojan@chromium.org>
1051
1052         Reviewed by Adam Barth.
1053
1054         [codereviewtool] make enter work when an individual line is focused
1055         https://bugs.webkit.org/show_bug.cgi?id=54843
1056
1057         * code-review.js:
1058
1059 2011-02-17  Ojan Vafai  <ojan@chromium.org>
1060
1061         Reviewed by Adam Barth.
1062
1063         use shift+j/k to focus next/previous line
1064         https://bugs.webkit.org/show_bug.cgi?id=54723
1065
1066         Tested on a large Hyatt change to confirm it performed fine.
1067         (https://bugs.webkit.org/attachment.cgi?id=82533&action=review)
1068
1069         * code-review.js:
1070
1071 2011-02-17  Ojan Vafai  <ojan@chromium.org>
1072
1073         Reviewed by Adam Barth.
1074
1075         fix escape handling in code review tool
1076         https://bugs.webkit.org/show_bug.cgi?id=54722
1077
1078         This broke with http://trac.webkit.org/changeset/78948.
1079         Apparently jquery events get a wrapped object that
1080         does not expose keyIdentifier.
1081
1082         * code-review.js:
1083
1084 2011-02-17  Ojan Vafai  <ojan@chromium.org>
1085
1086         Reviewed by Adam Barth.
1087
1088         create comments when hitting enter with a focused diff
1089         https://bugs.webkit.org/show_bug.cgi?id=54719
1090
1091         * code-review.js:
1092
1093 2011-02-17  Ojan Vafai  <ojan@chromium.org>
1094
1095         Reviewed by Adam Barth.
1096
1097         use actual browser focus in the code review tool
1098         https://bugs.webkit.org/show_bug.cgi?id=54726
1099
1100         This makes keyboard handling play better with focusable element
1101         (i.e. links/textareas/buttons). Also, in theory, this makes
1102         the review tool more amenable to screen readers (I think).
1103
1104         * PrettyPatch/PrettyPatch.rb:
1105         * code-review.js:
1106
1107 2011-02-17  Ojan Vafai  <ojan@chromium.org>
1108
1109         Reviewed by Antonio Gomes.
1110
1111         make event handling work in Gecko
1112         https://bugs.webkit.org/show_bug.cgi?id=54702
1113
1114         Gecko doesn't support the global event property. It's an IE-ism.
1115
1116         * code-review.js:
1117
1118 2011-02-14  Ojan Vafai  <ojan@chromium.org>
1119
1120         Reviewed by Adam Barth.
1121
1122         allow for modifying comments without the mouse
1123         https://bugs.webkit.org/show_bug.cgi?id=54433
1124
1125         Use n/p to navigate the comments.
1126         Enter to begin editing a comment.
1127         Escape to finish editing a comment.
1128
1129         * code-review.js:
1130
1131 2011-02-14  Ojan Vafai  <ojan@chromium.org>
1132
1133         Reviewed by Adam Barth.
1134
1135         improve line selection in the code review tool
1136         https://bugs.webkit.org/show_bug.cgi?id=54430
1137
1138         -shift+click now extends the comment context
1139         -selecting outside of a diff section no longer clears the selected lines.
1140          Instead it just restricts the selected lines to that diff section.
1141
1142         * code-review.js:
1143
1144 2011-02-10  Ojan Vafai  <ojan@chromium.org>
1145
1146         Reviewed by Adam Barth.
1147
1148         remove extra space from expansion lines
1149         https://bugs.webkit.org/show_bug.cgi?id=54266
1150
1151         * code-review.js:
1152
1153 2011-02-09  Ojan Vafai  <ojan@chromium.org>
1154
1155         Reviewed by Adam Barth.
1156
1157         only erase draft comments after publish is successful
1158         https://bugs.webkit.org/show_bug.cgi?id=54163
1159
1160         If there is a conflict or 500, then draft comments will survive.
1161         One drawback here is that the form post is now to the iframe,
1162         so to break out of the iframe we redirect to the bug page, which
1163         loses the information of who the email was sent to.
1164
1165         Once WebKit supports seamless iframes we should be able to avoid
1166         the redirect.
1167
1168         * code-review.js:
1169
1170 2011-02-09  Ojan Vafai  <ojan@chromium.org>
1171
1172         Reviewed by Adam Barth.
1173
1174         save overall comments when saving drafts in the review tool
1175         https://bugs.webkit.org/show_bug.cgi?id=54165
1176
1177         -save overall comments in localstorage as well
1178         -save all draft comments as you type
1179         -give a *subtle* indicator of saved state
1180
1181         The latter should also make it super easy if someone wanted to do
1182         the work to store draft comments in appengine/s3/bugzilla/etc.
1183
1184         * PrettyPatch/PrettyPatch.rb:
1185         * code-review-test.html:
1186         * code-review.js:
1187
1188 2011-02-08  Ojan Vafai  <ojan@chromium.org>
1189
1190         Reviewed by Adam Barth.
1191
1192         fix toolbar anchoring in the code review tool
1193         https://bugs.webkit.org/show_bug.cgi?id=54058
1194
1195         Avoid the anchoring cycle of doom when on the cusp
1196         of whether the toolbar needs to be anchored and
1197         speculatively avoid the Firefox crash when resizing.
1198
1199         * PrettyPatch/PrettyPatch.rb:
1200         * code-review.js:
1201
1202 2011-02-07  Ojan Vafai  <ojan@chromium.org>
1203
1204         Reviewed by Adam Barth.
1205
1206         bring back diff context lines into the review tool
1207         https://bugs.webkit.org/show_bug.cgi?id=53974
1208
1209         * code-review.js:
1210
1211 2011-02-01  Ojan Vafai  <ojan@chromium.org>
1212
1213         Reviewed by Adam Roben.
1214
1215         include svn revisions in git diffs for the code review tool to use
1216         https://bugs.webkit.org/show_bug.cgi?id=53569
1217
1218         * PrettyPatch/PrettyPatch.rb:
1219
1220 2011-02-01  Ojan Vafai  <ojan@chromium.org>
1221
1222         Reviewed by Adam Barth.
1223
1224         make draft comments focusable
1225         https://bugs.webkit.org/show_bug.cgi?id=53554
1226
1227         Makes frozen draft comments focusable. The ones that are currently being edited are not.
1228         I'm on the fence whether they should be, but this seems good enough for now.
1229
1230         * code-review.js:
1231
1232 2011-02-01  Ojan Vafai  <ojan@chromium.org>
1233
1234         Reviewed by Adam Barth.
1235
1236         avoid jitter when loading the comments to a patch
1237         https://bugs.webkit.org/show_bug.cgi?id=53570
1238
1239         * code-review.js:
1240
1241 2011-02-01  Ojan Vafai  <ojan@chromium.org>
1242
1243         Reviewed by Mihai Parparita.
1244
1245         [codereviewtool] extra space at the beginning of lines in side-by-side view
1246         https://bugs.webkit.org/show_bug.cgi?id=53550
1247
1248         The newline between the spans turns into user-visible space in side-by-side mode.
1249
1250         * PrettyPatch/PrettyPatch.rb:
1251
1252 2011-02-01  Ojan Vafai  <ojan@chromium.org>
1253
1254         Increment version number to cache-bust code-review.js.
1255
1256         * PrettyPatch/PrettyPatch.rb:
1257
1258 2011-01-31  Ojan Vafai  <ojan@chromium.org>
1259
1260         Reviewed by Adam Barth.
1261
1262         Store draft comments in localStorage
1263         https://bugs.webkit.org/show_bug.cgi?id=52866
1264
1265         * code-review.js:
1266         * code-review-test.html
1267
1268 2011-01-20  Ojan Vafai  <ojan@chromium.org>
1269
1270         Fix the review tool for image diffs. We would get a javascript error
1271         because image diffs don't have line numbers.
1272
1273         * code-review.js:
1274
1275 2011-01-20  Ojan Vafai  <ojan@chromium.org>
1276
1277         Reviewed by Oliver Hunt.
1278
1279         Could review tool include style failure info?
1280         https://bugs.webkit.org/show_bug.cgi?id=49049
1281
1282         This does not handle patches that have old paths (e.g. without the Source prefix).
1283         Although that would be fairly straightforward to workaround.
1284
1285         * code-review.js:
1286
1287 2011-01-20  Ojan Vafai  <ojan@chromium.org>
1288
1289         Reviewed by Adam Barth.
1290
1291         handle cases where PrettyPatch.rb doesn't linkify filenames
1292         https://bugs.webkit.org/show_bug.cgi?id=52834
1293
1294         * code-review.js:
1295
1296 2011-01-20  Ojan Vafai  <ojan@chromium.org>
1297
1298         Reviewed by Adam Barth.
1299
1300         fix goofup in http://trac.webkit.org/changeset/76082
1301         https://bugs.webkit.org/show_bug.cgi?id=52830
1302
1303         We were showing the revision number of the patch at the bottom
1304         of the diff and expanding from the bottom of the diff would fail.
1305
1306         Also, removed an unused variable and scoped queries appropriatly
1307         to the file_diff they should have been operating on.
1308
1309         * PrettyPatch/PrettyPatch.rb:
1310         * code-review.js:
1311
1312 2011-01-19  Ojan Vafai  <ojan@chromium.org>
1313
1314         Reviewed by Adam Barth.
1315
1316         [reviewtool] Add a link for annotated trac page on review page
1317         https://bugs.webkit.org/show_bug.cgi?id=52747
1318
1319         -add line numbers to all trac links to the first line in the file diff
1320         -add annotate/review log links
1321         -make file-diff links other than the file name only show when you mouseover the filediff
1322         -fix white-space to only wrap the .text element and it's childen, any of it's siblings/parents
1323
1324         * PrettyPatch/PrettyPatch.rb:
1325         * code-review.js:
1326
1327 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1328
1329         Reviewed by Adam Barth.
1330
1331         make line selection have a extend only from where you start the selection in the code review tool
1332         https://bugs.webkit.org/show_bug.cgi?id=52485
1333
1334         * code-review.js:
1335
1336 2011-01-18  Ojan Vafai  <ojan@chromium.org>
1337
1338         Reviewed by Eric Seidel.
1339
1340         make file expansion use the file at the svn revision the diff was created at
1341         https://bugs.webkit.org/show_bug.cgi?id=52650
1342
1343         This only works for patches created with SVN for now since the git
1344         diff does not include SVN revision numbers in it.
1345
1346         * PrettyPatch/PrettyPatch.rb:
1347         * code-review.js:
1348
1349 2011-01-15  Adam Barth  <abarth@webkit.org>
1350
1351         Rubber-stamped by Eric Seidel.
1352
1353         Move WebKit2 into Source
1354         https://bugs.webkit.org/show_bug.cgi?id=52438
1355
1356         Remove WebKit2 from the list of top-level source directories.
1357
1358         * PrettyPatch/PrettyPatch.rb:
1359
1360 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1361
1362         Reviewed by Adam Barth.
1363
1364         fix expanded lines in the code review tool
1365         https://bugs.webkit.org/show_bug.cgi?id=52498
1366
1367         Fixes them to work now that we have wrapper divs.
1368         Also fixes the long-standing bug that the line numbers
1369         were incorrect sometimes.
1370
1371         * code-review.js:
1372
1373 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1374
1375         Reviewed by Adam Barth.
1376
1377         put remove lines to the left of add lines in sidebyside view
1378         https://bugs.webkit.org/show_bug.cgi?id=52458
1379
1380         Also, sneaking in a fix to make long lines with no whitespace wrap.
1381
1382         * PrettyPatch/PrettyPatch.rb:
1383         * code-review.js:
1384
1385 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1386
1387         Fix bad merge in r75816.
1388
1389         * code-review.js:
1390
1391 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1392
1393         Reviewed by Adam Barth.
1394
1395         improve line selection with fast drags
1396         https://bugs.webkit.org/show_bug.cgi?id=52477
1397
1398         * code-review.js:
1399
1400 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1401
1402         Reviewed by Adam Barth.
1403
1404         tweak line selection in code review tool
1405         https://bugs.webkit.org/show_bug.cgi?id=52462
1406
1407         -Improve handling of data-comment-base-line code to deal with
1408         lines that have multiple values.
1409         -Make it so that if you click on the line immediately above
1410         a line that has a comment it will add a new comment instead of
1411         adding lines to the following comment. If the last selected line
1412         overlaps existing comment lines though they will still get
1413         added to the existing comment.
1414
1415         * code-review.js:
1416
1417 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1418
1419         Reviewed by Adam Roben.
1420
1421         fix bugs going back and forth between unified and sidebyside
1422         https://bugs.webkit.org/show_bug.cgi?id=52470
1423
1424         Now that we're appending DOM nodes directly, we only want to
1425         get the first ".text" element in the subtree. In the cases
1426         where there are multiple, their contents are the same.
1427
1428         Also, when making side-by-side expansion lines, we need to
1429         clone the contents so we have a second copy to append to the DOM.
1430
1431         * code-review.js:
1432
1433 2011-01-13  Ojan Vafai  <ojan@chromium.org>
1434
1435         Reviewed by Adam Barth.
1436
1437         simplify keyboard handling in code review tool
1438         https://bugs.webkit.org/show_bug.cgi?id=52407
1439
1440         Now that we have DiffBlock containers, the only things that are
1441         focusable are previousComment nodes and DiffBlock containers
1442         that contain add/remove lines.
1443
1444         Also, this means we show the focus border around the entire diff
1445         instead of just the first line.
1446
1447         * code-review.js:
1448
1449 2011-01-13  Ojan Vafai  <ojan@chromium.org>
1450
1451         Reviewed by Adam Barth.
1452
1453         add container divs for diff blocks
1454         https://bugs.webkit.org/show_bug.cgi?id=52400
1455
1456         This will help simplify a lot of code in code-review.js
1457         and make side-by-side diffs better (i.e. put removed lines
1458         to the left of corresponding added lines).
1459
1460         Also, allow for running the JS from a local file. Now you can modify code-review.js
1461         to point to a local file and then run:
1462         ruby prettify.rb < foo.diff > foo.html
1463
1464         foo.html will load a dummy code review matching foo.diff.
1465
1466         Before structure:
1467         Line
1468         Line remove
1469         Line add
1470         Line add
1471         Line
1472
1473         After structure:
1474         DiffBlock
1475           DiffBlockPart shared
1476             Line shared
1477         DiffBlock
1478           DiffBlockPart remove
1479             Line remove
1480           DiffBlockPart add
1481             Line add
1482             Line add
1483         DiffBlock
1484           DiffBlockPart shared
1485             Line shared
1486
1487         * PrettyPatch/PrettyPatch.rb:
1488         * code-review.js:
1489
1490 2011-01-12  Ojan Vafai  <ojan@chromium.org>
1491
1492         Reviewed by Mihai Parparita.
1493
1494         fix assorted bugs with expansion lines
1495         https://bugs.webkit.org/show_bug.cgi?id=52326
1496
1497         -Clicking "all" would collapse whitespace in the expanded lines.
1498         -Converted the remaining line creation code to using the DOM.
1499
1500         * PrettyPatch/PrettyPatch.rb:
1501         * code-review.js:
1502
1503 2011-01-12  Ojan Vafai  <ojan@chromium.org>
1504
1505         Reviewed by Adam Barth.
1506
1507         review tool formatted diff doesn't match the uploaded diff
1508         https://bugs.webkit.org/show_bug.cgi?id=51960
1509
1510         PrettyPatch works just fine if there is no newline at the end of the file.
1511
1512         * attachment.cgi:
1513
1514 2011-01-12  Ojan Vafai  <ojan@chromium.org>
1515
1516         Reviewed by Adam Barth.
1517
1518         Bugzilla: Add keyboard shortcuts to jump to next change
1519         https://bugs.webkit.org/show_bug.cgi?id=52305
1520
1521         Comments and diff blocks go in the same queue. If you have a 
1522         comment focused, then j/k will focus the next/prev diff block
1523         with respect to that comment.
1524
1525         * PrettyPatch/PrettyPatch.rb:
1526         * code-review.js:
1527
1528 2011-01-12  Ojan Vafai  <ojan@chromium.org>
1529
1530         Reviewed by Adam Barth.
1531
1532         show shared lines on both sides in code review tool
1533         https://bugs.webkit.org/show_bug.cgi?id=52308
1534
1535         We were appending the same DOM node twice. Instead, make
1536         the from column a clone of the original node.
1537
1538         * code-review.js:
1539
1540 2011-01-12  Ojan Vafai  <ojan@chromium.org>
1541
1542         Reviewed by Adam Barth.
1543
1544         fix wrapping bug in expansion lines in the code review tool
1545         https://bugs.webkit.org/show_bug.cgi?id=52270
1546
1547         * PrettyPatch/PrettyPatch.rb:
1548         * code-review.js:
1549
1550 2011-01-11  Ojan Vafai  <ojan@chromium.org>
1551
1552         Reviewed by Adam Barth.
1553
1554         fix difflink centering to be vertical, not horizontal
1555         https://bugs.webkit.org/show_bug.cgi?id=52263
1556
1557         * PrettyPatch/PrettyPatch.rb:
1558         * code-review.js:
1559
1560 2011-01-11  Ojan Vafai  <ojan@chromium.org>
1561
1562         Reviewed by Adam Barth.
1563
1564         remember diffstate for review tool
1565         https://bugs.webkit.org/show_bug.cgi?id=52253
1566
1567         If you use the global sidebyside/unified links, store the diff type
1568         in localstorage. Then, onload, use that diff type. That way, people
1569         who prefer one diff type or the other can always get that by default.
1570
1571         * code-review.js:
1572
1573 2011-01-11  Ojan Vafai  <ojan@chromium.org>
1574
1575         Reviewed by Adam Barth.
1576
1577         allow sidebysideifying individual files in the code review tool
1578         https://bugs.webkit.org/show_bug.cgi?id=52226
1579
1580         * code-review.js:
1581
1582 2011-01-11  Ojan Vafai  <ojan@chromium.org>
1583
1584         Reviewed by Mihai Parparita.
1585
1586         fix exception when adding a comment to a side-by-side diff
1587         https://bugs.webkit.org/show_bug.cgi?id=52240
1588
1589         prev() and next() won't get the previous and next lines in
1590         side-by-side mode. Instead do a query to find them.
1591
1592         * code-review.js:
1593
1594 2011-01-11  Ojan Vafai  <ojan@chromium.org>
1595
1596         Reviewed by Mihai Parparita.
1597
1598         maintain word diffs when converting to side-by-side and back
1599         https://bugs.webkit.org/show_bug.cgi?id=52244
1600
1601         Using textContent would lose the spans use to make the word-diff highlighting.
1602         Just move the span element itself.
1603
1604         * code-review.js:
1605
1606 2011-01-11  Ojan Vafai  <ojan@chromium.org>
1607
1608         Reviewed by Adam Barth.
1609
1610         fix layout error with comments in code review tool
1611         https://bugs.webkit.org/show_bug.cgi?id=52230
1612
1613         * PrettyPatch/PrettyPatch.rb:
1614
1615 2011-01-10  Ojan Vafai  <ojan@chromium.org>
1616
1617         Reviewed by Adam Barth.
1618
1619         convert back to unified from sidebyside diff
1620         https://bugs.webkit.org/show_bug.cgi?id=52180
1621
1622         Remove url fragment stuff. Having it be per-filediff is too complicated.
1623
1624         * PrettyPatch/PrettyPatch.rb:
1625         * code-review.js:
1626
1627 2011-01-07  Adam Barth  <abarth@webkit.org>
1628
1629         Rubber-stamped by Eric Seidel.
1630
1631         Move WebCore to Source
1632         https://bugs.webkit.org/show_bug.cgi?id=52050
1633
1634         Remove reference to old directory.
1635
1636         * PrettyPatch/PrettyPatch.rb:
1637
1638 2011-01-06  Ojan Vafai  <ojan@chromium.org>
1639
1640         Reviewed by Adam Barth.
1641
1642         side-by-side diffs in the code review tool
1643         https://bugs.webkit.org/show_bug.cgi?id=52019
1644
1645         Support for conversion from the formatted diff to a side-by-side diff.
1646         Maintains comments and new comments can be added.
1647
1648         The main architectural change is that Line elements are no longer necessarily
1649         siblings. Each physical line is now in a LineContainer and LineContainers are
1650         siblings. Each Line corresponds to a Line in the unified diff and has an id (e.g. line12).
1651         A Line can be a LineContainer or a child of a LineContainer.
1652
1653         In this way, converting to side-by-side and, in the future, back to unified is non-lossy.
1654
1655         * PrettyPatch/PrettyPatch.rb:
1656         * code-review.js:
1657
1658 2011-01-06  Ojan Vafai  <ojan@chromium.org>
1659
1660         Fix line context when replying to comments.
1661
1662         * code-review.js:
1663
1664 2011-01-05  Ojan Vafai  <ojan@chromium.org>
1665
1666         Reviewed by Adam Barth.
1667
1668         change the way we do comment highlighting in the code review tool
1669         https://bugs.webkit.org/show_bug.cgi?id=51971
1670
1671         Store a space-separated list of base line IDs on each line that has
1672         comments associated with that line. This allows for overlapping comments,
1673         but more importantly, makes adding side-by-side diff support easier.
1674
1675         * code-review.js:
1676
1677 2011-01-05  Ojan Vafai  <ojan@chromium.org>
1678
1679         Reviewed by Adam Barth.
1680
1681         minor code cleanup for code review tool
1682         https://bugs.webkit.org/show_bug.cgi?id=51962
1683
1684         Consolidates some queries to using shared functions.
1685
1686         * code-review.js:
1687
1688 2011-01-05  Ojan Vafai  <ojan@chromium.org>
1689
1690         Reviewed by Adam Barth.
1691
1692         assorted cleanups to prepare for side-by-side diffing
1693         https://bugs.webkit.org/show_bug.cgi?id=51961
1694
1695         Mostly, use selector queries instead of assuming comments are next siblings.
1696
1697         * code-review.js:
1698
1699 2011-01-05  Ojan Vafai  <ojan@chromium.org>
1700
1701         Reviewed by Adam Barth.
1702
1703         fix message container selector for code review tool
1704         https://bugs.webkit.org/show_bug.cgi?id=51959
1705
1706         * code-review.js:
1707
1708 2011-01-04  Ojan Vafai  <ojan@chromium.org>
1709
1710         Increment version number to avoid getting the cached JS file.
1711
1712         * PrettyPatch/PrettyPatch.rb:
1713
1714 2010-12-15  Ojan Vafai  <ojan@chromium.org>
1715  
1716         Reviewed by Adam Barth.
1717
1718         anchor the toolbar to the bottom of the diff if the diff doesn't take a full screen of height
1719         https://bugs.webkit.org/show_bug.cgi?id=51162
1720
1721         * PrettyPatch/PrettyPatch.rb:
1722         * code-review.js:
1723
1724 2011-01-01  Adam Barth  <abarth@webkit.org>
1725
1726         Reviewed by Eric Seidel.
1727
1728         Move JavaScriptCore to Source
1729         https://bugs.webkit.org/show_bug.cgi?id=51604
1730
1731         * PrettyPatch/PrettyPatch.rb:
1732           - Remove reference to JavaScriptCore as a source directory.
1733
1734 2011-01-01  Adam Barth  <abarth@webkit.org>
1735
1736         Reviewed by Eric Seidel.
1737
1738         Move Sources to Source
1739         https://bugs.webkit.org/show_bug.cgi?id=51794
1740
1741         * PrettyPatch/PrettyPatch.rb:
1742
1743 2010-12-31  Adam Barth  <abarth@webkit.org>
1744
1745         Rubber-stamped by Eric Seidel.
1746
1747         Move PageLoadTests to PerformanceTests/PageLoad
1748         https://bugs.webkit.org/show_bug.cgi?id=51771
1749
1750         Update list of top-level open-source directories.
1751
1752         * PrettyPatch/PrettyPatch.rb:
1753
1754 2010-12-31  Adam Barth  <abarth@webkit.org>
1755
1756         Rubber-stamped by Eric Seidel.
1757
1758         Move SunSpider into PerformanceTests
1759         https://bugs.webkit.org/show_bug.cgi?id=51769
1760
1761         Update list of top-level open-source directories.
1762
1763         * PrettyPatch/PrettyPatch.rb:
1764
1765 2010-12-26  Adam Barth  <abarth@webkit.org>
1766
1767         Reviewed by Eric Seidel.
1768
1769         Move autotools into Sources
1770         https://bugs.webkit.org/show_bug.cgi?id=51630
1771
1772         * PrettyPatch/PrettyPatch.rb:
1773           - Remove reference to non-existant autotools directory.
1774
1775 2010-12-26  Adam Barth  <abarth@webkit.org>
1776
1777         Reviewed by Eric Seidel.
1778
1779         Move cmake into Sources
1780         https://bugs.webkit.org/show_bug.cgi?id=51631
1781
1782         * PrettyPatch/PrettyPatch.rb:
1783
1784 2010-12-23  Adam Barth  <abarth@webkit.org>
1785
1786         Rubber-stamped by Eric Seidel.
1787
1788         Move JavaScriptGlue into Sources
1789         https://bugs.webkit.org/show_bug.cgi?id=51583
1790
1791         PrettyPatch keeps a list of the top-level directories that exist in
1792         trac.
1793
1794         * PrettyPatch/PrettyPatch.rb:
1795
1796 2010-12-20  Adam Barth  <abarth@webkit.org>
1797
1798         One more tweak caused by moving the web sites around.  I had this
1799         change in my patch originally, but I missed it because I had to use
1800         server-side svn move commands to move the bulk of the web site files.
1801
1802         * PrettyPatch/PrettyPatch.rb:
1803
1804 2010-12-18  Adam Barth  <abarth@webkit.org>
1805
1806         Reviewed by Sam Weinig.
1807
1808         Move WebKitExamplePlugins to Examples
1809         https://bugs.webkit.org/show_bug.cgi?id=51291
1810
1811         * PrettyPatch/PrettyPatch.rb:
1812
1813 2010-12-17  Dan Bernstein  <mitz@apple.com>
1814
1815         Reviewed by Simon Fraser.
1816
1817         Rename WebKitTools to Tools
1818         https://bugs.webkit.org/show_bug.cgi?id=49861
1819
1820         * PrettyPatch/PrettyPatch.rb:
1821         * committers-autocomplete.js:
1822
1823 2010-12-17  Ojan Vafai  <ojan@chromium.org>
1824
1825         Increment psuedo-version number to avoid pulling cached version.
1826
1827         * PrettyPatch/PrettyPatch.rb:
1828
1829 2010-12-17  Ojan Vafai  <ojan@chromium.org>
1830
1831         Speculative fix for the code review tool in Firefox.
1832         Fixes a JS error and makes the status bubble mostly work
1833         if postMessage is not supported.
1834
1835         * PrettyPatch/PrettyPatch.rb:
1836         * code-review.js:
1837
1838 2010-12-15  Ojan Vafai  <ojan@chromium.org>
1839
1840         Reviewed by Adam Barth.
1841
1842         size status bubble to it's contents on the code review page
1843         https://bugs.webkit.org/show_bug.cgi?id=51142
1844
1845         * PrettyPatch/PrettyPatch.rb:
1846         * code-review.js:
1847
1848 2010-12-15  Ojan Vafai  <ojan@chromium.org>
1849
1850         Reviewed by Adam Barth.
1851
1852         fix goofups from r74142 and r74130
1853         https://bugs.webkit.org/show_bug.cgi?id=51146
1854
1855         Accidentally shrank the inline comment boxes and made
1856         the toolbar not show up when there were image diffs.
1857
1858         * PrettyPatch/PrettyPatch.rb:
1859         * code-review.js:
1860
1861 2010-12-14  Ojan Vafai  <ojan@chromium.org>
1862
1863         Reviewed by Adam Barth.
1864
1865         add ability to view for file context to the review tool
1866         https://bugs.webkit.org/show_bug.cgi?id=51057
1867
1868         At the beginning/end of each file diff and between each
1869         hunk add links to expand the context. For now it grabs the
1870         tip of tree version of the file and tries to apply the diff 
1871         to that file. If it can't apply, then it gives up as we
1872         wouldn't want to show the wrong lines of context.
1873
1874         In the future, we can consider adding the upload svn revision
1875         to the diff itself, then we could fallback to the file at that
1876         revision if tip of tree doesn't apply.
1877
1878         * PrettyPatch/PrettyPatch.rb:
1879         * code-review.js:
1880
1881 2010-12-08  Ojan Vafai  <ojan@chromium.org>
1882
1883         Reviewed by Adam Barth.
1884
1885         [reviewtool] should always show overall comments text box
1886         https://bugs.webkit.org/show_bug.cgi?id=45870
1887
1888         * PrettyPatch/PrettyPatch.rb:
1889         * code-review.js:
1890
1891 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
1892
1893         Reviewed by Adam Barth.
1894
1895         Increase size of status bubbles
1896         https://bugs.webkit.org/show_bug.cgi?id=50496
1897
1898         The current available space isn't enough to display all
1899         pending results including position in the queue.
1900
1901         * PrettyPatch/PrettyPatch.rb:
1902
1903 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
1904
1905         Reviewed by Adam Barth.
1906
1907         Increase size of status bubbles
1908         https://bugs.webkit.org/show_bug.cgi?id=50496
1909
1910         The current available space isn't enough to display all
1911         pending results including position in the queue.
1912
1913         * template/en/custom/attachment/edit.html.tmpl:
1914         * template/en/custom/attachment/list.html.tmpl:
1915         * template/en/custom/attachment/reviewform.html.tmpl:
1916
1917 2010-11-08  Adam Barth  <abarth@webkit.org>
1918
1919         Update help text to match behavior change.
1920
1921         * code-review.js:
1922
1923 2010-11-08  Adam Barth  <abarth@webkit.org>
1924
1925         Reviewed by Eric Seidel.
1926
1927         Restore normal meaning of double-click on patch review page?
1928         https://bugs.webkit.org/show_bug.cgi?id=47641
1929
1930         After this patch, you need to click on the line numbers to add a
1931         comment.
1932
1933         * PrettyPatch/PrettyPatch.rb:
1934         * code-review.js:
1935
1936 2010-10-09  Adam Barth  <abarth@webkit.org>
1937
1938         Reviewed by Alexey Proskuryakov.
1939
1940         Exception in committers-autocomplete in Firefox
1941         https://bugs.webkit.org/show_bug.cgi?id=47456
1942
1943         Remove dependency on a WebKit-only quirk.  IMHO, this quirk is bad for
1944         the long-term health of the web, but that's a discussion for another
1945         patch.
1946
1947         * committers-autocomplete.js:
1948
1949 2010-10-08  Adam Barth  <abarth@webkit.org>
1950
1951         Reviewed by Oliver Hunt.
1952
1953         Bugmail for new attachments should link to the reviewtool
1954         https://bugs.webkit.org/show_bug.cgi?id=47440
1955
1956         As requested by olliej himself.  This patch was constructed by pure
1957         reason.  I have no way to actually run this code.
1958
1959         * Bugzilla/BugMail.pm:
1960
1961 2010-09-28  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
1962
1963         Reviewed by Csaba Osztrogonác.
1964
1965         Convert verbatim unicode values in comitter names to characters
1966
1967         * committers-autocomplete.js:
1968
1969 2010-09-24  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
1970
1971         Reviewed by Csaba Osztrogonác.
1972
1973         Support Unicode-strings in committers-autocomplete script
1974
1975         * committers-autocomplete.js:
1976
1977 2010-09-22  Adam Barth  <abarth@webkit.org>
1978
1979         Reviewed by John Sullivan.
1980
1981         New review-page design doesn't include name of reviewer
1982         https://bugs.webkit.org/show_bug.cgi?id=46271
1983
1984         * PrettyPatch/PrettyPatch.rb:
1985         * code-review.js:
1986
1987 2010-09-22  Adam Barth  <abarth@webkit.org>
1988
1989         Fix typo in CSS.
1990
1991         * PrettyPatch/PrettyPatch.rb:
1992
1993 2010-09-22  Adam Barth  <abarth@webkit.org>
1994
1995         Reviewed by Eric Seidel.
1996
1997         Add link to bug to review page
1998         https://bugs.webkit.org/show_bug.cgi?id=46192
1999
2000         To make room for the link, I moved the help text to the top of the page
2001         and tweaked the language to help folks discover that you can select
2002         context using the line numbers.
2003
2004         Also, move more text to sans-serif since that fits in better with the
2005         pretty-patch fonts.
2006
2007         * PrettyPatch/PrettyPatch.rb:
2008         * code-review.js:
2009
2010 2010-09-21  Adam Barth  <abarth@webkit.org>
2011
2012         Unreviewed.
2013
2014         [reviewtool] Publish button doesn't work
2015         https://bugs.webkit.org/show_bug.cgi?id=46168
2016
2017         Darin's recent patch had a runtime error.  Sadly, we don't have any
2018         testing for this code.
2019
2020         * code-review.js:
2021
2022 2010-09-20  Darin Adler  <darin@apple.com>
2023
2024         Reviewed by Adam Barth.
2025
2026         Add Preview button and link to bug to patch review bar
2027         https://bugs.webkit.org/show_bug.cgi?id=46153
2028
2029         * PrettyPatch/PrettyPatch.rb: Bump version number to make caches happy.
2030         * code-review.js: Added Preview button and link to bug.
2031
2032 2010-09-19  Adam Barth  <abarth@webkit.org>
2033
2034         Reviewed by Eric Seidel.
2035
2036         [reviewtool] Publish button should actually publish the review comments
2037         https://bugs.webkit.org/show_bug.cgi?id=46074
2038
2039         Instead of showing the confirmation lightbox, the Publish button now
2040         publishes the comments directly.  If there's demand for a "Preview"
2041         button, we can find a way to add that feature.
2042
2043         * code-review.js:
2044
2045 2010-09-19  Adam Barth  <abarth@webkit.org>
2046
2047         Reviewed by Eric Seidel.
2048
2049         Move reviewtool to action=review
2050         https://bugs.webkit.org/show_bug.cgi?id=46071
2051
2052         This patch removes the old Review Patch screen and moves the new
2053         reviewtool from Pretty Diff to Review Patch.
2054
2055         * attachment.cgi:
2056         * code-review.js:
2057
2058 2010-09-19  Adam Barth  <abarth@webkit.org>
2059
2060         Review tool UI tweak.  This patch lengthens the status bubbles so all
2061         the bubble fit (even when they have numbers inside).
2062
2063         * PrettyPatch/PrettyPatch.rb:
2064
2065 2010-09-19  Dan Bernstein  <mitz@apple.com>
2066
2067         Reviewed by Anders Carlsson.
2068
2069         Fixed a typo.
2070
2071         * code-review.js: Changed the label of the OK button from Ok to OK.
2072
2073 2010-09-15  Adam Barth  <abarth@webkit.org>
2074
2075         Unreviewed.
2076
2077         Only show the review-in-context link if there are contextual comments.
2078
2079         * code-review.js:
2080
2081 2010-09-15  Adam Barth  <abarth@webkit.org>
2082
2083         Reviewed by Eric Seidel.
2084
2085         [reviewtool] Show the status bubbles on the toolbar
2086         https://bugs.webkit.org/show_bug.cgi?id=45861
2087
2088         * PrettyPatch/PrettyPatch.rb:
2089         * code-review.js:
2090
2091 2010-09-15  Adam Barth  <abarth@webkit.org>
2092
2093         Reviewed by Eric Seidel.
2094
2095         [reviewtool] Allow setting review and commit-queue flags from main review screen
2096         https://bugs.webkit.org/show_bug.cgi?id=45860
2097
2098         This patch adds drop down menus for adjusting the review and
2099         commit-queue flags from the main review screen.  These controls don't
2100         appear immediately because we need to read the state of the flags off
2101         the details page.
2102
2103         It's a slight sadness that bugzilla doesn't have a clean way of
2104         identifying which select control is associated with a given flag.  The
2105         flags seem to have some sort of id, but it's unclear to me how to map
2106         that id back to a notion of "review" or "commit-queue".  Instead, we
2107         look for some magic strings in the title.  I'm sure this will come back
2108         to bit us at some point, but I'm not sure what to do that's better.
2109         (The webkitpy solution to this problem is to assume that the flags are
2110         in a given order, but this seems better.)
2111
2112         * PrettyPatch/PrettyPatch.rb:
2113             - Bump version number to bust throuh caches.
2114         * code-review.js:
2115
2116 2010-09-15  Adam Barth  <abarth@webkit.org>
2117
2118         Reviewed by Eric Seidel.
2119
2120         [reviewtool] Comments should quote previous comments on the same line
2121         https://bugs.webkit.org/show_bug.cgi?id=45847
2122
2123         Now when you reply to a previous comment, the tool will quote the
2124         previous comment in the bugs.webkit.org post.  This makes it eaiser for
2125         folks following along in email to understand the discussion.
2126
2127         While I was editing this code, I also cleaned up some of the whitespace
2128         handling in comments.
2129
2130         * code-review.js:
2131
2132 2010-09-15  Adam Barth  <abarth@webkit.org>
2133
2134         Minor UI tweak to the review tool.  We want to display newlines in
2135         previous and frozen comments instead of collapsing them.
2136
2137         * PrettyPatch/PrettyPatch.rb:
2138
2139 2010-09-14  Adam Barth  <abarth@webkit.org>
2140
2141         Reviewed by Eric Seidel.
2142
2143         bugs.webkit.org should autocomplete WebKit Committers and Reviewers
2144         https://bugs.webkit.org/show_bug.cgi?id=45799
2145
2146         This patch implements an autocomplete dropdown that helps users type
2147         the names of WebKit committers and reviewers.  The script grabs the
2148         data from SVN now that wms set up CORS for http://svn.webkit.org.
2149         Security technology for the win.
2150
2151         This script is based on (well, essentially copied from) the popular
2152         autocomplete extension:
2153
2154         https://chrome.google.com/extensions/detail/olaabhcgdogcbcoiolomlcodkngnemfb
2155
2156         I asked Ojan, and said he was fine licensing the code under the BSD
2157         license.  I'm not sure how the patch interacts with the extension, but
2158         we can retire the extension if they interact poorly.
2159
2160         * committers-autocomplete.js: Added.
2161         * template/en/default/global/footer.html.tmpl:
2162             - Add the script to every page on bugs.webkit.org
2163
2164 2010-09-14  Adam Barth  <abarth@webkit.org>
2165
2166         Add a license block to the reviewtool.  I should have added this from
2167         the start.  According to the SVN log, I'm the only one to have ever
2168         touched this file.
2169
2170         * code-review.js:
2171
2172 2010-09-07  Adam Barth  <abarth@webkit.org>
2173
2174         Minor tweaks to the reviewtool UI.
2175
2176         Rename Cancel to Discard to make it more clear what the button does.
2177         Also, move the style declaration for the focus ring later in the sheet
2178         so it works.
2179
2180         * PrettyPatch/PrettyPatch.rb:
2181         * code-review.js:
2182
2183 2010-09-06  Adam Barth  <abarth@webkit.org>
2184
2185         Small UI tweak to the reviewtool.  Make the Ok/Cancel buttons a bit
2186         bigger.
2187
2188         * PrettyPatch/PrettyPatch.rb:
2189
2190 2010-09-06  Adam Barth  <abarth@webkit.org>
2191
2192         Reviewed by Eric Seidel.
2193
2194         [reviewtool] Add a quick animation for opening/closing comment boxes
2195         https://bugs.webkit.org/show_bug.cgi?id=45278
2196
2197         The 200ms animation slides in and out the line-by-line comment box.
2198
2199         * code-review.js:
2200
2201 2010-09-06  Adam Barth  <abarth@webkit.org>
2202
2203         Reviewed by Eric Seidel.
2204
2205         [reviewtool] Add a field for overall comments
2206         https://bugs.webkit.org/show_bug.cgi?id=45273
2207
2208         This patch does a couple logically separate things that could be
2209         separated into smaller patches:
2210
2211         1) This patch adds an "overall comments" field where you can enter
2212            overall comments about the patch.  These comments appear at the top
2213            of the bugzilla posting.  Currently, these aren't redisplayed when
2214            viewing the patch, but I plan to add that in a future patch.
2215
2216         2) This patch renames some of the CSS classes to more consistently
2217            follow the camelCase style that PrettyPatch uses.
2218
2219         3) This patch moves the "prepare comments" button to the left of the
2220            toolbar and renames is to "publish comments".  This makes more sense
2221            when you scroll to the bottom of the page and enter in some overall
2222            comments.
2223
2224         4) When you attempt to add a comment to a line that already has a
2225            "frozen" comment, we now unfreeze the comment instead of doing
2226            nothing.  The old behavior was kind of frustrating if you didn't
2227            know that you could unfreeze a comment by clicking on it.
2228
2229         * PrettyPatch/PrettyPatch.rb:
2230             - Update CSS.
2231         * code-review.js:
2232
2233 2010-09-06  Adam Barth  <abarth@webkit.org>
2234
2235         [reviewtool] Tweak the ok button to cancel the comment if the comment
2236         is empty.  Previously we would get into a bad state where a line had a
2237         comment but there was no longer any way to access it.
2238
2239         * code-review.js:
2240
2241 2010-09-06  Adam Barth  <abarth@webkit.org>
2242
2243         Reviewed by Eric Seidel.
2244
2245         [reviewtool] Add an "ok" button that collapses review comments
2246         https://bugs.webkit.org/show_bug.cgi?id=45255
2247
2248         Many folks (including myself) are too trigger-happy with the "delete"
2249         button that we end up deleting perfectly good review comments.  This
2250         patch adds an "ok" button to absorb these clicks (and renames "delete"
2251         to "cancel").  When you click the ok button, the comment becomes
2252         non-editable and the box shrinks to fit the text.  You can click the
2253         text to get back to the editable version.
2254
2255         * code-review.js:
2256
2257 2010-08-31  Adam Barth  <abarth@webkit.org>
2258
2259         Reviewed by Eric Seidel.
2260
2261         [reviewtool] Make it easy to scroll through review comments
2262         https://bugs.webkit.org/show_bug.cgi?id=45002
2263
2264         This patch lets you scroll through review comments using "n" (for next)
2265         and "p" (for previous).  It also attributes comments to their authors.
2266
2267         * PrettyPatch/PrettyPatch.rb:
2268         * code-review.js:
2269
2270 2010-08-31  Adam Barth  <abarth@webkit.org>
2271
2272         Reviewed by Eric Seidel.
2273
2274         [reviewtool] Show previous comments inline in diff
2275         https://bugs.webkit.org/show_bug.cgi?id=44977
2276
2277         This patch adds basic support for showing previous comments inline in
2278         the diff.  We crawl the bugs.webkit.org comments about this attachment
2279         and extract comments related to specific lines.  We then show the
2280         comments inline in the diff.
2281
2282         This part of the tool needs a bunch of polish, but this at least is a
2283         starting point for further work.
2284
2285         * PrettyPatch/PrettyPatch.rb:
2286         * code-review.js:
2287
2288 2010-08-31  Adam Barth  <abarth@webkit.org>
2289
2290         Reviewed by Eric Seidel.
2291
2292         [reviewtool] Allow reviewer to select lines of context by dragging over the line numbers
2293         https://bugs.webkit.org/show_bug.cgi?id=44936
2294
2295         * PrettyPatch/PrettyPatch.rb:
2296             - Add a version number to bust through bugs.webkit.org's aggressive
2297               cacheing.
2298         * code-review.js:
2299
2300 2010-08-30  Adam Barth  <abarth@webkit.org>
2301
2302         Reviewed by Eric Seidel.
2303
2304         [review tool] Let reviewer select how much context to show in snippet
2305         https://bugs.webkit.org/show_bug.cgi?id=44905
2306
2307         We now highlight the context for a comment in yellow on the left (where
2308         the line numbers are).  Clicking a line number expands or contracts the
2309         amount of context, as appropriate.  Informal user testing indicates
2310         that we might want to support drag as well.
2311
2312         This patch also changes the "open a comment box here" action to
2313         double-click to avoid issues with mis-clicks.
2314
2315         * PrettyPatch/PrettyPatch.rb:
2316         * code-review.js:
2317
2318 2010-08-29  Adam Barth  <abarth@webkit.org>
2319
2320         Attempt to make Sam's life easier by not opening a comment text field
2321         if there's a selection.  This should make it easier to copy/paste text
2322         out of the diff without accidentally opening comment boxes.
2323
2324         * code-review.js:
2325
2326 2010-08-29  Adam Barth  <abarth@webkit.org>
2327
2328         Turns out we're supposed to use find instead of children here.
2329
2330         * code-review.js:
2331
2332 2010-08-29  Adam Barth  <abarth@webkit.org>
2333
2334         Reviewed by Eric Seidel.
2335
2336         Awesomify pretty-diff based review tool
2337         https://bugs.webkit.org/show_bug.cgi?id=44835
2338
2339         This patch replaces the existing inline comment feature with a new
2340         implementation.  Hopefully the new implementation is more awesome.
2341         It's not 100% done yet, but I'd like to get this version out there to
2342         see how it works.
2343
2344         Unlike the previous version this version works with the "Formatted
2345         Diff" link instead of the "Review Patch" link.  Hopefully that will
2346         avoid interfering with folks who like the old style review.
2347
2348         * PrettyPatch/PrettyPatch.rb:
2349         * code-review.js: Added.
2350         * prototype.js: Removed.
2351
2352 2010-08-19  Tony Chang  <tony@chromium.org>
2353
2354         Reviewed by Adam Barth.
2355
2356         webkit-patch barfed on upload with a new image test result
2357         https://bugs.webkit.org/show_bug.cgi?id=39097
2358
2359         * PrettyPatch/PrettyPatch.rb: don't use full path to git
2360
2361 2010-08-19  Tony Chang  <tony@chromium.org>
2362
2363         Unreviewed, setting svn:eol-style native on ChangeLog.
2364
2365 2010-08-19  Tony Chang  <tony@chromium.org>
2366
2367         Unreviewed.  Just removing \r's from ChangeLog.
2368
2369 2010-08-02  Adam Roben  <aroben@apple.com>
2370
2371         Tell the patch prettifier about some (not so) new directories
2372
2373         * PrettyPatch/PrettyPatch.rb: Added WebKit2, autotools, and cmake
2374         directories.
2375
2376 2010-07-27  Joseph Pecoraro  <joepeck@webkit.org>
2377
2378         Reviewed by David Kilzer.
2379
2380         Provide a Helpful Link for Web Inspector Bugs in Bugzilla
2381         https://bugs.webkit.org/show_bug.cgi?id=43065
2382
2383         * template/en/custom/global/choose-product.html.tmpl: Added. Explicitly added a link for the Web Inspector.
2384
2385 2010-06-09  Julie Parent  <jparent@chromium.org>
2386
2387         Reviewed by David Kilzer.
2388
2389         Bugzilla: Don't send mail for cancel/deny/granted for in-rietveld.
2390
2391         https://bugs.webkit.org/show_bug.cgi?id=40345
2392
2393         * Bugzilla/Flag.pm:
2394
2395 2010-05-17  Julie Parent  <jparent@chromium.org>
2396
2397         Reviewed by Ojan Vafai.
2398
2399         Rietveld review page should dedicate more space to Rietveld, less to high level comments form.
2400
2401         https://bugs.webkit.org/show_bug.cgi?id=39244
2402
2403         * template/en/custom/attachment/reviewform.html.tmpl:
2404         Delete a random BR amongst the hidden elements that was causing extra whitespace.
2405         * template/en/custom/attachment/rietveldreview.html.tmpl:
2406         Increase size of Rietveld frame to 80%, from 60%.
2407
2408 2010-05-14  Julie Parent  <jparent@chromium.org>
2409
2410         Reviewed by Ojan Vafai.
2411
2412         Bugzilla: Only show "Rietveld Review" link when in-rietveld is set to +.
2413         
2414         https://bugs.webkit.org/show_bug.cgi?id=39141
2415
2416         * template/en/custom/attachment/list.html.tmpl:
2417
2418 2010-05-13  Julie Parent  <jparent@chromium.org>
2419
2420         Reviewed by David Kilzer.
2421
2422         Bugzilla: Don't send mail if the only change is to the in-rietveld flag.
2423         
2424         https://bugs.webkit.org/show_bug.cgi?id=39090
2425
2426         * Bugzilla/BugMail.pm:
2427
2428 2010-05-12  Julie Parent  <jparent@chromium.org>
2429
2430         Reviewed by Ojan Vafai.
2431
2432         Hide the in-rietveld flag in the UI.  It is only used only by tooling and shouldn't clutter the UI.
2433         https://bugs.webkit.org/show_bug.cgi?id=39024
2434
2435         * template/en/custom/attachment/list.html.tmpl:
2436           Removes this flag entirely from the UI, not needed on this page.
2437         * template/en/custom/flag/list.html.tmpl: Added.
2438           Copied from template/en/default/flag/list.html.tmpl with only changes inside the if WEBKIT_CHANGES section.
2439           Hides the flag rather than removing it completely, so tools can still interact with it.  
2440
2441 2010-05-12  Julie Parent  <jparent@chromium.org>
2442
2443         Reviewed by David Kilzer.
2444
2445         Adds support for Rietveld reviews.  When there is an associated
2446         Rietveld id for a patch, adds a link to do a Rietveld review instead.
2447
2448         https://bugs.webkit.org/show_bug.cgi?id=38143
2449
2450         * attachment.cgi:
2451             Added support for handling action=rietveldreview.
2452         * template/en/custom/attachment/list.html.tmpl:
2453             Adds a link to use rietveld review iff the in_rietveld flag is set.
2454         * template/en/custom/attachment/reviewform.html.tmpl:
2455             Detects if we are in rietveld review mode due to query string paramter,
2456             and customizes the review form slightly:
2457                 - Changes text describing comment field.
2458                 - Comment field does not include quoted patch.
2459                 - Submit button fetches from Rietveld instead of just submitting.
2460         * template/en/custom/attachment/rietveldreview.html.tmpl: Added.
2461             New template for rietveldreview mode.  Hosts the rietveld frame in the
2462             top and the regular comments form in the bottom.
2463             Uses postMessage to communicate with Rietveld frame.
2464
2465 2010-05-08  Chris Jerdonek  <cjerdonek@webkit.org>
2466
2467         Reviewed by Maciej Stachowiak.
2468
2469         Allow long comment lines to wrap to the window width instead
2470         of to 80 characters.
2471
2472         https://bugs.webkit.org/show_bug.cgi?id=37792
2473
2474         * Bugzilla/Constants.pm:
2475           - Introduced a new constant COMMENT_COLS_WRAP to replace COMMENT_COLS
2476             inside the wrap_comment() subroutine.  The new constant effectively
2477             disables text wrapping by increasing the value used in wrap_comment()
2478             from 80 characters to a very large value (8000).
2479         * Bugzilla/Util.pm:
2480           - Replaced COMMENT_COLS with COMMENT_COLS_WRAP inside wrap_comment().
2481         * skins/custom/global.css:
2482         (.bz_comment_text):
2483           - Eliminate the width styling on comments to allow long lines
2484             to wrap at the window width.
2485         (.bz_comment_text > span.quote):
2486           - Inherit the "white-space" property.  This prevents long quoted
2487             lines in comments from causing non-quoted lines to extend beyond
2488             the window width.
2489
2490 2010-04-22  Adam Barth  <abarth@webkit.org>
2491
2492         Reviewed by Darin Adler.
2493
2494         The new review tools shouldn't clear the comment box automatically
2495         https://bugs.webkit.org/show_bug.cgi?id=38004
2496
2497         Instead, we now have a button for doing that explicitly.  If people
2498         like this tool, we can look at fancier solutions.
2499
2500         Also, tried to make the tool work on the edit page as well as the review page.
2501
2502         * PrettyPatch/PrettyPatch.rb:
2503
2504 2010-04-22  Adam Barth  <abarth@webkit.org>
2505
2506         Unreviewed.  Turns out we need to do more escaping because Ruby was
2507         explanding our newlines, creating a syntax error.
2508
2509         * PrettyPatch/PrettyPatch.rb:
2510
2511 2010-04-22  Adam Barth  <abarth@webkit.org>
2512
2513         Reviewed by Darin Adler.
2514
2515         Hack up PrettyDiff to allow line-by-line comments
2516         https://bugs.webkit.org/show_bug.cgi?id=37886
2517
2518         Admittedly a bit of a hack, this is a basic line-by-line editor we can
2519         play with thanks to Andrew Scherkus.  It's meant to integrate with the
2520         "review" page at bugs.webkit.org.
2521
2522         I changed a few things from Andrew's original version:
2523         1) Trigger text boxes on single click to make iPhone/iPad reviewing
2524         easier.
2525         2) Clear the main text box on load.
2526         3) Reference a version of prototype.js on bugs.webkit.org.
2527
2528         * PrettyPatch/PrettyPatch.rb:
2529         * prototype.js: Added.
2530
2531 2010-02-22  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
2532
2533         Reviewed by Maciej Stachowiak.
2534
2535         Don't reset the assignee back to default on component change
2536
2537         https://bugs.webkit.org/show_bug.cgi?id=35236
2538
2539         There's a risk of reseting a valid assignee for example when
2540         triaging a bug. All components in WebKit have the default 
2541         assignee set to webkit-unassigned@webkit.org so this should
2542         not cause problems for people relying on this functionality.
2543
2544         * template/en/custom/bug/edit.html.tmpl:
2545
2546 2010-01-22  Janusz Lewandowski  <lew21@xtreeme.org>
2547
2548         Reviewed by Maciej Stachowiak.
2549
2550         run-webkit-tests fails on Ruby 1.9
2551         https://bugs.webkit.org/show_bug.cgi?id=33554
2552         inject does not work with strings any more, so BugsSite/PrettyPatch/PrettyPatch.rb prints lots of errors.
2553
2554         * PrettyPatch/PrettyPatch.rb:
2555
2556 2010-01-20  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
2557
2558         Reviewed by Adam Barth.
2559
2560         Adjust height of status-bubbles to prevent them being cut.
2561
2562         * template/en/custom/attachment/edit.html.tmpl:
2563         * template/en/custom/attachment/list.html.tmpl:
2564         * template/en/custom/attachment/reviewform.html.tmpl:
2565
2566 2010-01-08  David Kilzer  <ddkilzer@apple.com>
2567
2568         <http://webkit.org/b/33410> Fix "View Plain Diff"/"View Formatted Diff" button
2569
2570         Reviewed by Adam Barth.
2571
2572         This button was broken during a previous Bugzilla upgrade after
2573         changes were made to the names of objects exposed on templates.
2574
2575         * template/en/custom/attachment/reviewform.html.tmpl: Changed
2576         attachid to attachment.id and ispatch to attachment.ispatch.
2577         Also tweaked the format to use 4 table columns instead of 5.
2578
2579 2010-01-08  David Kilzer  <ddkilzer@apple.com>
2580
2581         Included UNCONFIRMED bugs by default in advanced search query
2582
2583         Rubber-stamped by Alexey Proskuryakov.
2584
2585         * data/params: Prepended 'bug_status=UNCONFIRMED&' to
2586         'defaultquery' parameter.
2587
2588 2010-01-08  David Kilzer  <ddkilzer@apple.com>
2589
2590         FIX: Make patch status appear on reviewform.html.tmpl
2591
2592         * template/en/custom/attachment/reviewform.html.tmpl: Check
2593         attachment.ispatch (not just ispatch) when deciding whether
2594         to display patch status.  Also update layout a bit.
2595
2596 2010-01-08  David Kilzer  <ddkilzer@apple.com>
2597
2598         Update data/params parameters
2599
2600         * data/params: Updated 'timezone' to turn off DST.  Updated
2601         'usebugaliases' and 'usevotes' to turn them off.
2602
2603 2010-01-08  Adam Barth  <abarth@webkit.org>
2604
2605         Reviewed by Darin Adler.
2606
2607         Show patch status in bugs.webkit.org
2608         https://bugs.webkit.org/show_bug.cgi?id=33368
2609
2610         This patch adds an iframe to the show_bug page and the edit attachment
2611         page that displays whether the attachment passed the various bots.
2612
2613         * template/en/custom/attachment/edit.html.tmpl:
2614         * template/en/custom/attachment/list.html.tmpl:
2615         * template/en/custom/attachment/reviewform.html.tmpl:
2616
2617 2009-12-07  Shinichiro Hamaji  <hamaji@chromium.org>
2618
2619         Reviewed by David Kilzer.
2620
2621         Bugzilla should show images in git patches
2622         https://bugs.webkit.org/show_bug.cgi?id=31395
2623
2624         Attempt to go r51748 again using --directory option of git-apply.
2625
2626         * PrettyPatch/PrettyPatch.rb:
2627
2628 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
2629
2630         Unreviewed. Revert r51748.
2631
2632         Bugzilla should show images in git patches
2633         https://bugs.webkit.org/show_bug.cgi?id=31395
2634
2635         * PrettyPatch/PrettyPatch.rb:
2636
2637 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
2638
2639         Reviewed by David Kilzer.
2640
2641         Bugzilla should show images in git patches
2642         https://bugs.webkit.org/show_bug.cgi?id=31395
2643
2644         Show images in git patches using git-apply.
2645
2646         * PrettyPatch/PrettyPatch.rb:
2647
2648 2009-10-23  Eric Seidel  <eric@webkit.org>
2649
2650         Reviewed by Adam Roben.
2651
2652         PrettyPatch should show images even when they have the wrong mime type
2653         https://bugs.webkit.org/show_bug.cgi?id=29506
2654
2655         * PrettyPatch/PrettyPatch.rb:
2656
2657 2009-10-17  Alexey Proskuryakov  <ap@apple.com>
2658
2659         Reviewed by Eric Carlson.
2660
2661         https://bugs.webkit.org/show_bug.cgi?id=30470
2662         Make marking a bug as duplicate easier
2663
2664         * js/field.js: (setResolutionToDuplicate): Focus the edit field, too.
2665
2666 2009-10-16  Alexey Proskuryakov  <ap@apple.com>
2667
2668         Reviewed by Mark Rowe.
2669
2670         https://bugs.webkit.org/show_bug.cgi?id=30470
2671         Make marking a bug as duplicate easier
2672
2673         * js/field.js: (setResolutionToDuplicate): Show edit field right away, don't make the user
2674         click again.
2675
2676 2009-09-09  David Kilzer  <ddkilzer@apple.com>
2677
2678         Update data/params parameters
2679
2680         * data/params: Updated 'attachment_base' parameter for secure
2681         attachments.  Updated 'mybugstemplate' parameter to search for
2682         bugs in UNCONFIRMED, ASSIGNED and REOPENED status in addition to
2683         NEW status.
2684
2685 2009-08-11  David Kilzer  <ddkilzer@apple.com>
2686
2687         Updated data/params parameter
2688
2689         * data/params: Updated 'loginnetmask' parameter to allow users
2690         to choose whether their cookies are restricted to a single IP
2691         address or not.
2692
2693 2009-07-07  David Kilzer  <ddkilzer@apple.com>
2694
2695         Updated BugsSite with local changes on server
2696
2697         The new files were created by checksetup.pl during the upgrade
2698         to Bugzilla 3.2.3.  The rest are self-explanatory.
2699
2700         * data: Added "bugzilla-update.xml" to svn:ignore.
2701         * data/attachments: Added "groups.*" to svn:ignore.
2702         * data/params: Updated for a few changed parameters.
2703
2704         * lib/.htaccess: Added.
2705         * skins/contrib/Dusk/IE-fixes.css: Added.
2706         * skins/contrib/Dusk/admin.css: Added.
2707         * skins/contrib/Dusk/create_attachment.css: Added.
2708         * skins/contrib/Dusk/dependency-tree.css: Added.
2709         * skins/contrib/Dusk/duplicates.css: Added.
2710         * skins/contrib/Dusk/editusers.css: Added.
2711         * skins/contrib/Dusk/help.css: Added.
2712         * skins/contrib/Dusk/index.css: Added.
2713         * skins/contrib/Dusk/panel.css: Added.
2714         * skins/contrib/Dusk/params.css: Added.
2715         * skins/contrib/Dusk/release-notes.css: Added.
2716         * skins/contrib/Dusk/show_bug.css: Added.
2717         * skins/contrib/Dusk/show_multiple.css: Added.
2718         * skins/contrib/Dusk/summarize-time.css: Added.
2719         * skins/contrib/Dusk/voting.css: Added.
2720         * skins/contrib/Dusk/yui: Added.
2721         * skins/contrib/Dusk/yui/calendar.css: Added.
2722         * skins/custom/IE-fixes.css: Added.
2723         * skins/custom/admin.css: Added.
2724         * skins/custom/buglist.css: Added.
2725         * skins/custom/create_attachment.css: Added.
2726         * skins/custom/dependency-tree.css: Added.
2727         * skins/custom/duplicates.css: Added.
2728         * skins/custom/editusers.css: Added.
2729         * skins/custom/help.css: Added.
2730         * skins/custom/panel.css: Added.
2731         * skins/custom/params.css: Added.
2732         * skins/custom/release-notes.css: Added.
2733         * skins/custom/show_bug.css: Added.
2734         * skins/custom/show_multiple.css: Added.
2735         * skins/custom/summarize-time.css: Added.
2736         * skins/custom/voting.css: Added.
2737         * skins/custom/yui: Added.
2738         * skins/custom/yui/calendar.css: Added.
2739
2740 2009-07-03  David Kilzer  <ddkilzer@apple.com>
2741
2742         Bug 26958: Change edit link to review link in request messages
2743
2744         <https://bugs.webkit.org/show_bug.cgi?id=26958>
2745
2746         Reviewed by Dan Bernstein.
2747
2748         * template/en/custom/request/email.txt.tmpl: Copied from BugsSite/template/en/default/request/email.txt.tmpl.
2749         Changed link from "action=edit" to "action=review".
2750
2751 2009-07-03  David Kilzer  <ddkilzer@apple.com>
2752
2753         Bug 26950: Make the summary and alias fields support click-to-edit
2754         <https://bugs.webkit.org/show_bug.cgi?id=26950>
2755
2756         Reviewed by Maciej Stachowiak.
2757
2758         Original patch by Maciej Stachowiak.
2759
2760         * js/field.js:
2761         (hideEditableField): Updated to add click event listeners to the
2762         alias and short description elements to make them easier to
2763         edit.  Renamed field_id parameter to field2_id (short
2764         description id) and added a field1_id parameter (alias id).
2765         (showEditableField): If a third parameter is passed in the
2766         ContainerInputArray parameter, use it to find the element to
2767         focus.  Otherwise, fall back to the original behavior of
2768         focusing the first input field.
2769         (hideAliasAndSummary): Changed to pass the id for the alias
2770         element to hideEditableField().
2771
2772 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2773
2774         Config file for contrib/recode.pl when upgrading bugs.webkit.org
2775
2776         Current as of midday on Wednesday, July 1, 2009.
2777
2778         * contrib/recode-overrides.txt: Added.
2779
2780 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2781
2782         Workaround for WebKit Bug 9630 when running contrib/recode.pl
2783
2784         A number of WebKit nightly builds included Bug 9630 which caused
2785         non-breaking space characters (0xA0) to be submitted within the
2786         content of textarea elements.
2787
2788         * contrib/recode.pl: To work around these 0xA0 characters, try
2789         an encoding of cp1252 and use it if it works.
2790
2791 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2792
2793         Don't print "Failed to guess" warning if an override is available
2794
2795         * contrib/recode.pl: Check that %overrides does NOT contain a
2796         key matching the current digest before printing out the warning
2797         about a failed charset guess.
2798
2799 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2800
2801         Added --[no-]truncate switch to contrib/recode.pl
2802
2803         * contrib/recode.pl: Added --[no-]truncate switch to make
2804         debugging of failed charset guessing easier.  Often times the
2805         illegal character was truncated in the output.
2806
2807 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2808
2809         Changed Perl scripts to use #!/usr/bin/env perl
2810
2811         perl -pi -e 's@#\!/usr/bin/perl@#\!/usr/bin/env perl@' *.cgi *.pl */*.pl
2812
2813 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2814
2815         Replace Bugzilla favicon with webkit.org favicon
2816
2817         * favicon.ico: Removed.
2818         * images/favicon.ico: Replaced.  Copied from favicon.ico.
2819
2820 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
2821
2822         Merged BugsSite to Bugzilla-3.2.3
2823
2824         Updated to the latest-and-greatest stable version.
2825
2826 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
2827
2828         Merged BugsSite to Bugzilla-3.0.3
2829
2830         Nothing to see here.  Move along.
2831
2832 2009-06-09  Eric Seidel  <eric@webkit.org>
2833
2834         Reviewed by Adam Roben.
2835
2836         Add support for displaying added png files in PrettyPatch diffs
2837         https://bugs.webkit.org/show_bug.cgi?id=26210
2838
2839         Currently this is SVN only (git-send-bugzilla patches exclude binary data)
2840         and only works for PNG files but could easily be made to work for other images as needed.
2841
2842         * PrettyPatch/PrettyPatch.rb:
2843
2844 2009-05-19  Eric Seidel  <eric@webkit.org>
2845
2846         Reviewed by Adam Roben.
2847
2848         Make PrettyPatch understand quoted filenames in git diffs.
2849         https://bugs.webkit.org/show_bug.cgi?id=25881
2850
2851         * PrettyPatch/PrettyPatch.rb:
2852
2853 2009-05-15  Simon Fraser  <simon.fraser@apple.com>
2854
2855         Carrying forwards Darin Adler's rubber stamp
2856         
2857         Make the comment field wider too.
2858
2859         * skins/custom/global.css:
2860
2861 2009-05-14  Simon Fraser  <simon.fraser@apple.com>
2862
2863         Reviewed by Darin Adler
2864         
2865         Make the URL, Summary and Keyword fields wider to mitigate the
2866         effects of https://bugs.webkit.org/show_bug.cgi?id=25566 and make
2867         things generally better.
2868
2869         * skins/custom/global.css:
2870         * template/en/custom/bug/edit.html.tmpl:
2871
2872 2008-11-10  Darin Adler  <darin@apple.com>
2873
2874         - fix more of https://bugs.webkit.org/show_bug.cgi?id=21400
2875           "Edit" links for patches in comments, review queue, review emails
2876           should be replaced by "Review Patch" links
2877
2878         * globals.pl: Fix cases that are automatically generated, such as links
2879         in comments in bugs.
2880
2881         * template/en/custom/attachment/created.html.tmpl: Fix the link on the
2882         "I just created an attachment" page.
2883
2884         * template/en/custom/request/queue.html.tmpl: Fix the link in the queue
2885         page.
2886
2887 2008-11-10  Darin Adler  <darin@apple.com>
2888
2889         * template/en/default/request/queue.html.tmpl: Use review links instead
2890         of edit links in the review queue.
2891
2892 2008-11-10  Darin Adler  <darin@apple.com>
2893
2894         * template/en/custom/request/email.txt.tmpl: Send review links instead
2895         of edit links when flags are set on a bug.
2896
2897 2008-10-27  Darin Adler  <darin@apple.com>
2898
2899         * template/en/custom/attachment/reviewform.html.tmpl: Leave out the
2900         comment if it's untouched.
2901
2902 2008-10-23  Adam Roben  <aroben@apple.com>
2903
2904         Fix Bug 21401: Comments field on "Review Patch" page should be
2905         initially filled with quoted patch
2906
2907         <https://bugs.webkit.org/show_bug.cgi?id=21401>
2908
2909         Reviewed by Dave Hyatt.
2910
2911         * attachment.cgi:
2912         (edit): Retrieve the attachment data from the database instead of just
2913         its length. When the attachment is a patch, create a quotedpatch
2914         template variable that contains the patch data with each line
2915         prepended with "> ".
2916         * template/en/custom/attachment/reviewform.html.tmpl:
2917           - Changed the comments field to have a monospace font
2918           - Added an "Enter comments below:" caption above the comments field
2919             to make it clearer that this is where comments should go, now that
2920             the comments field is not initially empty
2921           - Fill the comments field with the quoted patch
2922
2923 2008-10-14  Adam Roben  <aroben@apple.com>
2924
2925         Fix Bug 21602: Bugzilla times out trying to display formatted diff for
2926         attachment 24345
2927         
2928         https://bugs.webkit.org/show_bug.cgi?id=21602
2929
2930         Reviewed by Dave Kilzer.
2931
2932         * PrettyPatch/PrettyPatch.rb:
2933         (PrettyPatch.BINARY_FILE_MARKER_FORMAT): Added.
2934         (PrettyPatch.FileDiff.initialize): If any of the lines in the diff
2935         are a binary file marker, mark this FileDiff as binary and stop trying
2936         to process the lines.
2937         (PrettyDiff.FileDiff.to_html): If we're binary, just print a string
2938         saying so.
2939
2940 2008-10-02  Adam Roben  <aroben@apple.com>
2941
2942         * template/en/custom/attachment/reviewform.html.tmpl: Fix typo.
2943
2944 2008-10-02  Adam Roben  <aroben@apple.com>
2945
2946         * template/en/custom/attachment/review.html.tmpl: Fix URLs
2947
2948 2008-10-02  Adam Roben  <aroben@apple.com>
2949
2950         Fix Bug 21315: bugs.webkit.org should have a nicer patch review
2951         interface
2952
2953         https://bugs.webkit.org/show_bug.cgi?id=21315
2954
2955         Reviewed by Tim Hatcher.
2956
2957         * attachment.cgi:
2958         (top level): Added support for the "review" and "reviewform" actions.
2959         (sub edit): Accept the template name as a parameter. If no name is
2960         give, use "edit" as before.
2961         * template/en/custom/attachment/list.html.tmpl: Added a "Review Patch"
2962         link for all patch attachments.
2963         * template/en/custom/attachment/review.html.tmpl: Added. Simple
2964         <frameset> page to show the patch on the top and a comment form on the
2965         bottom.
2966         * template/en/custom/attachment/reviewform.html.tmpl: Added. Simple
2967         comment form for reviewing patches.
2968
2969 2008-10-02  Adam Roben  <aroben@apple.com>
2970
2971         Remove references to some backup files I created when implementing
2972         PrettyPatch in Bugzilla
2973
2974         Reviewed by Tim Hatcher.
2975
2976         * attachment-aroben.cgi: Removed.
2977         * template/en/default/attachment/edit-aroben.html.tmpl: Removed.
2978
2979         * template/en/custom/attachment/edit.html.tmpl: Removed an erroneous
2980         reference to attachment-aroben.cgi. attachment.cgi will work just
2981         fine.
2982
2983 2008-06-25  David Kilzer  <ddkilzer@apple.com>
2984
2985         Make PrettyPatch handle diffs with no Index or diff headers
2986
2987         Part of Bug 19290: More patches not handled by PrettyPatch.rb
2988         <https://bugs.webkit.org/show_bug.cgi?id=19290>
2989
2990         Reviewed by Adam.
2991
2992         * PrettyPatch/PrettyPatch.rb:
2993         (PrettyPatch.DIFF_HEADER_FORMATS): Added regular expression to
2994         match on "+++ " lines for patches with no "Index" or "diff" header.
2995         (PrettyPatch.FileDiff.initialize): Look for filename on "+++ " line
2996         when the first line of a patch has no "Index" or "diff" header.
2997         (PrettyPatch.FileDiff.parse): Added haveSeenDiffHeader state
2998         variable to determine when no "Index" or "diff" header has been
2999         found, but a new patch has started with a "--- " line.
3000
3001 2008-06-08  Dan Bernstein  <mitz@apple.com>
3002
3003         Reviewed by Adam Roben.
3004
3005         - update trac URL in PrettyPatch
3006
3007         * PrettyPatch/PrettyPatch.rb:
3008
3009 2008-05-29  Adam Roben  <aroben@apple.com>
3010
3011         Update PrettyPatch directory list
3012
3013         * PrettyPatch/PrettyPatch.rb:
3014
3015 2008-05-28  Adam Roben  <aroben@apple.com>
3016
3017         Make PrettyPatch able to handle diffs taken from outside the WebKit
3018         source tree
3019
3020         Part of Bug 19290: More patches not handled by PrettyPatch.rb
3021         <https://bugs.webkit.org/show_bug.cgi?id=19290>
3022
3023         Reviewed by David Kilzer.
3024
3025         * PrettyPatch/PrettyPatch.rb:
3026         (PrettyPatch.find_url_and_path): Added. Searches the file path from
3027         the bottom up looking for a directory that exists in the source tree.
3028         (PrettyPatch.linkifyFilename): Changed to call find_url_and_path.
3029
3030 2008-05-28  Adam Roben  <aroben@apple.com>
3031
3032         Print exceptions from PrettyPatch to stdout
3033
3034         This will make it much easier to debug problems with PrettyPatch,
3035         since we'll be able to see the exceptions in the browser.
3036
3037         Reviewed by Sam Weinig.
3038
3039         * PrettyPatch/prettify.rb: Added a --html-exceptions option, which
3040         will print exceptions to stdout as HTML.
3041         * attachment.cgi: Changed to pass --html-exceptions to prettify.rb.
3042
3043 2008-04-26  David Kilzer  <ddkilzer@apple.com>
3044
3045         Removed temp files and added appropriate svn:ignore properties.
3046
3047         Rubber-stamped by Mark Rowe.
3048
3049         * data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
3050
3051         * data/template: Added svn:ignore property for "template" subdirectory.
3052         * data/template/template: Removed precompiled template subdirectory.
3053
3054         * data/versioncache: Removed.
3055         * data/versioncache.*: Removed backup versioncache files.
3056
3057         * data/webdot: Added svn:ignore property for "*.dot" files.
3058         * data/webdot/*.dot: Removed cached webdot files.
3059
3060 2008-04-08  Adam Roben  <aroben@apple.com>
3061
3062         Combine :equal operations with the following operation if they are
3063         fewer than 3 characters long
3064
3065         This keeps us from showing lots of small changes on long lines, just
3066         because some letters happened to be the same.
3067
3068         Inspired by
3069         http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/diffviewer/diffutils.py?r=1264#147
3070
3071         * PrettyPatch/PrettyPatch.rb:
3072         (PrettyPatch::DiffSection.initialize): Remove :equal operations fewer
3073         than 3 characters long. The characters from the removed operations
3074         become part of the subsequent operation.
3075         (PrettyPatch::CodeLine.text_as_html): Don't wrap 0-length strings in
3076         <ins>/<del> tags. Also removed the @fromLineNumber.nil? and
3077         @toLineNumber.nil? checks as they are no longer reliable now that
3078         we're removing operations.
3079
3080 2008-04-08  Adam Roben  <aroben@apple.com>
3081
3082         Change PrettyPatch to use DiffBuilder for intra-line diffs
3083
3084         This gives us much prettier intra-line diffs, largely because it can
3085         distinguish multiple changes on the same line. e.g., if a line changes
3086         from:
3087
3088         const int myConstant = 0;
3089
3090         to
3091
3092         static const unsigned myConstant;
3093
3094         You will see that "static " was inserted, "int" changed to "unsigned",
3095         and " = 0" was deleted.
3096
3097         This seems to have also gotten rid of some spurious instances of
3098         "<ins></ins>" and "<del></del>" at the end of a line.
3099
3100         * PrettyPatch/PrettyPatch.rb:
3101         (PrettyPatch::DiffSection.initialize): Use DiffBuilder instead of
3102         setChangeExtentFromLine (which had been copied from Trac's diffing
3103         code).
3104         (PrettyPatch::CodeLine):
3105          - Removed the changeExtent attribute
3106          - Added the operations attribute
3107          - Removed the setChangeExtentFromLine method
3108         (PrettyPatch::CodeLine.text_as_html): Uses @operations to determine
3109         what text to wrap in <ins> and <del> tags.
3110         * PrettyPatch/diff.rb: Deleted a bunch of code we don't use.
3111         (HTMLDiff::DiffBuilder.initialize): Moved code here from the old build
3112         method, but left out the calls to perform_operation, since we build
3113         the diff HTML outside of this class.
3114         (HTMLDiff::DiffBuilder.split_inputs_to_words): Removed calls to
3115         convert_html_to_list_of_words, since we're not diffing HTML.
3116
3117 2008-04-08  Adam Roben  <aroben@apple.com>
3118
3119         Check in diff.rb
3120
3121         This came from
3122         http://instiki.rubyforge.org/svn/instiki/trunk/lib/diff.rb
3123         revision 28521
3124
3125         * PrettyPatch/diff.rb: Added.
3126
3127 2008-03-04  Adam Roben  <aroben@apple.com>
3128
3129         * PrettyPatch/PrettyPatch.rb: Allow the extent of the previous file to
3130         be omitted from the hunk header.
3131