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