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

Unified Diff: chrome/browser/download/download_target_determiner.cc

Issue 2619603002: Remove android_java_ui as it is not used (Closed)
Patch Set: Rebase to master Created 3 years, 11 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/download/download_target_determiner.cc
diff --git a/chrome/browser/download/download_target_determiner.cc b/chrome/browser/download/download_target_determiner.cc
index fbc35cfeef8854132e5eca8142f25ca88397ccbe..f1d1845138c9e82acb617c042c178e5ba0136088 100644
--- a/chrome/browser/download/download_target_determiner.cc
+++ b/chrome/browser/download/download_target_determiner.cc
@@ -324,7 +324,7 @@ void DownloadTargetDeterminer::ReserveVirtualPathDone(
DVLOG(20) << "Reserved path: " << path.AsUTF8Unsafe()
<< " Verified:" << verified;
DCHECK_EQ(STATE_PROMPT_USER_FOR_DOWNLOAD_PATH, next_state_);
-#if BUILDFLAG(ANDROID_JAVA_UI)
+#if defined(OS_ANDROID)
if (!verified) {
if (path.empty()) {
DownloadManagerService::OnDownloadCanceled(
@@ -794,10 +794,10 @@ Profile* DownloadTargetDeterminer::GetProfile() const {
bool DownloadTargetDeterminer::ShouldPromptForDownload(
const base::FilePath& filename) const {
-#if BUILDFLAG(ANDROID_JAVA_UI)
- // Don't prompt user about saving path on Android.
- // TODO(qinmin): show an error toast to warn user in certain cases.
- return false;
+#if defined(OS_ANDROID)
+ // Don't prompt user about saving path on Android.
+ // TODO(qinmin): show an error toast to warn user in certain cases.
+ return false;
#endif
if (is_resumption_) {
// For resumed downloads, if the target disposition or prefs require

Powered by Google App Engine
This is Rietveld 408576698