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