Bug 132569 - Adapt inline SVG sizing behavior to Firefox and Blink
Summary: Adapt inline SVG sizing behavior to Firefox and Blink
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-05-05 09:45 PDT by Dirk Schulze
Modified: 2014-06-15 21:13 PDT (History)
26 users (show)

See Also:


Attachments
Checking for failing tests on bots (260.58 KB, patch)
2014-05-05 09:53 PDT, Dirk Schulze
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-01 for mac-mountainlion (787.71 KB, application/zip)
2014-05-05 11:34 PDT, Build Bot
no flags Details
Archive of layout-test-results from webkit-ews-10 for mac-mountainlion-wk2 (759.22 KB, application/zip)
2014-05-05 12:10 PDT, Build Bot
no flags Details
Patch (552.67 KB, patch)
2014-05-05 12:35 PDT, Dirk Schulze
no flags Details | Formatted Diff | Diff
Patch (552.67 KB, patch)
2014-05-05 12:58 PDT, Dirk Schulze
no flags Details | Formatted Diff | Diff
Patch (555.12 KB, patch)
2014-05-05 13:14 PDT, Dirk Schulze
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Schulze 2014-05-05 09:45:32 PDT
During the SVG F2F in Leipzig, the SVG WG agreed on a model to size inline SVGs in HTML content. The model is following the behavior of CSS2.1 for replaced elements and already implemented in Firefox. Blink followed the behavior of Firefox http://src.chromium.org/viewvc/blink?limit_changes=0&view=revision&revision=171958 .

An extensive test suite can be found here: http://w3c-test.org/html/rendering/replaced-elements/svg-inline-sizing/svg-inline.html
Comment 1 Dirk Schulze 2014-05-05 09:53:45 PDT
Created attachment 230834 [details]
Checking for failing tests on bots

Not for review.
Comment 2 WebKit Commit Bot 2014-05-05 09:55:50 PDT
Attachment 230834 [details] did not pass style-queue:


ERROR: Source/WebCore/rendering/RenderReplaced.cpp:287:  Multi line control clauses should use braces.  [whitespace/braces] [4]
ERROR: Source/WebCore/rendering/svg/RenderSVGRoot.cpp:91:  One line control clauses should not use braces.  [whitespace/braces] [4]
ERROR: Source/WebCore/svg/SVGSVGElement.cpp:289:  One line control clauses should not use braces.  [whitespace/braces] [4]
Total errors found: 3 in 110 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Build Bot 2014-05-05 11:34:24 PDT
Comment on attachment 230834 [details]
Checking for failing tests on bots

Attachment 230834 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/5976985020923904

New failing tests:
svg/text/non-bmp-positioning-lists.svg
fast/css/infinite-floating-value.html
svg/custom/object-sizing-width-50p-height-50p-on-target-svg.xhtml
svg/custom/object-sizing-width-75p-height-50p-on-target-svg.xhtml
fast/css3-text/css3-word-spacing-percentage/word-spacing-crash.html
svg/custom/object-sizing-width-50p-on-target-svg.xhtml
fast/css/remove-fixed-resizer-crash.html
svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute.xhtml
accessibility/svg-image.html
css2.1/20110323/replaced-intrinsic-001.htm
svg/custom/object-sizing-height-50p-on-target-svg-absolute.xhtml
css2.1/20110323/replaced-intrinsic-002.htm
http/tests/xmlviewer/dumpAsText/svg.xml
svg/hixie/intrinsic/002.html
svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute.xhtml
svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute.xhtml
svg/custom/object-sizing-height-50p-on-target-svg.xhtml
svg/hixie/intrinsic/001.html
svg/custom/object-sizing-width-50p-height-75p-on-target-svg.xhtml
fast/shapes/shape-outside-floats/shape-outside-relative-size-svg.html
Comment 4 Build Bot 2014-05-05 11:34:31 PDT
Created attachment 230840 [details]
Archive of layout-test-results from webkit-ews-01 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-01  Port: mac-mountainlion  Platform: Mac OS X 10.8.5
Comment 5 Build Bot 2014-05-05 12:10:32 PDT
Comment on attachment 230834 [details]
Checking for failing tests on bots

