231 lines
13 KiB
Diff
231 lines
13 KiB
Diff
From: uazo <uazo@users.noreply.github.com>
|
|
Date: Mon, 21 Aug 2023 14:03:25 +0000
|
|
Subject: Add option to disable snapshots
|
|
|
|
License: GPL-2.0-or-later - https://spdx.org/licenses/GPL-2.0-or-later.html
|
|
---
|
|
.../tab_management/TabGridViewBinder.java | 10 +++--
|
|
.../tasks/tab_management/TabProperties.java | 5 +++
|
|
.../java/res/xml/privacy_preferences.xml | 5 +++
|
|
.../chrome/browser/app/ChromeActivity.java | 3 +-
|
|
.../layouts/content/TabContentManager.java | 7 +++-
|
|
.../privacy/settings/PrivacySettings.java | 42 ++++++++++++++++++-
|
|
.../strings/android_chrome_strings.grd | 8 ++++
|
|
7 files changed, 73 insertions(+), 7 deletions(-)
|
|
|
|
diff --git a/chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGridViewBinder.java b/chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGridViewBinder.java
|
|
--- a/chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGridViewBinder.java
|
|
+++ b/chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGridViewBinder.java
|
|
@@ -327,9 +327,12 @@ class TabGridViewBinder {
|
|
// Adjust bitmap to thumbnail.
|
|
TabUtils.setBitmapAndUpdateImageMatrix(thumbnail, result, thumbnailSize);
|
|
} else if (isMostRecentRequest) {
|
|
- // If the most recent request is a null bitmap ensure a placeholder is
|
|
- // visible.
|
|
- thumbnail.setImageDrawable(null);
|
|
+ TabListFaviconProvider.TabFavicon tabFavicon = model.get(TabProperties.FAVICON);
|
|
+ if (tabFavicon != null) {
|
|
+ thumbnail.setScaleType(ImageView.ScaleType.CENTER);
|
|
+ thumbnail.setImageDrawable(
|
|
+ isSelected ? tabFavicon.getSelectedDrawable() : tabFavicon.getDefaultDrawable());
|
|
+ }
|
|
}
|
|
};
|
|
if (sThumbnailFetcherForTesting != null) {
|
|
@@ -368,6 +371,7 @@ class TabGridViewBinder {
|
|
ViewLookupCachingFrameLayout rootView,
|
|
PropertyModel model,
|
|
TabListFaviconProvider.TabFavicon favicon) {
|
|
+ model.set(TabProperties.FAVICON, favicon);
|
|
ImageView faviconView = (ImageView) rootView.fastFindViewById(R.id.tab_favicon);
|
|
if (favicon == null) {
|
|
faviconView.setImageDrawable(null);
|
|
diff --git a/chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabProperties.java b/chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabProperties.java
|
|
--- a/chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabProperties.java
|
|
+++ b/chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabProperties.java
|
|
@@ -63,6 +63,9 @@ public class TabProperties {
|
|
public static final WritableBooleanPropertyKey FAVICON_FETCHED =
|
|
new WritableBooleanPropertyKey();
|
|
|
|
+ public static final WritableObjectPropertyKey<TabListFaviconProvider.TabFavicon> FAVICON =
|
|
+ new WritableObjectPropertyKey<>();
|
|
+
|
|
/** Property for lazily fetching favicons when required by an item in a UI. */
|
|
public static final WritableObjectPropertyKey<TabListFaviconProvider.TabFaviconFetcher>
|
|
FAVICON_FETCHER = new WritableObjectPropertyKey<>();
|
|
@@ -160,6 +163,7 @@ public class TabProperties {
|
|
CARD_TYPE,
|
|
CONTENT_DESCRIPTION_STRING,
|
|
CLOSE_BUTTON_DESCRIPTION_STRING,
|
|
+ FAVICON,
|
|
SHOPPING_PERSISTED_TAB_DATA_FETCHER,
|
|
SHOULD_SHOW_PRICE_DROP_TOOLTIP,
|
|
QUICK_DELETE_ANIMATION_STATUS
|
|
@@ -172,6 +176,7 @@ public class TabProperties {
|
|
TAB_CLOSED_LISTENER,
|
|
FAVICON_FETCHED,
|
|
FAVICON_FETCHER,
|
|
+ FAVICON,
|
|
IS_SELECTED,
|
|
TITLE,
|
|
TABSTRIP_FAVICON_BACKGROUND_COLOR_ID,
|
|
diff --git a/chrome/android/java/res/xml/privacy_preferences.xml b/chrome/android/java/res/xml/privacy_preferences.xml
|
|
--- a/chrome/android/java/res/xml/privacy_preferences.xml
|
|
+++ b/chrome/android/java/res/xml/privacy_preferences.xml
|
|
@@ -54,6 +54,11 @@ found in the LICENSE file.
|
|
android:title="@string/open_external_links_incognito_title"
|
|
android:summary="@string/open_external_links_incognito_summary"
|
|
android:defaultValue="false" />
|
|
+ <org.chromium.components.browser_ui.settings.ChromeSwitchPreference
|
|
+ android:key="tabgrid_use_icons"
|
|
+ android:title="@string/tabgrid_use_icons_title"
|
|
+ android:summary="@string/tabgrid_use_icons_summary"
|
|
+ android:defaultValue="false" />
|
|
<Preference
|
|
android:key="preload_pages"
|
|
android:title="@string/preload_pages_title"
|
|
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/app/ChromeActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/app/ChromeActivity.java
|
|
--- a/chrome/android/java/src/org/chromium/chrome/browser/app/ChromeActivity.java
|
|
+++ b/chrome/android/java/src/org/chromium/chrome/browser/app/ChromeActivity.java
|
|
@@ -145,6 +145,7 @@ import org.chromium.chrome.browser.preferences.ChromePreferenceKeys;
|
|
import org.chromium.chrome.browser.preferences.ChromeSharedPreferences;
|
|
import org.chromium.chrome.browser.preferences.Pref;
|
|
import org.chromium.chrome.browser.printing.TabPrinter;
|
|
+import org.chromium.chrome.browser.privacy.settings.PrivacySettings;
|
|
import org.chromium.chrome.browser.profiles.Profile;
|
|
import org.chromium.chrome.browser.readaloud.ReadAloudController;
|
|
import org.chromium.chrome.browser.selection.SelectionPopupBackPressHandler;
|
|
@@ -704,7 +705,7 @@ public abstract class ChromeActivity<C extends ChromeActivityComponent>
|
|
new TabContentManager(
|
|
this,
|
|
mBrowserControlsManagerSupplier.get(),
|
|
- !SysUtils.isLowEndDevice(),
|
|
+ !ContextUtils.getAppSharedPreferences().getBoolean(PrivacySettings.PREF_TABGRID_USE_ICONS, false),
|
|
tabModelSelector != null ? tabModelSelector::getTabById : null));
|
|
|
|
getBrowserControlsManager()
|
|
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/content/TabContentManager.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/content/TabContentManager.java
|
|
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/content/TabContentManager.java
|
|
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/content/TabContentManager.java
|
|
@@ -326,11 +326,14 @@ public class TabContentManager {
|
|
*/
|
|
public void getTabThumbnailWithCallback(
|
|
@NonNull int tabId,
|
|
- @NonNull Size thumbnailSize,
|
|
+ @NonNull Size thumbnailSize, //
|
|
@NonNull Callback<Bitmap> callback,
|
|
boolean forceUpdate,
|
|
boolean writeBack) {
|
|
- if (!mSnapshotsEnabled) return;
|
|
+ if (!mSnapshotsEnabled) {
|
|
+ callback.onResult(null);
|
|
+ return;
|
|
+ }
|
|
|
|
// TODO(crbug/1444782): Remove forceUpdate and writeBack params once the following features
|
|
// launch:
|
|
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/privacy/settings/PrivacySettings.java b/chrome/android/java/src/org/chromium/chrome/browser/privacy/settings/PrivacySettings.java
|
|
--- a/chrome/android/java/src/org/chromium/chrome/browser/privacy/settings/PrivacySettings.java
|
|
+++ b/chrome/android/java/src/org/chromium/chrome/browser/privacy/settings/PrivacySettings.java
|
|
@@ -66,12 +66,25 @@ import org.chromium.chrome.browser.preferences.ChromeSharedPreferences;
|
|
import org.chromium.components.browser_ui.settings.ChromeSwitchPreference;
|
|
import org.chromium.components.browser_ui.settings.ManagedPreferenceDelegate;
|
|
|
|
+import org.chromium.chrome.browser.ui.messages.snackbar.SnackbarManager;
|
|
+import org.chromium.chrome.browser.ui.messages.snackbar.INeedSnackbarManager;
|
|
+import org.chromium.chrome.browser.ui.messages.snackbar.Snackbar;
|
|
+import org.chromium.chrome.browser.ApplicationLifetime;
|
|
+
|
|
import java.util.ArrayList;
|
|
import java.util.List;
|
|
|
|
/** Fragment to keep track of the all the privacy related preferences. */
|
|
public class PrivacySettings extends ChromeBaseSettingsFragment
|
|
- implements Preference.OnPreferenceChangeListener {
|
|
+ implements Preference.OnPreferenceChangeListener, INeedSnackbarManager {
|
|
+ private SnackbarManager mSnackbarManager;
|
|
+ private Snackbar mSnackbar;
|
|
+
|
|
+ @Override
|
|
+ public void setSnackbarManager(SnackbarManager manager) {
|
|
+ mSnackbarManager = manager;
|
|
+ }
|
|
+
|
|
private static final String PREF_CAN_MAKE_PAYMENT = "can_make_payment";
|
|
private static final String PREF_PRELOAD_PAGES = "preload_pages";
|
|
private static final String PREF_HTTPS_FIRST_MODE = "https_first_mode";
|
|
@@ -140,6 +153,21 @@ public class PrivacySettings extends ChromeBaseSettingsFragment
|
|
}
|
|
}
|
|
|
|
+ mSnackbar = Snackbar.make(getActivity().getString(R.string.ui_relaunch_notice),
|
|
+ new SnackbarManager.SnackbarController() {
|
|
+ @Override
|
|
+ public void onDismissNoAction(Object actionData) { }
|
|
+
|
|
+ @Override
|
|
+ public void onAction(Object actionData) {
|
|
+ ApplicationLifetime.terminate(true);
|
|
+ }
|
|
+ }, Snackbar.TYPE_NOTIFICATION, Snackbar.UMA_UNKNOWN)
|
|
+ .setSingleLine(false)
|
|
+ .setAction(getActivity().getString(R.string.relaunch),
|
|
+ /*actionData*/null)
|
|
+ .setDuration(/*durationMs*/70000);
|
|
+
|
|
Preference privacyGuidePreference = findPreference(PREF_PRIVACY_GUIDE);
|
|
// Record the launch of PG from the S&P link-row entry point
|
|
privacyGuidePreference.setOnPreferenceClickListener(
|
|
@@ -321,6 +349,7 @@ public class PrivacySettings extends ChromeBaseSettingsFragment
|
|
|
|
public static final String PREF_ALLOW_CUSTOM_TAB_INTENTS = "allow_custom_tab_intents";
|
|
public static final String PREF_OPEN_EXTERNAL_LINKS_INCOGNITO = "open_external_links_incognito";
|
|
+ public static final String PREF_TABGRID_USE_ICONS = "tabgrid_use_icons";
|
|
|
|
@Override
|
|
public boolean onPreferenceChange(Preference preference, Object newValue) {
|
|
@@ -346,6 +375,13 @@ public class PrivacySettings extends ChromeBaseSettingsFragment
|
|
SharedPreferences.Editor sharedPreferencesEditor = ContextUtils.getAppSharedPreferences().edit();
|
|
sharedPreferencesEditor.putBoolean(PREF_OPEN_EXTERNAL_LINKS_INCOGNITO, (boolean)newValue);
|
|
sharedPreferencesEditor.apply();
|
|
+ } else if (PREF_TABGRID_USE_ICONS.equals(key)) {
|
|
+ SharedPreferences.Editor sharedPreferencesEditor = ContextUtils.getAppSharedPreferences().edit();
|
|
+ sharedPreferencesEditor.putBoolean(PREF_TABGRID_USE_ICONS, (boolean)newValue);
|
|
+ sharedPreferencesEditor.apply();
|
|
+ if (!mSnackbarManager.isShowing()) {
|
|
+ mSnackbarManager.showSnackbar(mSnackbar);
|
|
+ }
|
|
}
|
|
return true;
|
|
}
|
|
@@ -386,6 +422,10 @@ public class PrivacySettings extends ChromeBaseSettingsFragment
|
|
: R.string.text_off);
|
|
}
|
|
|
|
+ Preference tabGridUseIcons =
|
|
+ (ChromeSwitchPreference) findPreference(PREF_TABGRID_USE_ICONS);
|
|
+ tabGridUseIcons.setOnPreferenceChangeListener(this);
|
|
+
|
|
Preference refererPolicyPref = findPreference(PREF_REFERER_POLICY);
|
|
refererPolicyPref.setSummary(
|
|
RefererSettingsFragment.getRefererSummaryString(getContext()));
|
|
diff --git a/chrome/browser/ui/android/strings/android_chrome_strings.grd b/chrome/browser/ui/android/strings/android_chrome_strings.grd
|
|
--- a/chrome/browser/ui/android/strings/android_chrome_strings.grd
|
|
+++ b/chrome/browser/ui/android/strings/android_chrome_strings.grd
|
|
@@ -5371,6 +5371,14 @@ To change this setting, <ph name="BEGIN_LINK">BEGIN_LINK</ph>clear the Chrome da
|
|
Your <ph name="FQDN">%1$s<ex>www.amazon.com</ex></ph> timer ran out. It'll start again tomorrow.
|
|
</message>
|
|
|
|
+ <!-- Tab Broup Disable Thumbnail -->
|
|
+ <message name="IDS_TABGRID_USE_ICONS_TITLE" desc="Text for 'Use icons in the tab grid' settings-privacy option.">
|
|
+ Use only icons in the tab grid
|
|
+ </message>
|
|
+ <message name="IDS_TABGRID_USE_ICONS_SUMMARY" desc="Summary text for 'Use icons in the tab grid' settings-privacy option.">
|
|
+ Disable the saving of snapshots to disk
|
|
+ </message>
|
|
+
|
|
<!-- Tab Group Tab Switcher strings -->
|
|
<message name="IDS_TAB_MANAGEMENT_MODULE_TITLE" desc="Text shown when the Tab Management module is referenced in install start, success, failure UI (e.g. in IDS_MODULE_INSTALL_START_TEXT, which will expand to 'Installing Tab management for Chrome…').">
|
|
Tab management
|
|
--
|