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