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