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