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