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