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