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