Opening page in main frame from its children iframe does not update reload button state
Reported by
mje...@opera.com,
Oct 23
|
||||||||
Issue descriptionUserAgent: 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.
,
Oct 23
,
Oct 24
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!
,
Oct 24
,
Oct 25
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!
,
Oct 25
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.
,
Dec 6
The pste.eu page has gone, so attaching the repro. Open index.html and click on the caption.
,
Dec 17
,
Jan 18
(5 days ago)
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 |
||||||||
Comment 1 by mje...@opera.com
, Oct 23