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

Unified Diff: chrome/browser/android/download/chrome_download_delegate.h

Issue 2014803002: Move DownloadControllerAndroid from content/ to chrome/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix failing tests/bugs Created 4 years, 6 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: chrome/browser/android/download/chrome_download_delegate.h
diff --git a/chrome/browser/android/download/chrome_download_delegate.h b/chrome/browser/android/download/chrome_download_delegate.h
index 1ad717c45da315fa9cc5828076ac4c8cdba1d1cf..864be455aca0aa1a43daf82b46d1e03b5a44272c 100644
--- a/chrome/browser/android/download/chrome_download_delegate.h
+++ b/chrome/browser/android/download/chrome_download_delegate.h
@@ -6,16 +6,42 @@
#define CHROME_BROWSER_ANDROID_DOWNLOAD_CHROME_DOWNLOAD_DELEGATE_H_
#include "base/android/jni_android.h"
-#include "base/android/jni_weak_ref.h"
-#include "base/android/scoped_java_ref.h"
+#include "content/public/browser/web_contents_user_data.h"
-class ChromeDownloadDelegate {
+class ChromeDownloadDelegate
+ : public content::WebContentsUserData<ChromeDownloadDelegate> {
public:
// Returns true iff this request resulted in the tab creating the download
// to close.
static bool EnqueueDownloadManagerRequest(jobject chrome_download_delegate,
bool overwrite,
jobject download_info);
+
+ void SetJavaRef(JNIEnv*, jobject obj);
+
+ void RequestHTTPGetDownload(const std::string& url,
+ const std::string& user_agent,
+ const std::string& content_disposition,
+ const std::string& mime_type,
+ const std::string& cookie,
+ const std::string& referer,
+ const base::string16& file_name,
+ int64_t content_length,
+ bool has_user_geature,
+ bool must_download);
+ void OnDownloadStarted(const std::string& filename,
+ const std::string& mime_type);
+ void OnDangerousDownload(const std::string& filename,
+ const std::string& guid);
+ void RequestFileAccess(intptr_t callback_id);
+
+ private:
+ explicit ChromeDownloadDelegate(content::WebContents* contents);
+ friend class content::WebContentsUserData<ChromeDownloadDelegate>;
+ ~ChromeDownloadDelegate() override;
+
+ // A reference to the Java ChromeDownloadDelegate object.
+ jobject java_ref_;
};
bool RegisterChromeDownloadDelegate(JNIEnv* env);
« no previous file with comments | « chrome/browser/android/chrome_jni_registrar.cc ('k') | chrome/browser/android/download/chrome_download_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698