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