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