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