Reland "Remove superfluous callback.h includes from //chrome headers" [chromium/src : main]

0 views
Skip to first unread message

Victor Vianna (Gerrit)

unread,
Nov 12, 2025, 12:51:10 PMNov 12
to Lei Zhang, AyeAye, Chromium LUCI CQ, Daniel Cheng, Thiabaud Engelbrecht, Tommy Li, chromium...@chromium.org, Kaan Icer, Nikhil Nayunigari, Sean Maher, Enterprise Policy Reviews, prerendering-reviews, devtools...@chromium.org, Mathias Bynens, Sophie Chang, Christian Biesinger, (Julie)Jeongeun Kim, Peter Beverloo, jasonrhee+w...@google.com, xlythe+wa...@google.com, francisjp...@google.com, kinuko+...@chromium.org, hsuregan+wa...@chromium.org, tranbaod...@chromium.org, zelin+watch-we...@chromium.org, filesapp...@chromium.org, keithle...@chromium.org, crost...@chromium.org, dibyapal+wa...@chromium.org, pushi+wat...@google.com, storage...@chromium.org, chrome-gr...@chromium.org, npm+...@chromium.org, yhanad...@chromium.org, hirokisa...@chromium.org, shuche...@chromium.org, feature-me...@chromium.org, marimatthew+watc...@google.com, yigu+...@chromium.org, loyso...@chromium.org, arc-review...@google.com, stevenjb+wa...@chromium.org, print-rev...@chromium.org, mfoltz+wa...@chromium.org, mreichhoff+watc...@chromium.org, gavin...@chromium.org, asumane...@google.com, chadduffin+w...@chromium.org, croissant-...@chromium.org, cros-print...@google.com, kaklilu+watch-...@chromium.org, cros-report...@google.com, omnibox-...@chromium.org, josiah...@chromium.org, jonmann+wa...@chromium.org, chromium-a...@chromium.org, mfoltz+wa...@chromium.org, extension...@chromium.org, yuzo+...@chromium.org, jackshira+w...@google.com, mtomas...@chromium.org, feature-v...@chromium.org, vasilii+watchlis...@chromium.org, dmurph+watc...@chromium.org, kuragin+web-ap...@chromium.org, jkeitel+watch-...@google.com, cros-ed...@google.com, bcb+watch-pl...@chromium.org, rhalava...@chromium.org, dtseng...@chromium.org, rrsilva+wat...@google.com, dclasson+w...@google.com, dtraino...@chromium.org, crisrael+w...@google.com, ejcaruso+wa...@chromium.org, yhanada+...@chromium.org, dewitt...@chromium.org, jackshira+w...@google.com, minap+watch-p...@google.com, cros-enterpris...@chromium.org, hansberry+w...@chromium.org, net-r...@chromium.org, hidehik...@chromium.org, tbarzi...@chromium.org, hais+wat...@google.com, philli...@chromium.org, bhartmire+w...@google.com, webap...@microsoft.com, asvitki...@chromium.org, hansenmichael...@google.com, jackshira+w...@google.com, kyungjunle...@google.com, abigailbk...@google.com, mgiuca...@chromium.org, suetfei+wa...@google.com, khorimoto+w...@chromium.org, rginda...@chromium.org, nektar...@chromium.org, hansberry+w...@chromium.org, tburkar...@chromium.org, nona+...@chromium.org, jiajunz+wa...@google.com, chromiumme...@microsoft.com, jdonnel...@chromium.org, gcasto+w...@chromium.org, ajayramamurth...@google.com, ajayramamurth...@google.com
Attention needed from Daniel Cheng and Lei Zhang

Victor Vianna voted

Code-Review+1
Commit-Queue+2
Open in Gerrit

Related details

