Commit 570d7557 authored by Xing Liu's avatar Xing Liu Committed by Commit Bot

Download location: Enable the feature flag.

This CL enables the download location feature as part of the rolling
out process.


Bug: 792775
Change-Id: I1f5f18d8eeb5c0d2a75c562672980380f2c11c2e
Reviewed-on: https://chromium-review.googlesource.com/1145474
Commit-Queue: Xing Liu <xingliu@chromium.org>
Reviewed-by: default avatarMin Qin <qinmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#577660}
parent 06995811
......@@ -282,6 +282,11 @@ void ChromeDownloadManagerDelegateTest::SetUp() {
delegate_->SetDownloadManager(download_manager_.get());
pref_service_ = profile()->GetTestingPrefService();
web_contents()->SetDelegate(&web_contents_delegate_);
#if defined(OS_ANDROID)
pref_service_->SetInteger(prefs::kPromptForDownloadAndroid,
static_cast<int>(DownloadPromptStatus::DONT_SHOW));
#endif
}
void ChromeDownloadManagerDelegateTest::TearDown() {
......@@ -1101,6 +1106,10 @@ class TestDownloadLocationDialogBridge : public DownloadLocationDialogBridge {
} // namespace
TEST_F(ChromeDownloadManagerDelegateTest, RequestConfirmation_Android) {
// TODO(xingliu): Delete this test when download location change feature
// flag is deleted.
base::test::ScopedFeatureList scoped_list;
scoped_list.InitAndDisableFeature(features::kDownloadsLocationChange);
EXPECT_FALSE(
base::FeatureList::IsEnabled(features::kDownloadsLocationChange));
......
......@@ -238,7 +238,7 @@ const base::Feature kDownloadsForeground{"DownloadsForeground",
#if defined(OS_ANDROID)
// Enable changing default downloads storage location on Android.
const base::Feature kDownloadsLocationChange{"DownloadsLocationChange",
base::FEATURE_DISABLED_BY_DEFAULT};
base::FEATURE_ENABLED_BY_DEFAULT};
#endif
// An experimental way of showing app banners, which has modal banners and gives
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment