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

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

Issue 1988933003: Revert "Introduce AncestorThrottle, which will process 'X-Frame-Options' headers." (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 years, 7 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_throttle.h
diff --git a/content/public/browser/navigation_throttle.h b/content/public/browser/navigation_throttle.h
index 2a62417066d14ef53a0438f3405c6d7fa7ba1095..5876343db0decd9bb6933f6243299aa81b825d06 100644
--- a/content/public/browser/navigation_throttle.h
+++ b/content/public/browser/navigation_throttle.h
@@ -32,11 +32,6 @@ class CONTENT_EXPORT NavigationThrottle {
// Cancels the navigation and makes the requester of the navigation acts
// like the request was never made.
CANCEL_AND_IGNORE,
-
- // Blocks a navigation due to rules asserted by a response (for instance,
- // embedding restrictions like 'X-Frame-Options'). This result will only
- // be returned from WillProcessResponse.
- BLOCK_RESPONSE,
};
NavigationThrottle(NavigationHandle* navigation_handle);
@@ -66,8 +61,7 @@ class CONTENT_EXPORT NavigationThrottle {
// throttle is associated with remain alive during the duration of this
// method. Failing to do so will result in use-after-free bugs. Should the
// implementer need to destroy the WebContents, it should return CANCEL,
- // CANCEL_AND_IGNORE, or BLOCK_RESPONSE and perform the destruction
- // asynchronously.
+ // CANCEL_AND_IGNORE and perform the destruction asynchronously.
virtual ThrottleCheckResult WillProcessResponse();
// The NavigationHandle that is tracking the information related to this
« no previous file with comments | « content/content_tests.gypi ('k') | third_party/WebKit/LayoutTests/http/tests/inspector/network/x-frame-options-deny.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698