Attention is currently required from:
  • Daniel Cheng
  • Lei Zhang
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: Ieabb6a041e69aef0a0cf567133f9a14b99517f7f
Gerrit-Change-Number: 7147125
Gerrit-PatchSet: 2
Gerrit-Owner: Lei Zhang <the...@chromium.org>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Daniel Cheng <dch...@chromium.org>
Gerrit-Reviewer: Lei Zhang <the...@chromium.org>
Gerrit-Reviewer: Victor Vianna <victor...@google.com>
Gerrit-CC: (Julie)Jeongeun Kim <je_jul...@chromium.org>
Gerrit-CC: Akihiro Ota <akihi...@chromium.org>
Gerrit-CC: Christian Biesinger <cbies...@chromium.org>
Gerrit-CC: Enterprise Policy Reviews <enterprise-p...@google.com>
Gerrit-CC: Kaan Icer <ic...@chromium.org>
Gerrit-CC: Mathias Bynens <mat...@chromium.org>
Gerrit-CC: Nikhil Nayunigari <nikh...@google.com>
Gerrit-CC: Peter Beverloo <pe...@chromium.org>
Gerrit-CC: Sean Maher <sp...@chromium.org>
Gerrit-CC: Sophie Chang <sophi...@chromium.org>
Gerrit-CC: Thiabaud Engelbrecht <thia...@google.com>
Gerrit-CC: Tommy Li <tomm...@chromium.org>
Gerrit-CC: prerendering-reviews <prerenderi...@chromium.org>
Gerrit-Attention: Lei Zhang <the...@chromium.org>
Gerrit-Attention: Daniel Cheng <dch...@chromium.org>
Gerrit-Comment-Date: Wed, 12 Nov 2025 17:50:57 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Chromium LUCI CQ (Gerrit)

unread,
Nov 12, 2025, 2:18:44 PMNov 12
to Lei Zhang, Victor Vianna, AyeAye, Daniel Cheng, Thiabaud Engelbrecht, Tommy Li, chromium...@chromium.org, Kaan Icer, Nikhil Nayunigari, Sean Maher, Akihiro Ota, Enterprise Policy Reviews, prerendering-reviews, devtools...@chromium.org, Mathias Bynens, Sophie Chang, Christian Biesinger, (Julie)Jeongeun Kim, Peter Beverloo, jasonrhee+w...@google.com, xlythe+wa...@google.com, francisjp...@google.com, kinuko+...@chromium.org, hsuregan+wa...@chromium.org, tranbaod...@chromium.org, zelin+watch-we...@chromium.org, filesapp...@chromium.org, keithle...@chromium.org, crost...@chromium.org, dibyapal+wa...@chromium.org, pushi+wat...@google.com, storage...@chromium.org, chrome-gr...@chromium.org, npm+...@chromium.org, yhanad...@chromium.org, hirokisa...@chromium.org, shuche...@chromium.org, feature-me...@chromium.org, marimatthew+watc...@google.com, yigu+...@chromium.org, loyso...@chromium.org, arc-review...@google.com, stevenjb+wa...@chromium.org, print-rev...@chromium.org, mfoltz+wa...@chromium.org, mreichhoff+watc...@chromium.org, gavin...@chromium.org, asumane...@google.com, chadduffin+w...@chromium.org, croissant-...@chromium.org, cros-print...@google.com, kaklilu+watch-...@chromium.org, cros-report...@google.com, omnibox-...@chromium.org, josiah...@chromium.org, jonmann+wa...@chromium.org, chromium-a...@chromium.org, mfoltz+wa...@chromium.org, extension...@chromium.org, yuzo+...@chromium.org, jackshira+w...@google.com, mtomas...@chromium.org, feature-v...@chromium.org, vasilii+watchlis...@chromium.org, dmurph+watc...@chromium.org, kuragin+web-ap...@chromium.org, jkeitel+watch-...@google.com, cros-ed...@google.com, bcb+watch-pl...@chromium.org, rhalava...@chromium.org, dtseng...@chromium.org, rrsilva+wat...@google.com, dclasson+w...@google.com, dtraino...@chromium.org, crisrael+w...@google.com, ejcaruso+wa...@chromium.org, yhanada+...@chromium.org, dewitt...@chromium.org, jackshira+w...@google.com, minap+watch-p...@google.com, cros-enterpris...@chromium.org, hansberry+w...@chromium.org, net-r...@chromium.org, hidehik...@chromium.org, tbarzi...@chromium.org, hais+wat...@google.com, philli...@chromium.org, bhartmire+w...@google.com, webap...@microsoft.com, asvitki...@chromium.org, hansenmichael...@google.com, jackshira+w...@google.com, kyungjunle...@google.com, abigailbk...@google.com, mgiuca...@chromium.org, suetfei+wa...@google.com, khorimoto+w...@chromium.org, rginda...@chromium.org, nektar...@chromium.org, hansberry+w...@chromium.org, tburkar...@chromium.org, nona+...@chromium.org, jiajunz+wa...@google.com, chromiumme...@microsoft.com, jdonnel...@chromium.org, gcasto+w...@chromium.org, ajayramamurth...@google.com, ajayramamurth...@google.com

