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