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

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

Issue 2619603002: Remove android_java_ui as it is not used (Closed)
Patch Set: 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..3a6c483743a34be32acf1d6bee336f90a90a5b42 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,7 +794,7 @@ Profile* DownloadTargetDeterminer::GetProfile() const {
bool DownloadTargetDeterminer::ShouldPromptForDownload(
const base::FilePath& filename) const {
-#if BUILDFLAG(ANDROID_JAVA_UI)
+#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;

Powered by Google App Engine
This is Rietveld 408576698