New issue
Advanced search Search tips

Issue 898134 link

Starred by 4 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Opening page in main frame from its children iframe does not update reload button state

Reported by mje...@opera.com, Oct 23

Issue description

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

Steps to reproduce the problem:
1. Install throbber_issue.7z
2. Go here https://pste.eu/p/Allf.html
3. Click on caption on this site.

What is the expected behavior?
Reload button changes to "stop" state

What went wrong?
Reload button does not change to "stop" state

Did this work before? N/A 

Chrome version: 70.0.3538.67  Channel: stable
OS Version: 10.0
Flash Version: 

You can also install some extension from store which appends some script on page load (e.g. synology-download-station)

Page from step 2 opens an invisible iframe after click on caption. Source site for this iframe opens https://developer.chrome.com in parent frame. This scenario imitates google search behavior for user agents other than chrome.
 
unexpected_behavior.gif
93.7 KB View Download
expected_behavior.gif
133 KB View Download
throbber_issue.7z
646 bytes Download
Issue happens when you have some extension installed which appends any script to DOM on page load. Attached throbber_issue.7z is an extension which appends empty script.
Labels: Needs-Triage-M70
Cc: viswa.karala@chromium.org
Labels: Needs-Bisect
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on chrome reported version# 70.0.3538.67 and on latest chrome# 72.0.3589.0 with steps provided in comment# 0 using Windows-10. This issue is not seen on M-60(60.0.3112.0). Will update the Bisect information and other OS behavior soon. Hence Untriaging it and adding Needs-Bisect label to it.

Thanks!
Labels: Triaged-ET
Components: -UI Blink>HTML>IFrame
Labels: -Type-Bug -Pri-2 -Needs-Bisect hasbisect-per-revision Target-70 Target-71 Target-72 M-72 FoundIn-71 FoundIn-70 FoundIn-72 RegressedIn-66 OS-Linux OS-Mac Pri-1 Type-Bug-Regression
Owner: ksakamoto@chromium.org
Status: Assigned (was: Untriaged)
Able to reproduce the issue on reported chrome version# 70.0.3538.67 and on latest chrome# 72.0.3589.0 using Windows-10, Mac 10.12.6 & Ubuntu 14.04, hence providing Bisect Info

Bisect Info:
================
Good build: 66.0.3334.0
Bad build: 66.0.3335.0

You are probably looking for a change made after 532768 (known good), but no later than 532769 (first known bad).
https://chromium.googlesource.com/chromium/src/+log/2e51d8e83f2d6bfc157b9f9411cd668051050931..3f5851c9aa393ca0f1ab9983ee0594c83a5d10b9
Change-Id: Iffc701df6997a0341839cf5b294694ffd5dc20d8
Reviewed-on: https://chromium-review.googlesource.com/887999

@Kunihiko Sakamoto: Please confirm the issue and help in re-assigning if it is not related to your change.

Thanks!
Cc: ksakamoto@chromium.org
Components: Blink>Loader
Labels: -Pri-1 -Type-Bug-Regression -RegressedIn-66 Pri-2 Type-Bug
Owner: ----
Status: Untriaged (was: Assigned)
I was able to reproduce this without using any extension.

Steps to reproduce the problem:
1. Go here https://pste.eu/p/xBs3.html
2. Click on caption on this site.

What is the expected behavior?
Reload button changes to "stop" state

What went wrong?
Reload button does not change to "stop" state


This reproduced on Linux build at position 532207 (== 66.0.3334.0), so my change did not introduce this bug but made it a bit more visible.

Removing the regression labels, and adding Blink>Loader for further triage.

The pste.eu page has gone, so attaching the repro.
Open index.html and click on the caption.
index.html
456 bytes View Download
frame.html
300 bytes View Download
Components: -Blink>HTML>IFrame

Comment 9 by horo@chromium.org, Jan 18 (5 days ago)

Cc: horo@chromium.org
Owner: est...@chromium.org
Status: Assigned (was: Untriaged)
I bisected on Linux:
Good:  486847
Bad: 486855
You are probably looking for a change made after 486847 (known good), but no later than 486855 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/defd8d7a39a0b275f3d7924f89e48e55563314c3..661ee9093d464215427009be3b18407ebf7262b4

661ee90 [subresource_filter] Don't return a filter if it is in an invalid state by Charles Harrison · 1 year, 6 months ago
c3b2602 Change meaning of "IsLoadingToDifferentDocument" in WebContents to only by Evan Stade · 1 year, 6 months ago
3543420 Speech recognition: BindOnce refactoring by Mark Pilgrim · 1 year, 6 months ago
57cb4a5 Implement HitTestAggregator by gklassen · 1 year, 6 months ago
1db86a0 Add a //services/viz/README.md describing the viz directory structure by Fady Samuel · 1 year, 6 months ago
99d0978 Roll src/third_party/skia/ fc4ee229a..8abb9f453 (1 commit) by skia-deps-roller@chromium.org · 1 year, 6 months ago
c29dde9a Use constexpr to avoid kInvalidSequenceNum duplicates by Bruce Dawson · 1 year, 6 months ago
a85699a [subresource_filter] Refactor unit test suite into separate test harness by Charles Harrison · 1 year, 6 months ago

I think c3b2602 is related to this issue.

estade@
Could you please handle this issue?

Sign in to add a comment