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