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