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