Migrate MemoryPressureListeners to the interface-based API [chromium/src : main]

0 views
Skip to first unread message

Patrick Monette (Gerrit)

unread,
Oct 10, 2025, 9:57:37 AM (2 days ago) Oct 10
to Francois Pierre Doray, AI Code Reviewer, Zijie He, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, Dirk Schulze, prerendering-reviews, Stephen Chenney, ios-r...@chromium.org, ios-revie...@chromium.org, marq+...@chromium.org, oshima...@chromium.org, fuchsia...@chromium.org, alexmo...@chromium.org, blink-reviews-p...@chromium.org, blink-...@chromium.org, cblume...@chromium.org, cc-...@chromium.org, chikamu...@chromium.org, christia...@chromium.org, creis...@chromium.org, dmurph+watching...@chromium.org, dmurph+watch...@chromium.org, drott+bl...@chromium.org, edgesto...@microsoft.com, feature-me...@chromium.org, fmalit...@chromium.org, fserb...@chromium.org, gavin...@chromium.org, gavinp...@chromium.org, gavin...@chromium.org, jdonnel...@chromium.org, kinuko...@chromium.org, navigation...@chromium.org, net-r...@chromium.org, network-ser...@chromium.org, omnibox-...@chromium.org, penghu...@chromium.org, penghuan...@chromium.org, performance-m...@chromium.org, scheduler-...@chromium.org, tburkar...@chromium.org, thiabaud+watch-d...@google.com

Patrick Monette voted Commit-Queue+2

Commit-Queue+2
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I3fb50600f1331326bd287016ff4974e134e9f426
Gerrit-Change-Number: 7016972
Gerrit-PatchSet: 25
Gerrit-Owner: Patrick Monette <pmon...@chromium.org>
Gerrit-Reviewer: Francois Pierre Doray <fdo...@chromium.org>
Gerrit-Reviewer: Patrick Monette <pmon...@chromium.org>
Gerrit-CC: AI Code Reviewer <peep-gen...@system.gserviceaccount.com>
Gerrit-CC: Dirk Schulze <dsch...@chromium.org>
Gerrit-CC: Stephen Chenney <sche...@chromium.org>
Gerrit-CC: Zijie He <zij...@google.com>
Gerrit-CC: prerendering-reviews <prerenderi...@chromium.org>
Gerrit-Comment-Date: Fri, 10 Oct 2025 13:57:31 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Chromium LUCI CQ (Gerrit)

unread,
Oct 10, 2025, 10:01:57 AM (2 days ago) Oct 10
to Patrick Monette, Francois Pierre Doray, AI Code Reviewer, Zijie He, AyeAye, chromium...@chromium.org, Dirk Schulze, prerendering-reviews, Stephen Chenney, ios-r...@chromium.org, ios-revie...@chromium.org, marq+...@chromium.org, oshima...@chromium.org, fuchsia...@chromium.org, alexmo...@chromium.org, blink-reviews-p...@chromium.org, blink-...@chromium.org, cblume...@chromium.org, cc-...@chromium.org, chikamu...@chromium.org, christia...@chromium.org, creis...@chromium.org, dmurph+watching...@chromium.org, dmurph+watch...@chromium.org, drott+bl...@chromium.org, edgesto...@microsoft.com, feature-me...@chromium.org, fmalit...@chromium.org, fserb...@chromium.org, gavin...@chromium.org, gavinp...@chromium.org, gavin...@chromium.org, jdonnel...@chromium.org, kinuko...@chromium.org, navigation...@chromium.org, net-r...@chromium.org, network-ser...@chromium.org, omnibox-...@chromium.org, penghu...@chromium.org, penghuan...@chromium.org, performance-m...@chromium.org, scheduler-...@chromium.org, tburkar...@chromium.org, thiabaud+watch-d...@google.com

Chromium LUCI CQ submitted the change

Unreviewed changes

21 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Change information