Chromium LUCI CQ submitted the change

Change information

Commit message:
Reland "Remove superfluous callback.h includes from //chrome headers"

This is a reland of commit 902f4bbfac86c31b03fa5db7c2fc9c68c371171c

The original CL got reverted because a gardener thought this caused a
test failure, which is not the case.

Original change's description:
> Remove superfluous callback.h includes from //chrome headers
>
> Remove unused includes mostly found by heuristics in headers that do not
> contain keywords like "Callback" or "Closure".
>
> Bug: 40318405
> Change-Id: I4cc8a9cb5a3aaf3b8d149360eb4ebd5cb002f92a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7139543
> Reviewed-by: Victor Vianna <victor...@google.com>
> Commit-Queue: Daniel Cheng <dch...@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1543439}
Bug: 40318405
Change-Id: Ieabb6a041e69aef0a0cf567133f9a14b99517f7f
Commit-Queue: Lei Zhang <the...@chromium.org>
Commit-Queue: Victor Vianna <victor...@google.com>
Auto-Submit: Lei Zhang <the...@chromium.org>
Reviewed-by: Victor Vianna <victor...@google.com>
Cr-Commit-Position: refs/heads/main@{#1543854}
Files:
  • M chrome/browser/ash/account_manager/account_manager_edu_coexistence_controller.h
  • M chrome/browser/ash/arc/input_method_manager/arc_input_method_state.h
  • M chrome/browser/ash/exo/chrome_data_exchange_delegate.h
  • M chrome/browser/ash/file_manager/io_task_controller.h
  • M chrome/browser/ash/file_system_provider/operation_request_manager.h
  • M chrome/browser/ash/growth/ui_action_performer.h
  • M chrome/browser/ash/input_method/text_field_contextual_info_fetcher.h
  • M chrome/browser/ash/login/demo_mode/demo_mode_idle_handler.h
  • M chrome/browser/ash/login/enrollment/timebound_user_context_holder.h
  • M chrome/browser/ash/login/smart_lock/smart_lock_auth_attempt.h
  • M chrome/browser/ash/login/smart_lock/smart_lock_notification_controller.h
  • M chrome/browser/ash/plugin_vm/plugin_vm_features.h
  • M chrome/browser/ash/printing/enterprise/print_servers_provider.h
  • M chrome/browser/ash/remote_apps/remote_apps_model.h
  • M chrome/browser/ash/wallpaper_handlers/mock_sea_pen_fetcher.h
  • M chrome/browser/component_updater/afp_blocked_domain_list_component_installer.h
  • M chrome/browser/component_updater/iwa_key_distribution_component_installer.h
  • M chrome/browser/contextual_tasks/contextual_tasks_ui_service.h
  • M chrome/browser/devtools/device/adb/mock_adb_server.h
  • M chrome/browser/download/android/download_controller_base.h
  • M chrome/browser/download/android/mock_download_controller.h
  • M chrome/browser/extensions/api/image_writer_private/zip_extractor.h
  • M chrome/browser/extensions/api/settings_private/settings_private_delegate.h
  • M chrome/browser/extensions/theme_installed_infobar_delegate.h
  • M chrome/browser/glic/host/context/glic_share_image_handler.h
  • M chrome/browser/media/router/providers/wired_display/wired_display_presentation_receiver.h
  • M chrome/browser/media_galleries/mtp_read_file_worker.h
  • M chrome/browser/metrics/desktop_session_duration/audible_contents_tracker.h
  • M chrome/browser/nearby_sharing/bluetooth_advertising_interval_client.h
  • M chrome/browser/nearby_sharing/contacts/nearby_share_contact_manager.h
  • M chrome/browser/nearby_sharing/contacts/nearby_share_contact_manager_impl.h
  • M chrome/browser/nearby_sharing/instantmessaging/stream_parser.h
  • M chrome/browser/new_tab_page/chrome_colors/chrome_colors_service.h
  • M chrome/browser/notifications/notification_display_queue.h
  • M chrome/browser/notifications/scheduler/internal/display_decider.h
  • M chrome/browser/password_manager/android/grouped_affiliations/acknowledge_grouped_credential_sheet_controller_test_helper.h
  • M chrome/browser/picture_in_picture/scoped_picture_in_picture_occlusion_observation.h
  • M chrome/browser/policy/messaging_layer/util/upload_response_parser.h
  • M chrome/browser/preloading/preview/preview_test_util.h
  • M chrome/browser/resource_coordinator/tab_load_tracker.h
  • M chrome/browser/screen_ai/screen_ai_install_state.h
  • M chrome/browser/ssl/chrome_security_state_tab_helper.h
  • M chrome/browser/sync_file_system/drive_backend/drive_backend_util.h
  • M chrome/browser/sync_file_system/local/syncable_file_operation_runner.h
  • M chrome/browser/ui/android/fast_checkout/fast_checkout_view_impl.h
  • M chrome/browser/ui/android/webid/account_selection_view_android.h
  • M chrome/browser/ui/ash/network/tether_notification_presenter.h
  • M chrome/browser/ui/autofill/autofill_suggestion_controller_test_base.h
  • M chrome/browser/ui/interventions/intervention_infobar_delegate.h
  • M chrome/browser/ui/omnibox/alternate_nav_infobar_delegate.h
  • M chrome/browser/ui/plus_addresses/android/all_plus_addresses_bottom_sheet_view.h
  • M chrome/browser/ui/screen_capture_notification_ui.h
  • M chrome/browser/ui/sharing_hub/fake_sharing_hub_bubble_controller.h
  • M chrome/browser/ui/tab_modal_confirm_dialog_delegate.h
  • M chrome/browser/ui/views/frame/browser_frame_header_chromeos.h
  • M chrome/browser/ui/views/safe_browsing/password_reuse_modal_warning_dialog.h
  • M chrome/browser/ui/views/safe_browsing/tailored_security_unconsented_modal.h
  • M chrome/browser/ui/views/toolbar/toolbar_action_view.h
  • M chrome/browser/ui/views/webid/account_selection_bubble_view.h
  • M chrome/browser/ui/webui/nearby_internals/quick_pair/quick_pair_handler.h
  • M chrome/browser/ui/webui/signin/managed_user_profile_notice_ui.h
  • M chrome/browser/ui/webui/top_chrome/preload_candidate_selector.h
  • M chrome/browser/unexpire_flags.h
  • M chrome/browser/vr/elements/transient_element.h
  • M chrome/browser/vr/elements/ui_element.h
  • M chrome/browser/vr/test/ui_utils.h
  • M chrome/browser/web_applications/os_integration/web_app_uninstallation_via_os_settings_registration.h
  • M chrome/common/profiler/native_unwinder_android_map_delegate_impl.h
  • M chrome/renderer/net/net_error_helper.h
  • M chrome/renderer/net/net_error_helper_core.h
  • M chrome/renderer/supervised_user/supervised_user_error_page_controller.h
  • M chrome/test/accessibility/ax_client/ax_client_uia.h
  • M chrome/test/chromedriver/chrome/web_view_impl.h
  • M chrome/updater/win/task_scheduler.h
  • M chrome/utility/safe_browsing/rar_analyzer.h
  • M chrome/utility/safe_browsing/seven_zip_analyzer.h
  • M chrome/utility/safe_browsing/zip_analyzer.h
Change size: M
Delta: 77 files changed, 0 insertions(+), 79 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Victor Vianna
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: Ieabb6a041e69aef0a0cf567133f9a14b99517f7f
Gerrit-Change-Number: 7147125
Gerrit-PatchSet: 3
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages