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