iframes don't show when in 3D cs transforms
Reported by
amir.co...@emaze.com,
Dec 5 2016
|
|||||||||||||||||||||||||
Issue descriptionChrome Version: 57.0.2937.0 canary (64-bit) on Windows 10 <b>Chrome OS Version: <From about:version: Platform x.x.x.x></b> <b>Chrome OS Platform: <Make/model of computer running Chrome OS></b> <b>Network info: <network, encryption type, router model (if known)></b> Please specify Cr-* of the system to which this bug/feature applies (add the label below). Steps To Reproduce: Open link: http://app.emazestaging.com/@AIZZQR#1 It is supposed to open 2 iframes but they don't open, they do open on Chrome version Version 54.0.2840.99 m Seems to be related to 3D transforms in which the iframes appear in (2) (3) Expected Result: Actual Result: How frequently does this problem reproduce? (Always, sometimes, hard to reproduce?) Always What is the impact to the user, and is there a workaround? If so, what is it? Please provide any additional information below. Attach a screen shot or log if possible. For graphics-related bugs, please copy/paste the contents of the about:gpu page at the end of this report.
,
Dec 9 2016
,
Dec 12 2016
I can't reproduce in 57.0.2949.0 canary on Win10 nor 56.0.2924.21 beta. I'm seeing 2 frames, one with video (that plays when clicked) and another with a social media page that opens when clicked. Is that what we're supposed to see? If not, please provide a screen shot to help us understand the problem.
,
Dec 13 2016
,
Dec 13 2016
Hi, I must say it also does not reproduce for me now. Initially it reproduced for me on version 57.0.2937.0 ,now on version 57.0.2949.0 it is fine, I also rebooted since if that has any relevance. Regards, Amir
,
Dec 19 2016
Hi, Happens again here: Link: http://emaze.me/summer#2 Version 57.0.2956.0 canary (64-bit) There is an iframe in the middle of the page that is not showing. It does show as expected on: Version 55.0.2883.87 m Thank You
,
Dec 19 2016
Which iframe? What is the DOM id of it in this example?
,
Dec 21 2016
It does not have an id. Its tag look like this: <iframe src="http://resources.emazestaging.com/widgets/instafeed/index.html?access_token=340900258.dc1ad33.b8015c9c107446d2b161edf0b108b8c0&imgboxsize=400&pad=12" data-src="http://resources.emazestaging.com/widgets/instafeed/index.html?access_token=340900258.dc1ad33.b8015c9c107446d2b161edf0b108b8c0&imgboxsize=400&pad=12" class="sd-element-embed sd-embedded sd-video-border-color_1" style="width: 100%; height: 100%;"></iframe>
,
Dec 28 2016
Thank you for providing more feedback. Adding requester "schenney@chromium.org" for another review and adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 29 2016
On Windows 10, Version 57.0.2966.1 canary SyzyASan, I see the attached screen for the page in comment #6. There are a couple of console errors related to same-origin rules that might be affecting the results. I'm not seeing that iframe src anywhere in the DevTools resource listing for the page or the network traffic on reload. Could you verify the screen shot and also check locally for console errors?
,
Jan 8 2017
I attached screenshots with devtools showing the iframe exists in the page. iframe2.JPG was taken on Chrome version 55.0.2883.87 iframe.JPG was taken on canary Version 57.0.2976.0 canary (64-bit) You can see in Chrome version 55.0.2883.87, the iframe shows (the image boxes in the middle of the page are from the iframe). In canary Version 57.0.2976.0, the iframe is not visible - and that is the problem
,
Jan 8 2017
The console errors are not nice but I do see how it is related, and they also appear on the Chrome version 55.0.2883.87, and there the iframe shows well
,
Jan 8 2017
Bisect using snapshots and http://emaze.me/summer#2 from #6: 429040 (good) - 429051 (bad), 56.0.2907.0 https://chromium.googlesource.com/chromium/src/+log/9c860c89..fbebed78?pretty=fuller
,
Jan 10 2017
https://chromium.googlesource.com/chromium/src/+/5e48a2e0284b930c68e73f44ea15807a9ffab541 is in that bisect range. -> ajuma.
,
Jan 10 2017
That patch was reverted very shortly thereafter in r429227 (and hasn't yet relanded) so that can't be it. Looking at the regression range, https://chromium.googlesource.com/chromium/src/+/3c364c812da0312a104e976ca5838507efe3488e ("Use intersection observer to control frame throttling") seems like the only iframe-related change, so passing to skyostil.
,
Jan 17 2017
Thank you for providing more feedback. Adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jan 17 2017
,
Jan 18 2017
,
Jan 18 2017
,
Jan 20 2017
Able to reproduce this issue on Mac 10.12.2, Win-10 and Ubuntu 14.04 using latest canary #57.0.2986.0. skyostil@ - Could you please have a look into this issue. Thanks...!!
,
Jan 20 2017
We're a few days away from the first stable cut for M56, moving back to M57 since there isn't a fix yet and were only accepting high priority merges (security/crashes).
,
Jan 21 2017
,
Jan 27 2017
With respect to the label added: Needs-triage-Mobile, triaged the issue and below are the observations. Able to reproduce the issue on Mobile device (Tablet and Handset) with the URL "http://emaze.me/summer#2" provided. Devices Used: ============= Device Nexus 7 Tablet Chrome Version: 57.0.2987.9 Android version: 5.0.2/LRX22G Device: HTC 4 Chrome Version: 58.0.2993.0 Android version: 6.0.1/MMB29M Observations on Mobile Device: =============================== 1) Loaded the URL "http://emaze.me/summer#2" 2) Observe the webpage 3) The iframe was not displayed on both handset and tablet. P.S: Issue was also seen when option "request desktop site" was also checked
,
Jan 27 2017
I dug into this and looks like the problem is that IntersectionObserver (well, really LayoutBox::mapToVisualRectInAncestorSpace) computes bogus intersection rectangles when 3D projections are involved. I took a quick stab at fixing this in two ways: 1. By using mapLocalToAncestor() -- but this doesn't take clips into account so it doesn't really help us. 2. By using clampedBoundsOfProjectedQuad() -- but it also somehow ended up in outer space, not sure why. It's as if it's missing the final camera transform. I'm tempted to first figure out a way to disable throttling if 3D projections are involved and then see if we can find a way to make everything work. WDYT?
,
Jan 27 2017
cc chrishtr@, who knows about mapLocalToAncestor
,
Jan 27 2017
Is this a dup of issue 590095 ?
,
Jan 27 2017
#26: partially -- I think we need to resolve the issue with throttling before fixing mapToVisibleRectInAncestorSpace.
,
Jan 27 2017
Which issue is the "issue with throttling"?
,
Jan 27 2017
Ah, sorry for being obscure. In M57 we switched frame throttling to determine visibility using IntersectionObserver. Previously I believe it worked with 3D transforms, but because IObs doesn't this has regressed. I'm proposing to opt 3D transformed frames out of throttling to work around this for now.
,
Jan 27 2017
I see. But just fixing 590095 would also fix the whole issue, right? It seems better to do that if we can get it done in time.
,
Jan 27 2017
That's true. I just wasn't sure how difficult it is going to be :)
,
Jan 27 2017
Do you have a reduced testcase to help with fixing 590095? I don't think it will be very hard to fix with a good testcase.
,
Jan 27 2017
,
Jan 31 2017
It's not really reduced, but the one in this bug isn't too bad. It's got three iframes which have perspective transforms, and if you add some logging to IntersectionGeometry you'll see the computed rects are bogus.
,
Feb 2 2017
amir.cogan@ : Could you please update the thread as per the above comments # 32 & 34 to help further.
,
Feb 8 2017
A friendly reminder that M57 Stable is launch is coming soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!
,
Feb 16 2017
A friendly reminder that M57 Stable is launch is coming VERY soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch (2987) ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!
,
Feb 21 2017
Gentle ping..! amir.cogan@: Could you please update further with your findings as per the above comments # 32 and 34.
,
Feb 21 2017
I don't think we need any more details here. We just need to fix the blocking bug 590095 .
,
Feb 22 2017
URGENT - PTAL ASAP. We're getting VERY close to M57 Stable promotion. And this issue is marked as M57 stable release blocker. Pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Know that this issue shouldn't block the release? Remove the ReleaseBlock-Stable label or move to M58. Thank you.
,
Feb 22 2017
Hi, From my end, I don't have a reduced testcase. I noticed now on Chrome 56.0.2924.87 it also is not showing the iframe properly. Regards,
,
Feb 23 2017
Given this bug is also in M56, perhaps we don't need to block M57 on it?
,
Feb 23 2017
Per comment #21 and #42, this bug exists on M56 and fix for blocking bug 590095 isn't ready yet. So punting to M58.
,
Mar 1 2017
Just to update! Issue is still seen on Mac 10.12.3 with chrome version 58.0.3026.0 skyostil@ Could you please take a look at this issue Thanks.
,
Mar 1 2017
Hi, We got lately hundreds of complaints on this issue, We have to direct all users to Firefox and Edge. When will this be fixed? Thanks
,
Mar 6 2017
skyostil@ Friendly ping to get an update on this. Thanks.
,
Mar 6 2017
,
Mar 7 2017
Verified that the fix for issue 590095 fixes this bug. Please comment or open a new bug if you still have an issue.
,
Mar 7 2017
[Auto-generated comment by a script] We noticed that this issue is targeted for M-58; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-58 label, otherwise remove Merge-TBD label. Thanks.
,
Mar 9 2017
This was merged to M58 here: https://bugs.chromium.org/p/chromium/issues/detail?id=590095#c18 Removing Merge-TBD. |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by amir.co...@emaze.com
, Dec 6 2016328 KB
328 KB View Download