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

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

Issue 2702503002: Block renderer-initiated main frame navigations to data URLs (Closed)
Patch Set: Fix downloads, plugin handling and browser side navigations 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
+ // 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