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