Attachment 230834 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/6008755699318784

New failing tests:
svg/text/non-bmp-positioning-lists.svg
fast/css/infinite-floating-value.html
svg/custom/object-sizing-width-50p-height-50p-on-target-svg.xhtml
svg/custom/object-sizing-width-75p-height-50p-on-target-svg.xhtml
fast/css3-text/css3-word-spacing-percentage/word-spacing-crash.html
svg/custom/object-sizing-width-50p-on-target-svg.xhtml
fast/css/remove-fixed-resizer-crash.html
svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute.xhtml
accessibility/svg-image.html
css2.1/20110323/replaced-intrinsic-001.htm
svg/custom/object-sizing-height-50p-on-target-svg-absolute.xhtml
css2.1/20110323/replaced-intrinsic-002.htm
http/tests/xmlviewer/dumpAsText/svg.xml
svg/hixie/intrinsic/002.html
svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute.xhtml
svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute.xhtml
svg/custom/object-sizing-height-50p-on-target-svg.xhtml
svg/hixie/intrinsic/001.html
svg/custom/object-sizing-width-50p-height-75p-on-target-svg.xhtml
fast/shapes/shape-outside-floats/shape-outside-relative-size-svg.html
Comment 6 Build Bot 2014-05-05 12:10:43 PDT
Created attachment 230843 [details]
Archive of layout-test-results from webkit-ews-10 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-10  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 7 Dirk Schulze 2014-05-05 12:35:58 PDT
Created attachment 230846 [details]
Patch
Comment 8 Dirk Schulze 2014-05-05 12:58:12 PDT
Created attachment 230850 [details]
Patch
Comment 9 Dirk Schulze 2014-05-05 13:14:57 PDT
Created attachment 230851 [details]
Patch
Comment 10 Dean Jackson 2014-05-05 14:03:51 PDT
Comment on attachment 230851 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=230851&action=review

> Source/WebCore/ChangeLog:14
> +        "The basis of this change is to map explicit width and height
> +        attributes to CSS properties, essentially promoting them to
> +        presentation attributes. Note that implicit "100%" width and height
> +        are not mapped.

You're missing an end quote mark.

> Source/WebCore/ChangeLog:39
> +        (svg:root):

Either remove this line or put the text above on it.

> Source/WebCore/ChangeLog:45
> +            And so it SVGSVGElement.

"is"
Comment 11 WebKit Commit Bot 2014-05-05 23:07:00 PDT
Comment on attachment 230851 [details]
Patch

Clearing flags on attachment: 230851

