2009-12-06 Shinichiro Hamaji <hamaji@chromium.org>
[WebKit-https.git] / BugsSite / ChangeLog
1 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
2
3         Unreviewed. Revert r51748.
4
5         Bugzilla should show images in git patches
6         https://bugs.webkit.org/show_bug.cgi?id=31395
7
8         * PrettyPatch/PrettyPatch.rb:
9
10 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
11
12         Reviewed by David Kilzer.
13
14         Bugzilla should show images in git patches
15         https://bugs.webkit.org/show_bug.cgi?id=31395
16
17         Show images in git patches using git-apply.
18
19         * PrettyPatch/PrettyPatch.rb:
20
21 2009-10-23  Eric Seidel  <eric@webkit.org>
22
23         Reviewed by Adam Roben.
24
25         PrettyPatch should show images even when they have the wrong mime type
26         https://bugs.webkit.org/show_bug.cgi?id=29506
27
28         * PrettyPatch/PrettyPatch.rb:
29
30 2009-10-17  Alexey Proskuryakov  <ap@apple.com>
31
32         Reviewed by Eric Carlson.
33
34         https://bugs.webkit.org/show_bug.cgi?id=30470
35         Make marking a bug as duplicate easier
36
37         * js/field.js: (setResolutionToDuplicate): Focus the edit field, too.
38
39 2009-10-16  Alexey Proskuryakov  <ap@apple.com>
40
41         Reviewed by Mark Rowe.
42
43         https://bugs.webkit.org/show_bug.cgi?id=30470
44         Make marking a bug as duplicate easier
45
46         * js/field.js: (setResolutionToDuplicate): Show edit field right away, don't make the user
47         click again.
48
49 2009-09-09  David Kilzer  <ddkilzer@apple.com>
50
51         Update data/params parameters
52
53         * data/params: Updated 'attachment_base' parameter for secure
54         attachments.  Updated 'mybugstemplate' parameter to search for
55         bugs in UNCONFIRMED, ASSIGNED and REOPENED status in addition to
56         NEW status.
57
58 2009-08-11  David Kilzer  <ddkilzer@apple.com>
59
60         Updated data/params parameter
61
62         * data/params: Updated 'loginnetmask' parameter to allow users
63         to choose whether their cookies are restricted to a single IP
64         address or not.
65
66 2009-07-07  David Kilzer  <ddkilzer@apple.com>
67
68         Updated BugsSite with local changes on server
69
70         The new files were created by checksetup.pl during the upgrade
71         to Bugzilla 3.2.3.  The rest are self-explanatory.
72
73         * data: Added "bugzilla-update.xml" to svn:ignore.
74         * data/attachments: Added "groups.*" to svn:ignore.
75         * data/params: Updated for a few changed parameters.
76
77         * lib/.htaccess: Added.
78         * skins/contrib/Dusk/IE-fixes.css: Added.
79         * skins/contrib/Dusk/admin.css: Added.
80         * skins/contrib/Dusk/create_attachment.css: Added.
81         * skins/contrib/Dusk/dependency-tree.css: Added.
82         * skins/contrib/Dusk/duplicates.css: Added.
83         * skins/contrib/Dusk/editusers.css: Added.
84         * skins/contrib/Dusk/help.css: Added.
85         * skins/contrib/Dusk/index.css: Added.
86         * skins/contrib/Dusk/panel.css: Added.
87         * skins/contrib/Dusk/params.css: Added.
88         * skins/contrib/Dusk/release-notes.css: Added.
89         * skins/contrib/Dusk/show_bug.css: Added.
90         * skins/contrib/Dusk/show_multiple.css: Added.
91         * skins/contrib/Dusk/summarize-time.css: Added.
92         * skins/contrib/Dusk/voting.css: Added.
93         * skins/contrib/Dusk/yui: Added.
94         * skins/contrib/Dusk/yui/calendar.css: Added.
95         * skins/custom/IE-fixes.css: Added.
96         * skins/custom/admin.css: Added.
97         * skins/custom/buglist.css: Added.
98         * skins/custom/create_attachment.css: Added.
99         * skins/custom/dependency-tree.css: Added.
100         * skins/custom/duplicates.css: Added.
101         * skins/custom/editusers.css: Added.
102         * skins/custom/help.css: Added.
103         * skins/custom/panel.css: Added.
104         * skins/custom/params.css: Added.
105         * skins/custom/release-notes.css: Added.
106         * skins/custom/show_bug.css: Added.
107         * skins/custom/show_multiple.css: Added.
108         * skins/custom/summarize-time.css: Added.
109         * skins/custom/voting.css: Added.
110         * skins/custom/yui: Added.
111         * skins/custom/yui/calendar.css: Added.
112
113 2009-07-03  David Kilzer  <ddkilzer@apple.com>
114
115         Bug 26958: Change edit link to review link in request messages
116
117         <https://bugs.webkit.org/show_bug.cgi?id=26958>
118
119         Reviewed by Dan Bernstein.
120
121         * template/en/custom/request/email.txt.tmpl: Copied from BugsSite/template/en/default/request/email.txt.tmpl.
122         Changed link from "action=edit" to "action=review".
123
124 2009-07-03  David Kilzer  <ddkilzer@apple.com>
125
126         Bug 26950: Make the summary and alias fields support click-to-edit
127         <https://bugs.webkit.org/show_bug.cgi?id=26950>
128
129         Reviewed by Maciej Stachowiak.
130
131         Original patch by Maciej Stachowiak.
132
133         * js/field.js:
134         (hideEditableField): Updated to add click event listeners to the
135         alias and short description elements to make them easier to
136         edit.  Renamed field_id parameter to field2_id (short
137         description id) and added a field1_id parameter (alias id).
138         (showEditableField): If a third parameter is passed in the
139         ContainerInputArray parameter, use it to find the element to
140         focus.  Otherwise, fall back to the original behavior of
141         focusing the first input field.
142         (hideAliasAndSummary): Changed to pass the id for the alias
143         element to hideEditableField().
144
145 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
146
147         Config file for contrib/recode.pl when upgrading bugs.webkit.org
148
149         Current as of midday on Wednesday, July 1, 2009.
150
151         * contrib/recode-overrides.txt: Added.
152
153 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
154
155         Workaround for WebKit Bug 9630 when running contrib/recode.pl
156
157         A number of WebKit nightly builds included Bug 9630 which caused
158         non-breaking space characters (0xA0) to be submitted within the
159         content of textarea elements.
160
161         * contrib/recode.pl: To work around these 0xA0 characters, try
162         an encoding of cp1252 and use it if it works.
163
164 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
165
166         Don't print "Failed to guess" warning if an override is available
167
168         * contrib/recode.pl: Check that %overrides does NOT contain a
169         key matching the current digest before printing out the warning
170         about a failed charset guess.
171
172 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
173
174         Added --[no-]truncate switch to contrib/recode.pl
175
176         * contrib/recode.pl: Added --[no-]truncate switch to make
177         debugging of failed charset guessing easier.  Often times the
178         illegal character was truncated in the output.
179
180 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
181
182         Changed Perl scripts to use #!/usr/bin/env perl
183
184         perl -pi -e 's@#\!/usr/bin/perl@#\!/usr/bin/env perl@' *.cgi *.pl */*.pl
185
186 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
187
188         Replace Bugzilla favicon with webkit.org favicon
189
190         * favicon.ico: Removed.
191         * images/favicon.ico: Replaced.  Copied from favicon.ico.
192
193 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
194
195         Merged BugsSite to Bugzilla-3.2.3
196
197         Updated to the latest-and-greatest stable version.
198
199 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
200
201         Merged BugsSite to Bugzilla-3.0.3
202
203         Nothing to see here.  Move along.
204
205 2009-06-09  Eric Seidel  <eric@webkit.org>
206
207         Reviewed by Adam Roben.
208
209         Add support for displaying added png files in PrettyPatch diffs
210         https://bugs.webkit.org/show_bug.cgi?id=26210
211
212         Currently this is SVN only (git-send-bugzilla patches exclude binary data)
213         and only works for PNG files but could easily be made to work for other images as needed.
214
215         * PrettyPatch/PrettyPatch.rb:
216
217 2009-05-19  Eric Seidel  <eric@webkit.org>
218
219         Reviewed by Adam Roben.
220
221         Make PrettyPatch understand quoted filenames in git diffs.
222         https://bugs.webkit.org/show_bug.cgi?id=25881
223
224         * PrettyPatch/PrettyPatch.rb:
225
226 2009-05-15  Simon Fraser  <simon.fraser@apple.com>
227
228         Carrying forwards Darin Adler's rubber stamp
229         
230         Make the comment field wider too.
231
232         * skins/custom/global.css:
233
234 2009-05-14  Simon Fraser  <simon.fraser@apple.com>
235
236         Reviewed by Darin Adler
237         
238         Make the URL, Summary and Keyword fields wider to mitigate the
239         effects of https://bugs.webkit.org/show_bug.cgi?id=25566 and make
240         things generally better.
241
242         * skins/custom/global.css:
243         * template/en/custom/bug/edit.html.tmpl:
244
245 2008-11-10  Darin Adler  <darin@apple.com>
246
247         - fix more of https://bugs.webkit.org/show_bug.cgi?id=21400
248           "Edit" links for patches in comments, review queue, review emails
249           should be replaced by "Review Patch" links
250
251         * globals.pl: Fix cases that are automatically generated, such as links
252         in comments in bugs.
253
254         * template/en/custom/attachment/created.html.tmpl: Fix the link on the
255         "I just created an attachment" page.
256
257         * template/en/custom/request/queue.html.tmpl: Fix the link in the queue
258         page.
259
260 2008-11-10  Darin Adler  <darin@apple.com>
261
262         * template/en/default/request/queue.html.tmpl: Use review links instead
263         of edit links in the review queue.
264
265 2008-11-10  Darin Adler  <darin@apple.com>
266
267         * template/en/custom/request/email.txt.tmpl: Send review links instead
268         of edit links when flags are set on a bug.
269
270 2008-10-27  Darin Adler  <darin@apple.com>
271
272         * template/en/custom/attachment/reviewform.html.tmpl: Leave out the
273         comment if it's untouched.
274
275 2008-10-23  Adam Roben  <aroben@apple.com>
276
277         Fix Bug 21401: Comments field on "Review Patch" page should be
278         initially filled with quoted patch
279
280         <https://bugs.webkit.org/show_bug.cgi?id=21401>
281
282         Reviewed by Dave Hyatt.
283
284         * attachment.cgi:
285         (edit): Retrieve the attachment data from the database instead of just
286         its length. When the attachment is a patch, create a quotedpatch
287         template variable that contains the patch data with each line
288         prepended with "> ".
289         * template/en/custom/attachment/reviewform.html.tmpl:
290           - Changed the comments field to have a monospace font
291           - Added an "Enter comments below:" caption above the comments field
292             to make it clearer that this is where comments should go, now that
293             the comments field is not initially empty
294           - Fill the comments field with the quoted patch
295
296 2008-10-14  Adam Roben  <aroben@apple.com>
297
298         Fix Bug 21602: Bugzilla times out trying to display formatted diff for
299         attachment 24345
300         
301         https://bugs.webkit.org/show_bug.cgi?id=21602
302
303         Reviewed by Dave Kilzer.
304
305         * PrettyPatch/PrettyPatch.rb:
306         (PrettyPatch.BINARY_FILE_MARKER_FORMAT): Added.
307         (PrettyPatch.FileDiff.initialize): If any of the lines in the diff
308         are a binary file marker, mark this FileDiff as binary and stop trying
309         to process the lines.
310         (PrettyDiff.FileDiff.to_html): If we're binary, just print a string
311         saying so.
312
313 2008-10-02  Adam Roben  <aroben@apple.com>
314
315         * template/en/custom/attachment/reviewform.html.tmpl: Fix typo.
316
317 2008-10-02  Adam Roben  <aroben@apple.com>
318
319         * template/en/custom/attachment/review.html.tmpl: Fix URLs
320
321 2008-10-02  Adam Roben  <aroben@apple.com>
322
323         Fix Bug 21315: bugs.webkit.org should have a nicer patch review
324         interface
325
326         https://bugs.webkit.org/show_bug.cgi?id=21315
327
328         Reviewed by Tim Hatcher.
329
330         * attachment.cgi:
331         (top level): Added support for the "review" and "reviewform" actions.
332         (sub edit): Accept the template name as a parameter. If no name is
333         give, use "edit" as before.
334         * template/en/custom/attachment/list.html.tmpl: Added a "Review Patch"
335         link for all patch attachments.
336         * template/en/custom/attachment/review.html.tmpl: Added. Simple
337         <frameset> page to show the patch on the top and a comment form on the
338         bottom.
339         * template/en/custom/attachment/reviewform.html.tmpl: Added. Simple
340         comment form for reviewing patches.
341
342 2008-10-02  Adam Roben  <aroben@apple.com>
343
344         Remove references to some backup files I created when implementing
345         PrettyPatch in Bugzilla
346
347         Reviewed by Tim Hatcher.
348
349         * attachment-aroben.cgi: Removed.
350         * template/en/default/attachment/edit-aroben.html.tmpl: Removed.
351
352         * template/en/custom/attachment/edit.html.tmpl: Removed an erroneous
353         reference to attachment-aroben.cgi. attachment.cgi will work just
354         fine.
355
356 2008-06-25  David Kilzer  <ddkilzer@apple.com>
357
358         Make PrettyPatch handle diffs with no Index or diff headers
359
360         Part of Bug 19290: More patches not handled by PrettyPatch.rb
361         <https://bugs.webkit.org/show_bug.cgi?id=19290>
362
363         Reviewed by Adam.
364
365         * PrettyPatch/PrettyPatch.rb:
366         (PrettyPatch.DIFF_HEADER_FORMATS): Added regular expression to
367         match on "+++ " lines for patches with no "Index" or "diff" header.
368         (PrettyPatch.FileDiff.initialize): Look for filename on "+++ " line
369         when the first line of a patch has no "Index" or "diff" header.
370         (PrettyPatch.FileDiff.parse): Added haveSeenDiffHeader state
371         variable to determine when no "Index" or "diff" header has been
372         found, but a new patch has started with a "--- " line.
373
374 2008-06-08  Dan Bernstein  <mitz@apple.com>
375
376         Reviewed by Adam Roben.
377
378         - update trac URL in PrettyPatch
379
380         * PrettyPatch/PrettyPatch.rb:
381
382 2008-05-29  Adam Roben  <aroben@apple.com>
383
384         Update PrettyPatch directory list
385
386         * PrettyPatch/PrettyPatch.rb:
387
388 2008-05-28  Adam Roben  <aroben@apple.com>
389
390         Make PrettyPatch able to handle diffs taken from outside the WebKit
391         source tree
392
393         Part of Bug 19290: More patches not handled by PrettyPatch.rb
394         <https://bugs.webkit.org/show_bug.cgi?id=19290>
395
396         Reviewed by David Kilzer.
397
398         * PrettyPatch/PrettyPatch.rb:
399         (PrettyPatch.find_url_and_path): Added. Searches the file path from
400         the bottom up looking for a directory that exists in the source tree.
401         (PrettyPatch.linkifyFilename): Changed to call find_url_and_path.
402
403 2008-05-28  Adam Roben  <aroben@apple.com>
404
405         Print exceptions from PrettyPatch to stdout
406
407         This will make it much easier to debug problems with PrettyPatch,
408         since we'll be able to see the exceptions in the browser.
409
410         Reviewed by Sam Weinig.
411
412         * PrettyPatch/prettify.rb: Added a --html-exceptions option, which
413         will print exceptions to stdout as HTML.
414         * attachment.cgi: Changed to pass --html-exceptions to prettify.rb.
415
416 2008-04-26  David Kilzer  <ddkilzer@apple.com>
417
418         Removed temp files and added appropriate svn:ignore properties.
419
420         Rubber-stamped by Mark Rowe.
421
422         * data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
423
424         * data/template: Added svn:ignore property for "template" subdirectory.
425         * data/template/template: Removed precompiled template subdirectory.
426
427         * data/versioncache: Removed.
428         * data/versioncache.*: Removed backup versioncache files.
429
430         * data/webdot: Added svn:ignore property for "*.dot" files.
431         * data/webdot/*.dot: Removed cached webdot files.
432
433 2008-04-08  Adam Roben  <aroben@apple.com>
434
435         Combine :equal operations with the following operation if they are
436         fewer than 3 characters long
437
438         This keeps us from showing lots of small changes on long lines, just
439         because some letters happened to be the same.
440
441         Inspired by
442         http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/diffviewer/diffutils.py?r=1264#147
443
444         * PrettyPatch/PrettyPatch.rb:
445         (PrettyPatch::DiffSection.initialize): Remove :equal operations fewer
446         than 3 characters long. The characters from the removed operations
447         become part of the subsequent operation.
448         (PrettyPatch::CodeLine.text_as_html): Don't wrap 0-length strings in
449         <ins>/<del> tags. Also removed the @fromLineNumber.nil? and
450         @toLineNumber.nil? checks as they are no longer reliable now that
451         we're removing operations.
452
453 2008-04-08  Adam Roben  <aroben@apple.com>
454
455         Change PrettyPatch to use DiffBuilder for intra-line diffs
456
457         This gives us much prettier intra-line diffs, largely because it can
458         distinguish multiple changes on the same line. e.g., if a line changes
459         from:
460
461         const int myConstant = 0;
462
463         to
464
465         static const unsigned myConstant;
466
467         You will see that "static " was inserted, "int" changed to "unsigned",
468         and " = 0" was deleted.
469
470         This seems to have also gotten rid of some spurious instances of
471         "<ins></ins>" and "<del></del>" at the end of a line.
472
473         * PrettyPatch/PrettyPatch.rb:
474         (PrettyPatch::DiffSection.initialize): Use DiffBuilder instead of
475         setChangeExtentFromLine (which had been copied from Trac's diffing
476         code).
477         (PrettyPatch::CodeLine):
478          - Removed the changeExtent attribute
479          - Added the operations attribute
480          - Removed the setChangeExtentFromLine method
481         (PrettyPatch::CodeLine.text_as_html): Uses @operations to determine
482         what text to wrap in <ins> and <del> tags.
483         * PrettyPatch/diff.rb: Deleted a bunch of code we don't use.
484         (HTMLDiff::DiffBuilder.initialize): Moved code here from the old build
485         method, but left out the calls to perform_operation, since we build
486         the diff HTML outside of this class.
487         (HTMLDiff::DiffBuilder.split_inputs_to_words): Removed calls to
488         convert_html_to_list_of_words, since we're not diffing HTML.
489
490 2008-04-08  Adam Roben  <aroben@apple.com>
491
492         Check in diff.rb
493
494         This came from
495         http://instiki.rubyforge.org/svn/instiki/trunk/lib/diff.rb
496         revision 28521
497
498         * PrettyPatch/diff.rb: Added.
499
500 2008-03-04  Adam Roben  <aroben@apple.com>
501
502         * PrettyPatch/PrettyPatch.rb: Allow the extent of the previous file to
503         be omitted from the hunk header.
504