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