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