The 'content' URL (image URL) gives me: "You need to sign in or sign up before continuing."

Does this still reproduce when the URL is not access restricted?

I suspect this isn't actually an SVG issue, but I'll put it in that component for the time being.

Sorry, stupid mistake to refer to our secured area... It *does* happen when the content URL is not restricted. I'll update the jsfiddle with a version that doesn't use a secured area

Thanks. I'm assuming that will be https://jsfiddle.net/L78wzjxh/1/ ? (Appears to)

And that one is served as text/plain =)

I stashed it on my local webserver, but then the problem did not reproduce.

Just updated the jsfiddle with an unrestricted resource URL.

You can also check the problem at this URL: https://lrmunoz.github.io/temp/

Regards,

Luis

To clarify, the updated jsfiddle is at: https://jsfiddle.net/L78wzjxh/4/


Also, worth noting that I'm clearing the cache to test the problem

Thanks! I could reproduce using that URL (actually, both). It does seems to be a bit flaky though. Bisected:

You are probably looking for a change made after 379799 (known good), but no later than 379801 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/84f2cfba1b8749ed115a0cca316c2dca0c12d674..3c7b1342b32dd6174efa9a47a4b476c940462b9c

So I guess it's related to r379801.

What is also strange, for me opening the test page from here or from `data:text/html,<a href="https://lrmunoz.github.io/temp/">click</a>` into a new tab by middle mouse button reliably yields correctly displayed page. Refresh breaks it again. Subsequent refreshes in Canary occasionally repairs it again, approx one in thirty refreshes.

51.0.2704.106 (Official Build) m (64-bit)
53.0.2782.0 (Official Build) canary (64-bit)

Deleted: New+Tab+Test+SVG+background+icons+-+Google+Chrome_-_20160628170508.png 12.6 KB

	New+Tab+Test+SVG+background+icons+-+Google+Chrome_-_20160628170508.png 12.6 KB View Download

Deleted: Reload+Test+SVG+background+icons+-+Google+Chrome_-_20160628170520.png 12.5 KB

	Reload+Test+SVG+background+icons+-+Google+Chrome_-_20160628170520.png 12.5 KB View Download

Yes, I observed the same. I suspect the cause has some relation to loading delay/resource handling.

I suspect this is:

// TODO(davve): Find out the default object size, if any, in this context.

in LayoutImageResourceStyleImage::imageSize().

The flakiness observed probably depends on when in the painting (invalidation) system the SVG is rendered.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/da6dc5be059c4dd6a6b7f1a4ba9c1f12be3d67ae

commit da6dc5be059c4dd6a6b7f1a4ba9c1f12be3d67ae
Author: davve <davve@opera.com>
Date: Fri Jul 08 14:25:03 2016

Fallback to 300x150 instead of 0x0 size for SVG inside content()

Prior to r379801, the fallback 300x150 in SVGImage::dataChanged() was
used. Post r379801 we instead used the empty rect as fallback. Both
are probably wrong but 300x150 matches what we did previously more
closely and we are less likely to end up with an empty image.

BUG= 623528 

Review-Url: https://codereview.chromium.org/2121973002
Cr-Commit-Position: refs/heads/master@{#404374}

[add] https://crrev.com/da6dc5be059c4dd6a6b7f1a4ba9c1f12be3d67ae/third_party/WebKit/LayoutTests/http/tests/misc/resources/svg-no-store.pl
[add] https://crrev.com/da6dc5be059c4dd6a6b7f1a4ba9c1f12be3d67ae/third_party/WebKit/LayoutTests/http/tests/misc/svg-image-in-content-expected.html
[add] https://crrev.com/da6dc5be059c4dd6a6b7f1a4ba9c1f12be3d67ae/third_party/WebKit/LayoutTests/http/tests/misc/svg-image-in-content.html
[modify] https://crrev.com/da6dc5be059c4dd6a6b7f1a4ba9c1f12be3d67ae/third_party/WebKit/Source/core/layout/LayoutImageResourceStyleImage.cpp
[modify] https://crrev.com/da6dc5be059c4dd6a6b7f1a4ba9c1f12be3d67ae/third_party/WebKit/Source/core/layout/LayoutImageResourceStyleImage.h
[modify] https://crrev.com/da6dc5be059c4dd6a6b7f1a4ba9c1f12be3d67ae/third_party/WebKit/Source/core/svg/graphics/SVGImage.cpp

Is there an estimate available for when this fix will be released?

Currently the fix is only in M54 (est. release mid-October), but we should probably try for a merge to M53 (est. release early September.) 

The sooner the better. This adversely affects our UI for a large number of users across all platforms running Chrome.

Agreed, this fix is critical and is a highly visible problem for all users.

Your change meets the bar and is auto-approved for M53 (branch: 2785)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7815a45a74afc00768f304acc60272ca653bec86

commit 7815a45a74afc00768f304acc60272ca653bec86
Author: Fredrik Söderquist <fs@opera.com>
Date: Wed Jul 20 11:56:26 2016

Fallback to 300x150 instead of 0x0 size for SVG inside content()

Prior to r379801, the fallback 300x150 in SVGImage::dataChanged() was
used. Post r379801 we instead used the empty rect as fallback. Both
are probably wrong but 300x150 matches what we did previously more
closely and we are less likely to end up with an empty image.

BUG= 623528 

Review-Url: https://codereview.chromium.org/2121973002
Cr-Commit-Position: refs/heads/master@{#404374}
(cherry picked from commit da6dc5be059c4dd6a6b7f1a4ba9c1f12be3d67ae)

Review URL: https://codereview.chromium.org/2168603002 .

Cr-Commit-Position: refs/branch-heads/2785@{#243}
Cr-Branched-From: 68623971be0cfc492a2cb0427d7f478e7b214c24-refs/heads/master@{#403382}

[add] https://crrev.com/7815a45a74afc00768f304acc60272ca653bec86/third_party/WebKit/LayoutTests/http/tests/misc/resources/svg-no-store.pl
[add] https://crrev.com/7815a45a74afc00768f304acc60272ca653bec86/third_party/WebKit/LayoutTests/http/tests/misc/svg-image-in-content-expected.html
[add] https://crrev.com/7815a45a74afc00768f304acc60272ca653bec86/third_party/WebKit/LayoutTests/http/tests/misc/svg-image-in-content.html
[modify] https://crrev.com/7815a45a74afc00768f304acc60272ca653bec86/third_party/WebKit/Source/core/layout/LayoutImageResourceStyleImage.cpp
[modify] https://crrev.com/7815a45a74afc00768f304acc60272ca653bec86/third_party/WebKit/Source/core/layout/LayoutImageResourceStyleImage.h
[modify] https://crrev.com/7815a45a74afc00768f304acc60272ca653bec86/third_party/WebKit/Source/core/svg/graphics/SVGImage.cpp

Tested the issue on windows 7, Linux Ubuntu 14.04 and Mac 10.11.5 using chrome version 53.0.2785.30.Able to see the 'X' icon in label box.
Please find the attached screen shot for the same.

Adding TE-Verified label.

Thanks,

Deleted: 623528.png 81.5 KB

	623528.png 81.5 KB View Download

 Issue 634325  has been merged into this issue.