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