Commit 9df334c1 authored by estade@chromium.org's avatar estade@chromium.org

revert r71405 as it is causing ui test failures:

Make new chrome UI security model the default.

allows us to update URL in chrome://settings when the user navigates within it.

BUG=57485
TEST=manual

Review URL: http://codereview.chromium.org/6277018

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72420 0039d316-1c4b-4281-b951-d872f2087c98
parent 42091493
...@@ -721,6 +721,7 @@ void BrowserRenderProcessHost::PropagateBrowserCommandLineToRenderer( ...@@ -721,6 +721,7 @@ void BrowserRenderProcessHost::PropagateBrowserCommandLineToRenderer(
switches::kRemoteShellPort, switches::kRemoteShellPort,
switches::kEnablePepperTesting, switches::kEnablePepperTesting,
switches::kAllowOutdatedPlugins, switches::kAllowOutdatedPlugins,
switches::kNewChromeUISecurityModel,
switches::kEnableRemoting, switches::kEnableRemoting,
switches::kEnableClickToPlay, switches::kEnableClickToPlay,
switches::kEnableResourceContentSettings, switches::kEnableResourceContentSettings,
......
...@@ -499,8 +499,7 @@ cr.define('options', function() { ...@@ -499,8 +499,7 @@ cr.define('options', function() {
OptionsPage.updateManagedBannerVisibility(); OptionsPage.updateManagedBannerVisibility();
// Recent webkit change no longer allows url change from "chrome://". // Recent webkit change no longer allows url change from "chrome://".
window.history.pushState({pageName: this.name}, this.title, window.history.pushState({pageName: this.name}, this.title);
'/' + this.name);
} }
if (this.tab) { if (this.tab) {
this.tab.classList.add('navbar-item-selected'); this.tab.classList.add('navbar-item-selected');
......
...@@ -850,6 +850,9 @@ const char kNaClLoaderProcess[] = "nacl-loader"; ...@@ -850,6 +850,9 @@ const char kNaClLoaderProcess[] = "nacl-loader";
// Causes the Native Client process to display a dialog on launch. // Causes the Native Client process to display a dialog on launch.
const char kNaClStartupDialog[] = "nacl-startup-dialog"; const char kNaClStartupDialog[] = "nacl-startup-dialog";
// Enables the new security model for "chrome" URLs.
const char kNewChromeUISecurityModel[] = "new-chrome-ui-security-model";
// Disables the default browser check. Useful for UI/browser tests where we // Disables the default browser check. Useful for UI/browser tests where we
// want to avoid having the default browser info-bar displayed. // want to avoid having the default browser info-bar displayed.
const char kNoDefaultBrowserCheck[] = "no-default-browser-check"; const char kNoDefaultBrowserCheck[] = "no-default-browser-check";
......
...@@ -244,6 +244,7 @@ extern const char kNaClDebugPorts[]; ...@@ -244,6 +244,7 @@ extern const char kNaClDebugPorts[];
extern const char kNaClBrokerProcess[]; extern const char kNaClBrokerProcess[];
extern const char kNaClLoaderProcess[]; extern const char kNaClLoaderProcess[];
extern const char kNaClStartupDialog[]; extern const char kNaClStartupDialog[];
extern const char kNewChromeUISecurityModel[];
extern const char kNoDefaultBrowserCheck[]; extern const char kNoDefaultBrowserCheck[];
extern const char kNoEvents[]; extern const char kNoEvents[];
extern const char kNoExperiments[]; extern const char kNoExperiments[];
......
...@@ -864,7 +864,12 @@ void RenderThread::EnsureWebKitInitialized() { ...@@ -864,7 +864,12 @@ void RenderThread::EnsureWebKitInitialized() {
// also be unable to script anything but themselves (to help limit the damage // also be unable to script anything but themselves (to help limit the damage
// that a corrupt chrome: page could cause). // that a corrupt chrome: page could cause).
WebString chrome_ui_scheme(ASCIIToUTF16(chrome::kChromeUIScheme)); WebString chrome_ui_scheme(ASCIIToUTF16(chrome::kChromeUIScheme));
WebSecurityPolicy::registerURLSchemeAsDisplayIsolated(chrome_ui_scheme); if (command_line.HasSwitch(switches::kNewChromeUISecurityModel)) {
WebSecurityPolicy::registerURLSchemeAsDisplayIsolated(chrome_ui_scheme);
} else {
WebSecurityPolicy::registerURLSchemeAsLocal(chrome_ui_scheme);
WebSecurityPolicy::registerURLSchemeAsNoAccess(chrome_ui_scheme);
}
// chrome-extension: resources shouldn't trigger insecure content warnings. // chrome-extension: resources shouldn't trigger insecure content warnings.
WebString extension_scheme(ASCIIToUTF16(chrome::kExtensionScheme)); WebString extension_scheme(ASCIIToUTF16(chrome::kExtensionScheme));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment