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