Discharge7443/brave-core
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[cr149] ToolbarButtonProvider now unowned data

Browse Source 
Chromium changes:
https://chromium.googlesource.com/chromium/src/+/d37c85afb67b54cef97da7097943b250dc8e6590

commit d37c85afb67b54cef97da7097943b250dc8e6590
Author: Thomas Lukaszewicz <tluk@chromium.org>
Date:   Mon Apr 20 11:08:23 2026 -0700

    [bedrock] Update ToolbarButtonProvider to support UnownedUserData

    Allows clients to depend on ToolbarButtonProvider without a
    direct dependency on BrowserView or Browser.

    Bug: 502745808
    Change-Id: If060854181eceba5942c69570f36e3cbb2ac7af9
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7770291
    Commit-Queue: Thomas Lukaszewicz <tluk@chromium.org>
    Reviewed-by: David Yeung <dayeung@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#1617621}
This commit is contained in:
Claudio DeSouza
2026-05-22 16:25:05 -04:00
committed by Max Karolinskiy
parent 824695407f
commit e45264bb6c
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
browser/ui/views/frame/brave_browser_view.cc
+1 -1
View File
@@ -679,7 +679,7 @@ void BraveBrowserView::ShowPlaylistBubble() {
#if BUILDFLAG(ENABLE_BRAVE_WAYBACK_MACHINE)
void BraveBrowserView::ShowWaybackMachineBubble() {
if (auto* anchor = toolbar_button_provider_->GetPageActionIconView(
if (auto* anchor = toolbar_button_provider()->GetPageActionIconView(
brave::kWaybackMachineActionIconType)) {
DCHECK(anchor->GetVisible());
// Launch bubble with this anchor.