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