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