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