diff --git a/chrome/android/java/src/org/chromium/chrome/browser/download/items/OfflineContentAggregatorNotificationBridgeUi.java b/chrome/android/java/src/org/chromium/chrome/browser/download/items/OfflineContentAggregatorNotificationBridgeUi.java index b97ba89eadf2d..924190f28a15e 100644 --- a/chrome/android/java/src/org/chromium/chrome/browser/download/items/OfflineContentAggregatorNotificationBridgeUi.java +++ b/chrome/android/java/src/org/chromium/chrome/browser/download/items/OfflineContentAggregatorNotificationBridgeUi.java @@ -9,6 +9,7 @@ import org.chromium.chrome.browser.download.DownloadNotifier; import org.chromium.chrome.browser.download.DownloadServiceDelegate; import org.chromium.components.offline_items_collection.ContentId; +import org.chromium.components.offline_items_collection.LegacyHelpers; import org.chromium.components.offline_items_collection.OfflineContentProvider; import org.chromium.components.offline_items_collection.OfflineItem; import org.chromium.components.offline_items_collection.OfflineItemState; @@ -147,6 +148,8 @@ private void getVisualsAndUpdateItem(OfflineItem item) { } private void pushItemToUi(OfflineItem item, OfflineItemVisuals visuals) { + if (!shouldShowNotification(item)) return; + DownloadInfo info = DownloadInfo.fromOfflineItem(item, visuals); switch (item.state) { case OfflineItemState.IN_PROGRESS: @@ -220,4 +223,10 @@ private boolean shouldCacheVisuals(OfflineItem item) { return false; } } + + private boolean shouldShowNotification(OfflineItem item) { + // Temporarily return immediately to prevent unnecessary notifications for offline pages + // until https://crbug.com/831083 and https://crbug.com/832282 are fixed. + return !item.isTransient && !LegacyHelpers.isLegacyOfflinePage(item.id); + } }