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