Committed r168350: <http://trac.webkit.org/changeset/168350>
Comment 12 WebKit Commit Bot 2014-05-05 23:07:14 PDT
All reviewed patches have been landed.  Closing bug.
Comment 13 Nikolas Zimmermann 2014-05-05 23:58:39 PDT
Wonderful patch :-) Finally that weird percentage intrinsic sizing is gone!
Comment 14 Simon Fraser (smfr) 2014-05-06 10:35:31 PDT
Does this fix the weird spacing around the images at http://dbaron.org/log/20110225-blur-radius ?
Comment 15 Dirk Schulze 2014-05-06 10:37:12 PDT
(In reply to comment #14)
> Does this fix the weird spacing around the images at http://dbaron.org/log/20110225-blur-radius ?

Yes. Indeed it does.
Comment 16 Alexey Proskuryakov 2014-05-08 22:55:50 PDT
This patch removed results for svg/text/non-bmp-positioning-lists.svg. Could you please re-land them?

http://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=svg%2Ftext%2Fnon-bmp-positioning-lists.svg
Comment 17 Alexey Proskuryakov 2014-05-08 23:17:15 PDT
Also, svg/as-object/sizing/svg-in-object-placeholder-height-percentage.html is flaky on some bots, and fails 100% of time on others.
Comment 19 Dirk Schulze 2014-05-09 00:30:40 PDT
Can't produce the svg-object test errors locally.

What does

Harness Error. harness_status.status = 2 , harness_status.message = null

mean? How can that be caused?
Comment 20 Dirk Schulze 2014-05-09 00:33:58 PDT
(In reply to comment #18)
> Actually, even more:
> 
> svg/as-object/sizing/svg-in-object-placeholder-height-fixed.html
> svg/as-object/sizing/svg-in-object-placeholder-height-percentage.html
> svg/as-object/sizing/svg-in-object-placeholder-height-auto.html
> 
> 
> http://build.webkit.org/results/Apple%20MountainLion%20(Leaks)/r168513%20(10394)/svg/as-object/sizing/svg-in-object-placeholder-height-fixed-diff.txt
> 
> http://build.webkit.org/results/Apple%20MountainLion%20(Leaks)/r168513%20(10394)/svg/as-object/sizing/svg-in-object-placeholder-height-percentage-diff.txt
> 
> http://build.webkit.org/results/Apple%20MountainLion%20(Leaks)/r168513%20(10394)/svg/as-object/sizing/svg-in-object-placeholder-height-auto-diff.txt

The tests need some time for running. Could these be timeout errors?
Comment 21 Dirk Schulze 2014-05-09 00:35:06 PDT
(In reply to comment #20)
> (In reply to comment #18)
> > Actually, even more:
> > 
> > svg/as-object/sizing/svg-in-object-placeholder-height-fixed.html
> > svg/as-object/sizing/svg-in-object-placeholder-height-percentage.html
> > svg/as-object/sizing/svg-in-object-placeholder-height-auto.html
> > 
> > 
> > http://build.webkit.org/results/Apple%20MountainLion%20(Leaks)/r168513%20(10394)/svg/as-object/sizing/svg-in-object-placeholder-height-fixed-diff.txt
> > 
> > http://build.webkit.org/results/Apple%20MountainLion%20(Leaks)/r168513%20(10394)/svg/as-object/sizing/svg-in-object-placeholder-height-percentage-diff.txt
> > 
> > http://build.webkit.org/results/Apple%20MountainLion%20(Leaks)/r168513%20(10394)/svg/as-object/sizing/svg-in-object-placeholder-height-auto-diff.txt
> 
> The tests need some time for running. Could these be timeout errors?

Oh found it: status = 2

means         status_text[Test.prototype.TIMEOUT] = "Timeout";

How can I give the tests more time for completion?
Comment 22 Alexey Proskuryakov 2014-05-09 10:56:19 PDT
I don't think that there is a way, the limit of 30 seconds is quite high already. We do not really want tests that take over 30 seconds to run each!
Comment 23 Alexey Proskuryakov 2014-05-10 17:12:13 PDT
(In reply to comment #16)
> This patch removed results for svg/text/non-bmp-positioning-lists.svg. Could you please re-land them?

Someone landed results in <http://trac.webkit.org/changeset/168543>, but looks like it happened by accident. Are the landed results correct?

> Actually, even more:

Filed bug 132791, and skipped the tests.

Dirk, please be more responsive when you make bots red. You knew about the failures for two days now.
Comment 24 Dirk Schulze 2014-05-11 08:30:20 PDT
(In reply to comment #23)
> (In reply to comment #16)
> > This patch removed results for svg/text/non-bmp-positioning-lists.svg. Could you please re-land them?
> 
> Someone landed results in <http://trac.webkit.org/changeset/168543>, but looks like it happened by accident. Are the landed results correct?
> 
> > Actually, even more:
> 
> Filed bug 132791, and skipped the tests.
> 
> Dirk, please be more responsive when you make bots red. You knew about the failures for two days now.

I apologize. I try to be more responsive. Needed to work out CSS Filters and device pixel ratio issues the last days. Will take care of the bug tomorrow and the day after.
Comment 25 zalan 2014-06-15 21:13:33 PDT
This regressed bug 133933