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