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