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