New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 755875 link

Starred by 4 users

Issue metadata

Status: WontFix
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

iframe's contentWindow resize event is not fired when iframe is inserted into DOM

Reported by parthivs...@gmail.com, Aug 16 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3112.90 Safari/537.36

Steps to reproduce the problem:
1. Open demo.html
2. Click on Click Here Button
3. Observe resize event is not fired after iframe is inserted into DOM

What is the expected behavior?
After clicking on Button, alert should appear with resize fired.

What went wrong?
Iframe resize event should fire when iframe is inserted into DOM

Did this work before? Yes 59.0.3071.86

Does this work in other browsers? Yes

Chrome version: 60.0.3112.90  Channel: stable
OS Version: 10.0
Flash Version:
 
demo.html
675 bytes View Download
demo.html
675 bytes View Download

Comment 1 by woxxom@gmail.com, Aug 16 2017

Bisect info: 471270 (good) - 471275 (bad)
https://chromium.googlesource.com/chromium/src/+log/e2e34549..54a03d79?pretty=fuller
Suspecting r471275 "Update WebView/FrameView size from LayoutView::UpdateAfterLayout"
Landed in 60.0.3098.0
Cc: susanjuniab@chromium.org manoranj...@chromium.org bustamante@chromium.org
Labels: -Pri-2 ReleaseBlock-Stable Needs-Triage-M60 hasbisect OS-Linux OS-Mac Pri-1
Owner: szager@chromium.org
Status: Assigned (was: Unconfirmed)
woxxom@ - Thanks for your feedback...!!

Able to reproduce the issue on Windows 7, Ubuntu 14.04 and Mac 10.12.6 using chrome stable version #60.0.3112.101 and latest canary #62.0.3189.0.

Bisect Information:
=====================
Good build: 60.0.3097.0	
Bad Build : 60.0.3098.0

As per comment #1, Change Log URL is: 
https://chromium.googlesource.com/chromium/src/+log/e2e34549..54a03d79?pretty=fuller

From the above change log suspecting below change
Review URL: https://codereview.chromium.org/2860433002

szager@ :Could you please check whether this issue is related with to your change, if not please help us in assigning it to the right owner. 

Adding 'Stable blocker' to this issue as this was regressed in M60 and for review if we plan any other stable refresh. 

Thanks..
Project Member

Comment 3 by sheriffbot@chromium.org, Aug 21 2017

This issue is marked as a release blocker with no milestone associated. Please add an appropriate milestone.

All release blocking issues should have milestones associated to it, so that the issue can tracked and the fixes can be pushed promptly.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 4 by sheriffbot@chromium.org, Aug 24 2017

This issue is marked as a release blocker with no milestone associated. Please add an appropriate milestone.

All release blocking issues should have milestones associated to it, so that the issue can tracked and the fixes can be pushed promptly.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: WontFix (was: Assigned)
As I understand it, the resize event should not fire when an iframe is inserted into DOM.  It should only fire if the size is later changed.  I believe the previous behavior, where the resize event was triggered, was actually broken.

I'm marking this WONTFIX, but please reopen it if you believe this behavior is wrong.

Sign in to add a comment