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

Unified Diff: android_webview/browser/renderer_host/aw_resource_dispatcher_host_delegate.cc

Issue 11017075: Don't intercept POST navigation in android_webview. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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: android_webview/browser/renderer_host/aw_resource_dispatcher_host_delegate.cc
diff --git a/android_webview/browser/renderer_host/aw_resource_dispatcher_host_delegate.cc b/android_webview/browser/renderer_host/aw_resource_dispatcher_host_delegate.cc
index 15e562bb11cb137bb5a641885ae0eb178d72d542..6bc3b73d38760da4976b5c2b7f938a6cb4083df7 100644
--- a/android_webview/browser/renderer_host/aw_resource_dispatcher_host_delegate.cc
+++ b/android_webview/browser/renderer_host/aw_resource_dispatcher_host_delegate.cc
@@ -94,7 +94,9 @@ void AwResourceDispatcherHostDelegate::RequestBeginning(
}
}
- if (resource_type == ResourceType::MAIN_FRAME) {
+ // We ignore POST requests because of BUG=155250.
+ if (resource_type == ResourceType::MAIN_FRAME &&
+ request->method() != "POST") {
benm (inactive) 2012/10/11 14:05:30 will it always be CAPS? is there a const we can us
mkosiba (inactive) 2012/10/11 14:22:53 apparently... these doesn't seem to be a constant
throttles->push_back(InterceptNavigationDelegate::CreateThrottleFor(
request));
}

Powered by Google App Engine
This is Rietveld 408576698