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