Commit a94fcea2 authored by Nasko Oskov's avatar Nasko Oskov Committed by Commit Bot

Remove unused ContentBrowserClient::IsURLAcceptableForWebUI API

This API is no longer used and the bug it references is closed.

Bug: 768526
Change-Id: I6570ce4531c7cccd47ab21e26ad7d570d10e6c8a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2091939
Commit-Queue: Nasko Oskov <nasko@chromium.org>
Reviewed-by: default avatardanakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748355}
parent d29b393e
...@@ -99,17 +99,11 @@ bool WebUIControllerFactoryRegistry::IsURLAcceptableForWebUI( ...@@ -99,17 +99,11 @@ bool WebUIControllerFactoryRegistry::IsURLAcceptableForWebUI(
// See http://crbug.com/42547 // See http://crbug.com/42547
url.spec() == url::kAboutBlankURL || url.spec() == url::kAboutBlankURL ||
// javascript: and debug URLs like chrome://kill are allowed. // javascript: and debug URLs like chrome://kill are allowed.
IsRendererDebugURL(url) || IsRendererDebugURL(url);
// Temporarily allow the embedder to whitelist URLs allowed in WebUI
// until crbug.com/768526 is resolved.
GetContentClient()->browser()->IsURLAcceptableForWebUI(browser_context,
url);
} }
WebUIControllerFactoryRegistry::WebUIControllerFactoryRegistry() { WebUIControllerFactoryRegistry::WebUIControllerFactoryRegistry() = default;
}
WebUIControllerFactoryRegistry::~WebUIControllerFactoryRegistry() { WebUIControllerFactoryRegistry::~WebUIControllerFactoryRegistry() = default;
}
} // namespace content } // namespace content
...@@ -184,13 +184,6 @@ bool ContentBrowserClient::CanCommitURL(RenderProcessHost* process_host, ...@@ -184,13 +184,6 @@ bool ContentBrowserClient::CanCommitURL(RenderProcessHost* process_host,
return true; return true;
} }
bool ContentBrowserClient::IsURLAcceptableForWebUI(
BrowserContext* browser_context,
const GURL& url) {
DCHECK(browser_context);
return false;
}
bool ContentBrowserClient::ShouldStayInParentProcessForNTP( bool ContentBrowserClient::ShouldStayInParentProcessForNTP(
const GURL& url, const GURL& url,
SiteInstance* parent_site_instance) { SiteInstance* parent_site_instance) {
......
...@@ -430,12 +430,6 @@ class CONTENT_EXPORT ContentBrowserClient { ...@@ -430,12 +430,6 @@ class CONTENT_EXPORT ContentBrowserClient {
// This is called on the UI thread. // This is called on the UI thread.
virtual bool CanCommitURL(RenderProcessHost* process_host, const GURL& url); virtual bool CanCommitURL(RenderProcessHost* process_host, const GURL& url);
// Returns whether a URL can be displayed within a WebUI for a given
// BrowserContext. Temporary workaround while crbug.com/768526 is resolved.
// Note: This is used by an internal Cast implementation of this class.
virtual bool IsURLAcceptableForWebUI(BrowserContext* browser_context,
const GURL& url);
// Allows the embedder to override parameters when navigating. Called for both // Allows the embedder to override parameters when navigating. Called for both
// opening new URLs and when transferring URLs across processes. // opening new URLs and when transferring URLs across processes.
virtual void OverrideNavigationParams( virtual void OverrideNavigationParams(
......
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