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

Unified Diff: Source/core/loader/PolicyChecker.cpp

Issue 14264012: Create errors (especially cancellation errors) internally to WebCore, rather (Closed) Base URL: svn://svn.chromium.org/blink/trunk/
Patch Set: Created 7 years, 8 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: Source/core/loader/PolicyChecker.cpp
===================================================================
--- Source/core/loader/PolicyChecker.cpp (revision 148582)
+++ Source/core/loader/PolicyChecker.cpp (working copy)
@@ -126,11 +126,6 @@
callback.cancel();
}
-void PolicyChecker::cannotShowMIMEType(const ResourceResponse& response)
-{
- handleUnimplementablePolicy(m_frame->loader()->client()->cannotShowMIMETypeError(response));
-}
-
void PolicyChecker::continueAfterNavigationPolicy(PolicyAction policy)
{
PolicyCallback callback = m_callback;
@@ -149,16 +144,8 @@
callback.clearRequest();
break;
}
- case PolicyUse: {
- ResourceRequest request(callback.request());
-
- if (!m_frame->loader()->client()->canHandleRequest(request)) {
- handleUnimplementablePolicy(m_frame->loader()->client()->cannotShowURLError(callback.request()));
- callback.clearRequest();
- shouldContinue = false;
- }
+ case PolicyUse:
break;
- }
}
callback.call(shouldContinue);
@@ -184,11 +171,4 @@
callback.call(policy == PolicyUse);
}
-void PolicyChecker::handleUnimplementablePolicy(const ResourceError& error)
-{
- m_delegateIsHandlingUnimplementablePolicy = true;
- m_frame->loader()->client()->dispatchUnableToImplementPolicy(error);
- m_delegateIsHandlingUnimplementablePolicy = false;
-}
-
} // namespace WebCore

Powered by Google App Engine
This is Rietveld 408576698