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

Unified Diff: chrome/browser/android/download_overwrite_infobar_delegate.h

Issue 580043002: [Android] Prompt with infobar on filename conflict (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fixed nits Created 5 years, 10 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_overwrite_infobar_delegate.h
diff --git a/chrome/browser/android/download_overwrite_infobar_delegate.h b/chrome/browser/android/download_overwrite_infobar_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..5d976513f5322f9dd4d01259ce5eed0f9b635534
--- /dev/null
+++ b/chrome/browser/android/download_overwrite_infobar_delegate.h
@@ -0,0 +1,80 @@
+// Copyright 2015 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.
+
+#ifndef CHROME_BROWSER_ANDROID_DOWNLOAD_OVERWRITE_INFOBAR_DELEGATE_H_
+#define CHROME_BROWSER_ANDROID_DOWNLOAD_OVERWRITE_INFOBAR_DELEGATE_H_
+
+#include "base/callback.h"
+#include "base/files/file_path.h"
+#include "chrome/browser/download/download_target_determiner_delegate.h"
+#include "components/infobars/core/infobar_delegate.h"
+
+class InfoBarService;
+
+namespace chrome {
+namespace android {
+
+// An infobar that asks if it is ok to overwrite an
+// existing file. Due to limited disk space on Android, two options are
+// presented to the user when downloading a file whose name conflicts with an
+// already present file:
+//
+// 1. Overwrite the file.
+// 2. Create a new file.
+//
+// Also, the user can dismiss the infobar to abort the download.
+//
+// Note that this infobar does not expire if the user subsequently navigates,
+// since such navigations won't automatically cancel the underlying download.
+class DownloadOverwriteInfoBarDelegate : public infobars::InfoBarDelegate {
+ public:
+ ~DownloadOverwriteInfoBarDelegate() override;
+
+ static void Create(
+ InfoBarService* infobar_service,
+ const base::FilePath& suggested_download_path,
+ const DownloadTargetDeterminerDelegate::FileSelectedCallback&
+ file_selected_callback);
+
+ // This is called when the user chooses to overwrite the existing file.
+ void OverwriteExistingFile();
+
+ // This is called when the user chooses to create a new file.
+ void CreateNewFile();
+
+ base::FilePath suggested_download_path() const {
+ return suggested_download_path_;
+ }
+
+ private:
+ DownloadOverwriteInfoBarDelegate(
+ const base::FilePath& suggested_path,
+ const DownloadTargetDeterminerDelegate::FileSelectedCallback& callback);
+
+ // infobars::InfoBarDelegate:
+ bool ShouldExpire(const NavigationDetails& details) const override;
+ void InfoBarDismissed() override;
+
+ // Creates a new file in content::BrowserThread::FILE thread, and when it's
+ // done, calls |callback| in the UI thread.
+ static void CreateNewFileInternal(
+ const base::FilePath& suggested_download_path,
+ const DownloadTargetDeterminerDelegate::FileSelectedCallback& callback);
+
+ // The suggested download path from download target determiner. This is used
+ // to show users the file name and the directory that will be used.
+ base::FilePath suggested_download_path_;
+
+ // A callback to download target determiner to notify that file selection
+ // is made (or cancelled).
+ DownloadTargetDeterminerDelegate::FileSelectedCallback
+ file_selected_callback_;
+
+ DISALLOW_COPY_AND_ASSIGN(DownloadOverwriteInfoBarDelegate);
+};
+
+} // namespace android
+} // namespace chrome
+
+#endif // CHROME_BROWSER_ANDROID_DOWNLOAD_OVERWRITE_INFOBAR_DELEGATE_H_

Powered by Google App Engine
This is Rietveld 408576698