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