[Offline pages] Wrap page deletion notification parameters into one struct

Desktop / Chromium - Cathy Li [chromium.org] - 12 July 2017 19:51 EDT

Also add request_origin to this struct.

Bug: 734753 Change-Id: I210bf0fa177f1379693b6b432c12f48b89d8cd28 Reviewed-on: https://chromium-review.googlesource.com/558564

631445b [Offline pages] Wrap page deletion notification parameters into one struct.
.../browser/offlinepages/DeletedPageInfo.java | 40 +++++++
.../browser/offlinepages/OfflinePageBridge.java | 12 ++-
.../SuggestionsOfflineModelObserver.java | 6 +-
chrome/android/java_sources.gni | 1 +
.../offlinepages/OfflinePageBridgeUnitTest.java | 8 +-
.../evaluation/offline_page_evaluation_bridge.cc | 3 +-
.../evaluation/offline_page_evaluation_bridge.h | 3 +-
.../android/offline_pages/offline_page_bridge.cc | 18 +++-
.../android/offline_pages/offline_page_bridge.h | 4 +-
.../offline_pages/recent_tab_helper_unittest.cc | 3 +-
.../ntp_snippets/download_suggestions_provider.cc | 7 +-
.../ntp_snippets/download_suggestions_provider.h | 5 +-
.../download_suggestions_provider_unittest.cc | 4 +-
.../recent_tab_suggestions_provider_unittest.cc | 4 +-
.../remote/prefetched_pages_tracker_impl.cc | 5 +-
.../remote/prefetched_pages_tracker_impl.h | 5 +-
.../prefetched_pages_tracker_impl_unittest.cc | 8 +-
.../core/downloads/download_ui_adapter.cc | 8 +-
.../core/downloads/download_ui_adapter.h | 4 +-
.../core/downloads/download_ui_adapter_unittest.cc | 5 +-
.../offline_pages/core/offline_page_model.cc | 8 ++
components/offline_pages/core/offline_page_model.h | 16 ++-
.../offline_pages/core/offline_page_model_impl.cc | 3 +-
.../core/offline_page_model_impl_unittest.cc | 118 ++++++++++++++++-----
24 files changed, 224 insertions(+), 74 deletions(-)

Upstream: git.chromium.org


  • Share