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