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