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