bazel/mac/app_helpers.bzl

@@ -5,13 +5,10 @@
 load("@bazel_skylib//rules:expand_template.bzl", "expand_template")
 load("@build_bazel_rules_apple//apple:macos.bzl", "macos_application")
 load("//bazel:variables.bzl", "VERSION_PLIST")
-load("//bazel/mac:variables.bzl",
-     "MACOS_DEPLOYMENT_TARGET",
-     "MACOS_BUNDLE_ID_BASE",
-     "CEF_FRAMEWORK_NAME",
+load("//bazel/mac:variables.bzl", "MACOS_DEPLOYMENT_TARGET", "MACOS_BUNDLE_ID_BASE",
                                   "COMMON_LINKOPTS")
 
-def _declare_helper_app(name, info_plist, deps, helper_base_name, helper_suffix, **kwargs):
+def _declare_helper_app(name, info_plist, deps, helper_base_name, helper_suffix):
     """
     Creates a Helper .app target.
     """
@@ -46,7 +43,6 @@ def _declare_helper_app(name, info_plist, deps, helper_base_name, helper_suffix,
         deps = [
             "@cef//:cef_sandbox",
         ] + deps,
-        **kwargs,
     )
 
 HELPERS = {
@@ -57,7 +53,7 @@ HELPERS = {
     "HelperRenderer": "Renderer",
 }
 
-def declare_all_helper_apps(name, info_plist, deps, **kwargs):
+def declare_all_helper_apps(name, info_plist, deps):
     """
     Creates all Helper .app targets.
     """
@@ -67,10 +63,9 @@ def declare_all_helper_apps(name, info_plist, deps, **kwargs):
         deps = deps,
         helper_base_name = h,
         helper_suffix = v,
-        **kwargs,
     ) for h, v in HELPERS.items()]
 
-def declare_main_app(name, info_plist, deps, resources, linkopts=[], **kwargs):
+def declare_main_app(name, info_plist, deps, resources, linkopts=[]):
     """
     Creates the main .app target.
     """
@@ -97,7 +92,6 @@ def declare_main_app(name, info_plist, deps, resources, linkopts=[], **kwargs):
             ":HelperGPU": "Frameworks",
             ":HelperPlugin": "Frameworks",
             ":HelperRenderer": "Frameworks",
-            "@cef//:cef_framework": "Frameworks/{}.framework".format(CEF_FRAMEWORK_NAME),
         },
         bundle_name = name,
         bundle_id = "{}.{}".format(MACOS_BUNDLE_ID_BASE, name.lower()),
@@ -108,6 +102,7 @@ def declare_main_app(name, info_plist, deps, resources, linkopts=[], **kwargs):
         target_compatible_with = [
             "@platforms//os:macos",
         ],
-        deps = deps,
-        **kwargs,
+        deps = [
+            "@cef//:cef_framework",
+        ] + deps,
     )

libcef/browser/chrome/browser_delegate.h

@@ -164,14 +164,13 @@ class BrowserDelegate : public content::WebContentsDelegate {
   virtual bool HasViewsHostedOpener() const { return false; }
 
   // Same as OpenURLFromTab but only taking |navigation_handle_callback|
-  // if the return value is false. Return false to cancel the navigation
-  // or true to proceed with default chrome handling.
-  virtual bool OpenURLFromTabEx(
+  // if the return value is non-nullptr.
+  virtual content::WebContents* OpenURLFromTabEx(
       content::WebContents* source,
       const content::OpenURLParams& params,
       base::OnceCallback<void(content::NavigationHandle&)>&
           navigation_handle_callback) {
-    return true;
+    return nullptr;
   }
 };
 

libcef/browser/chrome/chrome_browser_delegate.cc

@@ -526,7 +526,7 @@ void ChromeBrowserDelegate::WebContentsCreated(
                             /*is_devtools_popup=*/false, opener);
 }
 
