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

Unified Diff: content/public/browser/navigation_handle.h

Issue 2702503002: Block renderer-initiated main frame navigations to data URLs (Closed)
Patch Set: Cleanup Created 3 years, 9 months 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: content/public/browser/navigation_handle.h
diff --git a/content/public/browser/navigation_handle.h b/content/public/browser/navigation_handle.h
index b81bb398ff2d943f935716d3cf10ac430cc1a44a..317970ac2ce5ee73ce72c3defd680489e8f94882 100644
--- a/content/public/browser/navigation_handle.h
+++ b/content/public/browser/navigation_handle.h
@@ -183,6 +183,11 @@ class CONTENT_EXPORT NavigationHandle {
// errors that leave the user on the previous page.
virtual bool HasCommitted() = 0;
+ // Whether the navigation is a download. This is useful when the navigation
nasko 2017/03/28 19:58:01 nit: "navigation request", since navigations thems
meacer 2017/03/30 20:43:55 Done.
+ // hasn't committed yet, in which case HasCommitted() will return false even
+ // if the navigation is not a download.
+ virtual bool IsDownload() = 0;
+
// Whether the navigation resulted in an error page.
// Note that if an error page reloads, this will return true even though
// GetNetErrorCode will be net::OK.

Powered by Google App Engine
This is Rietveld 408576698