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