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