Regression: Chrome://settings page loads forever on back and forward navigation.
Reported by
rk...@etouch.net,
Jul 12 2016
|
||||
Issue descriptionChrome Version: 54.0.2794.0 Revision 02e8e6a146a2a17dcd8950b783130d5937ce87dd-refs/heads/master@{#404734} OS: Windows(7,8,10), Mac (10.10.5)(10.11.4), Linux(Ubuntu 14.04 LTS) What steps will reproduce the problem? (1) Launch chrome,open NTP and navigate to chrome://settings (2) Open dev tools,click on 'Toggle device mode' icon and then open source code of page by hitting Ctrl+U (3) Switch back to previous page(i.e.chrome://settings), click on back navigation button then forward navigation button. (4) Observe the chrome://settings page Chrome://settings page doesnot load after step 3 Chrome://settings page should be load. This is a regression issue, broken in 'M-54', below is bisect info: Good Build: 54.0.2793.0 Bad Build: 54.0.2794.0 Narrow Bisect: https://chromium.googlesource.com/chromium/src/+log/850dda6b73575be3e6e34ed9da82263f6cc1d5f2..38cb11390a316057b1990c5257a9379d6807da84?pretty=fuller&n=100 Suspecting: r404588
,
Jul 12 2016
Repro without devtools and viewsource: 1. open about:blank 2. open chrome://settings 3. duplicate chrome://settings tab from tab context menu 4. go back to chrome://settings tab, press back, press forward
,
Jul 12 2016
I was not able to reproduce with comment#2 instructions, was able to reproduce with #1 instructions (see screenshot for dev console errors).
,
Jul 12 2016
Aleksey is working on a fix.
,
Jul 14 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3ac9439a02b4185916378572f343f1870f71fc4b commit 3ac9439a02b4185916378572f343f1870f71fc4b Author: kozyatinskiy <kozyatinskiy@chromium.org> Date: Thu Jul 14 00:07:47 2016 [DevTools] Fix window object init on chrome://* pages BUG= 627407 R=dgozman@chromium.org Review-Url: https://codereview.chromium.org/2151573002 Cr-Commit-Position: refs/heads/master@{#405358} [modify] https://crrev.com/3ac9439a02b4185916378572f343f1870f71fc4b/chrome/browser/devtools/devtools_sanity_browsertest.cc [modify] https://crrev.com/3ac9439a02b4185916378572f343f1870f71fc4b/third_party/WebKit/Source/bindings/core/v8/ScriptController.cpp [modify] https://crrev.com/3ac9439a02b4185916378572f343f1870f71fc4b/third_party/WebKit/Source/core/loader/FrameLoader.cpp [modify] https://crrev.com/3ac9439a02b4185916378572f343f1870f71fc4b/third_party/WebKit/Source/devtools/front_end/Tests.js
,
Jul 14 2016
|
||||
►
Sign in to add a comment |
||||
Comment 1 by msrchandra@chromium.org
, Jul 12 2016