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

Unified Diff: Source/WebCore/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/WebCore/loader/PolicyChecker.cpp
===================================================================
--- Source/WebCore/loader/PolicyChecker.cpp (revision 148494)
+++ Source/WebCore/loader/PolicyChecker.cpp (working copy)
@@ -133,11 +133,6 @@
callback.cancel();
}
-void PolicyChecker::cannotShowMIMEType(const ResourceResponse& response)
-{
- handleUnimplementablePolicy(m_frame->loader()->client()->cannotShowMIMETypeError(response));
-}
-
void PolicyChecker::continueLoadAfterWillSubmitForm(PolicyAction)
{
// See header file for an explaination of why this function
@@ -163,16 +158,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);
@@ -205,11 +192,4 @@
callback.call(policy);
}
-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