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