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