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