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