Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(89)

Unified Diff: extensions/common/extension_urls.cc

Issue 2489233003: [extensions] Remove unnecessary checks in IsSensitiveURL (Closed)
Patch Set: more DomainIs Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: extensions/common/extension_urls.cc
diff --git a/extensions/common/extension_urls.cc b/extensions/common/extension_urls.cc
index 08fb32d62305b896ed32af8b4b16379fc7ea6d97..9f4dff95fd5bd70c437794bad89b559dc62aba1d 100644
--- a/extensions/common/extension_urls.cc
+++ b/extensions/common/extension_urls.cc
@@ -81,12 +81,8 @@ GURL GetWebstoreReportAbuseUrl(const std::string& extension_id,
bool IsWebstoreUpdateUrl(const GURL& update_url) {
GURL store_url = GetWebstoreUpdateUrl();
- if (update_url == store_url) {
- return true;
- } else {
- return (update_url.host() == store_url.host() &&
- update_url.path() == store_url.path());
- }
+ return (update_url.host_piece() == store_url.host_piece() &&
+ update_url.path_piece() == store_url.path_piece());
}
bool IsBlacklistUpdateUrl(const GURL& url) {
« no previous file with comments | « extensions/browser/api/web_request/web_request_permissions_unittest.cc ('k') | extensions/test/test_extensions_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698