Commit c4d403b7 authored by Sean Harrison's avatar Sean Harrison Committed by Commit Bot

Remove unused HasExtensionScheme function

Bug: 1007223
Change-Id: I9aa1f2db14ab501c2ed45900e28b40c99a9d8cdc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2005168Reviewed-by: default avatarChristian Dullweber <dullweber@chromium.org>
Commit-Queue: Sean Harrison <harrisonsean@chromium.org>
Cr-Commit-Position: refs/heads/master@{#732789}
parent 6135f8f3
......@@ -27,8 +27,3 @@ bool BrowsingDataHelper::HasWebScheme(const GURL& origin) {
bool BrowsingDataHelper::IsExtensionScheme(const std::string& scheme) {
return scheme == extensions::kExtensionScheme;
}
// Static
bool BrowsingDataHelper::HasExtensionScheme(const GURL& origin) {
return BrowsingDataHelper::IsExtensionScheme(origin.scheme());
}
......@@ -32,7 +32,6 @@ class BrowsingDataHelper {
// Returns true iff the provided scheme is an extension.
static bool IsExtensionScheme(const std::string& scheme);
static bool HasExtensionScheme(const GURL& origin);
private:
DISALLOW_IMPLICIT_CONSTRUCTORS(BrowsingDataHelper);
......
......@@ -39,8 +39,7 @@ class BrowsingDataHelperTest : public testing::Test {
bool IsExtensionScheme(const std::string& scheme) {
GURL test(scheme + "://example.com");
return (BrowsingDataHelper::HasExtensionScheme(test) &&
BrowsingDataHelper::IsExtensionScheme(scheme));
return (BrowsingDataHelper::IsExtensionScheme(scheme));
}
private:
......
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