New issue
Advanced search Search tips

Issue 646427 link

Starred by 7 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

embedded SVG is reloaded when css @media breakpoint changes layout

Reported by mhaec...@gmail.com, Sep 13 2016

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/601.7.8 (KHTML, like Gecko) Version/9.1.3 Safari/601.7.8

Example URL:
see attached reduction

Steps to reproduce the problem:
1. Unpack attached reproduction
2. cd $wherever_you_unpacked_it
3. python2 -m SimpleHTTPServer
4. open http://localhost:8000
5. Resize the browser window below and above 767px to trigger the bug

What is the expected behavior?
* The console should not show that unload is called again
* The gage-pointer should not disappear

What went wrong?
* The console shows that onload is called multiple times
* The gage-pointer disapears

Does it occur on multiple sites: Yes

Is it a problem with a plugin? No 

Did this work before? N/A 

Does this work in other browsers? No Safari (Version 9.1.3 (11601.7.8) and Technology Preview Edition Release 12 (Safari 9.1.2, WebKit 11603.1.3))

Chrome version: Version 51.0.2683.0 (64-bit)  Channel: n/a
OS Version: OS X 10.11.6
Flash Version: 

The problem is that the state that is in the SVG is lost. This is especially problematic if it is changed because of user interaction.

The workaround to this is to also save the state of the SVG somewhere outside of it, and then reapply it whenever onload is called on it again. This is quite a PITA.
 

Comment 1 by mhaec...@gmail.com, Sep 13 2016

Please see the attached reduction.
rerender-reproducer.zip
2.6 KB Download

Comment 2 by mhaec...@gmail.com, Sep 13 2016

Please see the webkit issue I filed: https://bugs.webkit.org/show_bug.cgi?id=161913

Comment 3 by rtoy@chromium.org, Sep 13 2016

Components: -Blink Blink>SVG
Status: Untriaged (was: Unconfirmed)

Comment 4 by f...@opera.com, Sep 14 2016

Components: -Blink>SVG Blink>HTML>Embed
Labels: -OS-Mac
Thanks for the report!

My guess would that when the MQ trigger, we need to reattach the <embed>s layout object - which loses the frame/SVG.

Not really an SVG issue as such, but probably in the  HTMLFrameOwnerElement/HTMLPluginElement/HTMLEmbedElement complex. Randomly punting to one of the HTML components for more triage.

Comment 5 by tkent@chromium.org, Sep 15 2016

Components: -Blink>HTML>Embed Blink>CSS Blink>Loader

Comment 6 by r...@opera.com, Sep 16 2016

Attached demo for embedded html file. State lost for embed and object, but not for iframe.

reattach.html
846 bytes View Download

Comment 7 by r...@opera.com, Sep 16 2016

Julien tried to fix this for  issue 240576 .

Comment 8 by sashab@chromium.org, Sep 19 2016

Owner: r...@opera.com
Status: Assigned (was: Untriaged)
Rune, could you assign this to Julien please? Sounds like they're the right person to work on this :)

Comment 9 by sashab@chromium.org, Sep 19 2016

Components: -Blink>CSS Blink>SVG
Components: -Blink>SVG

Comment 11 by r...@opera.com, Sep 23 2016

Owner: ----
Status: Available (was: Assigned)
Julien's no longer with the project.

I don't really know who should own this, but I suspect the fix will need to do something smart for LayoutObject reattachment for LayoutSVGRoot, etc. Some custom attachLayoutTree/detachLayoutTree for HTMLObjectElement, SVGSVGElement, etc.

Comment 12 by f...@opera.com, Nov 21 2016

 Issue 667121  has been merged into this issue.
Project Member

Comment 13 by sheriffbot@chromium.org, Nov 21 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Available (was: Untriaged)
Labels: -Hotlist-Recharge-Cold
Cc: dcheng@chromium.org
 Issue 902779  has been merged into this issue.

Sign in to add a comment