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