8f938c692351a8b65bd4bcdd8ae967c9b6343f4c
[WebKit-https.git] / Websites / bugs.webkit.org / ChangeLog
1 2010-12-26  Adam Barth  <abarth@webkit.org>
2
3         Reviewed by Eric Seidel.
4
5         Move cmake into Sources
6         https://bugs.webkit.org/show_bug.cgi?id=51631
7
8         * PrettyPatch/PrettyPatch.rb:
9
10 2010-12-23  Adam Barth  <abarth@webkit.org>
11
12         Rubber-stamped by Eric Seidel.
13
14         Move JavaScriptGlue into Sources
15         https://bugs.webkit.org/show_bug.cgi?id=51583
16
17         PrettyPatch keeps a list of the top-level directories that exist in
18         trac.
19
20         * PrettyPatch/PrettyPatch.rb:
21
22 2010-12-20  Adam Barth  <abarth@webkit.org>
23
24         One more tweak caused by moving the web sites around.  I had this
25         change in my patch originally, but I missed it because I had to use
26         server-side svn move commands to move the bulk of the web site files.
27
28         * PrettyPatch/PrettyPatch.rb:
29
30 2010-12-18  Adam Barth  <abarth@webkit.org>
31
32         Reviewed by Sam Weinig.
33
34         Move WebKitExamplePlugins to Examples
35         https://bugs.webkit.org/show_bug.cgi?id=51291
36
37         * PrettyPatch/PrettyPatch.rb:
38
39 2010-12-17  Dan Bernstein  <mitz@apple.com>
40
41         Reviewed by Simon Fraser.
42
43         Rename WebKitTools to Tools
44         https://bugs.webkit.org/show_bug.cgi?id=49861
45
46         * PrettyPatch/PrettyPatch.rb:
47         * committers-autocomplete.js:
48
49 2010-12-17  Ojan Vafai  <ojan@chromium.org>
50
51         Increment psuedo-version number to avoid pulling cached version.
52
53         * PrettyPatch/PrettyPatch.rb:
54
55 2010-12-17  Ojan Vafai  <ojan@chromium.org>
56
57         Speculative fix for the code review tool in Firefox.
58         Fixes a JS error and makes the status bubble mostly work
59         if postMessage is not supported.
60
61         * PrettyPatch/PrettyPatch.rb:
62         * code-review.js:
63
64 2010-12-15  Ojan Vafai  <ojan@chromium.org>
65
66         Reviewed by Adam Barth.
67
68         size status bubble to it's contents on the code review page
69         https://bugs.webkit.org/show_bug.cgi?id=51142
70
71         * PrettyPatch/PrettyPatch.rb:
72         * code-review.js:
73
74 2010-12-15  Ojan Vafai  <ojan@chromium.org>
75
76         Reviewed by Adam Barth.
77
78         fix goofups from r74142 and r74130
79         https://bugs.webkit.org/show_bug.cgi?id=51146
80
81         Accidentally shrank the inline comment boxes and made
82         the toolbar not show up when there were image diffs.
83
84         * PrettyPatch/PrettyPatch.rb:
85         * code-review.js:
86
87 2010-12-14  Ojan Vafai  <ojan@chromium.org>
88
89         Reviewed by Adam Barth.
90
91         add ability to view for file context to the review tool
92         https://bugs.webkit.org/show_bug.cgi?id=51057
93
94         At the beginning/end of each file diff and between each
95         hunk add links to expand the context. For now it grabs the
96         tip of tree version of the file and tries to apply the diff 
97         to that file. If it can't apply, then it gives up as we
98         wouldn't want to show the wrong lines of context.
99
100         In the future, we can consider adding the upload svn revision
101         to the diff itself, then we could fallback to the file at that
102         revision if tip of tree doesn't apply.
103
104         * PrettyPatch/PrettyPatch.rb:
105         * code-review.js:
106
107 2010-12-08  Ojan Vafai  <ojan@chromium.org>
108
109         Reviewed by Adam Barth.
110
111         [reviewtool] should always show overall comments text box
112         https://bugs.webkit.org/show_bug.cgi?id=45870
113
114         * PrettyPatch/PrettyPatch.rb:
115         * code-review.js:
116
117 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
118
119         Reviewed by Adam Barth.
120
121         Increase size of status bubbles
122         https://bugs.webkit.org/show_bug.cgi?id=50496
123
124         The current available space isn't enough to display all
125         pending results including position in the queue.
126
127         * PrettyPatch/PrettyPatch.rb:
128
129 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
130
131         Reviewed by Adam Barth.
132
133         Increase size of status bubbles
134         https://bugs.webkit.org/show_bug.cgi?id=50496
135
136         The current available space isn't enough to display all
137         pending results including position in the queue.
138
139         * template/en/custom/attachment/edit.html.tmpl:
140         * template/en/custom/attachment/list.html.tmpl:
141         * template/en/custom/attachment/reviewform.html.tmpl:
142
143 2010-11-08  Adam Barth  <abarth@webkit.org>
144
145         Update help text to match behavior change.
146
147         * code-review.js:
148
149 2010-11-08  Adam Barth  <abarth@webkit.org>
150
151         Reviewed by Eric Seidel.
152
153         Restore normal meaning of double-click on patch review page?
154         https://bugs.webkit.org/show_bug.cgi?id=47641
155
156         After this patch, you need to click on the line numbers to add a
157         comment.
158
159         * PrettyPatch/PrettyPatch.rb:
160         * code-review.js:
161
162 2010-10-09  Adam Barth  <abarth@webkit.org>
163
164         Reviewed by Alexey Proskuryakov.
165
166         Exception in committers-autocomplete in Firefox
167         https://bugs.webkit.org/show_bug.cgi?id=47456
168
169         Remove dependency on a WebKit-only quirk.  IMHO, this quirk is bad for
170         the long-term health of the web, but that's a discussion for another
171         patch.
172
173         * committers-autocomplete.js:
174
175 2010-10-08  Adam Barth  <abarth@webkit.org>
176
177         Reviewed by Oliver Hunt.
178
179         Bugmail for new attachments should link to the reviewtool
180         https://bugs.webkit.org/show_bug.cgi?id=47440
181
182         As requested by olliej himself.  This patch was constructed by pure
183         reason.  I have no way to actually run this code.
184
185         * Bugzilla/BugMail.pm:
186
187 2010-09-28  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
188
189         Reviewed by Csaba Osztrogonác.
190
191         Convert verbatim unicode values in comitter names to characters
192
193         * committers-autocomplete.js:
194
195 2010-09-24  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
196
197         Reviewed by Csaba Osztrogonác.
198
199         Support Unicode-strings in committers-autocomplete script
200
201         * committers-autocomplete.js:
202
203 2010-09-22  Adam Barth  <abarth@webkit.org>
204
205         Reviewed by John Sullivan.
206
207         New review-page design doesn't include name of reviewer
208         https://bugs.webkit.org/show_bug.cgi?id=46271
209
210         * PrettyPatch/PrettyPatch.rb:
211         * code-review.js:
212
213 2010-09-22  Adam Barth  <abarth@webkit.org>
214
215         Fix typo in CSS.
216
217         * PrettyPatch/PrettyPatch.rb:
218
219 2010-09-22  Adam Barth  <abarth@webkit.org>
220
221         Reviewed by Eric Seidel.
222
223         Add link to bug to review page
224         https://bugs.webkit.org/show_bug.cgi?id=46192
225
226         To make room for the link, I moved the help text to the top of the page
227         and tweaked the language to help folks discover that you can select
228         context using the line numbers.
229
230         Also, move more text to sans-serif since that fits in better with the
231         pretty-patch fonts.
232
233         * PrettyPatch/PrettyPatch.rb:
234         * code-review.js:
235
236 2010-09-21  Adam Barth  <abarth@webkit.org>
237
238         Unreviewed.
239
240         [reviewtool] Publish button doesn't work
241         https://bugs.webkit.org/show_bug.cgi?id=46168
242
243         Darin's recent patch had a runtime error.  Sadly, we don't have any
244         testing for this code.
245
246         * code-review.js:
247
248 2010-09-20  Darin Adler  <darin@apple.com>
249
250         Reviewed by Adam Barth.
251
252         Add Preview button and link to bug to patch review bar
253         https://bugs.webkit.org/show_bug.cgi?id=46153
254
255         * PrettyPatch/PrettyPatch.rb: Bump version number to make caches happy.
256         * code-review.js: Added Preview button and link to bug.
257
258 2010-09-19  Adam Barth  <abarth@webkit.org>
259
260         Reviewed by Eric Seidel.
261
262         [reviewtool] Publish button should actually publish the review comments
263         https://bugs.webkit.org/show_bug.cgi?id=46074
264
265         Instead of showing the confirmation lightbox, the Publish button now
266         publishes the comments directly.  If there's demand for a "Preview"
267         button, we can find a way to add that feature.
268
269         * code-review.js:
270
271 2010-09-19  Adam Barth  <abarth@webkit.org>
272
273         Reviewed by Eric Seidel.
274
275         Move reviewtool to action=review
276         https://bugs.webkit.org/show_bug.cgi?id=46071
277
278         This patch removes the old Review Patch screen and moves the new
279         reviewtool from Pretty Diff to Review Patch.
280
281         * attachment.cgi:
282         * code-review.js:
283
284 2010-09-19  Adam Barth  <abarth@webkit.org>
285
286         Review tool UI tweak.  This patch lengthens the status bubbles so all
287         the bubble fit (even when they have numbers inside).
288
289         * PrettyPatch/PrettyPatch.rb:
290
291 2010-09-19  Dan Bernstein  <mitz@apple.com>
292
293         Reviewed by Anders Carlsson.
294
295         Fixed a typo.
296
297         * code-review.js: Changed the label of the OK button from Ok to OK.
298
299 2010-09-15  Adam Barth  <abarth@webkit.org>
300
301         Unreviewed.
302
303         Only show the review-in-context link if there are contextual comments.
304
305         * code-review.js:
306
307 2010-09-15  Adam Barth  <abarth@webkit.org>
308
309         Reviewed by Eric Seidel.
310
311         [reviewtool] Show the status bubbles on the toolbar
312         https://bugs.webkit.org/show_bug.cgi?id=45861
313
314         * PrettyPatch/PrettyPatch.rb:
315         * code-review.js:
316
317 2010-09-15  Adam Barth  <abarth@webkit.org>
318
319         Reviewed by Eric Seidel.
320
321         [reviewtool] Allow setting review and commit-queue flags from main review screen
322         https://bugs.webkit.org/show_bug.cgi?id=45860
323
324         This patch adds drop down menus for adjusting the review and
325         commit-queue flags from the main review screen.  These controls don't
326         appear immediately because we need to read the state of the flags off
327         the details page.
328
329         It's a slight sadness that bugzilla doesn't have a clean way of
330         identifying which select control is associated with a given flag.  The
331         flags seem to have some sort of id, but it's unclear to me how to map
332         that id back to a notion of "review" or "commit-queue".  Instead, we
333         look for some magic strings in the title.  I'm sure this will come back
334         to bit us at some point, but I'm not sure what to do that's better.
335         (The webkitpy solution to this problem is to assume that the flags are
336         in a given order, but this seems better.)
337
338         * PrettyPatch/PrettyPatch.rb:
339             - Bump version number to bust throuh caches.
340         * code-review.js:
341
342 2010-09-15  Adam Barth  <abarth@webkit.org>
343
344         Reviewed by Eric Seidel.
345
346         [reviewtool] Comments should quote previous comments on the same line
347         https://bugs.webkit.org/show_bug.cgi?id=45847
348
349         Now when you reply to a previous comment, the tool will quote the
350         previous comment in the bugs.webkit.org post.  This makes it eaiser for
351         folks following along in email to understand the discussion.
352
353         While I was editing this code, I also cleaned up some of the whitespace
354         handling in comments.
355
356         * code-review.js:
357
358 2010-09-15  Adam Barth  <abarth@webkit.org>
359
360         Minor UI tweak to the review tool.  We want to display newlines in
361         previous and frozen comments instead of collapsing them.
362
363         * PrettyPatch/PrettyPatch.rb:
364
365 2010-09-14  Adam Barth  <abarth@webkit.org>
366
367         Reviewed by Eric Seidel.
368
369         bugs.webkit.org should autocomplete WebKit Committers and Reviewers
370         https://bugs.webkit.org/show_bug.cgi?id=45799
371
372         This patch implements an autocomplete dropdown that helps users type
373         the names of WebKit committers and reviewers.  The script grabs the
374         data from SVN now that wms set up CORS for http://svn.webkit.org.
375         Security technology for the win.
376
377         This script is based on (well, essentially copied from) the popular
378         autocomplete extension:
379
380         https://chrome.google.com/extensions/detail/olaabhcgdogcbcoiolomlcodkngnemfb
381
382         I asked Ojan, and said he was fine licensing the code under the BSD
383         license.  I'm not sure how the patch interacts with the extension, but
384         we can retire the extension if they interact poorly.
385
386         * committers-autocomplete.js: Added.
387         * template/en/default/global/footer.html.tmpl:
388             - Add the script to every page on bugs.webkit.org
389
390 2010-09-14  Adam Barth  <abarth@webkit.org>
391
392         Add a license block to the reviewtool.  I should have added this from
393         the start.  According to the SVN log, I'm the only one to have ever
394         touched this file.
395
396         * code-review.js:
397
398 2010-09-07  Adam Barth  <abarth@webkit.org>
399
400         Minor tweaks to the reviewtool UI.
401
402         Rename Cancel to Discard to make it more clear what the button does.
403         Also, move the style declaration for the focus ring later in the sheet
404         so it works.
405
406         * PrettyPatch/PrettyPatch.rb:
407         * code-review.js:
408
409 2010-09-06  Adam Barth  <abarth@webkit.org>
410
411         Small UI tweak to the reviewtool.  Make the Ok/Cancel buttons a bit
412         bigger.
413
414         * PrettyPatch/PrettyPatch.rb:
415
416 2010-09-06  Adam Barth  <abarth@webkit.org>
417
418         Reviewed by Eric Seidel.
419
420         [reviewtool] Add a quick animation for opening/closing comment boxes
421         https://bugs.webkit.org/show_bug.cgi?id=45278
422
423         The 200ms animation slides in and out the line-by-line comment box.
424
425         * code-review.js:
426
427 2010-09-06  Adam Barth  <abarth@webkit.org>
428
429         Reviewed by Eric Seidel.
430
431         [reviewtool] Add a field for overall comments
432         https://bugs.webkit.org/show_bug.cgi?id=45273
433
434         This patch does a couple logically separate things that could be
435         separated into smaller patches:
436
437         1) This patch adds an "overall comments" field where you can enter
438            overall comments about the patch.  These comments appear at the top
439            of the bugzilla posting.  Currently, these aren't redisplayed when
440            viewing the patch, but I plan to add that in a future patch.
441
442         2) This patch renames some of the CSS classes to more consistently
443            follow the camelCase style that PrettyPatch uses.
444
445         3) This patch moves the "prepare comments" button to the left of the
446            toolbar and renames is to "publish comments".  This makes more sense
447            when you scroll to the bottom of the page and enter in some overall
448            comments.
449
450         4) When you attempt to add a comment to a line that already has a
451            "frozen" comment, we now unfreeze the comment instead of doing
452            nothing.  The old behavior was kind of frustrating if you didn't
453            know that you could unfreeze a comment by clicking on it.
454
455         * PrettyPatch/PrettyPatch.rb:
456             - Update CSS.
457         * code-review.js:
458
459 2010-09-06  Adam Barth  <abarth@webkit.org>
460
461         [reviewtool] Tweak the ok button to cancel the comment if the comment
462         is empty.  Previously we would get into a bad state where a line had a
463         comment but there was no longer any way to access it.
464
465         * code-review.js:
466
467 2010-09-06  Adam Barth  <abarth@webkit.org>
468
469         Reviewed by Eric Seidel.
470
471         [reviewtool] Add an "ok" button that collapses review comments
472         https://bugs.webkit.org/show_bug.cgi?id=45255
473
474         Many folks (including myself) are too trigger-happy with the "delete"
475         button that we end up deleting perfectly good review comments.  This
476         patch adds an "ok" button to absorb these clicks (and renames "delete"
477         to "cancel").  When you click the ok button, the comment becomes
478         non-editable and the box shrinks to fit the text.  You can click the
479         text to get back to the editable version.
480
481         * code-review.js:
482
483 2010-08-31  Adam Barth  <abarth@webkit.org>
484
485         Reviewed by Eric Seidel.
486
487         [reviewtool] Make it easy to scroll through review comments
488         https://bugs.webkit.org/show_bug.cgi?id=45002
489
490         This patch lets you scroll through review comments using "n" (for next)
491         and "p" (for previous).  It also attributes comments to their authors.
492
493         * PrettyPatch/PrettyPatch.rb:
494         * code-review.js:
495
496 2010-08-31  Adam Barth  <abarth@webkit.org>
497
498         Reviewed by Eric Seidel.
499
500         [reviewtool] Show previous comments inline in diff
501         https://bugs.webkit.org/show_bug.cgi?id=44977
502
503         This patch adds basic support for showing previous comments inline in
504         the diff.  We crawl the bugs.webkit.org comments about this attachment
505         and extract comments related to specific lines.  We then show the
506         comments inline in the diff.
507
508         This part of the tool needs a bunch of polish, but this at least is a
509         starting point for further work.
510
511         * PrettyPatch/PrettyPatch.rb:
512         * code-review.js:
513
514 2010-08-31  Adam Barth  <abarth@webkit.org>
515
516         Reviewed by Eric Seidel.
517
518         [reviewtool] Allow reviewer to select lines of context by dragging over the line numbers
519         https://bugs.webkit.org/show_bug.cgi?id=44936
520
521         * PrettyPatch/PrettyPatch.rb:
522             - Add a version number to bust through bugs.webkit.org's aggressive
523               cacheing.
524         * code-review.js:
525
526 2010-08-30  Adam Barth  <abarth@webkit.org>
527
528         Reviewed by Eric Seidel.
529
530         [review tool] Let reviewer select how much context to show in snippet
531         https://bugs.webkit.org/show_bug.cgi?id=44905
532
533         We now highlight the context for a comment in yellow on the left (where
534         the line numbers are).  Clicking a line number expands or contracts the
535         amount of context, as appropriate.  Informal user testing indicates
536         that we might want to support drag as well.
537
538         This patch also changes the "open a comment box here" action to
539         double-click to avoid issues with mis-clicks.
540
541         * PrettyPatch/PrettyPatch.rb:
542         * code-review.js:
543
544 2010-08-29  Adam Barth  <abarth@webkit.org>
545
546         Attempt to make Sam's life easier by not opening a comment text field
547         if there's a selection.  This should make it easier to copy/paste text
548         out of the diff without accidentally opening comment boxes.
549
550         * code-review.js:
551
552 2010-08-29  Adam Barth  <abarth@webkit.org>
553
554         Turns out we're supposed to use find instead of children here.
555
556         * code-review.js:
557
558 2010-08-29  Adam Barth  <abarth@webkit.org>
559
560         Reviewed by Eric Seidel.
561
562         Awesomify pretty-diff based review tool
563         https://bugs.webkit.org/show_bug.cgi?id=44835
564
565         This patch replaces the existing inline comment feature with a new
566         implementation.  Hopefully the new implementation is more awesome.
567         It's not 100% done yet, but I'd like to get this version out there to
568         see how it works.
569
570         Unlike the previous version this version works with the "Formatted
571         Diff" link instead of the "Review Patch" link.  Hopefully that will
572         avoid interfering with folks who like the old style review.
573
574         * PrettyPatch/PrettyPatch.rb:
575         * code-review.js: Added.
576         * prototype.js: Removed.
577
578 2010-08-19  Tony Chang  <tony@chromium.org>
579
580         Reviewed by Adam Barth.
581
582         webkit-patch barfed on upload with a new image test result
583         https://bugs.webkit.org/show_bug.cgi?id=39097
584
585         * PrettyPatch/PrettyPatch.rb: don't use full path to git
586
587 2010-08-19  Tony Chang  <tony@chromium.org>
588
589         Unreviewed, setting svn:eol-style native on ChangeLog.
590
591 2010-08-19  Tony Chang  <tony@chromium.org>
592
593         Unreviewed.  Just removing \r's from ChangeLog.
594
595 2010-08-02  Adam Roben  <aroben@apple.com>
596
597         Tell the patch prettifier about some (not so) new directories
598
599         * PrettyPatch/PrettyPatch.rb: Added WebKit2, autotools, and cmake
600         directories.
601
602 2010-07-27  Joseph Pecoraro  <joepeck@webkit.org>
603
604         Reviewed by David Kilzer.
605
606         Provide a Helpful Link for Web Inspector Bugs in Bugzilla
607         https://bugs.webkit.org/show_bug.cgi?id=43065
608
609         * template/en/custom/global/choose-product.html.tmpl: Added. Explicitly added a link for the Web Inspector.
610
611 2010-06-09  Julie Parent  <jparent@chromium.org>
612
613         Reviewed by David Kilzer.
614
615         Bugzilla: Don't send mail for cancel/deny/granted for in-rietveld.
616
617         https://bugs.webkit.org/show_bug.cgi?id=40345
618
619         * Bugzilla/Flag.pm:
620
621 2010-05-17  Julie Parent  <jparent@chromium.org>
622
623         Reviewed by Ojan Vafai.
624
625         Rietveld review page should dedicate more space to Rietveld, less to high level comments form.
626
627         https://bugs.webkit.org/show_bug.cgi?id=39244
628
629         * template/en/custom/attachment/reviewform.html.tmpl:
630         Delete a random BR amongst the hidden elements that was causing extra whitespace.
631         * template/en/custom/attachment/rietveldreview.html.tmpl:
632         Increase size of Rietveld frame to 80%, from 60%.
633
634 2010-05-14  Julie Parent  <jparent@chromium.org>
635
636         Reviewed by Ojan Vafai.
637
638         Bugzilla: Only show "Rietveld Review" link when in-rietveld is set to +.
639         
640         https://bugs.webkit.org/show_bug.cgi?id=39141
641
642         * template/en/custom/attachment/list.html.tmpl:
643
644 2010-05-13  Julie Parent  <jparent@chromium.org>
645
646         Reviewed by David Kilzer.
647
648         Bugzilla: Don't send mail if the only change is to the in-rietveld flag.
649         
650         https://bugs.webkit.org/show_bug.cgi?id=39090
651
652         * Bugzilla/BugMail.pm:
653
654 2010-05-12  Julie Parent  <jparent@chromium.org>
655
656         Reviewed by Ojan Vafai.
657
658         Hide the in-rietveld flag in the UI.  It is only used only by tooling and shouldn't clutter the UI.
659         https://bugs.webkit.org/show_bug.cgi?id=39024
660
661         * template/en/custom/attachment/list.html.tmpl:
662           Removes this flag entirely from the UI, not needed on this page.
663         * template/en/custom/flag/list.html.tmpl: Added.
664           Copied from template/en/default/flag/list.html.tmpl with only changes inside the if WEBKIT_CHANGES section.
665           Hides the flag rather than removing it completely, so tools can still interact with it.  
666
667 2010-05-12  Julie Parent  <jparent@chromium.org>
668
669         Reviewed by David Kilzer.
670
671         Adds support for Rietveld reviews.  When there is an associated
672         Rietveld id for a patch, adds a link to do a Rietveld review instead.
673
674         https://bugs.webkit.org/show_bug.cgi?id=38143
675
676         * attachment.cgi:
677             Added support for handling action=rietveldreview.
678         * template/en/custom/attachment/list.html.tmpl:
679             Adds a link to use rietveld review iff the in_rietveld flag is set.
680         * template/en/custom/attachment/reviewform.html.tmpl:
681             Detects if we are in rietveld review mode due to query string paramter,
682             and customizes the review form slightly:
683                 - Changes text describing comment field.
684                 - Comment field does not include quoted patch.
685                 - Submit button fetches from Rietveld instead of just submitting.
686         * template/en/custom/attachment/rietveldreview.html.tmpl: Added.
687             New template for rietveldreview mode.  Hosts the rietveld frame in the
688             top and the regular comments form in the bottom.
689             Uses postMessage to communicate with Rietveld frame.
690
691 2010-05-08  Chris Jerdonek  <cjerdonek@webkit.org>
692
693         Reviewed by Maciej Stachowiak.
694
695         Allow long comment lines to wrap to the window width instead
696         of to 80 characters.
697
698         https://bugs.webkit.org/show_bug.cgi?id=37792
699
700         * Bugzilla/Constants.pm:
701           - Introduced a new constant COMMENT_COLS_WRAP to replace COMMENT_COLS
702             inside the wrap_comment() subroutine.  The new constant effectively
703             disables text wrapping by increasing the value used in wrap_comment()
704             from 80 characters to a very large value (8000).
705         * Bugzilla/Util.pm:
706           - Replaced COMMENT_COLS with COMMENT_COLS_WRAP inside wrap_comment().
707         * skins/custom/global.css:
708         (.bz_comment_text):
709           - Eliminate the width styling on comments to allow long lines
710             to wrap at the window width.
711         (.bz_comment_text > span.quote):
712           - Inherit the "white-space" property.  This prevents long quoted
713             lines in comments from causing non-quoted lines to extend beyond
714             the window width.
715
716 2010-04-22  Adam Barth  <abarth@webkit.org>
717
718         Reviewed by Darin Adler.
719
720         The new review tools shouldn't clear the comment box automatically
721         https://bugs.webkit.org/show_bug.cgi?id=38004
722
723         Instead, we now have a button for doing that explicitly.  If people
724         like this tool, we can look at fancier solutions.
725
726         Also, tried to make the tool work on the edit page as well as the review page.
727
728         * PrettyPatch/PrettyPatch.rb:
729
730 2010-04-22  Adam Barth  <abarth@webkit.org>
731
732         Unreviewed.  Turns out we need to do more escaping because Ruby was
733         explanding our newlines, creating a syntax error.
734
735         * PrettyPatch/PrettyPatch.rb:
736
737 2010-04-22  Adam Barth  <abarth@webkit.org>
738
739         Reviewed by Darin Adler.
740
741         Hack up PrettyDiff to allow line-by-line comments
742         https://bugs.webkit.org/show_bug.cgi?id=37886
743
744         Admittedly a bit of a hack, this is a basic line-by-line editor we can
745         play with thanks to Andrew Scherkus.  It's meant to integrate with the
746         "review" page at bugs.webkit.org.
747
748         I changed a few things from Andrew's original version:
749         1) Trigger text boxes on single click to make iPhone/iPad reviewing
750         easier.
751         2) Clear the main text box on load.
752         3) Reference a version of prototype.js on bugs.webkit.org.
753
754         * PrettyPatch/PrettyPatch.rb:
755         * prototype.js: Added.
756
757 2010-02-22  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
758
759         Reviewed by Maciej Stachowiak.
760
761         Don't reset the assignee back to default on component change
762
763         https://bugs.webkit.org/show_bug.cgi?id=35236
764
765         There's a risk of reseting a valid assignee for example when
766         triaging a bug. All components in WebKit have the default 
767         assignee set to webkit-unassigned@webkit.org so this should
768         not cause problems for people relying on this functionality.
769
770         * template/en/custom/bug/edit.html.tmpl:
771
772 2010-01-22  Janusz Lewandowski  <lew21@xtreeme.org>
773
774         Reviewed by Maciej Stachowiak.
775
776         run-webkit-tests fails on Ruby 1.9
777         https://bugs.webkit.org/show_bug.cgi?id=33554
778         inject does not work with strings any more, so BugsSite/PrettyPatch/PrettyPatch.rb prints lots of errors.
779
780         * PrettyPatch/PrettyPatch.rb:
781
782 2010-01-20  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
783
784         Reviewed by Adam Barth.
785
786         Adjust height of status-bubbles to prevent them being cut.
787
788         * template/en/custom/attachment/edit.html.tmpl:
789         * template/en/custom/attachment/list.html.tmpl:
790         * template/en/custom/attachment/reviewform.html.tmpl:
791
792 2010-01-08  David Kilzer  <ddkilzer@apple.com>
793
794         <http://webkit.org/b/33410> Fix "View Plain Diff"/"View Formatted Diff" button
795
796         Reviewed by Adam Barth.
797
798         This button was broken during a previous Bugzilla upgrade after
799         changes were made to the names of objects exposed on templates.
800
801         * template/en/custom/attachment/reviewform.html.tmpl: Changed
802         attachid to attachment.id and ispatch to attachment.ispatch.
803         Also tweaked the format to use 4 table columns instead of 5.
804
805 2010-01-08  David Kilzer  <ddkilzer@apple.com>
806
807         Included UNCONFIRMED bugs by default in advanced search query
808
809         Rubber-stamped by Alexey Proskuryakov.
810
811         * data/params: Prepended 'bug_status=UNCONFIRMED&' to
812         'defaultquery' parameter.
813
814 2010-01-08  David Kilzer  <ddkilzer@apple.com>
815
816         FIX: Make patch status appear on reviewform.html.tmpl
817
818         * template/en/custom/attachment/reviewform.html.tmpl: Check
819         attachment.ispatch (not just ispatch) when deciding whether
820         to display patch status.  Also update layout a bit.
821
822 2010-01-08  David Kilzer  <ddkilzer@apple.com>
823
824         Update data/params parameters
825
826         * data/params: Updated 'timezone' to turn off DST.  Updated
827         'usebugaliases' and 'usevotes' to turn them off.
828
829 2010-01-08  Adam Barth  <abarth@webkit.org>
830
831         Reviewed by Darin Adler.
832
833         Show patch status in bugs.webkit.org
834         https://bugs.webkit.org/show_bug.cgi?id=33368
835
836         This patch adds an iframe to the show_bug page and the edit attachment
837         page that displays whether the attachment passed the various bots.
838
839         * template/en/custom/attachment/edit.html.tmpl:
840         * template/en/custom/attachment/list.html.tmpl:
841         * template/en/custom/attachment/reviewform.html.tmpl:
842
843 2009-12-07  Shinichiro Hamaji  <hamaji@chromium.org>
844
845         Reviewed by David Kilzer.
846
847         Bugzilla should show images in git patches
848         https://bugs.webkit.org/show_bug.cgi?id=31395
849
850         Attempt to go r51748 again using --directory option of git-apply.
851
852         * PrettyPatch/PrettyPatch.rb:
853
854 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
855
856         Unreviewed. Revert r51748.
857
858         Bugzilla should show images in git patches
859         https://bugs.webkit.org/show_bug.cgi?id=31395
860
861         * PrettyPatch/PrettyPatch.rb:
862
863 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
864
865         Reviewed by David Kilzer.
866
867         Bugzilla should show images in git patches
868         https://bugs.webkit.org/show_bug.cgi?id=31395
869
870         Show images in git patches using git-apply.
871
872         * PrettyPatch/PrettyPatch.rb:
873
874 2009-10-23  Eric Seidel  <eric@webkit.org>
875
876         Reviewed by Adam Roben.
877
878         PrettyPatch should show images even when they have the wrong mime type
879         https://bugs.webkit.org/show_bug.cgi?id=29506
880
881         * PrettyPatch/PrettyPatch.rb:
882
883 2009-10-17  Alexey Proskuryakov  <ap@apple.com>
884
885         Reviewed by Eric Carlson.
886
887         https://bugs.webkit.org/show_bug.cgi?id=30470
888         Make marking a bug as duplicate easier
889
890         * js/field.js: (setResolutionToDuplicate): Focus the edit field, too.
891
892 2009-10-16  Alexey Proskuryakov  <ap@apple.com>
893
894         Reviewed by Mark Rowe.
895
896         https://bugs.webkit.org/show_bug.cgi?id=30470
897         Make marking a bug as duplicate easier
898
899         * js/field.js: (setResolutionToDuplicate): Show edit field right away, don't make the user
900         click again.
901
902 2009-09-09  David Kilzer  <ddkilzer@apple.com>
903
904         Update data/params parameters
905
906         * data/params: Updated 'attachment_base' parameter for secure
907         attachments.  Updated 'mybugstemplate' parameter to search for
908         bugs in UNCONFIRMED, ASSIGNED and REOPENED status in addition to
909         NEW status.
910
911 2009-08-11  David Kilzer  <ddkilzer@apple.com>
912
913         Updated data/params parameter
914
915         * data/params: Updated 'loginnetmask' parameter to allow users
916         to choose whether their cookies are restricted to a single IP
917         address or not.
918
919 2009-07-07  David Kilzer  <ddkilzer@apple.com>
920
921         Updated BugsSite with local changes on server
922
923         The new files were created by checksetup.pl during the upgrade
924         to Bugzilla 3.2.3.  The rest are self-explanatory.
925
926         * data: Added "bugzilla-update.xml" to svn:ignore.
927         * data/attachments: Added "groups.*" to svn:ignore.
928         * data/params: Updated for a few changed parameters.
929
930         * lib/.htaccess: Added.
931         * skins/contrib/Dusk/IE-fixes.css: Added.
932         * skins/contrib/Dusk/admin.css: Added.
933         * skins/contrib/Dusk/create_attachment.css: Added.
934         * skins/contrib/Dusk/dependency-tree.css: Added.
935         * skins/contrib/Dusk/duplicates.css: Added.
936         * skins/contrib/Dusk/editusers.css: Added.
937         * skins/contrib/Dusk/help.css: Added.
938         * skins/contrib/Dusk/index.css: Added.
939         * skins/contrib/Dusk/panel.css: Added.
940         * skins/contrib/Dusk/params.css: Added.
941         * skins/contrib/Dusk/release-notes.css: Added.
942         * skins/contrib/Dusk/show_bug.css: Added.
943         * skins/contrib/Dusk/show_multiple.css: Added.
944         * skins/contrib/Dusk/summarize-time.css: Added.
945         * skins/contrib/Dusk/voting.css: Added.
946         * skins/contrib/Dusk/yui: Added.
947         * skins/contrib/Dusk/yui/calendar.css: Added.
948         * skins/custom/IE-fixes.css: Added.
949         * skins/custom/admin.css: Added.
950         * skins/custom/buglist.css: Added.
951         * skins/custom/create_attachment.css: Added.
952         * skins/custom/dependency-tree.css: Added.
953         * skins/custom/duplicates.css: Added.
954         * skins/custom/editusers.css: Added.
955         * skins/custom/help.css: Added.
956         * skins/custom/panel.css: Added.
957         * skins/custom/params.css: Added.
958         * skins/custom/release-notes.css: Added.
959         * skins/custom/show_bug.css: Added.
960         * skins/custom/show_multiple.css: Added.
961         * skins/custom/summarize-time.css: Added.
962         * skins/custom/voting.css: Added.
963         * skins/custom/yui: Added.
964         * skins/custom/yui/calendar.css: Added.
965
966 2009-07-03  David Kilzer  <ddkilzer@apple.com>
967
968         Bug 26958: Change edit link to review link in request messages
969
970         <https://bugs.webkit.org/show_bug.cgi?id=26958>
971
972         Reviewed by Dan Bernstein.
973
974         * template/en/custom/request/email.txt.tmpl: Copied from BugsSite/template/en/default/request/email.txt.tmpl.
975         Changed link from "action=edit" to "action=review".
976
977 2009-07-03  David Kilzer  <ddkilzer@apple.com>
978
979         Bug 26950: Make the summary and alias fields support click-to-edit
980         <https://bugs.webkit.org/show_bug.cgi?id=26950>
981
982         Reviewed by Maciej Stachowiak.
983
984         Original patch by Maciej Stachowiak.
985
986         * js/field.js:
987         (hideEditableField): Updated to add click event listeners to the
988         alias and short description elements to make them easier to
989         edit.  Renamed field_id parameter to field2_id (short
990         description id) and added a field1_id parameter (alias id).
991         (showEditableField): If a third parameter is passed in the
992         ContainerInputArray parameter, use it to find the element to
993         focus.  Otherwise, fall back to the original behavior of
994         focusing the first input field.
995         (hideAliasAndSummary): Changed to pass the id for the alias
996         element to hideEditableField().
997
998 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
999
1000         Config file for contrib/recode.pl when upgrading bugs.webkit.org
1001
1002         Current as of midday on Wednesday, July 1, 2009.
1003
1004         * contrib/recode-overrides.txt: Added.
1005
1006 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1007
1008         Workaround for WebKit Bug 9630 when running contrib/recode.pl
1009
1010         A number of WebKit nightly builds included Bug 9630 which caused
1011         non-breaking space characters (0xA0) to be submitted within the
1012         content of textarea elements.
1013
1014         * contrib/recode.pl: To work around these 0xA0 characters, try
1015         an encoding of cp1252 and use it if it works.
1016
1017 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1018
1019         Don't print "Failed to guess" warning if an override is available
1020
1021         * contrib/recode.pl: Check that %overrides does NOT contain a
1022         key matching the current digest before printing out the warning
1023         about a failed charset guess.
1024
1025 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1026
1027         Added --[no-]truncate switch to contrib/recode.pl
1028
1029         * contrib/recode.pl: Added --[no-]truncate switch to make
1030         debugging of failed charset guessing easier.  Often times the
1031         illegal character was truncated in the output.
1032
1033 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1034
1035         Changed Perl scripts to use #!/usr/bin/env perl
1036
1037         perl -pi -e 's@#\!/usr/bin/perl@#\!/usr/bin/env perl@' *.cgi *.pl */*.pl
1038
1039 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
1040
1041         Replace Bugzilla favicon with webkit.org favicon
1042
1043         * favicon.ico: Removed.
1044         * images/favicon.ico: Replaced.  Copied from favicon.ico.
1045
1046 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
1047
1048         Merged BugsSite to Bugzilla-3.2.3
1049
1050         Updated to the latest-and-greatest stable version.
1051
1052 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
1053
1054         Merged BugsSite to Bugzilla-3.0.3
1055
1056         Nothing to see here.  Move along.
1057
1058 2009-06-09  Eric Seidel  <eric@webkit.org>
1059
1060         Reviewed by Adam Roben.
1061
1062         Add support for displaying added png files in PrettyPatch diffs
1063         https://bugs.webkit.org/show_bug.cgi?id=26210
1064
1065         Currently this is SVN only (git-send-bugzilla patches exclude binary data)
1066         and only works for PNG files but could easily be made to work for other images as needed.
1067
1068         * PrettyPatch/PrettyPatch.rb:
1069
1070 2009-05-19  Eric Seidel  <eric@webkit.org>
1071
1072         Reviewed by Adam Roben.
1073
1074         Make PrettyPatch understand quoted filenames in git diffs.
1075         https://bugs.webkit.org/show_bug.cgi?id=25881
1076
1077         * PrettyPatch/PrettyPatch.rb:
1078
1079 2009-05-15  Simon Fraser  <simon.fraser@apple.com>
1080
1081         Carrying forwards Darin Adler's rubber stamp
1082         
1083         Make the comment field wider too.
1084
1085         * skins/custom/global.css:
1086
1087 2009-05-14  Simon Fraser  <simon.fraser@apple.com>
1088
1089         Reviewed by Darin Adler
1090         
1091         Make the URL, Summary and Keyword fields wider to mitigate the
1092         effects of https://bugs.webkit.org/show_bug.cgi?id=25566 and make
1093         things generally better.
1094
1095         * skins/custom/global.css:
1096         * template/en/custom/bug/edit.html.tmpl:
1097
1098 2008-11-10  Darin Adler  <darin@apple.com>
1099
1100         - fix more of https://bugs.webkit.org/show_bug.cgi?id=21400
1101           "Edit" links for patches in comments, review queue, review emails
1102           should be replaced by "Review Patch" links
1103
1104         * globals.pl: Fix cases that are automatically generated, such as links
1105         in comments in bugs.
1106
1107         * template/en/custom/attachment/created.html.tmpl: Fix the link on the
1108         "I just created an attachment" page.
1109
1110         * template/en/custom/request/queue.html.tmpl: Fix the link in the queue
1111         page.
1112
1113 2008-11-10  Darin Adler  <darin@apple.com>
1114
1115         * template/en/default/request/queue.html.tmpl: Use review links instead
1116         of edit links in the review queue.
1117
1118 2008-11-10  Darin Adler  <darin@apple.com>
1119
1120         * template/en/custom/request/email.txt.tmpl: Send review links instead
1121         of edit links when flags are set on a bug.
1122
1123 2008-10-27  Darin Adler  <darin@apple.com>
1124
1125         * template/en/custom/attachment/reviewform.html.tmpl: Leave out the
1126         comment if it's untouched.
1127
1128 2008-10-23  Adam Roben  <aroben@apple.com>
1129
1130         Fix Bug 21401: Comments field on "Review Patch" page should be
1131         initially filled with quoted patch
1132
1133         <https://bugs.webkit.org/show_bug.cgi?id=21401>
1134
1135         Reviewed by Dave Hyatt.
1136
1137         * attachment.cgi:
1138         (edit): Retrieve the attachment data from the database instead of just
1139         its length. When the attachment is a patch, create a quotedpatch
1140         template variable that contains the patch data with each line
1141         prepended with "> ".
1142         * template/en/custom/attachment/reviewform.html.tmpl:
1143           - Changed the comments field to have a monospace font
1144           - Added an "Enter comments below:" caption above the comments field
1145             to make it clearer that this is where comments should go, now that
1146             the comments field is not initially empty
1147           - Fill the comments field with the quoted patch
1148
1149 2008-10-14  Adam Roben  <aroben@apple.com>
1150
1151         Fix Bug 21602: Bugzilla times out trying to display formatted diff for
1152         attachment 24345
1153         
1154         https://bugs.webkit.org/show_bug.cgi?id=21602
1155
1156         Reviewed by Dave Kilzer.
1157
1158         * PrettyPatch/PrettyPatch.rb:
1159         (PrettyPatch.BINARY_FILE_MARKER_FORMAT): Added.
1160         (PrettyPatch.FileDiff.initialize): If any of the lines in the diff
1161         are a binary file marker, mark this FileDiff as binary and stop trying
1162         to process the lines.
1163         (PrettyDiff.FileDiff.to_html): If we're binary, just print a string
1164         saying so.
1165
1166 2008-10-02  Adam Roben  <aroben@apple.com>
1167
1168         * template/en/custom/attachment/reviewform.html.tmpl: Fix typo.
1169
1170 2008-10-02  Adam Roben  <aroben@apple.com>
1171
1172         * template/en/custom/attachment/review.html.tmpl: Fix URLs
1173
1174 2008-10-02  Adam Roben  <aroben@apple.com>
1175
1176         Fix Bug 21315: bugs.webkit.org should have a nicer patch review
1177         interface
1178
1179         https://bugs.webkit.org/show_bug.cgi?id=21315
1180
1181         Reviewed by Tim Hatcher.
1182
1183         * attachment.cgi:
1184         (top level): Added support for the "review" and "reviewform" actions.
1185         (sub edit): Accept the template name as a parameter. If no name is
1186         give, use "edit" as before.
1187         * template/en/custom/attachment/list.html.tmpl: Added a "Review Patch"
1188         link for all patch attachments.
1189         * template/en/custom/attachment/review.html.tmpl: Added. Simple
1190         <frameset> page to show the patch on the top and a comment form on the
1191         bottom.
1192         * template/en/custom/attachment/reviewform.html.tmpl: Added. Simple
1193         comment form for reviewing patches.
1194
1195 2008-10-02  Adam Roben  <aroben@apple.com>
1196
1197         Remove references to some backup files I created when implementing
1198         PrettyPatch in Bugzilla
1199
1200         Reviewed by Tim Hatcher.
1201
1202         * attachment-aroben.cgi: Removed.
1203         * template/en/default/attachment/edit-aroben.html.tmpl: Removed.
1204
1205         * template/en/custom/attachment/edit.html.tmpl: Removed an erroneous
1206         reference to attachment-aroben.cgi. attachment.cgi will work just
1207         fine.
1208
1209 2008-06-25  David Kilzer  <ddkilzer@apple.com>
1210
1211         Make PrettyPatch handle diffs with no Index or diff headers
1212
1213         Part of Bug 19290: More patches not handled by PrettyPatch.rb
1214         <https://bugs.webkit.org/show_bug.cgi?id=19290>
1215
1216         Reviewed by Adam.
1217
1218         * PrettyPatch/PrettyPatch.rb:
1219         (PrettyPatch.DIFF_HEADER_FORMATS): Added regular expression to
1220         match on "+++ " lines for patches with no "Index" or "diff" header.
1221         (PrettyPatch.FileDiff.initialize): Look for filename on "+++ " line
1222         when the first line of a patch has no "Index" or "diff" header.
1223         (PrettyPatch.FileDiff.parse): Added haveSeenDiffHeader state
1224         variable to determine when no "Index" or "diff" header has been
1225         found, but a new patch has started with a "--- " line.
1226
1227 2008-06-08  Dan Bernstein  <mitz@apple.com>
1228
1229         Reviewed by Adam Roben.
1230
1231         - update trac URL in PrettyPatch
1232
1233         * PrettyPatch/PrettyPatch.rb:
1234
1235 2008-05-29  Adam Roben  <aroben@apple.com>
1236
1237         Update PrettyPatch directory list
1238
1239         * PrettyPatch/PrettyPatch.rb:
1240
1241 2008-05-28  Adam Roben  <aroben@apple.com>
1242
1243         Make PrettyPatch able to handle diffs taken from outside the WebKit
1244         source tree
1245
1246         Part of Bug 19290: More patches not handled by PrettyPatch.rb
1247         <https://bugs.webkit.org/show_bug.cgi?id=19290>
1248
1249         Reviewed by David Kilzer.
1250
1251         * PrettyPatch/PrettyPatch.rb:
1252         (PrettyPatch.find_url_and_path): Added. Searches the file path from
1253         the bottom up looking for a directory that exists in the source tree.
1254         (PrettyPatch.linkifyFilename): Changed to call find_url_and_path.
1255
1256 2008-05-28  Adam Roben  <aroben@apple.com>
1257
1258         Print exceptions from PrettyPatch to stdout
1259
1260         This will make it much easier to debug problems with PrettyPatch,
1261         since we'll be able to see the exceptions in the browser.
1262
1263         Reviewed by Sam Weinig.
1264
1265         * PrettyPatch/prettify.rb: Added a --html-exceptions option, which
1266         will print exceptions to stdout as HTML.
1267         * attachment.cgi: Changed to pass --html-exceptions to prettify.rb.
1268
1269 2008-04-26  David Kilzer  <ddkilzer@apple.com>
1270
1271         Removed temp files and added appropriate svn:ignore properties.
1272
1273         Rubber-stamped by Mark Rowe.
1274
1275         * data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
1276
1277         * data/template: Added svn:ignore property for "template" subdirectory.
1278         * data/template/template: Removed precompiled template subdirectory.
1279
1280         * data/versioncache: Removed.
1281         * data/versioncache.*: Removed backup versioncache files.
1282
1283         * data/webdot: Added svn:ignore property for "*.dot" files.
1284         * data/webdot/*.dot: Removed cached webdot files.
1285
1286 2008-04-08  Adam Roben  <aroben@apple.com>
1287
1288         Combine :equal operations with the following operation if they are
1289         fewer than 3 characters long
1290
1291         This keeps us from showing lots of small changes on long lines, just
1292         because some letters happened to be the same.
1293
1294         Inspired by
1295         http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/diffviewer/diffutils.py?r=1264#147
1296
1297         * PrettyPatch/PrettyPatch.rb:
1298         (PrettyPatch::DiffSection.initialize): Remove :equal operations fewer
1299         than 3 characters long. The characters from the removed operations
1300         become part of the subsequent operation.
1301         (PrettyPatch::CodeLine.text_as_html): Don't wrap 0-length strings in
1302         <ins>/<del> tags. Also removed the @fromLineNumber.nil? and
1303         @toLineNumber.nil? checks as they are no longer reliable now that
1304         we're removing operations.
1305
1306 2008-04-08  Adam Roben  <aroben@apple.com>
1307
1308         Change PrettyPatch to use DiffBuilder for intra-line diffs
1309
1310         This gives us much prettier intra-line diffs, largely because it can
1311         distinguish multiple changes on the same line. e.g., if a line changes
1312         from:
1313
1314         const int myConstant = 0;
1315
1316         to
1317
1318         static const unsigned myConstant;
1319
1320         You will see that "static " was inserted, "int" changed to "unsigned",
1321         and " = 0" was deleted.
1322
1323         This seems to have also gotten rid of some spurious instances of
1324         "<ins></ins>" and "<del></del>" at the end of a line.
1325
1326         * PrettyPatch/PrettyPatch.rb:
1327         (PrettyPatch::DiffSection.initialize): Use DiffBuilder instead of
1328         setChangeExtentFromLine (which had been copied from Trac's diffing
1329         code).
1330         (PrettyPatch::CodeLine):
1331          - Removed the changeExtent attribute
1332          - Added the operations attribute
1333          - Removed the setChangeExtentFromLine method
1334         (PrettyPatch::CodeLine.text_as_html): Uses @operations to determine
1335         what text to wrap in <ins> and <del> tags.
1336         * PrettyPatch/diff.rb: Deleted a bunch of code we don't use.
1337         (HTMLDiff::DiffBuilder.initialize): Moved code here from the old build
1338         method, but left out the calls to perform_operation, since we build
1339         the diff HTML outside of this class.
1340         (HTMLDiff::DiffBuilder.split_inputs_to_words): Removed calls to
1341         convert_html_to_list_of_words, since we're not diffing HTML.
1342
1343 2008-04-08  Adam Roben  <aroben@apple.com>
1344
1345         Check in diff.rb
1346
1347         This came from
1348         http://instiki.rubyforge.org/svn/instiki/trunk/lib/diff.rb
1349         revision 28521
1350
1351         * PrettyPatch/diff.rb: Added.
1352
1353 2008-03-04  Adam Roben  <aroben@apple.com>
1354
1355         * PrettyPatch/PrettyPatch.rb: Allow the extent of the previous file to
1356         be omitted from the hunk header.
1357