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

Unified Diff: webkit/glue/webframeloaderclient_impl.cc

Issue 271087: LTTF: Implement WebFrameLoaderClient::dispatchUnableToImplementPolicy() and so on (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 2 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: webkit/glue/webframeloaderclient_impl.cc
===================================================================
--- webkit/glue/webframeloaderclient_impl.cc (revision 29248)
+++ webkit/glue/webframeloaderclient_impl.cc (working copy)
@@ -849,8 +849,11 @@
// FIXME
}
-void WebFrameLoaderClient::dispatchUnableToImplementPolicy(const ResourceError&) {
- // FIXME
+void WebFrameLoaderClient::dispatchUnableToImplementPolicy(
+ const ResourceError& error) {
+ WebKit::WebURLError url_error =
+ webkit_glue::ResourceErrorToWebURLError(error);
+ webframe_->client()->unableToImplementPolicyWithError(webframe_, url_error);
}
void WebFrameLoaderClient::dispatchWillSubmitForm(FramePolicyFunction function,
@@ -999,10 +1002,13 @@
return ResourceError(net::kErrorDomain, net::ERR_ABORTED,
request.url().string(), String());
}
-ResourceError WebFrameLoaderClient::cannotShowURLError(const ResourceRequest&) {
- // FIXME
- return ResourceError();
+
+ResourceError WebFrameLoaderClient::cannotShowURLError(
+ const ResourceRequest& request) {
+ return webkit_glue::WebURLErrorToResourceError(
+ webframe_->client()->cannotShowURLError(WrappedResourceRequest(request)));
}
+
ResourceError WebFrameLoaderClient::interruptForPolicyChangeError(
const ResourceRequest& request) {
return ResourceError(kInternalErrorDomain, ERR_POLICY_CHANGE,
@@ -1034,10 +1040,9 @@
return error.errorCode() != net::ERR_ABORTED;
}
-bool WebFrameLoaderClient::canHandleRequest(const ResourceRequest&) const {
- // FIXME: this appears to be used only by the context menu code to determine
- // if "open" should be displayed in the menu when clicking on a link.
- return true;
+bool WebFrameLoaderClient::canHandleRequest(
+ const ResourceRequest& request) const {
+ return webframe_->client()->canHandleRequest(WrappedResourceRequest(request));
}
bool WebFrameLoaderClient::canShowMIMEType(const String& mime_type) const {

Powered by Google App Engine
This is Rietveld 408576698