Commit message:
Migrate MemoryPressureListeners to the interface-based API
Bug: 436324601
Change-Id: I3fb50600f1331326bd287016ff4974e134e9f426
Reviewed-by: Francois Pierre Doray <fdo...@chromium.org>
Owners-Override: Francois Pierre Doray <fdo...@chromium.org>
Commit-Queue: Patrick Monette <pmon...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1528115}
Files:
  • M base/memory/mock_memory_pressure_listener.cc
  • M base/memory/mock_memory_pressure_listener.h
  • M base/threading/hang_watcher.cc
  • M base/threading/hang_watcher.h
  • M cc/raster/staging_buffer_pool.cc
  • M cc/raster/staging_buffer_pool.h
  • M cc/resources/resource_pool.cc
  • M cc/resources/resource_pool.h
  • M cc/tiles/gpu_image_decode_cache.cc
  • M cc/tiles/gpu_image_decode_cache.h
  • M cc/trees/layer_tree_host_impl.cc
  • M cc/trees/layer_tree_host_impl.h
  • M chrome/browser/glic/public/glic_keyed_service.cc
  • M chrome/browser/glic/public/glic_keyed_service.h
  • M chrome/browser/performance_manager/policies/background_tab_loading_policy.cc
  • M chrome/browser/performance_manager/policies/background_tab_loading_policy.h
  • M chrome/browser/performance_manager/policies/urgent_page_discarding_policy.cc
  • M chrome/browser/performance_manager/policies/urgent_page_discarding_policy.h
  • M chrome/browser/performance_manager/policies/userspace_swap_policy_chromeos.cc
  • M chrome/browser/performance_manager/policies/userspace_swap_policy_chromeos.h
  • M chrome/browser/performance_manager/policies/working_set_trimmer_policy_chromeos.cc
  • M chrome/browser/performance_manager/policies/working_set_trimmer_policy_chromeos.h
  • M chrome/browser/sessions/tab_loader.cc
  • M chrome/browser/sessions/tab_loader.h
  • M chrome/browser/thumbnail/cc/thumbnail_cache.cc
  • M chrome/browser/thumbnail/cc/thumbnail_cache.h
  • M components/breadcrumbs/core/application_breadcrumbs_logger.cc
  • M components/breadcrumbs/core/application_breadcrumbs_logger.h
  • M components/discardable_memory/service/discardable_shared_memory_manager.cc
  • M components/discardable_memory/service/discardable_shared_memory_manager.h
  • M components/history/core/browser/history_backend.cc
  • M components/history/core/browser/history_backend.h
  • M components/omnibox/browser/on_device_tail_model_service.cc
  • M components/omnibox/browser/on_device_tail_model_service.h
  • M components/paint_preview/player/player_compositor_delegate.cc
  • M components/paint_preview/player/player_compositor_delegate.h
  • M components/performance_manager/graph/policies/bfcache_policy.cc
  • M components/performance_manager/graph/policies/bfcache_policy.h
  • M components/services/storage/shared_storage/shared_storage_manager.cc
  • M components/services/storage/shared_storage/shared_storage_manager.h
  • M components/viz/client/frame_eviction_manager.cc
  • M components/viz/client/frame_eviction_manager.h
  • M components/viz/common/gpu/vulkan_in_process_context_provider.cc
  • M components/viz/common/gpu/vulkan_in_process_context_provider.h
  • M components/viz/service/display_embedder/compositor_gpu_thread.cc
  • M components/viz/service/display_embedder/compositor_gpu_thread.h
  • M components/viz/service/display_embedder/skia_output_surface_impl.cc
  • M content/browser/cache_storage/cache_storage_manager.cc
  • M content/browser/cache_storage/cache_storage_manager.h
  • M content/browser/dom_storage/dom_storage_context_wrapper.cc
  • M content/browser/dom_storage/dom_storage_context_wrapper.h
  • M content/browser/gpu/gpu_process_host.cc
  • M content/browser/gpu/gpu_process_host.h
  • M content/browser/network_service_client.cc
  • M content/browser/network_service_client.h
  • M content/browser/preloading/prerender/prerender_host_registry.cc
  • M content/browser/preloading/prerender/prerender_host_registry.h
  • M content/browser/renderer_host/navigation_transitions/navigation_entry_screenshot_manager.cc
  • M content/browser/renderer_host/navigation_transitions/navigation_entry_screenshot_manager.h
  • M content/browser/renderer_host/spare_render_process_host_manager_impl.cc
  • M content/browser/renderer_host/spare_render_process_host_manager_impl.h
  • M content/gpu/gpu_child_thread.cc
  • M content/gpu/gpu_child_thread.h
  • M content/public/test/network_service_test_helper.cc
  • M content/renderer/render_thread_impl.cc
  • M content/renderer/render_thread_impl.h
  • M gpu/ipc/service/gpu_channel_manager.cc
  • M gpu/ipc/service/gpu_channel_manager.h
  • M ios/chrome/app/application_delegate/memory_warning_helper_unittest.mm
  • M ios/chrome/browser/crash_report/model/breadcrumbs/application_breadcrumbs_logger.h
  • M net/disk_cache/memory/mem_backend_impl.cc
  • M net/disk_cache/memory/mem_backend_impl.h
  • M net/http/http_network_session.cc
  • M net/http/http_network_session.h
  • M net/quic/quic_session_pool.cc
  • M net/quic/quic_session_pool.h
  • M net/ssl/ssl_client_session_cache.cc
  • M net/ssl/ssl_client_session_cache.h
  • M services/network/shared_dictionary/shared_dictionary_manager.cc
  • M services/network/shared_dictionary/shared_dictionary_manager.h
  • M services/network/shared_dictionary/shared_dictionary_storage_on_disk.cc
  • M services/network/shared_dictionary/shared_dictionary_storage_on_disk.h
  • M services/network/slop_bucket.cc
  • M storage/browser/blob/blob_memory_controller.cc
  • M storage/browser/blob/blob_memory_controller.h
  • M third_party/blink/renderer/controller/user_level_memory_pressure_signal_generator_test.cc
  • M third_party/blink/renderer/platform/graphics/image_decoding_store.cc
  • M third_party/blink/renderer/platform/graphics/image_decoding_store.h
  • M third_party/blink/renderer/platform/media/url_index.cc
  • M third_party/blink/renderer/platform/media/url_index.h
  • M third_party/blink/renderer/platform/scheduler/main_thread/memory_purge_manager_unittest.cc
  • M third_party/leveldatabase/leveldb_chrome.cc
Change size: L
Delta: 92 files changed, 251 insertions(+), 258 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Francois Pierre Doray
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I3fb50600f1331326bd287016ff4974e134e9f426
Gerrit-Change-Number: 7016972
Gerrit-PatchSet: 26
Gerrit-Owner: Patrick Monette <pmon...@chromium.org>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Francois Pierre Doray <fdo...@chromium.org>
Gerrit-Reviewer: Patrick Monette <pmon...@chromium.org>
Gerrit-CC: AI Code Reviewer <peep-gen...@system.gserviceaccount.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages