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