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

Unified Diff: chrome/browser/ui/android/infobars/download_overwrite_infobar.cc

Issue 580043002: [Android] Prompt with infobar on filename conflict (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed ted's comments Created 6 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: chrome/browser/ui/android/infobars/download_overwrite_infobar.cc
diff --git a/chrome/browser/ui/android/infobars/download_overwrite_infobar.cc b/chrome/browser/ui/android/infobars/download_overwrite_infobar.cc
new file mode 100644
index 0000000000000000000000000000000000000000..1652d78e24223c1a9375588ad9297b4dfaaec208
--- /dev/null
+++ b/chrome/browser/ui/android/infobars/download_overwrite_infobar.cc
@@ -0,0 +1,73 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/android/infobars/download_overwrite_infobar.h"
+
+#include "base/android/jni_android.h"
+#include "base/android/jni_array.h"
+#include "base/android/jni_string.h"
+#include "base/android/jni_weak_ref.h"
+#include "chrome/browser/android/download_overwrite_infobar_delegate.h"
+#include "jni/DownloadOverwriteInfoBarDelegate_jni.h"
+
+using base::android::ConvertUTF8ToJavaString;
+using chrome::android::DownloadOverwriteInfoBarDelegate;
+
+// static
+scoped_ptr<infobars::InfoBar> DownloadOverwriteInfoBar::CreateInfoBar(
+ scoped_ptr<DownloadOverwriteInfoBarDelegate> delegate) {
+ return scoped_ptr<infobars::InfoBar>(
+ new DownloadOverwriteInfoBar(delegate.Pass()));
+}
+
+DownloadOverwriteInfoBar::DownloadOverwriteInfoBar(
+ scoped_ptr<DownloadOverwriteInfoBarDelegate> delegate)
+ : InfoBarAndroid(delegate.PassAs<infobars::InfoBarDelegate>()),
+ java_delegate_() {
+}
+
+DownloadOverwriteInfoBar::~DownloadOverwriteInfoBar() {
Ted C 2014/10/15 00:19:44 same method ordering comment as the other file
Changwan Ryu 2014/10/15 22:23:24 Done.
+}
+
+ScopedJavaLocalRef<jobject> DownloadOverwriteInfoBar::CreateRenderInfoBar(
+ JNIEnv* env) {
+ java_delegate_.Reset(Java_DownloadOverwriteInfoBarDelegate_create(env));
+ DownloadOverwriteInfoBarDelegate* delegate = GetDelegate();
+ ScopedJavaLocalRef<jstring> j_file_name =
+ ConvertUTF8ToJavaString(env, delegate->GetFileName());
+ ScopedJavaLocalRef<jstring> j_dir_name =
+ ConvertUTF8ToJavaString(env, delegate->GetDirName());
+ ScopedJavaLocalRef<jstring> j_dir_full_path =
+ ConvertUTF8ToJavaString(env, delegate->GetDirFullPath());
+ return Java_DownloadOverwriteInfoBarDelegate_showDownloadOverwriteInfoBar(
+ env, java_delegate_.obj(), reinterpret_cast<intptr_t>(this),
+ j_file_name.obj(), j_dir_name.obj(), j_dir_full_path.obj());
+}
+
+void DownloadOverwriteInfoBar::ProcessButton(
+ int action, const std::string& action_value) {
+ if (!owner())
+ return; // We're closing; don't call anything, it might access the owner.
+
+ DownloadOverwriteInfoBarDelegate* delegate = GetDelegate();
+ if (action == InfoBarAndroid::ACTION_OK) {
+ delegate->Accept();
+ } else if (action == InfoBarAndroid::ACTION_CANCEL) {
+ delegate->Cancel();
+ } else {
+ DCHECK_EQ(InfoBarAndroid::ACTION_NONE, action);
+ }
+
+ RemoveSelf();
+}
+
+DownloadOverwriteInfoBarDelegate* DownloadOverwriteInfoBar::GetDelegate() {
+ return delegate()->AsDownloadOverwriteInfoBarDelegate();
+}
+
+// Native JNI methods ---------------------------------------------------------
+
+bool RegisterDownloadOverwriteInfoBarDelegate(JNIEnv* env) {
+ return RegisterNativesImpl(env);
+}

Powered by Google App Engine
This is Rietveld 408576698