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

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

Issue 2485253002: Remove unnecessary calls to GURL() (Closed)
Patch Set: Fix broken tests 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..297d78d4e75c68e0d015548e7e847861dd406b2b 100644
--- a/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc
+++ b/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc
@@ -326,7 +326,7 @@ void WebNavigationTabObserver::DidFinishLoad(
if (navigation_state_.GetUrl(render_frame_host) != validated_url &&
(navigation_state_.GetUrl(render_frame_host) !=
GURL(content::kAboutSrcDocURL) ||
Charlie Harrison 2016/11/09 21:12:58 Can you change GURL(content::kAboutSrcDocURL) too?
cfredric 2016/11/09 22:12:46 Done.
- validated_url != GURL(url::kAboutBlankURL))) {
+ validated_url != url::kAboutBlankURL)) {
return;
}

Powered by Google App Engine
This is Rietveld 408576698