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

Unified Diff: android_webview/native/aw_contents_client_bridge.cc

Issue 2425423004: Move NewDownload out of AwContentsIoThreadClientImpl to AwContentsClientBridge (Closed)
Patch Set: changed "even if" to "unless" Created 4 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/native/aw_contents_client_bridge.cc
diff --git a/android_webview/native/aw_contents_client_bridge.cc b/android_webview/native/aw_contents_client_bridge.cc
index 04c3eead7e866c3a2be19e543ee6da804075ab81..2c006ece771e5d60270c6253cdc39b3add1db3bf 100644
--- a/android_webview/native/aw_contents_client_bridge.cc
+++ b/android_webview/native/aw_contents_client_bridge.cc
@@ -363,6 +363,31 @@ bool AwContentsClientBridge::ShouldOverrideUrlLoading(const base::string16& url,
return did_override;
}
+void AwContentsClientBridge::NewDownload(const GURL& url,
+ const std::string& user_agent,
+ const std::string& content_disposition,
+ const std::string& mime_type,
+ int64_t content_length) {
+ DCHECK_CURRENTLY_ON(BrowserThread::UI);
+ JNIEnv* env = AttachCurrentThread();
+ ScopedJavaLocalRef<jobject> obj = java_ref_.get(env);
+ if (obj.is_null())
+ return;
+
+ ScopedJavaLocalRef<jstring> jstring_url =
+ ConvertUTF8ToJavaString(env, url.spec());
+ ScopedJavaLocalRef<jstring> jstring_user_agent =
+ ConvertUTF8ToJavaString(env, user_agent);
+ ScopedJavaLocalRef<jstring> jstring_content_disposition =
+ ConvertUTF8ToJavaString(env, content_disposition);
+ ScopedJavaLocalRef<jstring> jstring_mime_type =
+ ConvertUTF8ToJavaString(env, mime_type);
+
+ Java_AwContentsClientBridge_newDownload(
+ env, obj, jstring_url, jstring_user_agent, jstring_content_disposition,
+ jstring_mime_type, content_length);
+}
+
void AwContentsClientBridge::ConfirmJsResult(JNIEnv* env,
const JavaRef<jobject>&,
int id,

Powered by Google App Engine
This is Rietveld 408576698