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

Issue 657878 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: 2016-10-27
OS: Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Page is visually blank when multiple window.open calls are called at once

Reported by cstea...@onemedical.com, Oct 20 2016

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.59 Safari/537.36

Example URL:
https://plnkr.co/edit/BLnLFF0vNxVZ2y5nUiNV?p=preview

Steps to reproduce the problem:
1. Trigger a function that calls window.open() twice, back to back
2. Let the browser open 2 new tabs
3. Observe that the 2nd tab is visually empty
4. Switching tabs and coming back to the empty tab fixes the problem

What is the expected behavior?
The content in the new tab should visually render.

What went wrong?
The 2nd tab that is opened is visually empty. Switching tabs and coming back or minimizing and restoring fixes the problem.
Moving the cursor over the page shows that the cursor style can change so it appears that the content has loaded but just won't visually show up. Resizing the browser or scrolling does not fix the issue either.

Does it occur on multiple sites: Yes

Is it a problem with a plugin? No 

Did this work before? Yes Chrome 53

Does this work in other browsers? Yes

Chrome version: 54.0.2840.59  Channel: stable
OS Version: OS X 10.12.0
Flash Version: Shockwave Flash 23.0 r0

This issue was not reproducible on a Windows machine running Chrome 54.
 
Cc: chrishtr@chromium.org ccameron@chromium.org
Components: -Blink UI>Browser
Labels: M-56 Needs-Bisect
NextAction: 2016-10-27
I can reproduce this on 56.0.2886.0 / Canary on Mac.
I also cannot reproduce on Windows or Linux. Therefore I think this is
something wrong with the window loading system outside of Blink on the
Mac platform.
Cc: brajkumar@chromium.org
Labels: -Type-Bug -Needs-Bisect hasbisect-per-revision Type-Bug-Regression
Owner: enne@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 54.0.2836.0 (Revision: 413363).
Bad build: 54.0.2838.0 (Revision: 413929).

You are probably looking for a change made after 413600 (known good), but no later than 413601 (first known bad).

CHANGE-LOG URL:
----------------
https://chromium.googlesource.com/chromium/src/+log/233d75b83ccd14e44d51cc1a314337467622a67d..e5ce61d4966237e3572f3048a6ea39c7ba7d7c64

From the CL above, assigning the issue to the concern owner 
Review-Url: https://codereview.chromium.org/2246613002

@enne- Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Project Member

Comment 3 by bugdroid1@chromium.org, Oct 26 2016

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

commit 2051d912c9c52c61045908314d288a3ecca6c6c2
Author: enne <enne@chromium.org>
Date: Wed Oct 26 22:10:50 2016

Fix mac raciness with initial needs begin frame state

Because Mac doesn't delay begin frame messages (because it needs to
receive them during resize), this creates some startup raciness.
Fix this by just having the renderer host side assume that the
renderer is going to want begin frames (even before hearing about it),
so that if the SetNeedsBeginFrames(true) message is dropped it
won't matter.

Renderers when they start up send a SetNeedsBeginFrames(true) and
then a SetNeedsBeginFrames(false) once they've finished loading and
at that point the renderer and the host will be back in agreement
about the state.

BUG= 657878 

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

[modify] https://crrev.com/2051d912c9c52c61045908314d288a3ecca6c6c2/content/browser/renderer_host/render_widget_host_view_mac.mm
[modify] https://crrev.com/2051d912c9c52c61045908314d288a3ecca6c6c2/content/browser/renderer_host/render_widget_host_view_mac_unittest.mm

Comment 4 by enne@chromium.org, Oct 26 2016

Status: Fixed (was: Assigned)
Cc: ranjitkan@chromium.org
Labels: TE-Verified-M56 TE-Verified-56.0.2902.0
Rechecked this on chrome version 56.0.2902.0 on MAC 10.12. Fix is working as intended. 

Clicking the button from the sample url provided, opens two windows and all the content are displayed properly.

Adding TE-Verified labels.

Sign in to add a comment