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