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