PrettyDiff.rb fails to render image diffs with Ruby 1.9.3p194
[WebKit-https.git] / Websites / bugs.webkit.org / ChangeLog
1 2012-12-30  Martin Robinson  <mrobinson@igalia.com>
2
3         PrettyDiff.rb fails to render image diffs with Ruby 1.9.3p194
4         https://bugs.webkit.org/show_bug.cgi?id=104932
5
6         Reviewed by Eric Seidel.
7
8         Ruby 1.9 added the concept of string encodings, so the use of `match' with binary data
9         treats it as UTF-8. Forcing the png_bytes string to have the "binary" encoding avoids
10         this error.
11
12         * PrettyPatch/PrettyPatch.rb: Use force_encoding.
13
14 2012-12-06  Ojan Vafai  <ojan@chromium.org>
15
16         Review page is super flakey now
17         https://bugs.webkit.org/show_bug.cgi?id=104331
18
19         Committing without review since this is breaking the code review tool.
20
21         remove is only shipping in Chromium. Use jQuery's remove instead.
22
23         * PrettyPatch/PrettyPatch.rb:
24         * code-review.js:
25
26 2012-12-05  Ojan Vafai  <ojan@chromium.org>
27
28         Make the strip line numbers link into a checkbox
29         https://bugs.webkit.org/show_bug.cgi?id=104187
30
31         Reviewed by Tony Chang.
32
33         The link text was confusing.
34
35         * PrettyPatch/PrettyPatch.rb:
36         * code-review.js:
37
38 2012-12-05  Ojan Vafai  <ojan@chromium.org>
39
40         Sanitize content on copy in the code review tool
41         https://bugs.webkit.org/show_bug.cgi?id=104155
42
43         Reviewed by Tony Chang.
44
45         Always remove expand/header/annotate links. Provide an option
46         to remove line numbers as well. Store the option in localStorage so
47         people can always get whichever behavior they want.
48
49         A better solution would be to restructure the DOM, but that would require gutting
50         the whole code review tool and would make it difficult to include line numbers if
51         you wanted them.
52
53         * PrettyPatch/PrettyPatch.rb:
54         * code-review-test.html:
55         * code-review.js:
56
57 2012-12-04  Ojan Vafai  <ojan@chromium.org>
58
59         Use sticky positioning for the code review toolbar
60         https://bugs.webkit.org/show_bug.cgi?id=104056
61
62         Reviewed by Adam Barth.
63
64         This simplifies the code and gives a nicer user-experience.
65         Also, while here, I fixed up the CSS to not have toolbar items
66         overlap when you make the window too small.
67
68         * PrettyPatch/PrettyPatch.rb:
69         * code-review-test.html:
70         * code-review.js:
71
72 2012-12-04  Ojan Vafai  <ojan@chromium.org>
73
74         Properly create the header links in the code review tool
75         https://bugs.webkit.org/show_bug.cgi?id=104037
76
77         Reviewed by Adam Barth.
78
79         Right now, we just prepend "header" to the two existing links and change
80         the first link to point to the header. The problem is that jQuery's prepend
81         method just prepends to each item in the jQuery object instead of adding
82         the item to the beginning of the list as this code assumed.
83
84         * code-review-test.html:
85         * code-review.js:
86
87 2012-12-04  Ojan Vafai  <ojan@chromium.org>
88
89         Can't add followup comment to a previous comment
90         https://bugs.webkit.org/show_bug.cgi?id=104025
91
92         Reviewed by Adam Barth.
93
94         If we side-by-sidify a shared diff line, and then apply
95         a previous comment, we would incorrectly put the comment
96         on the Line instead of the LineContainer.
97
98         Also, get rid of global next_line_id to simplify testing.
99
100         * code-review-test.html:
101         * code-review.js:
102
103 2012-11-06  Ryosuke Niwa  <rniwa@webkit.org>
104
105         committers-autocomplete.js works only with WebKit based browsers
106         https://bugs.webkit.org/show_bug.cgi?id=66752
107
108         Reviewed by Csaba Osztrogonác.
109
110         Based on the patch posted by Peter Gal. Attach event listeners
111         to all input elements we want to enable auto-completions on.
112
113         * committers-autocomplete.js:
114
115 2012-10-03  Simon Fraser  <simon.fraser@apple.com>
116
117         Make the status bubble iframe big enough to show all the bubbles
118         https://bugs.webkit.org/show_bug.cgi?id=98329
119
120         Reviewed by Eric Seidel.
121
122         Make the status bubble iframe big enough.
123
124         * template/en/custom/attachment/edit.html.tmpl:
125         * template/en/custom/attachment/list.html.tmpl:
126
127 2012-10-03  Simon Pena  <spena@igalia.com>
128
129         ruby1.9 fails in PrettyPatch.rb with invalid byte sequence in UTF-8
130         https://bugs.webkit.org/show_bug.cgi?id=91212
131
132         Reviewed by Hajime Morita.
133
134         ruby1.9 changed the way it handles encoding, so the gsub method used
135         when normalizing line ending complains of invalid byte sequence in UTF-8.
136         This patch uses the "encode" string method for versions of Ruby >= 1.9
137         to provide the encoding conversion (keeping the current implementation in
138         Ruby 1.8).
139
140         * PrettyPatch/PrettyPatch.rb: Use the "encode" string method for encoding
141         conversion when the Ruby version is >= 1.9.
142
143 2012-09-26  Sheriff Bot  <webkit.review.bot@gmail.com>
144
145         Unreviewed, rolling out r129521.
146         http://trac.webkit.org/changeset/129521
147         https://bugs.webkit.org/show_bug.cgi?id=97710
148
149         Broke autocomplete on enter_bug.cgi (Requested by abarth on
150         #webkit).
151
152         * committers-autocomplete.js:
153
154 2012-09-25  Peter Gal  <galpeter@inf.u-szeged.hu>
155
156         committers-autocomplete.js works only with WebKit based browsers
157         https://bugs.webkit.org/show_bug.cgi?id=66752
158
159         Reviewed by Darin Adler.
160
161         Modified the code do only add event listeners to the given inputs,
162         and not to the full document.
163
164         * committers-autocomplete.js:
165
166 2012-09-24  Peter Beverloo  <peter@chromium.org>
167
168         Allow robots to index bugs on Bugzilla
169         https://bugs.webkit.org/show_bug.cgi?id=95688
170
171         Reviewed by Eric Seidel.
172
173         This allows robots to access show_bug.cgi, thus granting them access to
174         index contents and discussion on WebKit bugs. All other pages, except
175         for the site index itself, still have indexing disabled.
176
177         * robots.txt:
178
179 2012-07-13  Ojan Vafai  <ojan@chromium.org>
180
181         PrettyPatch.rb complains about missing checksum for new pixel results
182         https://bugs.webkit.org/show_bug.cgi?id=88368
183
184         Reviewed by Tony Chang.
185
186         When adding or removing a file, we incorrectly iterpreted not having an image
187         as not having a checksum.
188
189         * PrettyPatch/PrettyPatch.rb:
190         * PrettyPatch/PrettyPatch_test.rb:
191         I tried to fix the TempFile issue in these tests, but after a couple hours
192         of banging my head against this, I have no idea what's breaking.
193
194 2012-07-11  Alice Cheng  <alice_cheng@apple.com>
195
196         Deleting content at the top of prettypatch emails destroys HTML formatting
197         https://bugs.webkit.org/show_bug.cgi?id=90700
198         <rdar://problem/7488232>
199
200         Reviewed by David Kilzer.
201
202         * PrettyPatch/PrettyPatch.rb:
203
204 2012-07-10  Adam Barth  <abarth@webkit.org>
205
206         bugs.webkit.org has mixed content
207         https://bugs.webkit.org/show_bug.cgi?id=90907
208
209         Reviewed by Eric Seidel.
210
211         Now that we detect http XMLHttpRequests as mixed content, I've noticed
212         that we're loading committers.py over http instead of https.  For
213         better security, we should use https.
214
215         * committers-autocomplete.js:
216
217 2012-05-30  Ojan Vafai  <ojan@chromium.org>
218
219         Only include the image checksum error in pretty-diffs for -expected.png files.
220         https://bugs.webkit.org/show_bug.cgi?id=87881
221
222         Reviewed by Tony Chang.
223
224         * PrettyPatch/PrettyPatch.rb:
225         * PrettyPatch/PrettyPatch_test.rb:
226
227 2012-05-29  Ojan Vafai  <ojan@chromium.org>
228
229         Show an error in the pretty diff when an image lacks a checksum
230         https://bugs.webkit.org/show_bug.cgi?id=87791
231
232         Reviewed by Dirk Pranke.
233
234         * PrettyPatch/PrettyPatch.rb:
235         * PrettyPatch/PrettyPatch_test.rb:
236
237 2012-04-01  Adam Barth  <abarth@webkit.org>
238
239         Code review tool no longer needs to work around position:fixed handling on iPad
240         https://bugs.webkit.org/show_bug.cgi?id=82850
241
242         Reviewed by Eric Seidel.
243
244         We no longer need to avoid position:fixed on iPad because Mobile Safari
245         now does something reasonable with position:fixed elements.
246
247         * code-review.js:
248
249 2012-03-09  Ashod Nakashian  <ashodnakashian@yahoo.com>
250
251         Bash scripts should support LF endings only
252         https://bugs.webkit.org/show_bug.cgi?id=79509
253
254         Reviewed by David Kilzer.
255
256         * contrib/yp_nomail.sh: Added property svn:executable.
257
258 2012-03-05  Alexander Færøy  <alexander.faeroy@nokia.com>
259
260         The Bugzilla autocompleter doesn't support multiple Unicode characters per name
261         https://bugs.webkit.org/show_bug.cgi?id=80319
262
263         Reviewed by Adam Barth.
264
265         The current regular expression for matching Unicode characters uses
266         the global modifier, which means that it will never enter its second
267         iteration of the match-loop. This patch fixes this by removing the
268         global modifier from the regex.
269
270         * committers-autocomplete.js:
271         (WebKitCommitters):
272
273 2012-01-03  Ojan Vafai  <ojan@chromium.org>
274
275         Increment the code-review.js version number to bust browser
276         caches.
277
278         * PrettyPatch/PrettyPatch.rb:
279
280 2011-12-13  Ojan Vafai  <ojan@chromium.org>
281
282         Give a resize handle for sidebyside diffs.
283         https://bugs.webkit.org/show_bug.cgi?id=74465
284
285         Reviewed by Ryosuke Niwa.
286
287         This gives a drag handle so you can make the side-by-side
288         diff not be 50/50 on a per-file-diff basis. This is useful
289         for cases like new files where there's no value on the
290         left side.
291
292         * PrettyPatch/PrettyPatch.rb:
293         * code-review.js:
294
295 2011-12-13  Ojan Vafai  <ojan@chromium.org>
296
297         Expanding context is broken for prepare-ChangeLog in the code review tool.
298         https://bugs.webkit.org/show_bug.cgi?id=74458
299
300         Reviewed by Adam Barth.
301
302         * code-review-test.html:
303         -Moved all the tests into test* functions.
304         -Automated calling all test* functions.
305         -Added testIsChangeLog.
306         * code-review.js:
307         Made the check for whether it's a ChangeLog file more robust.
308
309 2011-12-13  Ojan Vafai  <ojan@chromium.org>
310
311         Fix bug in the code review tool when readding a discarded comment
312         https://bugs.webkit.org/show_bug.cgi?id=74450
313
314         Reviewed by Adam Barth.
315
316         If you discard a comment that has a corresponding previousComment,
317         then we would incorrectly remove the comment baseline. So, the next
318         time you added a comment by clicking on the previousComment, we
319         would get undefined as the start line for the new comment.
320
321         All of this works fine until you try to restore the comment from
322         localStorage, at which point we throw an error because the start
323         line is undefined.
324
325         Also added some failsafes to better handle the case of corrupted comments.
326
327         * code-review-test.html:
328         * code-review.js:
329
330 2011-11-15  Tony Chang  <tony@chromium.org>
331
332         set a max-width on the codereview overall comments textarea
333         https://bugs.webkit.org/show_bug.cgi?id=72415
334
335         Reviewed by Ojan Vafai.
336
337         Otherwise, when resizing the textarea you can end up making wider, but
338         are no longer able to click the resize gripper to make it smaller
339         (it's under another div).
340
341         * PrettyPatch/PrettyPatch.rb:
342
343 2011-10-13  Adam Barth  <abarth@webkit.org>
344
345         Would be nice if the review-tool offered to show the corresponding header
346         https://bugs.webkit.org/show_bug.cgi?id=63963
347
348         Reviewed by Ojan Vafai.
349
350         This is a first cut at adding links to the header file in the review
351         tool.  If folks like this feature, we can cover more cases in the
352         future.
353
354         * code-review.js:
355
356 2011-09-14  Ben Wells  <benwells@chromium.org>
357
358         PrettyPatch should handle "delta" patch mechanism in git binary patches
359         https://bugs.webkit.org/show_bug.cgi?id=67628
360
361         Git patches are encoded using two mechanisms - "literal" and "delta".
362         For details of these mechanisms, see the function emit_binary_diff_body
363         in the git source file diff.c (https://github.com/git/git/blob/master/diff.c).
364
365         When determining if a binary file patch is an image or not we should accept
366         both literal and delta patch encodings.
367
368         When reconstructing the images from the patches, if we have a delta patch
369         we may download the previous revision from svn.webkit.org to get the image data.
370
371         Reviewed by Adam Roben.
372
373         * PrettyPatch/PrettyPatch.rb:
374         * PrettyPatch/PrettyPatch_test.rb:
375
376 2011-09-06  Sheriff Bot  <webkit.review.bot@gmail.com>
377
378         Unreviewed, rolling out r94554.
379         http://trac.webkit.org/changeset/94554
380         https://bugs.webkit.org/show_bug.cgi?id=67631
381
382         This patch did not quite fix the problem (Requested by
383         benwells on #webkit).
384
385         * PrettyPatch/PrettyPatch.rb:
386         * PrettyPatch/PrettyPatch_test.rb:
387
388 2011-09-06  Ben Wells  <benwells@chromium.org>
389
390         PrettyPatch should handle "delta" patch mechanism in git binary patches
391         https://bugs.webkit.org/show_bug.cgi?id=67628
392
393         Git patches are encoded using two mechanisms - "literal" and "delta".
394         See this email from the git mailing list archive for info
395         http://marc.info/?l=git&m=114682417113315&w=2
396
397         When determining if a binary file patch is an image or not we should accept
398         both literal and delta patch encodings.
399
400         Reviewed by Shinichiro Hamaji.
401
402         * PrettyPatch/PrettyPatch.rb:
403         * PrettyPatch/PrettyPatch_test.rb:
404
405 2011-06-30  Adam Barth  <abarth@webkit.org>
406
407         Reviewed by Eric Seidel.
408
409         [reviewtool] Expanding lines "below" does not remove purple context line
410         https://bugs.webkit.org/show_bug.cgi?id=63695
411
412         Honestly, this is the last bug in the review tool that bothers me.
413
414         * PrettyPatch/PrettyPatch.rb:
415             - Jump straight to version 42, because, well, you know.
416         * code-review.js:
417             - In addition to removing the context bar when we expand all, we
418               now remove it when we expand below (but not when we expand
419               above).
420
421 2011-06-21  Adam Barth  <abarth@webkit.org>
422
423         Reviewed by Darin Adler.
424
425         bugs.webkit.org should use Strict-Transport-Security
426         https://bugs.webkit.org/show_bug.cgi?id=63097
427
428         Strict-Transport-Security forces all connections to bugs.webkit.org to
429         use HTTPS, preventing sslstrip and other attacks.
430
431         * .htaccess:
432
433 2011-05-04  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
434
435         Reviewed by Adam Roben.
436
437         REGRESSION (r85657): webkitpy.common.prettypatch_unittest.PrettyPatchTest.test_pretty_diff_encodings failing on Leopard
438         https://bugs.webkit.org/show_bug.cgi?id=60164
439
440         Use Array#length to get the number of elements in an array. Array#count
441         was added in Ruby 1.8.7, but Leopard have Ruby 1.8.6.
442
443         * PrettyPatch/PrettyPatch.rb:
444
445 2011-05-03  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
446
447         Reviewed by Adam Roben.
448
449         Add tests to PrettyPatch
450         https://bugs.webkit.org/show_bug.cgi?id=57298
451
452         Verify that 'prettify' parse the right number of files and parts
453         (add/remove/shared) by checking a set of patches available in
454         bugs.webkit.org.
455
456         * PrettyPatch/PrettyPatch.rb: exported a couple a globals with
457         statistics information for our test script.
458         * PrettyPatch/PrettyPatch_test.rb: Added.
459
460 2011-04-22  Alexey Proskuryakov  <ap@apple.com>
461
462         Code changes by Adam Barth. Reviewed by Ojan Vafai.
463
464         Bugzilla autocomplete should support frequent contributors who are not committers
465         https://bugs.webkit.org/show_bug.cgi?id=59209
466
467         * committers-autocomplete.js: Support Contributor role in committers.py.
468
469 2011-04-19  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
470
471         Reviewed by Adam Roben.
472
473         PrettyPatch tries to show images for deleted files
474         https://bugs.webkit.org/show_bug.cgi?id=28457
475
476         When the image data is empty, instead of generating an <img> tag, generate
477         a text to indicate the image was deleted.
478
479         * PrettyPatch/PrettyPatch.rb:
480
481 2011-04-14  Tony Chang  <tony@chromium.org>
482
483         Reviewed by Adam Roben.
484
485         make PrettyPatch.rb aware of checksums in png files
486         https://bugs.webkit.org/show_bug.cgi?id=58387
487
488         Patch v2 with nil checks in the case of file adds and deletes.
489
490         * PrettyPatch/PrettyPatch.rb:
491
492 2011-04-13  Simon Fraser  <simon.fraser@apple.com>
493
494         Revert http://trac.webkit.org/changeset/83750 because it broke PrettyPatch.
495         https://bugs.webkit.org/show_bug.cgi?id=58510        
496
497         * PrettyPatch/PrettyPatch.rb:
498
499 2011-04-12  Tony Chang  <tony@chromium.org>
500
501         Reviewed by Adam Roben.
502
503         make PrettyPatch.rb aware of checksums in png files
504         https://bugs.webkit.org/show_bug.cgi?id=58387
505
506         * PrettyPatch/PrettyPatch.rb:
507
508 2011-04-12  Adam Barth  <abarth@webkit.org>
509
510         Reviewed by Eric Seidel.
511
512         Review tool should work well on iPad
513         https://bugs.webkit.org/show_bug.cgi?id=58351
514
515         On the iPad, the toolbar ends up at some random position in the middle
516         of the page because of how position:fixed interacts with scrolling on
517         the iPad.  I feel a bit dirty using a user agent detect, but I'm not
518         sure if there's a better way to detect whether the user agent has this
519         behavior.
520
521         * code-review.js:
522
523 2011-03-15  David Levin  <levin@chromium.org>
524
525         Reviewed by Adam Barth.
526
527         Links from code reviews to trac are incorrect for files in Source/WebKit.
528         https://bugs.webkit.org/show_bug.cgi?id=56404
529
530         * PrettyPatch/PrettyPatch.rb: Remove WebKit from the list of top level
531           directories since it is now under Source.
532
533 2011-03-10  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
534
535         Reviewed by Adam Roben.
536
537         Formatted Diff for attachment 23920 is mangled
538         https://bugs.webkit.org/show_bug.cgi?id=21222
539
540         The mangled diff files mentioned in the bug were using Mac line
541         ending and this was causing problems for String#each_line. Now we
542         normalize the line endings in patch data before processing it.
543
544         * PrettyPatch/PrettyPatch.rb:
545
546 2011-03-10  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
547
548         Reviewed by Adam Roben.
549
550         PrettyPatch displays last two lines of a git-format-patch diff strangely
551         https://bugs.webkit.org/show_bug.cgi?id=29317
552
553         If the diff section has full range information, we parse only enough to cover the
554         range. This avoids incorrectly showing trailing lines (like git signature) as part
555         of the patch.
556
557         * PrettyPatch/PrettyPatch.rb:
558
559 2011-03-10  Adam Roben  <aroben@apple.com>
560
561         Don't search for intra-line diffs in really long lines
562
563         Doing so can lead to hangs (or at least really slow execution).
564
565         Fixes <http://webkit.org/b/56109> run-webkit-tests sometimes times out on Windows XP Debug
566         (Tests) after fast/text/large-text-composed-char-dos.html fails (due to PrettyPatch
567         hanging?)
568
569         Reviewed by David Kilzer.
570
571         * PrettyPatch/PrettyPatch.rb:
572         (PrettyPatch.MAXIMUM_INTRALINE_DIFF_LINE_LENGTH): Added this new constant.
573         (PrettyPatch.DiffSection.initialize): Don't bother looking for intra-line diffs in lines
574         longer than the maximum length.
575
576 2011-02-25  Ojan Vafai  <ojan@chromium.org>
577
578         Reviewed by Adam Barth.
579
580         [codereviewtool] fix expand links on git diffs
581         https://bugs.webkit.org/show_bug.cgi?id=55208
582
583         In git diffs, we don't have a "revision" span at the end
584         of each FileDiff.
585         * code-review.js:
586
587 2011-02-23  Ojan Vafai  <ojan@chromium.org>
588
589         Reviewed by Tony Chang.
590
591         autocomplete=off prevents refilling form fields on back/forward navigation
592         https://bugs.webkit.org/show_bug.cgi?id=47463
593
594         I haven't actually run this patch due to the difficulty of getting
595         a bugzilla instance running. It's simple enough code that I'm not too worried
596         about it.
597
598         * committers-autocomplete.js:
599
600 2011-02-21  Ojan Vafai  <ojan@chromium.org>
601
602         Reviewed by Adam Barth.
603
604         [codereviewtool] focus first visible node if no node is focused
605         https://bugs.webkit.org/show_bug.cgi?id=54935
606
607         Now hitting j/k/n/p will focus the first/last visible node
608         if no other node is focused. Also tweaked the scroll into view
609         logic a bit to avoid scrolling in this case.
610
611         * code-review.js:
612
613 2011-02-21  Ojan Vafai  <ojan@chromium.org>
614
615         Reviewed by Adam Barth.
616
617         [codereviewtool] remove patch fuzzing
618         https://bugs.webkit.org/show_bug.cgi?id=54940
619
620         This code is buggy and not as necessary now that we
621         include svn revision numbers in uploaded patches.
622
623         * code-review.js:
624
625 2011-02-21  Ojan Vafai  <ojan@chromium.org>
626
627         Reviewed by Adam Barth.
628
629         [codereviewtool] focus comment on accept
630         https://bugs.webkit.org/show_bug.cgi?id=54930
631
632         * code-review.js:
633
634 2011-02-20  Ojan Vafai  <ojan@chromium.org>
635
636         Reviewed by Adam Barth.
637
638         [codereviewtool] avoid overriding ctrl/cmd + r
639         https://bugs.webkit.org/show_bug.cgi?id=54852
640
641         * code-review.js:
642
643 2011-02-16  Ojan Vafai  <ojan@chromium.org>
644
645         Reviewed by Adam Barth.
646
647         keyboard support for extending/shrinking comment context
648         https://bugs.webkit.org/show_bug.cgi?id=54612
649
650         ctrl+shift+up/down will extend/shrink the comment context when
651         a comment is focused or when one is being edited.
652
653         Also, switch over to using keydown instead of keypress events.
654         This lets us share code for handling escape and other key events.
655         Also, keypress is evil and should die.
656
657         * code-review.js:
658
659 2011-02-20  Ojan Vafai  <ojan@chromium.org>
660
661         Recommit accidental revert.
662         r79180 accidentally revereted r79178.
663         * code-review.js:
664
665 2011-02-20  Ojan Vafai  <ojan@chromium.org>
666
667         Reviewed by Adam Barth.
668
669         [codereviewtool] add more help information
670         https://bugs.webkit.org/show_bug.cgi?id=54847
671
672         * code-review.js:
673
674 2011-02-20  Ojan Vafai  <ojan@chromium.org>
675
676         Reviewed by Adam Barth.
677
678         [codereviewtool] use 'r' to focus the review select element
679         https://bugs.webkit.org/show_bug.cgi?id=54845
680
681         This allows for quick keyboard access to the toolbar items.
682         Hit r to get to the review select element. Then you can
683         tab to the cq/preview/publish elements.
684
685         * code-review.js:
686
687 2011-02-20  Ojan Vafai  <ojan@chromium.org>
688
689         Reviewed by Adam Barth.
690
691         [codereviewtool] don't let lines/diffblocks be mouse focusable
692         https://bugs.webkit.org/show_bug.cgi?id=54851
693
694         Unfortunately, this will only work once https://bugs.webkit.org/show_bug.cgi?id=54727
695         is committed. In the interim, it won't break anything.
696
697         * code-review.js:
698
699 2011-02-20  Ojan Vafai  <ojan@chromium.org>
700
701         Reviewed by Adam Barth.
702
703         [codereviewtool] use keydown instead of keypress
704         https://bugs.webkit.org/show_bug.cgi?id=54849
705
706         There is no functional change. This is just a cleanup
707         to make future patches (e.g. https://bugs.webkit.org/show_bug.cgi?id=54612)
708         cleaner.
709
710         * code-review.js:
711
712 2011-02-20  Ojan Vafai  <ojan@chromium.org>
713
714         Reviewed by Adam Barth.
715
716         [codereviewtool] fix layout nit
717         https://bugs.webkit.org/show_bug.cgi?id=54848
718
719         There is occasionally an off-by-one in the 50% width
720         calculation. This ensures that the add lines always
721         line up with the shared context lines.
722
723         * PrettyPatch/PrettyPatch.rb:
724
725 2011-02-20  Ojan Vafai  <ojan@chromium.org>
726
727         Reviewed by Adam Barth.
728
729         [codereviewtool] make escape hide the preview form
730         https://bugs.webkit.org/show_bug.cgi?id=54844
731
732         * code-review.js:
733
734 2011-02-20  Ojan Vafai  <ojan@chromium.org>
735
736         Reviewed by Adam Barth.
737
738         [codereviewtool] make enter work when an individual line is focused
739         https://bugs.webkit.org/show_bug.cgi?id=54843
740
741         * code-review.js:
742
743 2011-02-17  Ojan Vafai  <ojan@chromium.org>
744
745         Reviewed by Adam Barth.
746
747         use shift+j/k to focus next/previous line
748         https://bugs.webkit.org/show_bug.cgi?id=54723
749
750         Tested on a large Hyatt change to confirm it performed fine.
751         (https://bugs.webkit.org/attachment.cgi?id=82533&action=review)
752
753         * code-review.js:
754
755 2011-02-17  Ojan Vafai  <ojan@chromium.org>
756
757         Reviewed by Adam Barth.
758
759         fix escape handling in code review tool
760         https://bugs.webkit.org/show_bug.cgi?id=54722
761
762         This broke with http://trac.webkit.org/changeset/78948.
763         Apparently jquery events get a wrapped object that
764         does not expose keyIdentifier.
765
766         * code-review.js:
767
768 2011-02-17  Ojan Vafai  <ojan@chromium.org>
769
770         Reviewed by Adam Barth.
771
772         create comments when hitting enter with a focused diff
773         https://bugs.webkit.org/show_bug.cgi?id=54719
774
775         * code-review.js:
776
777 2011-02-17  Ojan Vafai  <ojan@chromium.org>
778
779         Reviewed by Adam Barth.
780
781         use actual browser focus in the code review tool
782         https://bugs.webkit.org/show_bug.cgi?id=54726
783
784         This makes keyboard handling play better with focusable element
785         (i.e. links/textareas/buttons). Also, in theory, this makes
786         the review tool more amenable to screen readers (I think).
787
788         * PrettyPatch/PrettyPatch.rb:
789         * code-review.js:
790
791 2011-02-17  Ojan Vafai  <ojan@chromium.org>
792
793         Reviewed by Antonio Gomes.
794
795         make event handling work in Gecko
796         https://bugs.webkit.org/show_bug.cgi?id=54702
797
798         Gecko doesn't support the global event property. It's an IE-ism.
799
800         * code-review.js:
801
802 2011-02-14  Ojan Vafai  <ojan@chromium.org>
803
804         Reviewed by Adam Barth.
805
806         allow for modifying comments without the mouse
807         https://bugs.webkit.org/show_bug.cgi?id=54433
808
809         Use n/p to navigate the comments.
810         Enter to begin editing a comment.
811         Escape to finish editing a comment.
812
813         * code-review.js:
814
815 2011-02-14  Ojan Vafai  <ojan@chromium.org>
816
817         Reviewed by Adam Barth.
818
819         improve line selection in the code review tool
820         https://bugs.webkit.org/show_bug.cgi?id=54430
821
822         -shift+click now extends the comment context
823         -selecting outside of a diff section no longer clears the selected lines.
824          Instead it just restricts the selected lines to that diff section.
825
826         * code-review.js:
827
828 2011-02-10  Ojan Vafai  <ojan@chromium.org>
829
830         Reviewed by Adam Barth.
831
832         remove extra space from expansion lines
833         https://bugs.webkit.org/show_bug.cgi?id=54266
834
835         * code-review.js:
836
837 2011-02-09  Ojan Vafai  <ojan@chromium.org>
838
839         Reviewed by Adam Barth.
840
841         only erase draft comments after publish is successful
842         https://bugs.webkit.org/show_bug.cgi?id=54163
843
844         If there is a conflict or 500, then draft comments will survive.
845         One drawback here is that the form post is now to the iframe,
846         so to break out of the iframe we redirect to the bug page, which
847         loses the information of who the email was sent to.
848
849         Once WebKit supports seamless iframes we should be able to avoid
850         the redirect.
851
852         * code-review.js:
853
854 2011-02-09  Ojan Vafai  <ojan@chromium.org>
855
856         Reviewed by Adam Barth.
857
858         save overall comments when saving drafts in the review tool
859         https://bugs.webkit.org/show_bug.cgi?id=54165
860
861         -save overall comments in localstorage as well
862         -save all draft comments as you type
863         -give a *subtle* indicator of saved state
864
865         The latter should also make it super easy if someone wanted to do
866         the work to store draft comments in appengine/s3/bugzilla/etc.
867
868         * PrettyPatch/PrettyPatch.rb:
869         * code-review-test.html:
870         * code-review.js:
871
872 2011-02-08  Ojan Vafai  <ojan@chromium.org>
873
874         Reviewed by Adam Barth.
875
876         fix toolbar anchoring in the code review tool
877         https://bugs.webkit.org/show_bug.cgi?id=54058
878
879         Avoid the anchoring cycle of doom when on the cusp
880         of whether the toolbar needs to be anchored and
881         speculatively avoid the Firefox crash when resizing.
882
883         * PrettyPatch/PrettyPatch.rb:
884         * code-review.js:
885
886 2011-02-07  Ojan Vafai  <ojan@chromium.org>
887
888         Reviewed by Adam Barth.
889
890         bring back diff context lines into the review tool
891         https://bugs.webkit.org/show_bug.cgi?id=53974
892
893         * code-review.js:
894
895 2011-02-01  Ojan Vafai  <ojan@chromium.org>
896
897         Reviewed by Adam Roben.
898
899         include svn revisions in git diffs for the code review tool to use
900         https://bugs.webkit.org/show_bug.cgi?id=53569
901
902         * PrettyPatch/PrettyPatch.rb:
903
904 2011-02-01  Ojan Vafai  <ojan@chromium.org>
905
906         Reviewed by Adam Barth.
907
908         make draft comments focusable
909         https://bugs.webkit.org/show_bug.cgi?id=53554
910
911         Makes frozen draft comments focusable. The ones that are currently being edited are not.
912         I'm on the fence whether they should be, but this seems good enough for now.
913
914         * code-review.js:
915
916 2011-02-01  Ojan Vafai  <ojan@chromium.org>
917
918         Reviewed by Adam Barth.
919
920         avoid jitter when loading the comments to a patch
921         https://bugs.webkit.org/show_bug.cgi?id=53570
922
923         * code-review.js:
924
925 2011-02-01  Ojan Vafai  <ojan@chromium.org>
926
927         Reviewed by Mihai Parparita.
928
929         [codereviewtool] extra space at the beginning of lines in side-by-side view
930         https://bugs.webkit.org/show_bug.cgi?id=53550
931
932         The newline between the spans turns into user-visible space in side-by-side mode.
933
934         * PrettyPatch/PrettyPatch.rb:
935
936 2011-02-01  Ojan Vafai  <ojan@chromium.org>
937
938         Increment version number to cache-bust code-review.js.
939
940         * PrettyPatch/PrettyPatch.rb:
941
942 2011-01-31  Ojan Vafai  <ojan@chromium.org>
943
944         Reviewed by Adam Barth.
945
946         Store draft comments in localStorage
947         https://bugs.webkit.org/show_bug.cgi?id=52866
948
949         * code-review.js:
950         * code-review-test.html
951
952 2011-01-20  Ojan Vafai  <ojan@chromium.org>
953
954         Fix the review tool for image diffs. We would get a javascript error
955         because image diffs don't have line numbers.
956
957         * code-review.js:
958
959 2011-01-20  Ojan Vafai  <ojan@chromium.org>
960
961         Reviewed by Oliver Hunt.
962
963         Could review tool include style failure info?
964         https://bugs.webkit.org/show_bug.cgi?id=49049
965
966         This does not handle patches that have old paths (e.g. without the Source prefix).
967         Although that would be fairly straightforward to workaround.
968
969         * code-review.js:
970
971 2011-01-20  Ojan Vafai  <ojan@chromium.org>
972
973         Reviewed by Adam Barth.
974
975         handle cases where PrettyPatch.rb doesn't linkify filenames
976         https://bugs.webkit.org/show_bug.cgi?id=52834
977
978         * code-review.js:
979
980 2011-01-20  Ojan Vafai  <ojan@chromium.org>
981
982         Reviewed by Adam Barth.
983
984         fix goofup in http://trac.webkit.org/changeset/76082
985         https://bugs.webkit.org/show_bug.cgi?id=52830
986
987         We were showing the revision number of the patch at the bottom
988         of the diff and expanding from the bottom of the diff would fail.
989
990         Also, removed an unused variable and scoped queries appropriatly
991         to the file_diff they should have been operating on.
992
993         * PrettyPatch/PrettyPatch.rb:
994         * code-review.js:
995
996 2011-01-19  Ojan Vafai  <ojan@chromium.org>
997
998         Reviewed by Adam Barth.
999
1000         [reviewtool] Add a link for annotated trac page on review page
1001         https://bugs.webkit.org/show_bug.cgi?id=52747
1002
1003         -add line numbers to all trac links to the first line in the file diff
1004         -add annotate/review log links
1005         -make file-diff links other than the file name only show when you mouseover the filediff
1006         -fix white-space to only wrap the .text element and it's childen, any of it's siblings/parents
1007
1008         * PrettyPatch/PrettyPatch.rb:
1009         * code-review.js:
1010
1011 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1012
1013         Reviewed by Adam Barth.
1014
1015         make line selection have a extend only from where you start the selection in the code review tool
1016         https://bugs.webkit.org/show_bug.cgi?id=52485
1017
1018         * code-review.js:
1019
1020 2011-01-18  Ojan Vafai  <ojan@chromium.org>
1021
1022         Reviewed by Eric Seidel.
1023
1024         make file expansion use the file at the svn revision the diff was created at
1025         https://bugs.webkit.org/show_bug.cgi?id=52650
1026
1027         This only works for patches created with SVN for now since the git
1028         diff does not include SVN revision numbers in it.
1029
1030         * PrettyPatch/PrettyPatch.rb:
1031         * code-review.js:
1032
1033 2011-01-15  Adam Barth  <abarth@webkit.org>
1034
1035         Rubber-stamped by Eric Seidel.
1036
1037         Move WebKit2 into Source
1038         https://bugs.webkit.org/show_bug.cgi?id=52438
1039
1040         Remove WebKit2 from the list of top-level source directories.
1041
1042         * PrettyPatch/PrettyPatch.rb:
1043
1044 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1045
1046         Reviewed by Adam Barth.
1047
1048         fix expanded lines in the code review tool
1049         https://bugs.webkit.org/show_bug.cgi?id=52498
1050
1051         Fixes them to work now that we have wrapper divs.
1052         Also fixes the long-standing bug that the line numbers
1053         were incorrect sometimes.
1054
1055         * code-review.js:
1056
1057 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1058
1059         Reviewed by Adam Barth.
1060
1061         put remove lines to the left of add lines in sidebyside view
1062         https://bugs.webkit.org/show_bug.cgi?id=52458
1063
1064         Also, sneaking in a fix to make long lines with no whitespace wrap.
1065
1066         * PrettyPatch/PrettyPatch.rb:
1067         * code-review.js:
1068
1069 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1070
1071         Fix bad merge in r75816.
1072
1073         * code-review.js:
1074
1075 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1076
1077         Reviewed by Adam Barth.
1078
1079         improve line selection with fast drags
1080         https://bugs.webkit.org/show_bug.cgi?id=52477
1081
1082         * code-review.js:
1083
1084 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1085
1086         Reviewed by Adam Barth.
1087
1088         tweak line selection in code review tool
1089         https://bugs.webkit.org/show_bug.cgi?id=52462
1090
1091         -Improve handling of data-comment-base-line code to deal with
1092         lines that have multiple values.
1093         -Make it so that if you click on the line immediately above
1094         a line that has a comment it will add a new comment instead of
1095         adding lines to the following comment. If the last selected line
1096         overlaps existing comment lines though they will still get
1097         added to the existing comment.
1098
1099         * code-review.js:
1100
1101 2011-01-14  Ojan Vafai  <ojan@chromium.org>
1102
1103         Reviewed by Adam Roben.
1104
1105         fix bugs going back and forth between unified and sidebyside
1106         https://bugs.webkit.org/show_bug.cgi?id=52470
1107
1108         Now that we're appending DOM nodes directly, we only want to
1109         get the first ".text" element in the subtree. In the cases
1110         where there are multiple, their contents are the same.
1111
1112         Also, when making side-by-side expansion lines, we need to
1113         clone the contents so we have a second copy to append to the DOM.
1114
1115         * code-review.js:
1116
1117 2011-01-13  Ojan Vafai  <ojan@chromium.org>
1118
1119         Reviewed by Adam Barth.
1120
1121         simplify keyboard handling in code review tool
1122         https://bugs.webkit.org/show_bug.cgi?id=52407
1123
1124         Now that we have DiffBlock containers, the only things that are
1125         focusable are previousComment nodes and DiffBlock containers
1126         that contain add/remove lines.
1127
1128         Also, this means we show the focus border around the entire diff
1129         instead of just the first line.
1130
1131         * code-review.js:
1132
1133 2011-01-13  Ojan Vafai  <ojan@chromium.org>
1134
1135         Reviewed by Adam Barth.
1136
1137         add container divs for diff blocks
1138         https://bugs.webkit.org/show_bug.cgi?id=52400
1139
1140         This will help simplify a lot of code in code-review.js
1141         and make side-by-side diffs better (i.e. put removed lines
1142         to the left of corresponding added lines).
1143
1144         Also, allow for running the JS from a local file. Now you can modify code-review.js
1145         to point to a local file and then run:
1146         ruby prettify.rb < foo.diff > foo.html
1147
1148         foo.html will load a dummy code review matching foo.diff.
1149
1150         Before structure:
1151         Line
1152         Line remove
1153         Line add
1154         Line add
1155         Line
1156
1157         After structure:
1158         DiffBlock
1159           DiffBlockPart shared
1160             Line shared
1161         DiffBlock
1162           DiffBlockPart remove
1163             Line remove
1164           DiffBlockPart add
1165             Line add
1166             Line add
1167         DiffBlock
1168           DiffBlockPart shared
1169             Line shared
1170
1171         * PrettyPatch/PrettyPatch.rb:
1172         * code-review.js:
1173
1174 2011-01-12  Ojan Vafai  <ojan@chromium.org>
1175
1176         Reviewed by Mihai Parparita.
1177
1178         fix assorted bugs with expansion lines
1179         https://bugs.webkit.org/show_bug.cgi?id=52326
1180
1181         -Clicking "all" would collapse whitespace in the expanded lines.
1182         -Converted the remaining line creation code to using the DOM.
1183
1184         * PrettyPatch/PrettyPatch.rb:
1185         * code-review.js:
1186
1187 2011-01-12  Ojan Vafai  <ojan@chromium.org>
1188
1189         Reviewed by Adam Barth.
1190
1191         review tool formatted diff doesn't match the uploaded diff
1192         https://bugs.webkit.org/show_bug.cgi?id=51960
1193
1194         PrettyPatch works just fine if there is no newline at the end of the file.
1195
1196         * attachment.cgi:
1197
1198 2011-01-12  Ojan Vafai  <ojan@chromium.org>
1199
1200         Reviewed by Adam Barth.
1201
1202         Bugzilla: Add keyboard shortcuts to jump to next change
1203         https://bugs.webkit.org/show_bug.cgi?id=52305
1204
1205         Comments and diff blocks go in the same queue. If you have a 
1206         comment focused, then j/k will focus the next/prev diff block
1207         with respect to that comment.
1208
1209         * PrettyPatch/PrettyPatch.rb:
1210         * code-review.js:
1211
1212 2011-01-12  Ojan Vafai  <ojan@chromium.org>
1213
1214         Reviewed by Adam Barth.
1215
1216         show shared lines on both sides in code review tool
1217         https://bugs.webkit.org/show_bug.cgi?id=52308
1218
1219         We were appending the same DOM node twice. Instead, make
1220         the from column a clone of the original node.
1221
1222         * code-review.js:
1223
1224 2011-01-12  Ojan Vafai  <ojan@chromium.org>
1225
1226         Reviewed by Adam Barth.
1227
1228         fix wrapping bug in expansion lines in the code review tool
1229         https://bugs.webkit.org/show_bug.cgi?id=52270
1230
1231         * PrettyPatch/PrettyPatch.rb:
1232         * code-review.js:
1233
1234 2011-01-11  Ojan Vafai  <ojan@chromium.org>
1235
1236         Reviewed by Adam Barth.
1237
1238         fix difflink centering to be vertical, not horizontal
1239         https://bugs.webkit.org/show_bug.cgi?id=52263
1240
1241         * PrettyPatch/PrettyPatch.rb:
1242         * code-review.js:
1243
1244 2011-01-11  Ojan Vafai  <ojan@chromium.org>
1245
1246         Reviewed by Adam Barth.
1247
1248         remember diffstate for review tool
1249         https://bugs.webkit.org/show_bug.cgi?id=52253
1250
1251         If you use the global sidebyside/unified links, store the diff type
1252         in localstorage. Then, onload, use that diff type. That way, people
1253         who prefer one diff type or the other can always get that by default.
1254
1255         * code-review.js:
1256
1257 2011-01-11  Ojan Vafai  <ojan@chromium.org>
1258
1259         Reviewed by Adam Barth.
1260
1261         allow sidebysideifying individual files in the code review tool
1262         https://bugs.webkit.org/show_bug.cgi?id=52226
1263
1264         * code-review.js:
1265
1266 2011-01-11  Ojan Vafai  <ojan@chromium.org>
1267
1268         Reviewed by Mihai Parparita.
1269
1270         fix exception when adding a comment to a side-by-side diff
1271         https://bugs.webkit.org/show_bug.cgi?id=52240
1272
1273         prev() and next() won't get the previous and next lines in
1274         side-by-side mode. Instead do a query to find them.
1275
1276         * code-review.js:
1277
1278 2011-01-11  Ojan Vafai  <ojan@chromium.org>
1279
1280         Reviewed by Mihai Parparita.
1281
1282         maintain word diffs when converting to side-by-side and back
1283         https://bugs.webkit.org/show_bug.cgi?id=52244
1284
1285         Using textContent would lose the spans use to make the word-diff highlighting.
1286         Just move the span element itself.
1287
1288         * code-review.js:
1289
1290 2011-01-11  Ojan Vafai  <ojan@chromium.org>
1291
1292         Reviewed by Adam Barth.
1293
1294         fix layout error with comments in code review tool
1295         https://bugs.webkit.org/show_bug.cgi?id=52230
1296
1297         * PrettyPatch/PrettyPatch.rb:
1298
1299 2011-01-10  Ojan Vafai  <ojan@chromium.org>
1300
1301         Reviewed by Adam Barth.
1302
1303         convert back to unified from sidebyside diff
1304         https://bugs.webkit.org/show_bug.cgi?id=52180
1305
1306         Remove url fragment stuff. Having it be per-filediff is too complicated.
1307
1308         * PrettyPatch/PrettyPatch.rb:
1309         * code-review.js:
1310
1311 2011-01-07  Adam Barth  <abarth@webkit.org>
1312
1313         Rubber-stamped by Eric Seidel.
1314
1315         Move WebCore to Source
1316         https://bugs.webkit.org/show_bug.cgi?id=52050
1317
1318         Remove reference to old directory.
1319
1320         * PrettyPatch/PrettyPatch.rb:
1321
1322 2011-01-06  Ojan Vafai  <ojan@chromium.org>
1323
1324         Reviewed by Adam Barth.
1325
1326         side-by-side diffs in the code review tool
1327         https://bugs.webkit.org/show_bug.cgi?id=52019
1328
1329         Support for conversion from the formatted diff to a side-by-side diff.
1330         Maintains comments and new comments can be added.
1331
1332         The main architectural change is that Line elements are no longer necessarily
1333         siblings. Each physical line is now in a LineContainer and LineContainers are
1334         siblings. Each Line corresponds to a Line in the unified diff and has an id (e.g. line12).
1335         A Line can be a LineContainer or a child of a LineContainer.
1336
1337         In this way, converting to side-by-side and, in the future, back to unified is non-lossy.
1338
1339         * PrettyPatch/PrettyPatch.rb:
1340         * code-review.js:
1341
1342 2011-01-06  Ojan Vafai  <ojan@chromium.org>
1343
1344         Fix line context when replying to comments.
1345
1346         * code-review.js:
1347
1348 2011-01-05  Ojan Vafai  <ojan@chromium.org>
1349
1350         Reviewed by Adam Barth.
1351
1352         change the way we do comment highlighting in the code review tool
1353         https://bugs.webkit.org/show_bug.cgi?id=51971
1354
1355         Store a space-separated list of base line IDs on each line that has
1356         comments associated with that line. This allows for overlapping comments,
1357         but more importantly, makes adding side-by-side diff support easier.
1358
1359         * code-review.js:
1360
1361 2011-01-05  Ojan Vafai  <ojan@chromium.org>
1362
1363         Reviewed by Adam Barth.
1364
1365         minor code cleanup for code review tool
1366         https://bugs.webkit.org/show_bug.cgi?id=51962
1367
1368         Consolidates some queries to using shared functions.
1369
1370         * code-review.js:
1371
1372 2011-01-05  Ojan Vafai  <ojan@chromium.org>
1373
1374         Reviewed by Adam Barth.
1375
1376         assorted cleanups to prepare for side-by-side diffing
1377         https://bugs.webkit.org/show_bug.cgi?id=51961
1378
1379         Mostly, use selector queries instead of assuming comments are next siblings.
1380
1381         * code-review.js:
1382
1383 2011-01-05  Ojan Vafai  <ojan@chromium.org>
1384
1385         Reviewed by Adam Barth.
1386
1387         fix message container selector for code review tool
1388         https://bugs.webkit.org/show_bug.cgi?id=51959
1389
1390         * code-review.js:
1391
1392 2011-01-04  Ojan Vafai  <ojan@chromium.org>
1393
1394         Increment version number to avoid getting the cached JS file.
1395
1396         * PrettyPatch/PrettyPatch.rb:
1397
1398 2010-12-15  Ojan Vafai  <ojan@chromium.org>
1399  
1400         Reviewed by Adam Barth.
1401
1402         anchor the toolbar to the bottom of the diff if the diff doesn't take a full screen of height
1403         https://bugs.webkit.org/show_bug.cgi?id=51162
1404
1405         * PrettyPatch/PrettyPatch.rb:
1406         * code-review.js:
1407
1408 2011-01-01  Adam Barth  <abarth@webkit.org>
1409
1410         Reviewed by Eric Seidel.
1411
1412         Move JavaScriptCore to Source
1413         https://bugs.webkit.org/show_bug.cgi?id=51604
1414
1415         * PrettyPatch/PrettyPatch.rb:
1416           - Remove reference to JavaScriptCore as a source directory.
1417
1418 2011-01-01  Adam Barth  <abarth@webkit.org>
1419
1420         Reviewed by Eric Seidel.
1421
1422         Move Sources to Source
1423         https://bugs.webkit.org/show_bug.cgi?id=51794
1424
1425         * PrettyPatch/PrettyPatch.rb:
1426
1427 2010-12-31  Adam Barth  <abarth@webkit.org>
1428
1429         Rubber-stamped by Eric Seidel.
1430
1431         Move PageLoadTests to PerformanceTests/PageLoad
1432         https://bugs.webkit.org/show_bug.cgi?id=51771
1433
1434         Update list of top-level open-source directories.
1435
1436         * PrettyPatch/PrettyPatch.rb:
1437
1438 2010-12-31  Adam Barth  <abarth@webkit.org>
1439
1440         Rubber-stamped by Eric Seidel.
1441
1442         Move SunSpider into PerformanceTests
1443         https://bugs.webkit.org/show_bug.cgi?id=51769
1444
1445         Update list of top-level open-source directories.
1446
1447         * PrettyPatch/PrettyPatch.rb:
1448
1449 2010-12-26  Adam Barth  <abarth@webkit.org>
1450
1451         Reviewed by Eric Seidel.
1452
1453         Move autotools into Sources
1454         https://bugs.webkit.org/show_bug.cgi?id=51630
1455
1456         * PrettyPatch/PrettyPatch.rb:
1457           - Remove reference to non-existant autotools directory.
1458
1459 2010-12-26  Adam Barth  <abarth@webkit.org>
1460
1461         Reviewed by Eric Seidel.
1462
1463         Move cmake into Sources
1464         https://bugs.webkit.org/show_bug.cgi?id=51631
1465
1466         * PrettyPatch/PrettyPatch.rb:
1467
1468 2010-12-23  Adam Barth  <abarth@webkit.org>
1469
1470         Rubber-stamped by Eric Seidel.
1471
1472         Move JavaScriptGlue into Sources
1473         https://bugs.webkit.org/show_bug.cgi?id=51583
1474
1475         PrettyPatch keeps a list of the top-level directories that exist in
1476         trac.
1477
1478         * PrettyPatch/PrettyPatch.rb:
1479
1480 2010-12-20  Adam Barth  <abarth@webkit.org>
1481
1482         One more tweak caused by moving the web sites around.  I had this
1483         change in my patch originally, but I missed it because I had to use
1484         server-side svn move commands to move the bulk of the web site files.
1485
1486         * PrettyPatch/PrettyPatch.rb:
1487
1488 2010-12-18  Adam Barth  <abarth@webkit.org>
1489
1490         Reviewed by Sam Weinig.
1491
1492         Move WebKitExamplePlugins to Examples
1493         https://bugs.webkit.org/show_bug.cgi?id=51291
1494
1495         * PrettyPatch/PrettyPatch.rb:
1496
1497 2010-12-17  Dan Bernstein  <mitz@apple.com>
1498
1499         Reviewed by Simon Fraser.
1500
1501         Rename WebKitTools to Tools
1502         https://bugs.webkit.org/show_bug.cgi?id=49861
1503
1504         * PrettyPatch/PrettyPatch.rb:
1505         * committers-autocomplete.js:
1506
1507 2010-12-17  Ojan Vafai  <ojan@chromium.org>
1508
1509         Increment psuedo-version number to avoid pulling cached version.
1510
1511         * PrettyPatch/PrettyPatch.rb:
1512
1513 2010-12-17  Ojan Vafai  <ojan@chromium.org>
1514
1515         Speculative fix for the code review tool in Firefox.
1516         Fixes a JS error and makes the status bubble mostly work
1517         if postMessage is not supported.
1518
1519         * PrettyPatch/PrettyPatch.rb:
1520         * code-review.js:
1521
1522 2010-12-15  Ojan Vafai  <ojan@chromium.org>
1523
1524         Reviewed by Adam Barth.
1525
1526         size status bubble to it's contents on the code review page
1527         https://bugs.webkit.org/show_bug.cgi?id=51142
1528
1529         * PrettyPatch/PrettyPatch.rb:
1530         * code-review.js:
1531
1532 2010-12-15  Ojan Vafai  <ojan@chromium.org>
1533
1534         Reviewed by Adam Barth.
1535
1536         fix goofups from r74142 and r74130
1537         https://bugs.webkit.org/show_bug.cgi?id=51146
1538
1539         Accidentally shrank the inline comment boxes and made
1540         the toolbar not show up when there were image diffs.
1541
1542         * PrettyPatch/PrettyPatch.rb:
1543         * code-review.js:
1544
1545 2010-12-14  Ojan Vafai  <ojan@chromium.org>
1546
1547         Reviewed by Adam Barth.
1548
1549         add ability to view for file context to the review tool
1550         https://bugs.webkit.org/show_bug.cgi?id=51057
1551
1552         At the beginning/end of each file diff and between each
1553         hunk add links to expand the context. For now it grabs the
1554         tip of tree version of the file and tries to apply the diff 
1555         to that file. If it can't apply, then it gives up as we
1556         wouldn't want to show the wrong lines of context.
1557
1558         In the future, we can consider adding the upload svn revision
1559         to the diff itself, then we could fallback to the file at that
1560         revision if tip of tree doesn't apply.
1561
1562         * PrettyPatch/PrettyPatch.rb:
1563         * code-review.js:
1564
1565 2010-12-08  Ojan Vafai  <ojan@chromium.org>
1566
1567         Reviewed by Adam Barth.
1568
1569         [reviewtool] should always show overall comments text box
1570         https://bugs.webkit.org/show_bug.cgi?id=45870
1571
1572         * PrettyPatch/PrettyPatch.rb:
1573         * code-review.js:
1574
1575 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
1576
1577         Reviewed by Adam Barth.
1578
1579         Increase size of status bubbles
1580         https://bugs.webkit.org/show_bug.cgi?id=50496
1581
1582         The current available space isn't enough to display all
1583         pending results including position in the queue.
1584
1585         * PrettyPatch/PrettyPatch.rb:
1586
1587 2010-12-04  Patrick Gansterer  <paroga@webkit.org>
1588
1589         Reviewed by Adam Barth.
1590
1591         Increase size of status bubbles
1592         https://bugs.webkit.org/show_bug.cgi?id=50496
1593
1594         The current available space isn't enough to display all
1595         pending results including position in the queue.
1596
1597         * template/en/custom/attachment/edit.html.tmpl:
1598         * template/en/custom/attachment/list.html.tmpl:
1599         * template/en/custom/attachment/reviewform.html.tmpl:
1600
1601 2010-11-08  Adam Barth  <abarth@webkit.org>
1602
1603         Update help text to match behavior change.
1604
1605         * code-review.js:
1606
1607 2010-11-08  Adam Barth  <abarth@webkit.org>
1608
1609         Reviewed by Eric Seidel.
1610
1611         Restore normal meaning of double-click on patch review page?
1612         https://bugs.webkit.org/show_bug.cgi?id=47641
1613
1614         After this patch, you need to click on the line numbers to add a
1615         comment.
1616
1617         * PrettyPatch/PrettyPatch.rb:
1618         * code-review.js:
1619
1620 2010-10-09  Adam Barth  <abarth@webkit.org>
1621
1622         Reviewed by Alexey Proskuryakov.
1623
1624         Exception in committers-autocomplete in Firefox
1625         https://bugs.webkit.org/show_bug.cgi?id=47456
1626
1627         Remove dependency on a WebKit-only quirk.  IMHO, this quirk is bad for
1628         the long-term health of the web, but that's a discussion for another
1629         patch.
1630
1631         * committers-autocomplete.js:
1632
1633 2010-10-08  Adam Barth  <abarth@webkit.org>
1634
1635         Reviewed by Oliver Hunt.
1636
1637         Bugmail for new attachments should link to the reviewtool
1638         https://bugs.webkit.org/show_bug.cgi?id=47440
1639
1640         As requested by olliej himself.  This patch was constructed by pure
1641         reason.  I have no way to actually run this code.
1642
1643         * Bugzilla/BugMail.pm:
1644
1645 2010-09-28  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
1646
1647         Reviewed by Csaba Osztrogonác.
1648
1649         Convert verbatim unicode values in comitter names to characters
1650
1651         * committers-autocomplete.js:
1652
1653 2010-09-24  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
1654
1655         Reviewed by Csaba Osztrogonác.
1656
1657         Support Unicode-strings in committers-autocomplete script
1658
1659         * committers-autocomplete.js:
1660
1661 2010-09-22  Adam Barth  <abarth@webkit.org>
1662
1663         Reviewed by John Sullivan.
1664
1665         New review-page design doesn't include name of reviewer
1666         https://bugs.webkit.org/show_bug.cgi?id=46271
1667
1668         * PrettyPatch/PrettyPatch.rb:
1669         * code-review.js:
1670
1671 2010-09-22  Adam Barth  <abarth@webkit.org>
1672
1673         Fix typo in CSS.
1674
1675         * PrettyPatch/PrettyPatch.rb:
1676
1677 2010-09-22  Adam Barth  <abarth@webkit.org>
1678
1679         Reviewed by Eric Seidel.
1680
1681         Add link to bug to review page
1682         https://bugs.webkit.org/show_bug.cgi?id=46192
1683
1684         To make room for the link, I moved the help text to the top of the page
1685         and tweaked the language to help folks discover that you can select
1686         context using the line numbers.
1687
1688         Also, move more text to sans-serif since that fits in better with the
1689         pretty-patch fonts.
1690
1691         * PrettyPatch/PrettyPatch.rb:
1692         * code-review.js:
1693
1694 2010-09-21  Adam Barth  <abarth@webkit.org>
1695
1696         Unreviewed.
1697
1698         [reviewtool] Publish button doesn't work
1699         https://bugs.webkit.org/show_bug.cgi?id=46168
1700
1701         Darin's recent patch had a runtime error.  Sadly, we don't have any
1702         testing for this code.
1703
1704         * code-review.js:
1705
1706 2010-09-20  Darin Adler  <darin@apple.com>
1707
1708         Reviewed by Adam Barth.
1709
1710         Add Preview button and link to bug to patch review bar
1711         https://bugs.webkit.org/show_bug.cgi?id=46153
1712
1713         * PrettyPatch/PrettyPatch.rb: Bump version number to make caches happy.
1714         * code-review.js: Added Preview button and link to bug.
1715
1716 2010-09-19  Adam Barth  <abarth@webkit.org>
1717
1718         Reviewed by Eric Seidel.
1719
1720         [reviewtool] Publish button should actually publish the review comments
1721         https://bugs.webkit.org/show_bug.cgi?id=46074
1722
1723         Instead of showing the confirmation lightbox, the Publish button now
1724         publishes the comments directly.  If there's demand for a "Preview"
1725         button, we can find a way to add that feature.
1726
1727         * code-review.js:
1728
1729 2010-09-19  Adam Barth  <abarth@webkit.org>
1730
1731         Reviewed by Eric Seidel.
1732
1733         Move reviewtool to action=review
1734         https://bugs.webkit.org/show_bug.cgi?id=46071
1735
1736         This patch removes the old Review Patch screen and moves the new
1737         reviewtool from Pretty Diff to Review Patch.
1738
1739         * attachment.cgi:
1740         * code-review.js:
1741
1742 2010-09-19  Adam Barth  <abarth@webkit.org>
1743
1744         Review tool UI tweak.  This patch lengthens the status bubbles so all
1745         the bubble fit (even when they have numbers inside).
1746
1747         * PrettyPatch/PrettyPatch.rb:
1748
1749 2010-09-19  Dan Bernstein  <mitz@apple.com>
1750
1751         Reviewed by Anders Carlsson.
1752
1753         Fixed a typo.
1754
1755         * code-review.js: Changed the label of the OK button from Ok to OK.
1756
1757 2010-09-15  Adam Barth  <abarth@webkit.org>
1758
1759         Unreviewed.
1760
1761         Only show the review-in-context link if there are contextual comments.
1762
1763         * code-review.js:
1764
1765 2010-09-15  Adam Barth  <abarth@webkit.org>
1766
1767         Reviewed by Eric Seidel.
1768
1769         [reviewtool] Show the status bubbles on the toolbar
1770         https://bugs.webkit.org/show_bug.cgi?id=45861
1771
1772         * PrettyPatch/PrettyPatch.rb:
1773         * code-review.js:
1774
1775 2010-09-15  Adam Barth  <abarth@webkit.org>
1776
1777         Reviewed by Eric Seidel.
1778
1779         [reviewtool] Allow setting review and commit-queue flags from main review screen
1780         https://bugs.webkit.org/show_bug.cgi?id=45860
1781
1782         This patch adds drop down menus for adjusting the review and
1783         commit-queue flags from the main review screen.  These controls don't
1784         appear immediately because we need to read the state of the flags off
1785         the details page.
1786
1787         It's a slight sadness that bugzilla doesn't have a clean way of
1788         identifying which select control is associated with a given flag.  The
1789         flags seem to have some sort of id, but it's unclear to me how to map
1790         that id back to a notion of "review" or "commit-queue".  Instead, we
1791         look for some magic strings in the title.  I'm sure this will come back
1792         to bit us at some point, but I'm not sure what to do that's better.
1793         (The webkitpy solution to this problem is to assume that the flags are
1794         in a given order, but this seems better.)
1795
1796         * PrettyPatch/PrettyPatch.rb:
1797             - Bump version number to bust throuh caches.
1798         * code-review.js:
1799
1800 2010-09-15  Adam Barth  <abarth@webkit.org>
1801
1802         Reviewed by Eric Seidel.
1803
1804         [reviewtool] Comments should quote previous comments on the same line
1805         https://bugs.webkit.org/show_bug.cgi?id=45847
1806
1807         Now when you reply to a previous comment, the tool will quote the
1808         previous comment in the bugs.webkit.org post.  This makes it eaiser for
1809         folks following along in email to understand the discussion.
1810
1811         While I was editing this code, I also cleaned up some of the whitespace
1812         handling in comments.
1813
1814         * code-review.js:
1815
1816 2010-09-15  Adam Barth  <abarth@webkit.org>
1817
1818         Minor UI tweak to the review tool.  We want to display newlines in
1819         previous and frozen comments instead of collapsing them.
1820
1821         * PrettyPatch/PrettyPatch.rb:
1822
1823 2010-09-14  Adam Barth  <abarth@webkit.org>
1824
1825         Reviewed by Eric Seidel.
1826
1827         bugs.webkit.org should autocomplete WebKit Committers and Reviewers
1828         https://bugs.webkit.org/show_bug.cgi?id=45799
1829
1830         This patch implements an autocomplete dropdown that helps users type
1831         the names of WebKit committers and reviewers.  The script grabs the
1832         data from SVN now that wms set up CORS for http://svn.webkit.org.
1833         Security technology for the win.
1834
1835         This script is based on (well, essentially copied from) the popular
1836         autocomplete extension:
1837
1838         https://chrome.google.com/extensions/detail/olaabhcgdogcbcoiolomlcodkngnemfb
1839
1840         I asked Ojan, and said he was fine licensing the code under the BSD
1841         license.  I'm not sure how the patch interacts with the extension, but
1842         we can retire the extension if they interact poorly.
1843
1844         * committers-autocomplete.js: Added.
1845         * template/en/default/global/footer.html.tmpl:
1846             - Add the script to every page on bugs.webkit.org
1847
1848 2010-09-14  Adam Barth  <abarth@webkit.org>
1849
1850         Add a license block to the reviewtool.  I should have added this from
1851         the start.  According to the SVN log, I'm the only one to have ever
1852         touched this file.
1853
1854         * code-review.js:
1855
1856 2010-09-07  Adam Barth  <abarth@webkit.org>
1857
1858         Minor tweaks to the reviewtool UI.
1859
1860         Rename Cancel to Discard to make it more clear what the button does.
1861         Also, move the style declaration for the focus ring later in the sheet
1862         so it works.
1863
1864         * PrettyPatch/PrettyPatch.rb:
1865         * code-review.js:
1866
1867 2010-09-06  Adam Barth  <abarth@webkit.org>
1868
1869         Small UI tweak to the reviewtool.  Make the Ok/Cancel buttons a bit
1870         bigger.
1871
1872         * PrettyPatch/PrettyPatch.rb:
1873
1874 2010-09-06  Adam Barth  <abarth@webkit.org>
1875
1876         Reviewed by Eric Seidel.
1877
1878         [reviewtool] Add a quick animation for opening/closing comment boxes
1879         https://bugs.webkit.org/show_bug.cgi?id=45278
1880
1881         The 200ms animation slides in and out the line-by-line comment box.
1882
1883         * code-review.js:
1884
1885 2010-09-06  Adam Barth  <abarth@webkit.org>
1886
1887         Reviewed by Eric Seidel.
1888
1889         [reviewtool] Add a field for overall comments
1890         https://bugs.webkit.org/show_bug.cgi?id=45273
1891
1892         This patch does a couple logically separate things that could be
1893         separated into smaller patches:
1894
1895         1) This patch adds an "overall comments" field where you can enter
1896            overall comments about the patch.  These comments appear at the top
1897            of the bugzilla posting.  Currently, these aren't redisplayed when
1898            viewing the patch, but I plan to add that in a future patch.
1899
1900         2) This patch renames some of the CSS classes to more consistently
1901            follow the camelCase style that PrettyPatch uses.
1902
1903         3) This patch moves the "prepare comments" button to the left of the
1904            toolbar and renames is to "publish comments".  This makes more sense
1905            when you scroll to the bottom of the page and enter in some overall
1906            comments.
1907
1908         4) When you attempt to add a comment to a line that already has a
1909            "frozen" comment, we now unfreeze the comment instead of doing
1910            nothing.  The old behavior was kind of frustrating if you didn't
1911            know that you could unfreeze a comment by clicking on it.
1912
1913         * PrettyPatch/PrettyPatch.rb:
1914             - Update CSS.
1915         * code-review.js:
1916
1917 2010-09-06  Adam Barth  <abarth@webkit.org>
1918
1919         [reviewtool] Tweak the ok button to cancel the comment if the comment
1920         is empty.  Previously we would get into a bad state where a line had a
1921         comment but there was no longer any way to access it.
1922
1923         * code-review.js:
1924
1925 2010-09-06  Adam Barth  <abarth@webkit.org>
1926
1927         Reviewed by Eric Seidel.
1928
1929         [reviewtool] Add an "ok" button that collapses review comments
1930         https://bugs.webkit.org/show_bug.cgi?id=45255
1931
1932         Many folks (including myself) are too trigger-happy with the "delete"
1933         button that we end up deleting perfectly good review comments.  This
1934         patch adds an "ok" button to absorb these clicks (and renames "delete"
1935         to "cancel").  When you click the ok button, the comment becomes
1936         non-editable and the box shrinks to fit the text.  You can click the
1937         text to get back to the editable version.
1938
1939         * code-review.js:
1940
1941 2010-08-31  Adam Barth  <abarth@webkit.org>
1942
1943         Reviewed by Eric Seidel.
1944
1945         [reviewtool] Make it easy to scroll through review comments
1946         https://bugs.webkit.org/show_bug.cgi?id=45002
1947
1948         This patch lets you scroll through review comments using "n" (for next)
1949         and "p" (for previous).  It also attributes comments to their authors.
1950
1951         * PrettyPatch/PrettyPatch.rb:
1952         * code-review.js:
1953
1954 2010-08-31  Adam Barth  <abarth@webkit.org>
1955
1956         Reviewed by Eric Seidel.
1957
1958         [reviewtool] Show previous comments inline in diff
1959         https://bugs.webkit.org/show_bug.cgi?id=44977
1960
1961         This patch adds basic support for showing previous comments inline in
1962         the diff.  We crawl the bugs.webkit.org comments about this attachment
1963         and extract comments related to specific lines.  We then show the
1964         comments inline in the diff.
1965
1966         This part of the tool needs a bunch of polish, but this at least is a
1967         starting point for further work.
1968
1969         * PrettyPatch/PrettyPatch.rb:
1970         * code-review.js:
1971
1972 2010-08-31  Adam Barth  <abarth@webkit.org>
1973
1974         Reviewed by Eric Seidel.
1975
1976         [reviewtool] Allow reviewer to select lines of context by dragging over the line numbers
1977         https://bugs.webkit.org/show_bug.cgi?id=44936
1978
1979         * PrettyPatch/PrettyPatch.rb:
1980             - Add a version number to bust through bugs.webkit.org's aggressive
1981               cacheing.
1982         * code-review.js:
1983
1984 2010-08-30  Adam Barth  <abarth@webkit.org>
1985
1986         Reviewed by Eric Seidel.
1987
1988         [review tool] Let reviewer select how much context to show in snippet
1989         https://bugs.webkit.org/show_bug.cgi?id=44905
1990
1991         We now highlight the context for a comment in yellow on the left (where
1992         the line numbers are).  Clicking a line number expands or contracts the
1993         amount of context, as appropriate.  Informal user testing indicates
1994         that we might want to support drag as well.
1995
1996         This patch also changes the "open a comment box here" action to
1997         double-click to avoid issues with mis-clicks.
1998
1999         * PrettyPatch/PrettyPatch.rb:
2000         * code-review.js:
2001
2002 2010-08-29  Adam Barth  <abarth@webkit.org>
2003
2004         Attempt to make Sam's life easier by not opening a comment text field
2005         if there's a selection.  This should make it easier to copy/paste text
2006         out of the diff without accidentally opening comment boxes.
2007
2008         * code-review.js:
2009
2010 2010-08-29  Adam Barth  <abarth@webkit.org>
2011
2012         Turns out we're supposed to use find instead of children here.
2013
2014         * code-review.js:
2015
2016 2010-08-29  Adam Barth  <abarth@webkit.org>
2017
2018         Reviewed by Eric Seidel.
2019
2020         Awesomify pretty-diff based review tool
2021         https://bugs.webkit.org/show_bug.cgi?id=44835
2022
2023         This patch replaces the existing inline comment feature with a new
2024         implementation.  Hopefully the new implementation is more awesome.
2025         It's not 100% done yet, but I'd like to get this version out there to
2026         see how it works.
2027
2028         Unlike the previous version this version works with the "Formatted
2029         Diff" link instead of the "Review Patch" link.  Hopefully that will
2030         avoid interfering with folks who like the old style review.
2031
2032         * PrettyPatch/PrettyPatch.rb:
2033         * code-review.js: Added.
2034         * prototype.js: Removed.
2035
2036 2010-08-19  Tony Chang  <tony@chromium.org>
2037
2038         Reviewed by Adam Barth.
2039
2040         webkit-patch barfed on upload with a new image test result
2041         https://bugs.webkit.org/show_bug.cgi?id=39097
2042
2043         * PrettyPatch/PrettyPatch.rb: don't use full path to git
2044
2045 2010-08-19  Tony Chang  <tony@chromium.org>
2046
2047         Unreviewed, setting svn:eol-style native on ChangeLog.
2048
2049 2010-08-19  Tony Chang  <tony@chromium.org>
2050
2051         Unreviewed.  Just removing \r's from ChangeLog.
2052
2053 2010-08-02  Adam Roben  <aroben@apple.com>
2054
2055         Tell the patch prettifier about some (not so) new directories
2056
2057         * PrettyPatch/PrettyPatch.rb: Added WebKit2, autotools, and cmake
2058         directories.
2059
2060 2010-07-27  Joseph Pecoraro  <joepeck@webkit.org>
2061
2062         Reviewed by David Kilzer.
2063
2064         Provide a Helpful Link for Web Inspector Bugs in Bugzilla
2065         https://bugs.webkit.org/show_bug.cgi?id=43065
2066
2067         * template/en/custom/global/choose-product.html.tmpl: Added. Explicitly added a link for the Web Inspector.
2068
2069 2010-06-09  Julie Parent  <jparent@chromium.org>
2070
2071         Reviewed by David Kilzer.
2072
2073         Bugzilla: Don't send mail for cancel/deny/granted for in-rietveld.
2074
2075         https://bugs.webkit.org/show_bug.cgi?id=40345
2076
2077         * Bugzilla/Flag.pm:
2078
2079 2010-05-17  Julie Parent  <jparent@chromium.org>
2080
2081         Reviewed by Ojan Vafai.
2082
2083         Rietveld review page should dedicate more space to Rietveld, less to high level comments form.
2084
2085         https://bugs.webkit.org/show_bug.cgi?id=39244
2086
2087         * template/en/custom/attachment/reviewform.html.tmpl:
2088         Delete a random BR amongst the hidden elements that was causing extra whitespace.
2089         * template/en/custom/attachment/rietveldreview.html.tmpl:
2090         Increase size of Rietveld frame to 80%, from 60%.
2091
2092 2010-05-14  Julie Parent  <jparent@chromium.org>
2093
2094         Reviewed by Ojan Vafai.
2095
2096         Bugzilla: Only show "Rietveld Review" link when in-rietveld is set to +.
2097         
2098         https://bugs.webkit.org/show_bug.cgi?id=39141
2099
2100         * template/en/custom/attachment/list.html.tmpl:
2101
2102 2010-05-13  Julie Parent  <jparent@chromium.org>
2103
2104         Reviewed by David Kilzer.
2105
2106         Bugzilla: Don't send mail if the only change is to the in-rietveld flag.
2107         
2108         https://bugs.webkit.org/show_bug.cgi?id=39090
2109
2110         * Bugzilla/BugMail.pm:
2111
2112 2010-05-12  Julie Parent  <jparent@chromium.org>
2113
2114         Reviewed by Ojan Vafai.
2115
2116         Hide the in-rietveld flag in the UI.  It is only used only by tooling and shouldn't clutter the UI.
2117         https://bugs.webkit.org/show_bug.cgi?id=39024
2118
2119         * template/en/custom/attachment/list.html.tmpl:
2120           Removes this flag entirely from the UI, not needed on this page.
2121         * template/en/custom/flag/list.html.tmpl: Added.
2122           Copied from template/en/default/flag/list.html.tmpl with only changes inside the if WEBKIT_CHANGES section.
2123           Hides the flag rather than removing it completely, so tools can still interact with it.  
2124
2125 2010-05-12  Julie Parent  <jparent@chromium.org>
2126
2127         Reviewed by David Kilzer.
2128
2129         Adds support for Rietveld reviews.  When there is an associated
2130         Rietveld id for a patch, adds a link to do a Rietveld review instead.
2131
2132         https://bugs.webkit.org/show_bug.cgi?id=38143
2133
2134         * attachment.cgi:
2135             Added support for handling action=rietveldreview.
2136         * template/en/custom/attachment/list.html.tmpl:
2137             Adds a link to use rietveld review iff the in_rietveld flag is set.
2138         * template/en/custom/attachment/reviewform.html.tmpl:
2139             Detects if we are in rietveld review mode due to query string paramter,
2140             and customizes the review form slightly:
2141                 - Changes text describing comment field.
2142                 - Comment field does not include quoted patch.
2143                 - Submit button fetches from Rietveld instead of just submitting.
2144         * template/en/custom/attachment/rietveldreview.html.tmpl: Added.
2145             New template for rietveldreview mode.  Hosts the rietveld frame in the
2146             top and the regular comments form in the bottom.
2147             Uses postMessage to communicate with Rietveld frame.
2148
2149 2010-05-08  Chris Jerdonek  <cjerdonek@webkit.org>
2150
2151         Reviewed by Maciej Stachowiak.
2152
2153         Allow long comment lines to wrap to the window width instead
2154         of to 80 characters.
2155
2156         https://bugs.webkit.org/show_bug.cgi?id=37792
2157
2158         * Bugzilla/Constants.pm:
2159           - Introduced a new constant COMMENT_COLS_WRAP to replace COMMENT_COLS
2160             inside the wrap_comment() subroutine.  The new constant effectively
2161             disables text wrapping by increasing the value used in wrap_comment()
2162             from 80 characters to a very large value (8000).
2163         * Bugzilla/Util.pm:
2164           - Replaced COMMENT_COLS with COMMENT_COLS_WRAP inside wrap_comment().
2165         * skins/custom/global.css:
2166         (.bz_comment_text):
2167           - Eliminate the width styling on comments to allow long lines
2168             to wrap at the window width.
2169         (.bz_comment_text > span.quote):
2170           - Inherit the "white-space" property.  This prevents long quoted
2171             lines in comments from causing non-quoted lines to extend beyond
2172             the window width.
2173
2174 2010-04-22  Adam Barth  <abarth@webkit.org>
2175
2176         Reviewed by Darin Adler.
2177
2178         The new review tools shouldn't clear the comment box automatically
2179         https://bugs.webkit.org/show_bug.cgi?id=38004
2180
2181         Instead, we now have a button for doing that explicitly.  If people
2182         like this tool, we can look at fancier solutions.
2183
2184         Also, tried to make the tool work on the edit page as well as the review page.
2185
2186         * PrettyPatch/PrettyPatch.rb:
2187
2188 2010-04-22  Adam Barth  <abarth@webkit.org>
2189
2190         Unreviewed.  Turns out we need to do more escaping because Ruby was
2191         explanding our newlines, creating a syntax error.
2192
2193         * PrettyPatch/PrettyPatch.rb:
2194
2195 2010-04-22  Adam Barth  <abarth@webkit.org>
2196
2197         Reviewed by Darin Adler.
2198
2199         Hack up PrettyDiff to allow line-by-line comments
2200         https://bugs.webkit.org/show_bug.cgi?id=37886
2201
2202         Admittedly a bit of a hack, this is a basic line-by-line editor we can
2203         play with thanks to Andrew Scherkus.  It's meant to integrate with the
2204         "review" page at bugs.webkit.org.
2205
2206         I changed a few things from Andrew's original version:
2207         1) Trigger text boxes on single click to make iPhone/iPad reviewing
2208         easier.
2209         2) Clear the main text box on load.
2210         3) Reference a version of prototype.js on bugs.webkit.org.
2211
2212         * PrettyPatch/PrettyPatch.rb:
2213         * prototype.js: Added.
2214
2215 2010-02-22  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
2216
2217         Reviewed by Maciej Stachowiak.
2218
2219         Don't reset the assignee back to default on component change
2220
2221         https://bugs.webkit.org/show_bug.cgi?id=35236
2222
2223         There's a risk of reseting a valid assignee for example when
2224         triaging a bug. All components in WebKit have the default 
2225         assignee set to webkit-unassigned@webkit.org so this should
2226         not cause problems for people relying on this functionality.
2227
2228         * template/en/custom/bug/edit.html.tmpl:
2229
2230 2010-01-22  Janusz Lewandowski  <lew21@xtreeme.org>
2231
2232         Reviewed by Maciej Stachowiak.
2233
2234         run-webkit-tests fails on Ruby 1.9
2235         https://bugs.webkit.org/show_bug.cgi?id=33554
2236         inject does not work with strings any more, so BugsSite/PrettyPatch/PrettyPatch.rb prints lots of errors.
2237
2238         * PrettyPatch/PrettyPatch.rb:
2239
2240 2010-01-20  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
2241
2242         Reviewed by Adam Barth.
2243
2244         Adjust height of status-bubbles to prevent them being cut.
2245
2246         * template/en/custom/attachment/edit.html.tmpl:
2247         * template/en/custom/attachment/list.html.tmpl:
2248         * template/en/custom/attachment/reviewform.html.tmpl:
2249
2250 2010-01-08  David Kilzer  <ddkilzer@apple.com>
2251
2252         <http://webkit.org/b/33410> Fix "View Plain Diff"/"View Formatted Diff" button
2253
2254         Reviewed by Adam Barth.
2255
2256         This button was broken during a previous Bugzilla upgrade after
2257         changes were made to the names of objects exposed on templates.
2258
2259         * template/en/custom/attachment/reviewform.html.tmpl: Changed
2260         attachid to attachment.id and ispatch to attachment.ispatch.
2261         Also tweaked the format to use 4 table columns instead of 5.
2262
2263 2010-01-08  David Kilzer  <ddkilzer@apple.com>
2264
2265         Included UNCONFIRMED bugs by default in advanced search query
2266
2267         Rubber-stamped by Alexey Proskuryakov.
2268
2269         * data/params: Prepended 'bug_status=UNCONFIRMED&' to
2270         'defaultquery' parameter.
2271
2272 2010-01-08  David Kilzer  <ddkilzer@apple.com>
2273
2274         FIX: Make patch status appear on reviewform.html.tmpl
2275
2276         * template/en/custom/attachment/reviewform.html.tmpl: Check
2277         attachment.ispatch (not just ispatch) when deciding whether
2278         to display patch status.  Also update layout a bit.
2279
2280 2010-01-08  David Kilzer  <ddkilzer@apple.com>
2281
2282         Update data/params parameters
2283
2284         * data/params: Updated 'timezone' to turn off DST.  Updated
2285         'usebugaliases' and 'usevotes' to turn them off.
2286
2287 2010-01-08  Adam Barth  <abarth@webkit.org>
2288
2289         Reviewed by Darin Adler.
2290
2291         Show patch status in bugs.webkit.org
2292         https://bugs.webkit.org/show_bug.cgi?id=33368
2293
2294         This patch adds an iframe to the show_bug page and the edit attachment
2295         page that displays whether the attachment passed the various bots.
2296
2297         * template/en/custom/attachment/edit.html.tmpl:
2298         * template/en/custom/attachment/list.html.tmpl:
2299         * template/en/custom/attachment/reviewform.html.tmpl:
2300
2301 2009-12-07  Shinichiro Hamaji  <hamaji@chromium.org>
2302
2303         Reviewed by David Kilzer.
2304
2305         Bugzilla should show images in git patches
2306         https://bugs.webkit.org/show_bug.cgi?id=31395
2307
2308         Attempt to go r51748 again using --directory option of git-apply.
2309
2310         * PrettyPatch/PrettyPatch.rb:
2311
2312 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
2313
2314         Unreviewed. Revert r51748.
2315
2316         Bugzilla should show images in git patches
2317         https://bugs.webkit.org/show_bug.cgi?id=31395
2318
2319         * PrettyPatch/PrettyPatch.rb:
2320
2321 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
2322
2323         Reviewed by David Kilzer.
2324
2325         Bugzilla should show images in git patches
2326         https://bugs.webkit.org/show_bug.cgi?id=31395
2327
2328         Show images in git patches using git-apply.
2329
2330         * PrettyPatch/PrettyPatch.rb:
2331
2332 2009-10-23  Eric Seidel  <eric@webkit.org>
2333
2334         Reviewed by Adam Roben.
2335
2336         PrettyPatch should show images even when they have the wrong mime type
2337         https://bugs.webkit.org/show_bug.cgi?id=29506
2338
2339         * PrettyPatch/PrettyPatch.rb:
2340
2341 2009-10-17  Alexey Proskuryakov  <ap@apple.com>
2342
2343         Reviewed by Eric Carlson.
2344
2345         https://bugs.webkit.org/show_bug.cgi?id=30470
2346         Make marking a bug as duplicate easier
2347
2348         * js/field.js: (setResolutionToDuplicate): Focus the edit field, too.
2349
2350 2009-10-16  Alexey Proskuryakov  <ap@apple.com>
2351
2352         Reviewed by Mark Rowe.
2353
2354         https://bugs.webkit.org/show_bug.cgi?id=30470
2355         Make marking a bug as duplicate easier
2356
2357         * js/field.js: (setResolutionToDuplicate): Show edit field right away, don't make the user
2358         click again.
2359
2360 2009-09-09  David Kilzer  <ddkilzer@apple.com>
2361
2362         Update data/params parameters
2363
2364         * data/params: Updated 'attachment_base' parameter for secure
2365         attachments.  Updated 'mybugstemplate' parameter to search for
2366         bugs in UNCONFIRMED, ASSIGNED and REOPENED status in addition to
2367         NEW status.
2368
2369 2009-08-11  David Kilzer  <ddkilzer@apple.com>
2370
2371         Updated data/params parameter
2372
2373         * data/params: Updated 'loginnetmask' parameter to allow users
2374         to choose whether their cookies are restricted to a single IP
2375         address or not.
2376
2377 2009-07-07  David Kilzer  <ddkilzer@apple.com>
2378
2379         Updated BugsSite with local changes on server
2380
2381         The new files were created by checksetup.pl during the upgrade
2382         to Bugzilla 3.2.3.  The rest are self-explanatory.
2383
2384         * data: Added "bugzilla-update.xml" to svn:ignore.
2385         * data/attachments: Added "groups.*" to svn:ignore.
2386         * data/params: Updated for a few changed parameters.
2387
2388         * lib/.htaccess: Added.
2389         * skins/contrib/Dusk/IE-fixes.css: Added.
2390         * skins/contrib/Dusk/admin.css: Added.
2391         * skins/contrib/Dusk/create_attachment.css: Added.
2392         * skins/contrib/Dusk/dependency-tree.css: Added.
2393         * skins/contrib/Dusk/duplicates.css: Added.
2394         * skins/contrib/Dusk/editusers.css: Added.
2395         * skins/contrib/Dusk/help.css: Added.
2396         * skins/contrib/Dusk/index.css: Added.
2397         * skins/contrib/Dusk/panel.css: Added.
2398         * skins/contrib/Dusk/params.css: Added.
2399         * skins/contrib/Dusk/release-notes.css: Added.
2400         * skins/contrib/Dusk/show_bug.css: Added.
2401         * skins/contrib/Dusk/show_multiple.css: Added.
2402         * skins/contrib/Dusk/summarize-time.css: Added.
2403         * skins/contrib/Dusk/voting.css: Added.
2404         * skins/contrib/Dusk/yui: Added.
2405         * skins/contrib/Dusk/yui/calendar.css: Added.
2406         * skins/custom/IE-fixes.css: Added.
2407         * skins/custom/admin.css: Added.
2408         * skins/custom/buglist.css: Added.
2409         * skins/custom/create_attachment.css: Added.
2410         * skins/custom/dependency-tree.css: Added.
2411         * skins/custom/duplicates.css: Added.
2412         * skins/custom/editusers.css: Added.
2413         * skins/custom/help.css: Added.
2414         * skins/custom/panel.css: Added.
2415         * skins/custom/params.css: Added.
2416         * skins/custom/release-notes.css: Added.
2417         * skins/custom/show_bug.css: Added.
2418         * skins/custom/show_multiple.css: Added.
2419         * skins/custom/summarize-time.css: Added.
2420         * skins/custom/voting.css: Added.
2421         * skins/custom/yui: Added.
2422         * skins/custom/yui/calendar.css: Added.
2423
2424 2009-07-03  David Kilzer  <ddkilzer@apple.com>
2425
2426         Bug 26958: Change edit link to review link in request messages
2427
2428         <https://bugs.webkit.org/show_bug.cgi?id=26958>
2429
2430         Reviewed by Dan Bernstein.
2431
2432         * template/en/custom/request/email.txt.tmpl: Copied from BugsSite/template/en/default/request/email.txt.tmpl.
2433         Changed link from "action=edit" to "action=review".
2434
2435 2009-07-03  David Kilzer  <ddkilzer@apple.com>
2436
2437         Bug 26950: Make the summary and alias fields support click-to-edit
2438         <https://bugs.webkit.org/show_bug.cgi?id=26950>
2439
2440         Reviewed by Maciej Stachowiak.
2441
2442         Original patch by Maciej Stachowiak.
2443
2444         * js/field.js:
2445         (hideEditableField): Updated to add click event listeners to the
2446         alias and short description elements to make them easier to
2447         edit.  Renamed field_id parameter to field2_id (short
2448         description id) and added a field1_id parameter (alias id).
2449         (showEditableField): If a third parameter is passed in the
2450         ContainerInputArray parameter, use it to find the element to
2451         focus.  Otherwise, fall back to the original behavior of
2452         focusing the first input field.
2453         (hideAliasAndSummary): Changed to pass the id for the alias
2454         element to hideEditableField().
2455
2456 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2457
2458         Config file for contrib/recode.pl when upgrading bugs.webkit.org
2459
2460         Current as of midday on Wednesday, July 1, 2009.
2461
2462         * contrib/recode-overrides.txt: Added.
2463
2464 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2465
2466         Workaround for WebKit Bug 9630 when running contrib/recode.pl
2467
2468         A number of WebKit nightly builds included Bug 9630 which caused
2469         non-breaking space characters (0xA0) to be submitted within the
2470         content of textarea elements.
2471
2472         * contrib/recode.pl: To work around these 0xA0 characters, try
2473         an encoding of cp1252 and use it if it works.
2474
2475 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2476
2477         Don't print "Failed to guess" warning if an override is available
2478
2479         * contrib/recode.pl: Check that %overrides does NOT contain a
2480         key matching the current digest before printing out the warning
2481         about a failed charset guess.
2482
2483 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2484
2485         Added --[no-]truncate switch to contrib/recode.pl
2486
2487         * contrib/recode.pl: Added --[no-]truncate switch to make
2488         debugging of failed charset guessing easier.  Often times the
2489         illegal character was truncated in the output.
2490
2491 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2492
2493         Changed Perl scripts to use #!/usr/bin/env perl
2494
2495         perl -pi -e 's@#\!/usr/bin/perl@#\!/usr/bin/env perl@' *.cgi *.pl */*.pl
2496
2497 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
2498
2499         Replace Bugzilla favicon with webkit.org favicon
2500
2501         * favicon.ico: Removed.
2502         * images/favicon.ico: Replaced.  Copied from favicon.ico.
2503
2504 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
2505
2506         Merged BugsSite to Bugzilla-3.2.3
2507
2508         Updated to the latest-and-greatest stable version.
2509
2510 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
2511
2512         Merged BugsSite to Bugzilla-3.0.3
2513
2514         Nothing to see here.  Move along.
2515
2516 2009-06-09  Eric Seidel  <eric@webkit.org>
2517
2518         Reviewed by Adam Roben.
2519
2520         Add support for displaying added png files in PrettyPatch diffs
2521         https://bugs.webkit.org/show_bug.cgi?id=26210
2522
2523         Currently this is SVN only (git-send-bugzilla patches exclude binary data)
2524         and only works for PNG files but could easily be made to work for other images as needed.
2525
2526         * PrettyPatch/PrettyPatch.rb:
2527
2528 2009-05-19  Eric Seidel  <eric@webkit.org>
2529
2530         Reviewed by Adam Roben.
2531
2532         Make PrettyPatch understand quoted filenames in git diffs.
2533         https://bugs.webkit.org/show_bug.cgi?id=25881
2534
2535         * PrettyPatch/PrettyPatch.rb:
2536
2537 2009-05-15  Simon Fraser  <simon.fraser@apple.com>
2538
2539         Carrying forwards Darin Adler's rubber stamp
2540         
2541         Make the comment field wider too.
2542
2543         * skins/custom/global.css:
2544
2545 2009-05-14  Simon Fraser  <simon.fraser@apple.com>
2546
2547         Reviewed by Darin Adler
2548         
2549         Make the URL, Summary and Keyword fields wider to mitigate the
2550         effects of https://bugs.webkit.org/show_bug.cgi?id=25566 and make
2551         things generally better.
2552
2553         * skins/custom/global.css:
2554         * template/en/custom/bug/edit.html.tmpl:
2555
2556 2008-11-10  Darin Adler  <darin@apple.com>
2557
2558         - fix more of https://bugs.webkit.org/show_bug.cgi?id=21400
2559           "Edit" links for patches in comments, review queue, review emails
2560           should be replaced by "Review Patch" links
2561
2562         * globals.pl: Fix cases that are automatically generated, such as links
2563         in comments in bugs.
2564
2565         * template/en/custom/attachment/created.html.tmpl: Fix the link on the
2566         "I just created an attachment" page.
2567
2568         * template/en/custom/request/queue.html.tmpl: Fix the link in the queue
2569         page.
2570
2571 2008-11-10  Darin Adler  <darin@apple.com>
2572
2573         * template/en/default/request/queue.html.tmpl: Use review links instead
2574         of edit links in the review queue.
2575
2576 2008-11-10  Darin Adler  <darin@apple.com>
2577
2578         * template/en/custom/request/email.txt.tmpl: Send review links instead
2579         of edit links when flags are set on a bug.
2580
2581 2008-10-27  Darin Adler  <darin@apple.com>
2582
2583         * template/en/custom/attachment/reviewform.html.tmpl: Leave out the
2584         comment if it's untouched.
2585
2586 2008-10-23  Adam Roben  <aroben@apple.com>
2587
2588         Fix Bug 21401: Comments field on "Review Patch" page should be
2589         initially filled with quoted patch
2590
2591         <https://bugs.webkit.org/show_bug.cgi?id=21401>
2592
2593         Reviewed by Dave Hyatt.
2594
2595         * attachment.cgi:
2596         (edit): Retrieve the attachment data from the database instead of just
2597         its length. When the attachment is a patch, create a quotedpatch
2598         template variable that contains the patch data with each line
2599         prepended with "> ".
2600         * template/en/custom/attachment/reviewform.html.tmpl:
2601           - Changed the comments field to have a monospace font
2602           - Added an "Enter comments below:" caption above the comments field
2603             to make it clearer that this is where comments should go, now that
2604             the comments field is not initially empty
2605           - Fill the comments field with the quoted patch
2606
2607 2008-10-14  Adam Roben  <aroben@apple.com>
2608
2609         Fix Bug 21602: Bugzilla times out trying to display formatted diff for
2610         attachment 24345
2611         
2612         https://bugs.webkit.org/show_bug.cgi?id=21602
2613
2614         Reviewed by Dave Kilzer.
2615
2616         * PrettyPatch/PrettyPatch.rb:
2617         (PrettyPatch.BINARY_FILE_MARKER_FORMAT): Added.
2618         (PrettyPatch.FileDiff.initialize): If any of the lines in the diff
2619         are a binary file marker, mark this FileDiff as binary and stop trying
2620         to process the lines.
2621         (PrettyDiff.FileDiff.to_html): If we're binary, just print a string
2622         saying so.
2623
2624 2008-10-02  Adam Roben  <aroben@apple.com>
2625
2626         * template/en/custom/attachment/reviewform.html.tmpl: Fix typo.
2627
2628 2008-10-02  Adam Roben  <aroben@apple.com>
2629
2630         * template/en/custom/attachment/review.html.tmpl: Fix URLs
2631
2632 2008-10-02  Adam Roben  <aroben@apple.com>
2633
2634         Fix Bug 21315: bugs.webkit.org should have a nicer patch review
2635         interface
2636
2637         https://bugs.webkit.org/show_bug.cgi?id=21315
2638
2639         Reviewed by Tim Hatcher.
2640
2641         * attachment.cgi:
2642         (top level): Added support for the "review" and "reviewform" actions.
2643         (sub edit): Accept the template name as a parameter. If no name is
2644         give, use "edit" as before.
2645         * template/en/custom/attachment/list.html.tmpl: Added a "Review Patch"
2646         link for all patch attachments.
2647         * template/en/custom/attachment/review.html.tmpl: Added. Simple
2648         <frameset> page to show the patch on the top and a comment form on the
2649         bottom.
2650         * template/en/custom/attachment/reviewform.html.tmpl: Added. Simple
2651         comment form for reviewing patches.
2652
2653 2008-10-02  Adam Roben  <aroben@apple.com>
2654
2655         Remove references to some backup files I created when implementing
2656         PrettyPatch in Bugzilla
2657
2658         Reviewed by Tim Hatcher.
2659
2660         * attachment-aroben.cgi: Removed.
2661         * template/en/default/attachment/edit-aroben.html.tmpl: Removed.
2662
2663         * template/en/custom/attachment/edit.html.tmpl: Removed an erroneous
2664         reference to attachment-aroben.cgi. attachment.cgi will work just
2665         fine.
2666
2667 2008-06-25  David Kilzer  <ddkilzer@apple.com>
2668
2669         Make PrettyPatch handle diffs with no Index or diff headers
2670
2671         Part of Bug 19290: More patches not handled by PrettyPatch.rb
2672         <https://bugs.webkit.org/show_bug.cgi?id=19290>
2673
2674         Reviewed by Adam.
2675
2676         * PrettyPatch/PrettyPatch.rb:
2677         (PrettyPatch.DIFF_HEADER_FORMATS): Added regular expression to
2678         match on "+++ " lines for patches with no "Index" or "diff" header.
2679         (PrettyPatch.FileDiff.initialize): Look for filename on "+++ " line
2680         when the first line of a patch has no "Index" or "diff" header.
2681         (PrettyPatch.FileDiff.parse): Added haveSeenDiffHeader state
2682         variable to determine when no "Index" or "diff" header has been
2683         found, but a new patch has started with a "--- " line.
2684
2685 2008-06-08  Dan Bernstein  <mitz@apple.com>
2686
2687         Reviewed by Adam Roben.
2688
2689         - update trac URL in PrettyPatch
2690
2691         * PrettyPatch/PrettyPatch.rb:
2692
2693 2008-05-29  Adam Roben  <aroben@apple.com>
2694
2695         Update PrettyPatch directory list
2696
2697         * PrettyPatch/PrettyPatch.rb:
2698
2699 2008-05-28  Adam Roben  <aroben@apple.com>
2700
2701         Make PrettyPatch able to handle diffs taken from outside the WebKit
2702         source tree
2703
2704         Part of Bug 19290: More patches not handled by PrettyPatch.rb
2705         <https://bugs.webkit.org/show_bug.cgi?id=19290>
2706
2707         Reviewed by David Kilzer.
2708
2709         * PrettyPatch/PrettyPatch.rb:
2710         (PrettyPatch.find_url_and_path): Added. Searches the file path from
2711         the bottom up looking for a directory that exists in the source tree.
2712         (PrettyPatch.linkifyFilename): Changed to call find_url_and_path.
2713
2714 2008-05-28  Adam Roben  <aroben@apple.com>
2715
2716         Print exceptions from PrettyPatch to stdout
2717
2718         This will make it much easier to debug problems with PrettyPatch,
2719         since we'll be able to see the exceptions in the browser.
2720
2721         Reviewed by Sam Weinig.
2722
2723         * PrettyPatch/prettify.rb: Added a --html-exceptions option, which
2724         will print exceptions to stdout as HTML.
2725         * attachment.cgi: Changed to pass --html-exceptions to prettify.rb.
2726
2727 2008-04-26  David Kilzer  <ddkilzer@apple.com>
2728
2729         Removed temp files and added appropriate svn:ignore properties.
2730
2731         Rubber-stamped by Mark Rowe.
2732
2733         * data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
2734
2735         * data/template: Added svn:ignore property for "template" subdirectory.
2736         * data/template/template: Removed precompiled template subdirectory.
2737
2738         * data/versioncache: Removed.
2739         * data/versioncache.*: Removed backup versioncache files.
2740
2741         * data/webdot: Added svn:ignore property for "*.dot" files.
2742         * data/webdot/*.dot: Removed cached webdot files.
2743
2744 2008-04-08  Adam Roben  <aroben@apple.com>
2745
2746         Combine :equal operations with the following operation if they are
2747         fewer than 3 characters long
2748
2749         This keeps us from showing lots of small changes on long lines, just
2750         because some letters happened to be the same.
2751
2752         Inspired by
2753         http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/diffviewer/diffutils.py?r=1264#147
2754
2755         * PrettyPatch/PrettyPatch.rb:
2756         (PrettyPatch::DiffSection.initialize): Remove :equal operations fewer
2757         than 3 characters long. The characters from the removed operations
2758         become part of the subsequent operation.
2759         (PrettyPatch::CodeLine.text_as_html): Don't wrap 0-length strings in
2760         <ins>/<del> tags. Also removed the @fromLineNumber.nil? and
2761         @toLineNumber.nil? checks as they are no longer reliable now that
2762         we're removing operations.
2763
2764 2008-04-08  Adam Roben  <aroben@apple.com>
2765
2766         Change PrettyPatch to use DiffBuilder for intra-line diffs
2767
2768         This gives us much prettier intra-line diffs, largely because it can
2769         distinguish multiple changes on the same line. e.g., if a line changes
2770         from:
2771
2772         const int myConstant = 0;
2773
2774         to
2775
2776         static const unsigned myConstant;
2777
2778         You will see that "static " was inserted, "int" changed to "unsigned",
2779         and " = 0" was deleted.
2780
2781         This seems to have also gotten rid of some spurious instances of
2782         "<ins></ins>" and "<del></del>" at the end of a line.
2783
2784         * PrettyPatch/PrettyPatch.rb:
2785         (PrettyPatch::DiffSection.initialize): Use DiffBuilder instead of
2786         setChangeExtentFromLine (which had been copied from Trac's diffing
2787         code).
2788         (PrettyPatch::CodeLine):
2789          - Removed the changeExtent attribute
2790          - Added the operations attribute
2791          - Removed the setChangeExtentFromLine method
2792         (PrettyPatch::CodeLine.text_as_html): Uses @operations to determine
2793         what text to wrap in <ins> and <del> tags.
2794         * PrettyPatch/diff.rb: Deleted a bunch of code we don't use.
2795         (HTMLDiff::DiffBuilder.initialize): Moved code here from the old build
2796         method, but left out the calls to perform_operation, since we build
2797         the diff HTML outside of this class.
2798         (HTMLDiff::DiffBuilder.split_inputs_to_words): Removed calls to
2799         convert_html_to_list_of_words, since we're not diffing HTML.
2800
2801 2008-04-08  Adam Roben  <aroben@apple.com>
2802
2803         Check in diff.rb
2804
2805         This came from
2806         http://instiki.rubyforge.org/svn/instiki/trunk/lib/diff.rb
2807         revision 28521
2808
2809         * PrettyPatch/diff.rb: Added.
2810
2811 2008-03-04  Adam Roben  <aroben@apple.com>
2812
2813         * PrettyPatch/PrettyPatch.rb: Allow the extent of the previous file to
2814         be omitted from the hunk header.
2815