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