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