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