Index: chrome/common/extensions/url_pattern.cc |
=================================================================== |
--- chrome/common/extensions/url_pattern.cc (revision 128756) |
+++ chrome/common/extensions/url_pattern.cc (working copy) |
@@ -97,14 +97,12 @@ |
URLPattern::URLPattern() |
: valid_schemes_(SCHEME_NONE), |
match_all_urls_(false), |
- partial_filesystem_support_hack_(false), |
match_subdomains_(false), |
port_("*") {} |
URLPattern::URLPattern(int valid_schemes) |
: valid_schemes_(valid_schemes), |
match_all_urls_(false), |
- partial_filesystem_support_hack_(false), |
match_subdomains_(false), |
port_("*") {} |
@@ -113,7 +111,6 @@ |
// appropriate when we know |pattern| is valid. |
: valid_schemes_(valid_schemes), |
match_all_urls_(false), |
- partial_filesystem_support_hack_(false), |
match_subdomains_(false), |
port_("*") { |
if (PARSE_SUCCESS != Parse(pattern)) |
@@ -298,27 +295,14 @@ |
} |
bool URLPattern::MatchesURL(const GURL& test) const { |
- const GURL* test_url = &test; |
- bool has_inner_url = test.inner_url() != NULL; |
- |
- if (partial_filesystem_support_hack_ != has_inner_url) |
+ if (!MatchesScheme(test.scheme())) |
return false; |
- if (has_inner_url) |
- test_url = test.inner_url(); |
- |
- if (!MatchesScheme(test_url->scheme())) |
- return false; |
- |
if (match_all_urls_) |
return true; |
- std::string path_for_request = test.PathForRequest(); |
- if (has_inner_url) |
- path_for_request = test_url->path() + path_for_request; |
- |
- return MatchesSecurityOriginHelper(*test_url) && |
- MatchesPath(path_for_request); |
+ return MatchesSecurityOriginHelper(test) && |
+ MatchesPath(test.PathForRequest()); |
} |
bool URLPattern::MatchesSecurityOrigin(const GURL& test) const { |
@@ -449,10 +433,6 @@ |
DCHECK(path_.find('*') == path_.size() - 1); |
DCHECK(other.path().find('*') == other.path().size() - 1); |
- if (partial_filesystem_support_hack_ != |
- other.partial_filesystem_support_hack()) |
- return false; |
- |
if (!MatchesPath(other.path().substr(0, other.path().size() - 1)) && |
!other.MatchesPath(path_.substr(0, path_.size() - 1))) |
return false; |