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

Issue 621344 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocking:
issue 614758



Sign in to add a comment

enable-md-settings doesn't work with enable-settings-window

Project Member Reported by michae...@chromium.org, Jun 19 2016

Issue description

Version: 53.0.2772.0 

With both --enable-settings-window (as on Chrome OS) and --enable-features=MaterialDesignSetting from chrome://flags, Settings (even chrome://settings) loads the old options page.
 
Blocking: 614758
Labels: Hotlist-MD-Settings-General
Cc: dbeam@chromium.org michae...@chromium.org
Owner: ----
Didn't mean to give this an owner yet, one of us can take it on Monday
Status: Available (was: Untriaged)
Owner: michae...@chromium.org
Status: Started (was: Available)
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 21 2016

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

commit 8e75cccd2ff5224bdd82662d47af49a26e247d39
Author: michaelpg <michaelpg@chromium.org>
Date: Tue Jun 21 20:26:37 2016

Fix Material Settings Help and Settings redirect

"About in Settings" shouldn't affect loading of MD Settings or
prevent Help from loading at all.

On Chrome OS, chrome://settings wasn't redirecting; on other
platforms, chrome://help wasn't loading.

BUG= 621390 , 621344 

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

[modify] https://crrev.com/8e75cccd2ff5224bdd82662d47af49a26e247d39/chrome/browser/browser_about_handler.cc
[modify] https://crrev.com/8e75cccd2ff5224bdd82662d47af49a26e247d39/chrome/browser/browser_about_handler_unittest.cc
[modify] https://crrev.com/8e75cccd2ff5224bdd82662d47af49a26e247d39/chrome/browser/ui/webui/settings/md_settings_ui.cc

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
Bulk verified

Sign in to add a comment