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