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