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