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

Issue 778125 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression

Blocking:
issue 769402



Sign in to add a comment

Regression: Chrome://extensions Page gets blank after reloading it.

Reported by db...@etouch.net, Oct 25 2017

Issue description

Chrome Version: 63.0.3239.18 Revision 975787a36cf8dbe16eaf42134aaee20fe9504ba2-refs/branch-heads/3239@{#188}(32/64 bit)
OS: Windows(7,8,10), Mac(10.12.6, 10.13.1) and Linux(14.04 LTS).

Pre-condition: Enabled 'Enable Material Design extensions' flag from chrome://flags

Test URL: https://chrome.google.com/webstore/detail/momentum/laookkfknpbbblfpciffpaejjkokdgca/related?hl=en

What steps will reproduce the problem?
(1) Launch chrome, navigate to above URL and click on ADD TO CHROME button.
(2) Right click on Added extension icon and select Manage extension option(navigated to chrome://extensions page)
(3) Again right click on Added extension icon and select 'Remove from chrome' option
(4) Click on back navigation of chrome://extension page and then click on back navigation near omnibox, Reload page and observe.

Actual: Chrome://extensions Page gets blank after reloading page.

Expected: Chrome://extensions Page should not gets blank after reloading page.

This is a regression issue, broken in 'M-62', below is bisect info:

Good Build: 62.0.3180.0
Bad Build: 62.0.3181.0

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

CHANGELOG URL:

The script might not always return single CL as suspect as some perf builds might get missing due to failure.

https://chromium.googlesource.com/chromium/src/+log/7a6f47e3a5e77e77fcd6334285dcb942db0916d2..52d7ff244f41063ccbbb9e8adf73186bbf457d87

Suspect: https://chromium.googlesource.com/chromium/src/+/52d7ff244f41063ccbbb9e8adf73186bbf457d87
 
Actual_Extensions.mov
8.7 MB Download
Expected_Extensions.mov
2.1 MB Download
The step that causes the problem is visiting an extensions URL with non-existent/removed ID (via refresh or prev. page button), it should redirect to the root page.
Owner: dpa...@chromium.org

Comment 3 by dpa...@chromium.org, Oct 26 2017

Components: -Platform>Extensions UI>Browser>ExtensionsManagement

Comment 4 by dpa...@chromium.org, Oct 28 2017

Blocking: 769402

Comment 5 by dpa...@chromium.org, Oct 30 2017

Owner: ----
Status: Available (was: Assigned)
Marking as Available since I am still finishing up some performance related work.
Owner: dschuyler@chromium.org
Status: Assigned (was: Available)
It's not a big deal, but the Expected_Extensions.mov above doesn't look related to the bug. The bug is clear enough without the expected.mov though, so np.
Status: Started (was: Assigned)
CL at https://chromium-review.googlesource.com/c/chromium/src/+/749114
Project Member

Comment 9 by bugdroid1@chromium.org, Nov 4 2017

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

commit d12aa0cfc5a466ab51f6d1ee866821a0c42b1cf5
Author: Dave Schuyler <dschuyler@chromium.org>
Date: Fri Nov 03 23:46:42 2017

[MD extensions] navigate to extensions list if ID not found

This CL changes what happens when navigating to the details page for an
app or extension that is not installed (i.e. maybe recently removed and
the url is still in the history). With this CL, the user will arrive at
the main extensions list page, rather than seeing a (mostly) blank page.

Bug:  778125 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I7e017042f6ecab94d4dbfe3219d8d198992a40b0
Reviewed-on: https://chromium-review.googlesource.com/749114
Commit-Queue: Dave Schuyler <dschuyler@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#513964}
[modify] https://crrev.com/d12aa0cfc5a466ab51f6d1ee866821a0c42b1cf5/chrome/browser/resources/md_extensions/manager.js
[modify] https://crrev.com/d12aa0cfc5a466ab51f6d1ee866821a0c42b1cf5/chrome/test/data/webui/extensions/extension_manager_test.js

Status: Fixed (was: Started)

Sign in to add a comment