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