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