[Android] Prompt with infobar on filename conflict

chrome/browser/android/download_overwrite_infobar_delegate.h

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..e604ec5e161a46bfe264f71a13c7afdf6d1536da
--- /dev/null
+++ b/chrome/browser/android/download_overwrite_infobar_delegate.h
@@ -0,0 +1,64 @@
+// 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.
+
+#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"
+
+namespace content {
+class DownloadItem;
+}
+
+namespace chrome {
+namespace android {
+
+// A class to implement an infobar that asks if it is ok to overwrite an exist

[Ted C, 2014/10/27 15:30:12]

an exist[ing]

[Changwan Ryu, 2014/10/28 04:59:44]

Done.

+// file. In Android, since we have limited disk space, we give users two options
+// when we are downloading a file but there exists another file with the same

[Ted C, 2014/10/27 15:30:11]

I think this might be easier to read:

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.) Do not download.

The existing one is a bit confusing, but I'm not 100% happy with this either.

[Changwan Ryu, 2014/10/28 04:59:44]

Done. Please let me know if the comment still needs fixing.

+// file name: 1) Overwrite the file. 2) Do not download.
+class DownloadOverwriteInfoBarDelegate : public infobars::InfoBarDelegate {
+ public:
+  ~DownloadOverwriteInfoBarDelegate() override;
+
+  static infobars::InfoBar* Create(
+      content::DownloadItem* download,
+      const base::FilePath& suggested_download_path,
+      const DownloadTargetDeterminerDelegate::FileSelectedCallback&
+          file_selected_callback);
+
+  // This is called when the user accepts overwriting.
+  bool AcceptOverwrite();
+
+  // This is called when the user rejects overwriting and cancels downloading.
+  bool CancelDownload();
+  std::string GetFileName() const;
+  std::string GetDirName() const;
+  std::string GetDirFullPath() const;
+
+ private:
+  DownloadOverwriteInfoBarDelegate(
+      content::DownloadItem* download,
+      const base::FilePath& suggested_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_