-bool ChromeBrowserDelegate::OpenURLFromTabEx(
+content::WebContents* ChromeBrowserDelegate::OpenURLFromTabEx(
     content::WebContents* source,
     const content::OpenURLParams& params,
     base::OnceCallback<void(content::NavigationHandle&)>&
@@ -535,31 +535,17 @@ bool ChromeBrowserDelegate::OpenURLFromTabEx(
   // Reading List sidebar. In that case we default to using the Browser's
   // currently active WebContents.
   if (!source) {
-    // GetActiveWebContents() may return nullptr if we're in a new Browser
-    // created using ScopedTabbedBrowserDisplayer. This new Browser does
-    // not have a WebContents yet.
     source = browser_->tab_strip_model()->GetActiveWebContents();
-  }
-  if (!source) {
-    LOG(WARNING) << "Failed to identify target browser for "
-                 << params.url.spec();
-    // Proceed with default chrome handling.
-    return true;
+    DCHECK(source);
   }
 
+  // Return nullptr to cancel the navigation. Otherwise, proceed with default
+  // chrome handling.
   if (auto delegate = GetDelegateForWebContents(source)) {
-    // Returns nullptr to cancel the navigation.
-    const bool cancel =
-        delegate->OpenURLFromTabEx(source, params,
-                                   navigation_handle_callback) == nullptr;
-    if (cancel) {
-      // Cancel the navigation.
-      return false;
+    return delegate->OpenURLFromTabEx(source, params,
+                                      navigation_handle_callback);
   }
-  }
-
-  // Proceed with default chrome handling.
-  return true;
+  return nullptr;
 }
 
 void ChromeBrowserDelegate::LoadingStateChanged(content::WebContents* source,

libcef/browser/chrome/chrome_browser_delegate.h

@@ -90,7 +90,8 @@ class ChromeBrowserDelegate : public cef::BrowserDelegate {
   const std::optional<SkRegion> GetDraggableRegion() const override;
   void WindowFullscreenStateChanged() override;
   bool HasViewsHostedOpener() const override;
-  bool OpenURLFromTabEx(content::WebContents* source,
+  content::WebContents* OpenURLFromTabEx(
+      content::WebContents* source,
       const content::OpenURLParams& params,
       base::OnceCallback<void(content::NavigationHandle&)>&
           navigation_handle_callback) override;

patch/patch.cfg

@@ -278,10 +278,6 @@ patches = [
   {
     # chrome: Support custom DownloadManagerDelegate handling.
     # https://github.com/chromiumembedded/cef/issues/3681
-    #
-    # chrome: Allow routing of Download bubble file open to non-Tabbed
-    # source browser.
-    # https://github.com/chromiumembedded/cef/issues/3750
     'name': 'chrome_browser_download',
   },
   {

patch/patches/chrome_browser_browser.patch

@@ -173,7 +173,7 @@ index c54ec37e56ad7..fda899ec278ef 100644
      ]
    }
 diff --git chrome/browser/ui/browser.cc chrome/browser/ui/browser.cc
-index 5513b0d21ea63..8b10a082d24a9 100644
+index 5513b0d21ea63..0c8291992f194 100644
 --- chrome/browser/ui/browser.cc
 +++ chrome/browser/ui/browser.cc
 @@ -271,6 +271,25 @@
@@ -271,14 +271,16 @@ index 5513b0d21ea63..8b10a082d24a9 100644
  }
  
  bool Browser::TabsNeedBeforeUnloadFired() const {
-@@ -1785,6 +1834,14 @@ WebContents* Browser::OpenURLFromTab(
+@@ -1785,6 +1834,16 @@ WebContents* Browser::OpenURLFromTab(
    }
  #endif  // BUILDFLAG(IS_CHROMEOS_ASH)
  
 +#if BUILDFLAG(ENABLE_CEF)
-+  if (cef_browser_delegate_ &&
-+      !cef_browser_delegate_->OpenURLFromTabEx(source, params,
-+                                               navigation_handle_callback)) {
++  if (cef_browser_delegate_) {
++    auto web_contents =
++        cef_browser_delegate_->OpenURLFromTabEx(source, params,
++                                                navigation_handle_callback);
++    if (!web_contents)
 +      return nullptr;
 +  }
 +#endif
@@ -286,7 +288,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
    NavigateParams nav_params(this, params.url, params.transition);
    nav_params.FillNavigateParamsFromOpenURLParams(params);
    nav_params.source_contents = source;
-@@ -1947,6 +2004,8 @@ void Browser::LoadingStateChanged(WebContents* source,
+@@ -1947,6 +2006,8 @@ void Browser::LoadingStateChanged(WebContents* source,
                                    bool should_show_loading_ui) {
    ScheduleUIUpdate(source, content::INVALIDATE_TYPE_LOAD);
    UpdateWindowForLoadingStateChanged(source, should_show_loading_ui);
@@ -295,7 +297,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
  }
  
  void Browser::CloseContents(WebContents* source) {
-@@ -1975,6 +2034,8 @@ void Browser::SetContentsBounds(WebContents* source, const gfx::Rect& bounds) {
+@@ -1975,6 +2036,8 @@ void Browser::SetContentsBounds(WebContents* source, const gfx::Rect& bounds) {
  }
  
  void Browser::UpdateTargetURL(WebContents* source, const GURL& url) {
@@ -304,7 +306,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
    if (!GetStatusBubble())
      return;
  
-@@ -1982,6 +2043,17 @@ void Browser::UpdateTargetURL(WebContents* source, const GURL& url) {
+@@ -1982,6 +2045,17 @@ void Browser::UpdateTargetURL(WebContents* source, const GURL& url) {
      GetStatusBubble()->SetURL(url);
  }
  
@@ -322,7 +324,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
  void Browser::ContentsMouseEvent(WebContents* source, const ui::Event& event) {
    const ui::EventType type = event.type();
    const bool exited = type == ui::EventType::kMouseExited;
-@@ -2010,6 +2082,19 @@ bool Browser::TakeFocus(content::WebContents* source, bool reverse) {
+@@ -2010,6 +2084,19 @@ bool Browser::TakeFocus(content::WebContents* source, bool reverse) {
    return false;
  }
  
@@ -342,7 +344,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
  void Browser::BeforeUnloadFired(WebContents* web_contents,
                                  bool proceed,
                                  bool* proceed_to_fire_unload) {
-@@ -2109,12 +2194,24 @@ void Browser::WebContentsCreated(WebContents* source_contents,
+@@ -2109,12 +2196,24 @@ void Browser::WebContentsCreated(WebContents* source_contents,
  
    // Make the tab show up in the task manager.
    task_manager::WebContentsTags::CreateForTabContents(new_contents);
@@ -367,7 +369,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
    // Don't show the page hung dialog when a HTML popup hangs because
    // the dialog will take the focus and immediately close the popup.
    RenderWidgetHostView* view = render_widget_host->GetView();
-@@ -2127,6 +2224,13 @@ void Browser::RendererUnresponsive(
+@@ -2127,6 +2226,13 @@ void Browser::RendererUnresponsive(
  void Browser::RendererResponsive(
      WebContents* source,
      content::RenderWidgetHost* render_widget_host) {
@@ -381,7 +383,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
    RenderWidgetHostView* view = render_widget_host->GetView();
    if (view && !render_widget_host->GetView()->IsHTMLFormPopup()) {
      TabDialogs::FromWebContents(source)->HideHungRendererDialog(
-@@ -2136,6 +2240,15 @@ void Browser::RendererResponsive(
+@@ -2136,6 +2242,15 @@ void Browser::RendererResponsive(
  
  content::JavaScriptDialogManager* Browser::GetJavaScriptDialogManager(
      WebContents* source) {
@@ -397,7 +399,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
    return javascript_dialogs::TabModalDialogManager::FromWebContents(source);
  }
  
-@@ -2171,6 +2284,11 @@ void Browser::DraggableRegionsChanged(
+@@ -2171,6 +2286,11 @@ void Browser::DraggableRegionsChanged(
    if (app_controller_) {
      app_controller_->DraggableRegionsChanged(regions, contents);
    }
@@ -409,7 +411,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
  }
  
  void Browser::DidFinishNavigation(
-@@ -2251,11 +2369,15 @@ void Browser::EnterFullscreenModeForTab(
+@@ -2251,11 +2371,15 @@ void Browser::EnterFullscreenModeForTab(
      const blink::mojom::FullscreenOptions& options) {
    exclusive_access_manager_->fullscreen_controller()->EnterFullscreenModeForTab(
        requesting_frame, options.display_id);
@@ -425,7 +427,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
  }
  
  bool Browser::IsFullscreenForTabOrPending(const WebContents* web_contents) {
-@@ -2456,6 +2578,15 @@ void Browser::RequestMediaAccessPermission(
+@@ -2456,6 +2580,15 @@ void Browser::RequestMediaAccessPermission(
      content::WebContents* web_contents,
      const content::MediaStreamRequest& request,
      content::MediaResponseCallback callback) {
@@ -441,7 +443,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
    const extensions::Extension* extension =
        GetExtensionForOrigin(profile_, request.security_origin);
    MediaCaptureDevicesDispatcher::GetInstance()->ProcessMediaAccessRequest(
-@@ -2998,9 +3129,11 @@ void Browser::RemoveScheduledUpdatesFor(WebContents* contents) {
+@@ -2998,9 +3131,11 @@ void Browser::RemoveScheduledUpdatesFor(WebContents* contents) {
  // Browser, Getters for UI (private):
  
  StatusBubble* Browser::GetStatusBubble() {
@@ -454,7 +456,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
    }
  
    // We hide the status bar for web apps windows as this matches native
-@@ -3008,6 +3141,12 @@ StatusBubble* Browser::GetStatusBubble() {
+@@ -3008,6 +3143,12 @@ StatusBubble* Browser::GetStatusBubble() {
    // mode, as the minimal browser UI includes the status bar.
    if (web_app::AppBrowserController::IsWebApp(this) &&
        !app_controller()->HasMinimalUiButtons()) {
@@ -467,7 +469,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
      return nullptr;
    }
  
-@@ -3157,6 +3296,8 @@ void Browser::SetAsDelegate(WebContents* web_contents, bool set_delegate) {
+@@ -3157,6 +3298,8 @@ void Browser::SetAsDelegate(WebContents* web_contents, bool set_delegate) {
      BookmarkTabHelper::FromWebContents(web_contents)->RemoveObserver(this);
      web_contents_collection_.StopObserving(web_contents);
    }
@@ -476,7 +478,7 @@ index 5513b0d21ea63..8b10a082d24a9 100644
  }
  
  void Browser::TabDetachedAtImpl(content::WebContents* contents,
-@@ -3311,6 +3452,14 @@ bool Browser::PictureInPictureBrowserSupportsWindowFeature(
+@@ -3311,6 +3454,14 @@ bool Browser::PictureInPictureBrowserSupportsWindowFeature(
  
  bool Browser::SupportsWindowFeatureImpl(WindowFeature feature,
                                          bool check_can_support) const {

patch/patches/chrome_browser_download.patch

@@ -1,16 +1,8 @@
 diff --git chrome/browser/download/chrome_download_manager_delegate.cc chrome/browser/download/chrome_download_manager_delegate.cc
-index 1becf94d357e9..10c169b092a3d 100644
+index 1becf94d357e9..9cb014e242917 100644
 --- chrome/browser/download/chrome_download_manager_delegate.cc
 +++ chrome/browser/download/chrome_download_manager_delegate.cc
-@@ -31,6 +31,7 @@
- #include "base/time/time.h"
- #include "build/build_config.h"
- #include "build/chromeos_buildflags.h"
-+#include "cef/libcef/features/features.h"
- #include "chrome/browser/browser_process.h"
- #include "chrome/browser/download/bubble/download_bubble_prefs.h"
- #include "chrome/browser/download/download_core_service.h"
-@@ -157,6 +158,10 @@
+@@ -157,6 +157,10 @@
  #include "chrome/browser/ash/policy/skyvault/skyvault_rename_handler.h"
  #endif
  
@@ -21,7 +13,7 @@ index 1becf94d357e9..10c169b092a3d 100644
  using content::BrowserThread;
  using content::DownloadManager;
  using download::DownloadItem;
-@@ -513,6 +518,11 @@ ChromeDownloadManagerDelegate::ChromeDownloadManagerDelegate(Profile* profile)
+@@ -513,6 +517,11 @@ ChromeDownloadManagerDelegate::ChromeDownloadManagerDelegate(Profile* profile)
    download_dialog_bridge_ = std::make_unique<DownloadDialogBridge>();
    download_message_bridge_ = std::make_unique<DownloadMessageBridge>();
  #endif
@@ -33,7 +25,7 @@ index 1becf94d357e9..10c169b092a3d 100644
  }
  
  ChromeDownloadManagerDelegate::~ChromeDownloadManagerDelegate() {
-@@ -572,6 +582,9 @@ void ChromeDownloadManagerDelegate::Shutdown() {
+@@ -572,6 +581,9 @@ void ChromeDownloadManagerDelegate::Shutdown() {
      download_manager_->RemoveObserver(this);
      download_manager_ = nullptr;
    }
@@ -43,7 +35,7 @@ index 1becf94d357e9..10c169b092a3d 100644
  }
  
  void ChromeDownloadManagerDelegate::OnDownloadCanceledAtShutdown(
-@@ -640,6 +653,12 @@ bool ChromeDownloadManagerDelegate::DetermineDownloadTarget(
+@@ -640,6 +652,12 @@ bool ChromeDownloadManagerDelegate::DetermineDownloadTarget(
      ReportPDFLoadStatus(PDFLoadStatus::kTriggeredNoGestureDriveByDownload);
    }
  
@@ -56,20 +48,6 @@ index 1becf94d357e9..10c169b092a3d 100644
    DownloadTargetDeterminer::CompletionCallback target_determined_callback =
        base::BindOnce(&ChromeDownloadManagerDelegate::OnDownloadTargetDetermined,
                       weak_ptr_factory_.GetWeakPtr(), download->GetId(),
-@@ -1027,8 +1046,11 @@ void ChromeDownloadManagerDelegate::OpenDownload(DownloadItem* download) {
-   Browser* browser =
-       web_contents ? chrome::FindBrowserWithTab(web_contents) : nullptr;
-   std::unique_ptr<chrome::ScopedTabbedBrowserDisplayer> browser_displayer;
--  if (!browser ||
--      !browser->CanSupportWindowFeature(Browser::FEATURE_TABSTRIP)) {
-+  if (!browser
-+#if !BUILDFLAG(ENABLE_CEF)
-+      || !browser->CanSupportWindowFeature(Browser::FEATURE_TABSTRIP)
-+#endif
-+    ) {
-     browser_displayer =
-         std::make_unique<chrome::ScopedTabbedBrowserDisplayer>(profile_);
-     browser = browser_displayer->browser();
 diff --git chrome/browser/download/chrome_download_manager_delegate.h chrome/browser/download/chrome_download_manager_delegate.h
 index 2c99baa2c9fa8..b721db7058d8f 100644
 --- chrome/browser/download/chrome_download_manager_delegate.h

tools/distrib/bazel/BUILD.bazel

@@ -7,8 +7,8 @@ package(default_visibility = [
     "//visibility:public",
 ])
 
-load("@aspect_bazel_lib//lib:copy_directory.bzl", "copy_directory")
 load("@bazel_skylib//lib:selects.bzl", "selects")
+load("@build_bazel_rules_apple//apple:apple.bzl", "apple_dynamic_framework_import")
 load("//bazel:library_helpers.bzl", "declare_cc_library", "declare_objc_library")
 load("//bazel/win:variables.bzl",
      WIN_DLLS="DLLS",
@@ -309,15 +309,10 @@ alias(
     }),
 )
 
-# Copy the CEF framework into the app bundle but do not link it. See
-# https://groups.google.com/g/cef-announce/c/Fith0A3kWtw/m/6ds_mJVMCQAJ
-# for background. Use `copy_directory` instead of `filegroup` to remove
-# the Debug/Release path prefix.
-copy_directory(
+apple_dynamic_framework_import(
     name = "cef_framework",
-    src = select({
-        "@cef//:dbg": "Debug/{}.framework".format(CEF_FRAMEWORK_NAME),
-        "//conditions:default": "Release/{}.framework".format(CEF_FRAMEWORK_NAME),
+    framework_imports = select({
+        "@cef//:dbg": glob(["Debug/{}.framework/**".format(CEF_FRAMEWORK_NAME)]),
+        "//conditions:default": glob(["Release/{}.framework/**".format(CEF_FRAMEWORK_NAME)]),
     }),
-    out = "{}.framework".format(CEF_FRAMEWORK_NAME),
 )

tools/distrib/bazel/MODULE.bazel.in

@@ -15,4 +15,3 @@ bazel_dep(name = "rules_apple", version = "3.6.0", repo_name = "build_bazel_rule
 bazel_dep(name = "rules_cc", version = "0.0.9")
 
 # Add other dependencies here.
-bazel_dep(name = "aspect_bazel_lib", version = "2.7.9")