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