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