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

Unified Diff: chrome/browser/extensions/api/web_navigation/web_navigation_api.cc

Issue 2485253002: Remove unnecessary calls to GURL() (Closed)
Patch Set: Assert that StringPiece must always be canonicalized. Fix some constants. 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: chrome/browser/extensions/api/web_navigation/web_navigation_api.cc
diff --git a/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc b/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc
index f525af6bb57b0c714772b23faede33cc64b4fad4..d8e40accf18cd602a23572eb78099002ebaff25b 100644
--- a/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc
+++ b/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc
@@ -325,8 +325,8 @@ void WebNavigationTabObserver::DidFinishLoad(
// srcdoc iframes will report a url of about:blank, still let it through.
if (navigation_state_.GetUrl(render_frame_host) != validated_url &&
(navigation_state_.GetUrl(render_frame_host) !=
- GURL(content::kAboutSrcDocURL) ||
- validated_url != GURL(url::kAboutBlankURL))) {
+ content::kAboutSrcDocURL ||
+ validated_url != url::kAboutBlankURL)) {
return;
}
« no previous file with comments | « chrome/browser/extensions/api/web_navigation/frame_navigation_state.cc ('k') | chrome/browser/extensions/tab_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698