RESOLVED FIXED 107848
[CSS Filters] Using negative drop-shadow radius values has slow performance
https://bugs.webkit.org/show_bug.cgi?id=107848
Summary [CSS Filters] Using negative drop-shadow radius values has slow performance
Alexandru Chiculita
Reported 2013-01-24 11:33:59 PST
Using a negative radius for the drop-shadow filters has slow repaint performance. Also added a bug on the spec to check what's the right behavior for negative radius lengths: https://www.w3.org/Bugs/Public/show_bug.cgi?id=20762 Example: -webkit-filter: drop-shadow(10px 10px -1px red)
Attachments
Patch (31.74 KB, patch)
2013-03-01 14:49 PST, Michelangelo De Simone
no flags
Patch (30.12 KB, patch)
2013-03-05 13:19 PST, Michelangelo De Simone
no flags
Patch (30.57 KB, patch)
2013-03-05 16:48 PST, Michelangelo De Simone
no flags
Patch (30.65 KB, patch)
2013-03-07 10:06 PST, Michelangelo De Simone
no flags
Patch (30.75 KB, patch)
2013-03-07 10:13 PST, Michelangelo De Simone
no flags
Patch (30.54 KB, patch)
2013-03-22 06:45 PDT, Michelangelo De Simone
no flags
Archive of layout-test-results from gce-cr-linux-08 (1.33 MB, application/zip)
2013-03-22 07:30 PDT, WebKit Review Bot
no flags
Patch (30.49 KB, patch)
2013-03-22 09:58 PDT, Michelangelo De Simone
no flags
Radar WebKit Bug Importer
Comment 1 2013-01-30 14:21:13 PST
Dean Jackson
Comment 2 2013-02-25 11:06:00 PST
Alex, do you think you'll address this soon? It seems like an easy fix.
Alexandru Chiculita
Comment 3 2013-02-25 15:48:45 PST
I will take a look at it.
Alexandru Chiculita
Comment 4 2013-02-25 16:26:57 PST
FEDropShadow and FEGaussianBlur are also affected, meaning that SVGFEDropShadowElement and SVGFEGaussianBlurElement also reproduce the issue.
Michelangelo De Simone
Comment 5 2013-02-25 16:32:42 PST
Alex's alter ego will take a look at this.:)
Michelangelo De Simone
Comment 6 2013-03-01 14:49:00 PST
WebKit Review Bot
Comment 7 2013-03-01 14:56:15 PST
Attachment 191040 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/css3/filters/effect-blur-negative-radius-expected.html', u'LayoutTests/css3/filters/effect-blur-negative-radius.html', u'LayoutTests/css3/filters/effect-drop-shadow-negative-radius-expected.html', u'LayoutTests/css3/filters/effect-drop-shadow-negative-radius.html', u'LayoutTests/css3/filters/filter-property-parsing-invalid-expected.txt', u'LayoutTests/css3/filters/script-tests/filter-property-parsing-invalid.js', u'LayoutTests/fast/box-shadow/box-shadow-negative-radius-expected.html', u'LayoutTests/fast/box-shadow/box-shadow-negative-radius.html', u'LayoutTests/platform/chromium-mac/svg/filters/feDropShadow-zero-deviation-expected.png', u'LayoutTests/platform/chromium-mac/svg/filters/feDropShadow-zero-deviation-expected.txt', u'LayoutTests/platform/chromium-mac/svg/filters/feGaussianBlur-zero-deviation-expected.png', u'LayoutTests/platform/chromium-mac/svg/filters/feGaussianBlur-zero-deviation-expected.txt', u'LayoutTests/platform/mac/svg/filters/feDropShadow-zero-deviation-expected.png', u'LayoutTests/platform/mac/svg/filters/feDropShadow-zero-deviation-expected.txt', u'LayoutTests/platform/mac/svg/filters/feGaussianBlur-zero-deviation-expected.png', u'LayoutTests/platform/mac/svg/filters/feGaussianBlur-zero-deviation-expected.txt', u'LayoutTests/svg/filters/feDropShadow-negative-deviation-expected.svg', u'LayoutTests/svg/filters/feDropShadow-negative-deviation.svg', u'LayoutTests/svg/filters/feDropShadow-zero-deviation.svg', u'LayoutTests/svg/filters/feGaussianBlur-negative-deviation-expected.svg', u'LayoutTests/svg/filters/feGaussianBlur-negative-deviation.svg', u'LayoutTests/svg/filters/feGaussianBlur-zero-deviation.svg', u'Source/WebCore/ChangeLog', u'Source/WebCore/css/CSSParser.cpp', u'Source/WebCore/platform/graphics/filters/FEGaussianBlur.cpp', u'Source/WebCore/svg/SVGFEDropShadowElement.cpp', u'Source/WebCore/svg/SVGFEGaussianBlurElement.cpp']" exit_code: 1 LayoutTests/platform/mac/svg/filters/feGaussianBlur-zero-deviation-expected.png:0: Have to enable auto props in the subversion config file (/home/alancutter/.subversion/config "enable-auto-props = yes"). Have to set the svn:mime-type in the subversion config file (/home/alancutter/.subversion/config "*.png = svn:mime-type=image/png"). [image/png] [5] Total errors found: 1 in 25 files If any of these errors are false positives, please file a bug against check-webkit-style.
Simon Fraser (smfr)
Comment 8 2013-03-01 15:02:47 PST
Comment on attachment 191040 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=191040&action=review > Source/WebCore/css/CSSParser.cpp:6276 > + } else if (validUnit(val, FLength | FNonNeg, CSSStrictMode)) { Do the specs say that negative radii are invalid, or clamped to zero?
Michelangelo De Simone
Comment 9 2013-03-01 15:15:32 PST
(In reply to comment #8) > Do the specs say that negative radii are invalid, or clamped to zero? Specs claim "A negative value is an error...", so I guess it's not to be clamped.
Simon Fraser (smfr)
Comment 10 2013-03-01 15:17:07 PST
Then that should cause the entire property to be ignored. Is that the behavior of your patch?
Michelangelo De Simone
Comment 11 2013-03-01 15:27:28 PST
(In reply to comment #10) > Then that should cause the entire property to be ignored. Is that the behavior of your patch? Yep. Actually something like blur(-1px) was already covered by existing parsing tests.
Dirk Schulze
Comment 12 2013-03-01 17:48:43 PST
Comment on attachment 191040 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=191040&action=review The patch looks good to me in general. Just some change requests. Please check why the bots seems to have problems. > Source/WebCore/ChangeLog:28 > + (WebCore::FEGaussianBlur::calculateUnscaledKernelSize): Added "non-negative" assertion. > + * svg/SVGFEDropShadowElement.cpp: > + (WebCore::SVGFEDropShadowElement::build): If a negative standard deviation is encountered, don't Can you write some more lines what you did? IIRC there even was a problem with not returning early here? >> Source/WebCore/css/CSSParser.cpp:6276 >> + } else if (validUnit(val, FLength | FNonNeg, CSSStrictMode)) { > > Do the specs say that negative radii are invalid, or clamped to zero? Your comment should really say that. The spec follows CSS background and borders. And CSS3BG says that negative values for the radius is not allowed. > LayoutTests/css3/filters/effect-blur-negative-radius.html:10 > + -webkit-filter: blur(-1px); I wonder if we do not have negative tests for the -webkit-filter prefix? Maybe it is not necessary to add a ref test just for a parser check. > LayoutTests/css3/filters/script-tests/filter-property-parsing-invalid.js:96 > +testInvalidFilterRule("Negative radius", "drop-shadow(10px 10px -1px red)"); Here you go. This makes the other test unnecessary. > LayoutTests/fast/box-shadow/box-shadow-negative-radius.html:35 > + The following divs have box-shadow associated with them; every shadow is declared > + using a mix of invalid negative values, which are clearly wrong. Ditto. Don't we have box-shadow parser tests? Since it should fail on parsing, this test seems to be no necessary. Thank you for taking the time to write this test :).
Michelangelo De Simone
Comment 13 2013-03-05 13:19:54 PST
Michelangelo De Simone
Comment 14 2013-03-05 13:20:37 PST
(In reply to comment #12) I've uploaded a revised patch according to your comments. > The patch looks good to me in general. Just some change requests. Please check why the bots seems to have problems. The check-webkit-style seems to be complaining about svn auto-props for PNGs, which AFAICT is a local configuration thingy not really related to the patch. Anyhow, I've just filed bug 111474. I'll eventually land this manually. > Can you write some more lines what you did? IIRC there even was a problem with not returning early here? Done. > Your comment should really say that. The spec follows CSS background and borders. And CSS3BG says that negative values for the radius is not allowed. Done. > > LayoutTests/css3/filters/script-tests/filter-property-parsing-invalid.js:96 > > +testInvalidFilterRule("Negative radius", "drop-shadow(10px 10px -1px red)"); > Here you go. This makes the other test unnecessary. I've removed the blur-negative test, which was redundant. > > LayoutTests/fast/box-shadow/box-shadow-negative-radius.html:35 > > + The following divs have box-shadow associated with them; every shadow is declared > > + using a mix of invalid negative values, which are clearly wrong. > Ditto. Don't we have box-shadow parser tests? Since it should fail on parsing, this test seems to be no necessary. Thank you for taking the time to write this test :). I didn't find any existing parsing test for box-shadow so I thought to leave this test as it is; let me know if there's something I didn't check out. Thanks!
WebKit Review Bot
Comment 15 2013-03-05 13:22:33 PST
Attachment 191548 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/css3/filters/effect-drop-shadow-negative-radius-expected.html', u'LayoutTests/css3/filters/effect-drop-shadow-negative-radius.html', u'LayoutTests/css3/filters/filter-property-parsing-invalid-expected.txt', u'LayoutTests/css3/filters/script-tests/filter-property-parsing-invalid.js', u'LayoutTests/fast/box-shadow/box-shadow-negative-radius-expected.html', u'LayoutTests/fast/box-shadow/box-shadow-negative-radius.html', u'LayoutTests/platform/chromium-mac/svg/filters/feDropShadow-zero-deviation-expected.png', u'LayoutTests/platform/chromium-mac/svg/filters/feDropShadow-zero-deviation-expected.txt', u'LayoutTests/platform/chromium-mac/svg/filters/feGaussianBlur-zero-deviation-expected.png', u'LayoutTests/platform/chromium-mac/svg/filters/feGaussianBlur-zero-deviation-expected.txt', u'LayoutTests/platform/mac/svg/filters/feDropShadow-zero-deviation-expected.png', u'LayoutTests/platform/mac/svg/filters/feDropShadow-zero-deviation-expected.txt', u'LayoutTests/platform/mac/svg/filters/feGaussianBlur-zero-deviation-expected.png', u'LayoutTests/platform/mac/svg/filters/feGaussianBlur-zero-deviation-expected.txt', u'LayoutTests/svg/filters/feDropShadow-negative-deviation-expected.svg', u'LayoutTests/svg/filters/feDropShadow-negative-deviation.svg', u'LayoutTests/svg/filters/feDropShadow-zero-deviation.svg', u'LayoutTests/svg/filters/feGaussianBlur-negative-deviation-expected.svg', u'LayoutTests/svg/filters/feGaussianBlur-negative-deviation.svg', u'LayoutTests/svg/filters/feGaussianBlur-zero-deviation.svg', u'Source/WebCore/ChangeLog', u'Source/WebCore/css/CSSParser.cpp', u'Source/WebCore/platform/graphics/filters/FEGaussianBlur.cpp', u'Source/WebCore/svg/SVGFEDropShadowElement.cpp', u'Source/WebCore/svg/SVGFEGaussianBlurElement.cpp']" exit_code: 1 LayoutTests/platform/mac/svg/filters/feGaussianBlur-zero-deviation-expected.png:0: Have to enable auto props in the subversion config file (/home/alancutter/.subversion/config "enable-auto-props = yes"). Have to set the svn:mime-type in the subversion config file (/home/alancutter/.subversion/config "*.png = svn:mime-type=image/png"). [image/png] [5] Total errors found: 1 in 23 files If any of these errors are false positives, please file a bug against check-webkit-style.
Dirk Schulze
Comment 16 2013-03-05 13:47:49 PST
Comment on attachment 191548 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=191548&action=review Please look onto the style issue. Looks like you need to set the mime type manually? There is just one little change request. Other than that the patch looks great! > LayoutTests/ChangeLog:16 > + * fast/box-shadow/box-shadow-negative-radius-expected.html: Added. > + * fast/box-shadow/box-shadow-negative-radius.html: Added. box shadow seems really to miss parsing tests. That is really really bad. Would it be possible that you create a box-shadow-parsing-invalid.html similar to other tests where you test just your part. Create a new bug report to add parser tests for box-shadow and add a FIXME into this new created file. I really appreciate if you do that. Thanks.
Michelangelo De Simone
Comment 17 2013-03-05 16:48:31 PST
Michelangelo De Simone
Comment 18 2013-03-07 10:06:18 PST
Michelangelo De Simone
Comment 19 2013-03-07 10:13:32 PST
Michelangelo De Simone
Comment 20 2013-03-07 10:14:08 PST
(In reply to comment #16) > Please look onto the style issue. Looks like you need to set the mime type manually? There is just one little change request. Other than that the patch looks great! As far as I know webkit-patch land should take care of setting the mime type on PNGs automatically (see bug 75825). I've a git only repo here so, in case this is not the case, I'll handle this manually. > > LayoutTests/ChangeLog:16 > > + * fast/box-shadow/box-shadow-negative-radius-expected.html: Added. > > + * fast/box-shadow/box-shadow-negative-radius.html: Added. > box shadow seems really to miss parsing tests. That is really really bad. Would it be possible that you create a box-shadow-parsing-invalid.html similar to other tests where you test just your part. Create a new bug report to add parser tests for box-shadow and add a FIXME into this new created file. I really appreciate if you do that. Thanks. As requested, I've added a new test for box-shadow invalid parsing test and filed a new bug for it. Thanks.
WebKit Review Bot
Comment 21 2013-03-07 10:23:26 PST
Attachment 192039 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/css3/filters/effect-drop-shadow-negative-radius-expected.html', u'LayoutTests/css3/filters/effect-drop-shadow-negative-radius.html', u'LayoutTests/css3/filters/filter-property-parsing-invalid-expected.txt', u'LayoutTests/css3/filters/script-tests/filter-property-parsing-invalid.js', u'LayoutTests/fast/box-shadow/box-shadow-parsing-invalid-expected.txt', u'LayoutTests/fast/box-shadow/box-shadow-parsing-invalid.html', u'LayoutTests/fast/box-shadow/script-tests/box-shadow-parsing-invalid.js', u'LayoutTests/platform/chromium-mac/svg/filters/feDropShadow-zero-deviation-expected.png', u'LayoutTests/platform/chromium-mac/svg/filters/feDropShadow-zero-deviation-expected.txt', u'LayoutTests/platform/chromium-mac/svg/filters/feGaussianBlur-zero-deviation-expected.png', u'LayoutTests/platform/chromium-mac/svg/filters/feGaussianBlur-zero-deviation-expected.txt', u'LayoutTests/platform/mac/svg/filters/feDropShadow-zero-deviation-expected.png', u'LayoutTests/platform/mac/svg/filters/feDropShadow-zero-deviation-expected.txt', u'LayoutTests/platform/mac/svg/filters/feGaussianBlur-zero-deviation-expected.png', u'LayoutTests/platform/mac/svg/filters/feGaussianBlur-zero-deviation-expected.txt', u'LayoutTests/svg/filters/feDropShadow-negative-deviation-expected.svg', u'LayoutTests/svg/filters/feDropShadow-negative-deviation.svg', u'LayoutTests/svg/filters/feDropShadow-zero-deviation.svg', u'LayoutTests/svg/filters/feGaussianBlur-negative-deviation-expected.svg', u'LayoutTests/svg/filters/feGaussianBlur-negative-deviation.svg', u'LayoutTests/svg/filters/feGaussianBlur-zero-deviation.svg', u'Source/WebCore/ChangeLog', u'Source/WebCore/css/CSSParser.cpp', u'Source/WebCore/platform/graphics/filters/FEGaussianBlur.cpp', u'Source/WebCore/svg/SVGFEDropShadowElement.cpp', u'Source/WebCore/svg/SVGFEGaussianBlurElement.cpp']" exit_code: 1 LayoutTests/platform/mac/svg/filters/feGaussianBlur-zero-deviation-expected.png:0: Have to enable auto props in the subversion config file (/home/alancutter/.subversion/config "enable-auto-props = yes"). Have to set the svn:mime-type in the subversion config file (/home/alancutter/.subversion/config "*.png = svn:mime-type=image/png"). [image/png] [5] Total errors found: 1 in 24 files If any of these errors are false positives, please file a bug against check-webkit-style.
Dirk Schulze
Comment 22 2013-03-07 14:56:26 PST
Comment on attachment 192039 [details] Patch LGTM. r=me. Let's see if the landing bot can handle it.
WebKit Review Bot
Comment 23 2013-03-07 20:35:29 PST
Comment on attachment 192039 [details] Patch Rejecting attachment 192039 [details] from commit-queue. New failing tests: fast/repaint/shadow-multiple-strict-vertical.html fast/box-shadow/inset-shadow-large-offset.html fast/box-shadow/spread-multiple-inset.html fast/repaint/box-shadow-inset-repaint.html http/tests/cache/subresource-failover-to-network.html fast/borders/border-radius-with-box-shadow-01.html fast/borders/border-shadow-large-radius.html compositing/filters/sw-shadow-overlaps-hw-shadow.html fast/box-shadow/spread.html fast/repaint/shadow-multiple-strict-horizontal.html fast/box-shadow/spread-multiple-normal.html fast/dom/title-directionality-removeChild.html fast/regions/element-region-overset-state-vertical-rl.html fast/loader/text-document-wrapping.html fast/repaint/box-shadow-h.html fast/repaint/shadow-multiple-horizontal.html fast/css/shadow-multiple.html compositing/filters/sw-layer-overlaps-hw-shadow.html fast/dom/title-directionality.html css3/filters/filter-repaint-shadow-rotated.html fast/box-shadow/inset.html fast/repaint/box-shadow-v.html fast/box-shadow/box-shadow-clipped-slices.html fast/repaint/shadow-multiple-vertical.html fast/box-shadow/inset-box-shadows.html fast/box-shadow/no-blur-multiple-offsets.html fast/loader/javascript-url-in-object.html css3/filters/filter-repaint-shadow-clipped.html compositing/filters/sw-nested-shadow-overlaps-hw-nested-shadow.html Full output: http://webkit-commit-queue.appspot.com/results/16991477
Build Bot
Comment 24 2013-03-08 13:08:58 PST
Comment on attachment 192039 [details] Patch Attachment 192039 [details] did not pass mac-ews (mac): Output: http://webkit-commit-queue.appspot.com/results/17025482 New failing tests: platform/mac/editing/deleting/deletionUI-single-instance.html compositing/filters/sw-layer-overlaps-hw-shadow.html svg/css/shadow-changes.svg svg/filters/shadow-on-rect-with-filter.svg http/tests/security/mixedContent/redirect-https-to-http-iframe-in-main-frame.html fast/repaint/box-shadow-h.html compositing/filters/sw-nested-shadow-overlaps-hw-nested-shadow.html fast/repaint/box-shadow-v.html fast/regions/element-region-overset-state-vertical-rl.html svg/css/text-shadow-multiple.xhtml svg/css/shadow-with-negative-offset.svg fast/loader/javascript-url-in-object.html transitions/cubic-bezier-overflow-shadow.html media/video-played-collapse.html svg/css/shadow-and-opacity.svg compositing/filters/sw-shadow-overlaps-hw-shadow.html http/tests/plugins/third-party-cookie-accept-policy.html
Michelangelo De Simone
Comment 25 2013-03-08 15:16:06 PST
Looks like there are "few" tests that need to be rebased.:-/
Dean Jackson
Comment 26 2013-03-08 15:30:01 PST
(In reply to comment #25) > Looks like there are "few" tests that need to be rebased.:-/ Do these reproduce for you locally?
WebKit Review Bot
Comment 27 2013-03-09 13:00:08 PST
Comment on attachment 192039 [details] Patch Attachment 192039 [details] did not pass chromium-ews (chromium-xvfb): Output: http://webkit-commit-queue.appspot.com/results/17000570 New failing tests: fast/repaint/shadow-multiple-strict-vertical.html fast/box-shadow/inset-shadow-large-offset.html fast/box-shadow/spread-multiple-inset.html fast/repaint/box-shadow-inset-repaint.html fast/hidpi/video-controls-in-hidpi.html http/tests/cache/subresource-failover-to-network.html fast/borders/border-radius-with-box-shadow-01.html fast/borders/border-shadow-large-radius.html compositing/filters/sw-shadow-overlaps-hw-shadow.html fast/box-shadow/spread.html fast/repaint/shadow-multiple-strict-horizontal.html fast/preloader/document-write-noscript.html fast/regions/element-region-overset-state-vertical-rl.html fast/loader/text-document-wrapping.html fast/repaint/box-shadow-h.html fast/repaint/shadow-multiple-horizontal.html fast/css/shadow-multiple.html compositing/filters/sw-layer-overlaps-hw-shadow.html css3/filters/filter-repaint-shadow-rotated.html fast/box-shadow/inset.html fast/repaint/box-shadow-v.html fast/box-shadow/spread-multiple-normal.html fast/box-shadow/box-shadow-clipped-slices.html fast/repaint/shadow-multiple-vertical.html fast/box-shadow/inset-box-shadows.html fast/box-shadow/no-blur-multiple-offsets.html fast/loader/javascript-url-in-object.html css3/filters/filter-repaint-shadow-clipped.html compositing/filters/sw-nested-shadow-overlaps-hw-nested-shadow.html fast/repaint/text-shadow-horizontal.html
Michelangelo De Simone
Comment 28 2013-03-11 12:16:28 PDT
(In reply to comment #26) > Do these reproduce for you locally? Almost all of them. I'm going over each one manually.
Build Bot
Comment 29 2013-03-19 04:45:31 PDT
Comment on attachment 192039 [details] Patch Attachment 192039 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-commit-queue.appspot.com/results/17136630 New failing tests: compositing/filters/sw-layer-overlaps-hw-shadow.html svg/filters/shadow-on-rect-with-filter.svg svg/css/shadow-changes.svg fast/repaint/box-shadow-h.html compositing/filters/sw-nested-shadow-overlaps-hw-nested-shadow.html svg/css/shadow-and-opacity.svg fast/regions/element-region-overset-state-vertical-rl.html svg/css/shadow-with-negative-offset.svg transitions/multiple-shadow-transitions.html svg/css/text-shadow-multiple.xhtml transitions/svg-text-shadow-transition.html transitions/cubic-bezier-overflow-shadow.html fast/repaint/box-shadow-v.html compositing/filters/sw-shadow-overlaps-hw-shadow.html
Michelangelo De Simone
Comment 30 2013-03-22 06:45:18 PDT
WebKit Review Bot
Comment 31 2013-03-22 07:30:12 PDT
Comment on attachment 194530 [details] Patch Attachment 194530 [details] did not pass chromium-ews (chromium-xvfb): Output: http://webkit-commit-queue.appspot.com/results/17141553 New failing tests: svg/filters/feGaussianBlur-zero-deviation.svg svg/filters/feDropShadow-zero-deviation.svg
WebKit Review Bot
Comment 32 2013-03-22 07:30:18 PDT
Created attachment 194539 [details] Archive of layout-test-results from gce-cr-linux-08 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: gce-cr-linux-08 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-3.3.8-gcg-201212281604-x86_64-with-GCEL-10.04-gcel_10.04
Michelangelo De Simone
Comment 33 2013-03-22 09:28:03 PDT
(In reply to comment #31) > New failing tests: > svg/filters/feGaussianBlur-zero-deviation.svg > svg/filters/feDropShadow-zero-deviation.svg These two tests seem to be false positives.
Michelangelo De Simone
Comment 34 2013-03-22 09:58:41 PDT
Dirk Schulze
Comment 35 2013-03-25 05:02:36 PDT
Comment on attachment 194577 [details] Patch LGTM. r=me
WebKit Review Bot
Comment 36 2013-03-25 06:12:42 PDT
Comment on attachment 194577 [details] Patch Clearing flags on attachment: 194577 Committed r146762: <http://trac.webkit.org/changeset/146762>
WebKit Review Bot
Comment 37 2013-03-25 06:12:49 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.