Commit 332deb89 authored by Natalie Chouinard's avatar Natalie Chouinard Committed by Commit Bot

[Settings Refactor] Move Download settings

Move Download settings to org.chromium.chrome.browser.download package.
This package rename will be followed by modularization
(crbug.com/1047896).

Bug: 1047357
Change-Id: Iaa12fab75234a1f51f4429536ad867eedebf26c6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2076440Reviewed-by: default avatarDavid Trainor <dtrainor@chromium.org>
Commit-Queue: Natalie Chouinard <chouinard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#749411}
parent d6357380
...@@ -664,6 +664,11 @@ chrome_java_sources = [ ...@@ -664,6 +664,11 @@ chrome_java_sources = [
"java/src/org/chromium/chrome/browser/download/items/OfflineContentAggregatorNotificationBridgeUiFactory.java", "java/src/org/chromium/chrome/browser/download/items/OfflineContentAggregatorNotificationBridgeUiFactory.java",
"java/src/org/chromium/chrome/browser/download/service/DownloadBackgroundTask.java", "java/src/org/chromium/chrome/browser/download/service/DownloadBackgroundTask.java",
"java/src/org/chromium/chrome/browser/download/service/DownloadTaskScheduler.java", "java/src/org/chromium/chrome/browser/download/service/DownloadTaskScheduler.java",
"java/src/org/chromium/chrome/browser/download/settings/DownloadDirectoryAdapter.java",
"java/src/org/chromium/chrome/browser/download/settings/DownloadLocationPreference.java",
"java/src/org/chromium/chrome/browser/download/settings/DownloadLocationPreferenceAdapter.java",
"java/src/org/chromium/chrome/browser/download/settings/DownloadLocationPreferenceDialog.java",
"java/src/org/chromium/chrome/browser/download/settings/DownloadSettings.java",
"java/src/org/chromium/chrome/browser/engagement/SiteEngagementService.java", "java/src/org/chromium/chrome/browser/engagement/SiteEngagementService.java",
"java/src/org/chromium/chrome/browser/explore_sites/CategoryCardAdapter.java", "java/src/org/chromium/chrome/browser/explore_sites/CategoryCardAdapter.java",
"java/src/org/chromium/chrome/browser/explore_sites/CategoryCardViewHolderFactory.java", "java/src/org/chromium/chrome/browser/explore_sites/CategoryCardViewHolderFactory.java",
...@@ -1386,11 +1391,6 @@ chrome_java_sources = [ ...@@ -1386,11 +1391,6 @@ chrome_java_sources = [
"java/src/org/chromium/chrome/browser/settings/SettingsLauncher.java", "java/src/org/chromium/chrome/browser/settings/SettingsLauncher.java",
"java/src/org/chromium/chrome/browser/settings/accessibility/AccessibilitySettings.java", "java/src/org/chromium/chrome/browser/settings/accessibility/AccessibilitySettings.java",
"java/src/org/chromium/chrome/browser/settings/accessibility/TextScalePreference.java", "java/src/org/chromium/chrome/browser/settings/accessibility/TextScalePreference.java",
"java/src/org/chromium/chrome/browser/settings/download/DownloadDirectoryAdapter.java",
"java/src/org/chromium/chrome/browser/settings/download/DownloadLocationPreference.java",
"java/src/org/chromium/chrome/browser/settings/download/DownloadLocationPreferenceAdapter.java",
"java/src/org/chromium/chrome/browser/settings/download/DownloadLocationPreferenceDialog.java",
"java/src/org/chromium/chrome/browser/settings/download/DownloadSettings.java",
"java/src/org/chromium/chrome/browser/settings/homepage/HomepageEditor.java", "java/src/org/chromium/chrome/browser/settings/homepage/HomepageEditor.java",
"java/src/org/chromium/chrome/browser/settings/homepage/HomepageMetricsEnums.java", "java/src/org/chromium/chrome/browser/settings/homepage/HomepageMetricsEnums.java",
"java/src/org/chromium/chrome/browser/settings/homepage/HomepageSettings.java", "java/src/org/chromium/chrome/browser/settings/homepage/HomepageSettings.java",
......
...@@ -4,7 +4,7 @@ ...@@ -4,7 +4,7 @@
found in the LICENSE file. --> found in the LICENSE file. -->
<PreferenceScreen xmlns:android="http://schemas.android.com/apk/res/android"> <PreferenceScreen xmlns:android="http://schemas.android.com/apk/res/android">
<org.chromium.chrome.browser.settings.download.DownloadLocationPreference <org.chromium.chrome.browser.download.settings.DownloadLocationPreference
android:key="location_change" android:key="location_change"
android:title="@string/downloads_location_selector_title" android:title="@string/downloads_location_selector_title"
android:positiveButtonText="@string/done" android:positiveButtonText="@string/done"
......
...@@ -91,7 +91,7 @@ ...@@ -91,7 +91,7 @@
android:order="16" android:order="16"
android:title="@string/data_reduction_title_lite_mode"/> android:title="@string/data_reduction_title_lite_mode"/>
<org.chromium.chrome.browser.settings.ChromeBasePreference <org.chromium.chrome.browser.settings.ChromeBasePreference
android:fragment="org.chromium.chrome.browser.settings.download.DownloadSettings" android:fragment="org.chromium.chrome.browser.download.settings.DownloadSettings"
android:key="downloads" android:key="downloads"
android:order="17" android:order="17"
android:title="@string/menu_downloads"/> android:title="@string/menu_downloads"/>
......
...@@ -4,7 +4,7 @@ ...@@ -4,7 +4,7 @@
package org.chromium.chrome.browser.download; package org.chromium.chrome.browser.download;
import static org.chromium.chrome.browser.settings.download.DownloadDirectoryAdapter.NO_SELECTED_ITEM_ID; import static org.chromium.chrome.browser.download.settings.DownloadDirectoryAdapter.NO_SELECTED_ITEM_ID;
import android.content.Context; import android.content.Context;
import android.util.AttributeSet; import android.util.AttributeSet;
...@@ -18,7 +18,7 @@ import android.widget.TextView; ...@@ -18,7 +18,7 @@ import android.widget.TextView;
import androidx.annotation.Nullable; import androidx.annotation.Nullable;
import org.chromium.chrome.browser.settings.download.DownloadDirectoryAdapter; import org.chromium.chrome.browser.download.settings.DownloadDirectoryAdapter;
import org.chromium.chrome.download.R; import org.chromium.chrome.download.R;
import org.chromium.components.browser_ui.widget.text.AlertDialogEditText; import org.chromium.components.browser_ui.widget.text.AlertDialogEditText;
......
...@@ -9,10 +9,10 @@ import android.content.Context; ...@@ -9,10 +9,10 @@ import android.content.Context;
import org.chromium.chrome.browser.GlobalDiscardableReferencePool; import org.chromium.chrome.browser.GlobalDiscardableReferencePool;
import org.chromium.chrome.browser.download.items.OfflineContentAggregatorFactory; import org.chromium.chrome.browser.download.items.OfflineContentAggregatorFactory;
import org.chromium.chrome.browser.download.settings.DownloadSettings;
import org.chromium.chrome.browser.feature_engagement.TrackerFactory; import org.chromium.chrome.browser.feature_engagement.TrackerFactory;
import org.chromium.chrome.browser.profiles.Profile; import org.chromium.chrome.browser.profiles.Profile;
import org.chromium.chrome.browser.settings.SettingsLauncher; import org.chromium.chrome.browser.settings.SettingsLauncher;
import org.chromium.chrome.browser.settings.download.DownloadSettings;
import org.chromium.chrome.browser.ui.messages.snackbar.SnackbarManager; import org.chromium.chrome.browser.ui.messages.snackbar.SnackbarManager;
import org.chromium.ui.modaldialog.ModalDialogManager; import org.chromium.ui.modaldialog.ModalDialogManager;
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be // Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file. // found in the LICENSE file.
package org.chromium.chrome.browser.settings.download; package org.chromium.chrome.browser.download.settings;
import android.content.Context; import android.content.Context;
import android.view.LayoutInflater; import android.view.LayoutInflater;
...@@ -233,8 +233,8 @@ public class DownloadDirectoryAdapter extends ArrayAdapter<Object> { ...@@ -233,8 +233,8 @@ public class DownloadDirectoryAdapter extends ArrayAdapter<Object> {
break; break;
case DirectoryOption.DownloadLocationDirectoryType.ADDITIONAL: case DirectoryOption.DownloadLocationDirectoryType.ADDITIONAL:
String directoryName = (numOtherAdditionalDirectories > 0) String directoryName = (numOtherAdditionalDirectories > 0)
? mContext.getString(org.chromium.chrome.R.string ? mContext.getString(
.downloads_location_sd_card_number, org.chromium.chrome.R.string.downloads_location_sd_card_number,
numOtherAdditionalDirectories + 1) numOtherAdditionalDirectories + 1)
: mContext.getString( : mContext.getString(
org.chromium.chrome.R.string.downloads_location_sd_card); org.chromium.chrome.R.string.downloads_location_sd_card);
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be // Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file. // found in the LICENSE file.
package org.chromium.chrome.browser.settings.download; package org.chromium.chrome.browser.download.settings;
import android.content.Context; import android.content.Context;
import android.support.v7.preference.DialogPreference; import android.support.v7.preference.DialogPreference;
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be // Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file. // found in the LICENSE file.
package org.chromium.chrome.browser.settings.download; package org.chromium.chrome.browser.download.settings;
import android.content.Context; import android.content.Context;
import android.view.LayoutInflater; import android.view.LayoutInflater;
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be // Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file. // found in the LICENSE file.
package org.chromium.chrome.browser.settings.download; package org.chromium.chrome.browser.download.settings;
import android.os.Bundle; import android.os.Bundle;
import android.support.v7.preference.PreferenceDialogFragmentCompat; import android.support.v7.preference.PreferenceDialogFragmentCompat;
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be // Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file. // found in the LICENSE file.
package org.chromium.chrome.browser.settings.download; package org.chromium.chrome.browser.download.settings;
import android.os.Bundle; import android.os.Bundle;
import android.support.v7.preference.Preference; import android.support.v7.preference.Preference;
......
...@@ -29,9 +29,9 @@ import org.chromium.base.StrictModeContext; ...@@ -29,9 +29,9 @@ import org.chromium.base.StrictModeContext;
import org.chromium.base.test.util.CommandLineFlags; import org.chromium.base.test.util.CommandLineFlags;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.chrome.browser.download.DownloadTestRule.CustomMainActivityStart; import org.chromium.chrome.browser.download.DownloadTestRule.CustomMainActivityStart;
import org.chromium.chrome.browser.download.settings.DownloadDirectoryAdapter;
import org.chromium.chrome.browser.flags.ChromeFeatureList; import org.chromium.chrome.browser.flags.ChromeFeatureList;
import org.chromium.chrome.browser.flags.ChromeSwitches; import org.chromium.chrome.browser.flags.ChromeSwitches;
import org.chromium.chrome.browser.settings.download.DownloadDirectoryAdapter;
import org.chromium.chrome.download.R; import org.chromium.chrome.download.R;
import org.chromium.chrome.test.ChromeJUnit4ClassRunner; import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
import org.chromium.chrome.test.util.browser.Features; import org.chromium.chrome.test.util.browser.Features;
......
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