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