2021-04-11 22:10:11 +02:00
|
|
|
diff --git chrome/browser/ui/browser_command_controller.cc chrome/browser/ui/browser_command_controller.cc
|
2024-01-04 15:42:19 +01:00
|
|
|
index 2abd329e1ba7d..e812968372741 100644
|
2021-04-11 22:10:11 +02:00
|
|
|
--- chrome/browser/ui/browser_command_controller.cc
|
|
|
|
+++ chrome/browser/ui/browser_command_controller.cc
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -400,6 +400,7 @@ bool BrowserCommandController::ExecuteCommandWithDisposition(
|
2021-04-11 22:10:11 +02:00
|
|
|
// choose to not implement CommandUpdaterDelegate inside this class and
|
|
|
|
// therefore command_updater_ doesn't have the delegate set).
|
2023-01-30 18:43:54 +01:00
|
|
|
if (!SupportsCommand(id) || !IsCommandEnabled(id)) {
|
2021-04-11 22:10:11 +02:00
|
|
|
+ LOG(WARNING) << "Invalid/disabled command " << id;
|
|
|
|
return false;
|
2023-01-30 18:43:54 +01:00
|
|
|
}
|
2021-04-11 22:10:11 +02:00
|
|
|
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -416,6 +417,13 @@ bool BrowserCommandController::ExecuteCommandWithDisposition(
|
2022-03-22 19:31:30 +01:00
|
|
|
DCHECK(command_updater_.IsCommandEnabled(id))
|
|
|
|
<< "Invalid/disabled command " << id;
|
|
|
|
|
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+ if (browser_->cef_delegate() &&
|
|
|
|
+ browser_->cef_delegate()->HandleCommand(id, disposition)) {
|
|
|
|
+ return true;
|
|
|
|
+ }
|
|
|
|
+#endif
|
|
|
|
+
|
|
|
|
// The order of commands in this switch statement must match the function
|
|
|
|
// declaration order in browser.h!
|
|
|
|
switch (id) {
|
2023-11-27 21:56:19 +01:00
|
|
|
@@ -1172,11 +1180,13 @@ void BrowserCommandController::TabRestoreServiceLoaded(
|
2021-04-11 22:10:11 +02:00
|
|
|
// BrowserCommandController, private:
|
|
|
|
|
|
|
|
bool BrowserCommandController::IsShowingMainUI() {
|
|
|
|
- return browser_->SupportsWindowFeature(Browser::FEATURE_TABSTRIP);
|
|
|
|
+ return browser_->SupportsWindowFeature(Browser::FEATURE_TABSTRIP) ||
|
|
|
|
+ browser_->toolbar_overridden();
|
|
|
|
}
|
|
|
|
|
|
|
|
bool BrowserCommandController::IsShowingLocationBar() {
|
|
|
|
- return browser_->SupportsWindowFeature(Browser::FEATURE_LOCATIONBAR);
|
|
|
|
+ return browser_->SupportsWindowFeature(Browser::FEATURE_LOCATIONBAR) ||
|
|
|
|
+ browser_->toolbar_overridden();
|
|
|
|
}
|
|
|
|
|
2022-02-21 23:23:40 +01:00
|
|
|
bool BrowserCommandController::IsWebAppOrCustomTab() const {
|
2023-03-07 00:21:57 +01:00
|
|
|
diff --git chrome/browser/ui/toolbar/app_menu_model.cc chrome/browser/ui/toolbar/app_menu_model.cc
|
2024-01-04 15:42:19 +01:00
|
|
|
index 38b59cea39875..9106229719db6 100644
|
2023-03-07 00:21:57 +01:00
|
|
|
--- chrome/browser/ui/toolbar/app_menu_model.cc
|
|
|
|
+++ chrome/browser/ui/toolbar/app_menu_model.cc
|
2023-11-30 15:45:56 +01:00
|
|
|
@@ -594,6 +594,57 @@ SaveAndShareSubMenuModel::SaveAndShareSubMenuModel(
|
2023-05-30 10:55:32 +02:00
|
|
|
}
|
2023-03-07 00:21:57 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+using IsVisibleCallback = base::RepeatingCallback<bool(int)>;
|
|
|
|
+
|
|
|
|
+void FilterMenuModel(ui::SimpleMenuModel* model,
|
|
|
|
+ const IsVisibleCallback& is_visible) {
|
|
|
|
+ absl::optional<size_t> last_separator;
|
|
|
|
+ size_t visible_ct = 0;
|
|
|
|
+ for (size_t i = 0; i < model->GetItemCount(); ++i) {
|
|
|
|
+ const auto type = model->GetTypeAt(i);
|
|
|
|
+ if (type == ui::MenuModel::TYPE_SEPARATOR) {
|
|
|
|
+ if (last_separator) {
|
|
|
|
+ // Remove multiple separators in a row. Prefer to remove a NORMAL
|
|
|
|
+ // separator if possible (as compared to zoom/edit controls which use
|
|
|
|
+ // UPPER/LOWER separators).
|
|
|
|
+ if (model->GetSeparatorTypeAt(*last_separator) ==
|
|
|
|
+ ui::NORMAL_SEPARATOR) {
|
|
|
|
+ model->RemoveItemAt(*last_separator);
|
|
|
|
+ i--;
|
|
|
|
+ last_separator = i;
|
|
|
|
+ } else {
|
|
|
|
+ model->RemoveItemAt(i);
|
|
|
|
+ i--;
|
|
|
|
+ }
|
|
|
|
+ } else if (visible_ct == 0) {
|
|
|
|
+ // Remove leading separator.
|
|
|
|
+ model->RemoveItemAt(i);
|
|
|
|
+ i--;
|
|
|
|
+ } else {
|
|
|
|
+ last_separator = i;
|
|
|
|
+ }
|
|
|
|
+ visible_ct = 0;
|
|
|
|
+ } else if (is_visible.Run(model->GetCommandIdAt(i))) {
|
|
|
|
+ last_separator = absl::nullopt;
|
|
|
|
+ visible_ct++;
|
|
|
|
+
|
|
|
|
+ if (type == ui::MenuModel::TYPE_SUBMENU) {
|
|
|
|
+ // Filter sub-menu.
|
|
|
|
+ auto sub_model =
|
|
|
|
+ static_cast<ui::SimpleMenuModel*>(model->GetSubmenuModelAt(i));
|
|
|
|
+ FilterMenuModel(sub_model, is_visible);
|
|
|
|
+ }
|
|
|
|
+ }
|
|
|
|
+ }
|
|
|
|
+
|
|
|
|
+ if (last_separator) {
|
|
|
|
+ // Remove trailing separator.
|
|
|
|
+ model->RemoveItemAt(*last_separator);
|
|
|
|
+ }
|
|
|
|
+}
|
|
|
|
+#endif // BUILDFLAG(ENABLE_CEF)
|
|
|
|
+
|
|
|
|
} // namespace
|
|
|
|
|
|
|
|
////////////////////////////////////////////////////////////////////////////////
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -1450,7 +1501,7 @@ bool AppMenuModel::IsCommandIdChecked(int command_id) const {
|
2023-03-07 00:21:57 +01:00
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
-bool AppMenuModel::IsCommandIdEnabled(int command_id) const {
|
|
|
|
+bool AppMenuModel::IsCommandIdEnabledInternal(int command_id) const {
|
|
|
|
GlobalError* error =
|
|
|
|
GlobalErrorServiceFactory::GetForProfile(browser_->profile())
|
|
|
|
->GetGlobalErrorByMenuItemCommandID(command_id);
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -1465,6 +1516,30 @@ bool AppMenuModel::IsCommandIdEnabled(int command_id) const {
|
2023-03-07 00:21:57 +01:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
+bool AppMenuModel::IsCommandIdEnabled(int command_id) const {
|
|
|
|
+ if (!IsCommandIdEnabledInternal(command_id)) {
|
|
|
|
+ return false;
|
|
|
|
+ }
|
|
|
|
+
|
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+ if (browser_->cef_delegate()) {
|
|
|
|
+ return browser_->cef_delegate()->IsAppMenuItemEnabled(command_id);
|
|
|
|
+ }
|
|
|
|
+#endif
|
|
|
|
+
|
|
|
|
+ return true;
|
|
|
|
+}
|
|
|
|
+
|
|
|
|
+bool AppMenuModel::IsCommandIdVisible(int command_id) const {
|
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+ if (browser_->cef_delegate()) {
|
|
|
|
+ return browser_->cef_delegate()->IsAppMenuItemVisible(command_id);
|
|
|
|
+ }
|
|
|
|
+#endif
|
|
|
|
+
|
|
|
|
+ return true;
|
|
|
|
+}
|
|
|
|
+
|
|
|
|
bool AppMenuModel::IsCommandIdAlerted(int command_id) const {
|
2023-11-21 20:17:55 +01:00
|
|
|
if (command_id == IDC_VIEW_PASSWORDS ||
|
|
|
|
command_id == IDC_SHOW_PASSWORD_MANAGER) {
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -1630,11 +1705,15 @@ void AppMenuModel::Build() {
|
2023-07-21 00:11:34 +02:00
|
|
|
kDefaultIconSize));
|
2023-03-07 00:21:57 +01:00
|
|
|
}
|
|
|
|
|
2023-04-26 21:55:59 +02:00
|
|
|
- AddSeparator(features::IsChromeRefresh2023() ? ui::NORMAL_SEPARATOR
|
|
|
|
- : ui::LOWER_SEPARATOR);
|
2023-03-07 00:21:57 +01:00
|
|
|
- CreateZoomMenu();
|
2023-04-26 21:55:59 +02:00
|
|
|
- AddSeparator(features::IsChromeRefresh2023() ? ui::NORMAL_SEPARATOR
|
|
|
|
- : ui::UPPER_SEPARATOR);
|
2023-03-07 00:21:57 +01:00
|
|
|
+ if (IsCommandIdVisible(IDC_ZOOM_MENU)) {
|
2023-04-26 21:55:59 +02:00
|
|
|
+ AddSeparator(features::IsChromeRefresh2023() ? ui::NORMAL_SEPARATOR
|
|
|
|
+ : ui::LOWER_SEPARATOR);
|
2023-03-07 00:21:57 +01:00
|
|
|
+ CreateZoomMenu();
|
2023-04-26 21:55:59 +02:00
|
|
|
+ AddSeparator(features::IsChromeRefresh2023() ? ui::NORMAL_SEPARATOR
|
|
|
|
+ : ui::UPPER_SEPARATOR);
|
2023-03-07 00:21:57 +01:00
|
|
|
+ } else {
|
|
|
|
+ AddSeparator(ui::NORMAL_SEPARATOR);
|
|
|
|
+ }
|
|
|
|
|
|
|
|
AddItemWithStringId(IDC_PRINT, IDS_PRINT);
|
|
|
|
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -1723,9 +1802,13 @@ void AppMenuModel::Build() {
|
2023-03-07 00:21:57 +01:00
|
|
|
kMoreToolsMenuItem);
|
|
|
|
|
2023-04-26 21:55:59 +02:00
|
|
|
if (!features::IsChromeRefresh2023()) {
|
|
|
|
- AddSeparator(ui::LOWER_SEPARATOR);
|
|
|
|
- CreateCutCopyPasteMenu();
|
|
|
|
- AddSeparator(ui::UPPER_SEPARATOR);
|
|
|
|
+ if (IsCommandIdVisible(IDC_EDIT_MENU)) {
|
|
|
|
+ AddSeparator(ui::LOWER_SEPARATOR);
|
|
|
|
+ CreateCutCopyPasteMenu();
|
|
|
|
+ AddSeparator(ui::UPPER_SEPARATOR);
|
|
|
|
+ } else {
|
|
|
|
+ AddSeparator(ui::NORMAL_SEPARATOR);
|
|
|
|
+ }
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!features::IsChromeRefresh2023()) {
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -1812,6 +1895,11 @@ void AppMenuModel::Build() {
|
2023-05-30 10:55:32 +02:00
|
|
|
SetCommandIcon(this, IDC_EXIT, kExitMenuIcon);
|
2023-03-07 00:21:57 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+ FilterMenuModel(this, base::BindRepeating(&AppMenuModel::IsCommandIdVisible,
|
|
|
|
+ base::Unretained(this)));
|
|
|
|
+#endif
|
|
|
|
+
|
|
|
|
uma_action_recorded_ = false;
|
|
|
|
}
|
|
|
|
|
|
|
|
diff --git chrome/browser/ui/toolbar/app_menu_model.h chrome/browser/ui/toolbar/app_menu_model.h
|
2024-01-04 15:42:19 +01:00
|
|
|
index bba8a3887c023..967f424d88ad2 100644
|
2023-03-07 00:21:57 +01:00
|
|
|
--- chrome/browser/ui/toolbar/app_menu_model.h
|
|
|
|
+++ chrome/browser/ui/toolbar/app_menu_model.h
|
2023-11-30 15:45:56 +01:00
|
|
|
@@ -214,6 +214,7 @@ class AppMenuModel : public ui::SimpleMenuModel,
|
2023-06-26 12:13:38 +02:00
|
|
|
void ExecuteCommand(int command_id, int event_flags) override;
|
|
|
|
bool IsCommandIdChecked(int command_id) const override;
|
|
|
|
bool IsCommandIdEnabled(int command_id) const override;
|
|
|
|
+ bool IsCommandIdVisible(int command_id) const override;
|
|
|
|
bool IsCommandIdAlerted(int command_id) const override;
|
2023-11-21 20:17:55 +01:00
|
|
|
bool IsElementIdAlerted(ui::ElementIdentifier element_id) const override;
|
2023-06-26 12:13:38 +02:00
|
|
|
bool GetAcceleratorForCommandId(int command_id,
|
2023-11-30 15:45:56 +01:00
|
|
|
@@ -255,6 +256,8 @@ class AppMenuModel : public ui::SimpleMenuModel,
|
|
|
|
absl::optional<safety_hub::SafetyHubModuleType> expected_module =
|
|
|
|
absl::nullopt);
|
2023-03-07 00:21:57 +01:00
|
|
|
|
|
|
|
+ bool IsCommandIdEnabledInternal(int command_id) const;
|
|
|
|
+
|
|
|
|
private:
|
2023-04-26 21:55:59 +02:00
|
|
|
// Adds actionable global error menu items to the menu.
|
|
|
|
// Examples: Extension permissions and sign in errors.
|
2023-04-12 20:34:39 +02:00
|
|
|
diff --git chrome/browser/ui/views/find_bar_host.cc chrome/browser/ui/views/find_bar_host.cc
|
2023-04-26 21:55:59 +02:00
|
|
|
index 59024587ef6b7..0c30aa71768cf 100644
|
2023-04-12 20:34:39 +02:00
|
|
|
--- chrome/browser/ui/views/find_bar_host.cc
|
|
|
|
+++ chrome/browser/ui/views/find_bar_host.cc
|
|
|
|
@@ -412,6 +412,12 @@ void FindBarHost::GetWidgetBounds(gfx::Rect* bounds) {
|
|
|
|
// The BrowserView does Layout for the components that we care about
|
|
|
|
// positioning relative to, so we ask it to tell us where we should go.
|
|
|
|
*bounds = browser_view()->GetFindBarBoundingBox();
|
|
|
|
+
|
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+ if (browser_view()->browser() && browser_view()->browser()->cef_delegate()) {
|
|
|
|
+ browser_view()->browser()->cef_delegate()->UpdateFindBarBoundingBox(bounds);
|
|
|
|
+ }
|
|
|
|
+#endif
|
|
|
|
}
|
|
|
|
|
|
|
|
void FindBarHost::RegisterAccelerators() {
|
2021-02-18 02:58:25 +01:00
|
|
|
diff --git chrome/browser/ui/views/frame/browser_frame.cc chrome/browser/ui/views/frame/browser_frame.cc
|
2023-11-14 18:16:43 +01:00
|
|
|
index bb4b3821bcfda..4014f61a7057f 100644
|
2021-02-18 02:58:25 +01:00
|
|
|
--- chrome/browser/ui/views/frame/browser_frame.cc
|
|
|
|
+++ chrome/browser/ui/views/frame/browser_frame.cc
|
2023-09-15 21:51:43 +02:00
|
|
|
@@ -114,15 +114,23 @@ ui::ColorProviderKey::SchemeVariant GetSchemeVariant(
|
2021-02-18 02:58:25 +01:00
|
|
|
////////////////////////////////////////////////////////////////////////////////
|
|
|
|
// BrowserFrame, public:
|
|
|
|
|
|
|
|
+BrowserFrame::BrowserFrame() : BrowserFrame(nullptr) {}
|
|
|
|
+
|
|
|
|
BrowserFrame::BrowserFrame(BrowserView* browser_view)
|
|
|
|
: native_browser_frame_(nullptr),
|
|
|
|
root_view_(nullptr),
|
|
|
|
browser_frame_view_(nullptr),
|
|
|
|
- browser_view_(browser_view) {
|
|
|
|
- browser_view_->set_frame(this);
|
|
|
|
+ browser_view_(nullptr) {
|
|
|
|
set_is_secondary_widget(false);
|
|
|
|
// Don't focus anything on creation, selecting a tab will set the focus.
|
|
|
|
set_focus_on_creation(false);
|
|
|
|
+ if (browser_view)
|
|
|
|
+ InitBrowserView(browser_view);
|
|
|
|
+}
|
|
|
|
+
|
|
|
|
+void BrowserFrame::InitBrowserView(BrowserView* browser_view) {
|
|
|
|
+ browser_view_ = browser_view;
|
|
|
|
+ browser_view_->set_frame(this);
|
|
|
|
}
|
|
|
|
|
|
|
|
BrowserFrame::~BrowserFrame() {}
|
2023-11-14 18:16:43 +01:00
|
|
|
@@ -228,10 +236,20 @@ void BrowserFrame::LayoutWebAppWindowTitle(
|
2021-02-18 02:58:25 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
int BrowserFrame::GetTopInset() const {
|
|
|
|
+ if (!browser_frame_view_) {
|
|
|
|
+ // With CEF the browser may already be part of a larger Views layout. Zero
|
|
|
|
+ // out the adjustment in BrowserView::GetTopInsetInBrowserView() so that
|
|
|
|
+ // the browser isn't shifted to the top of the window.
|
|
|
|
+ return browser_view_->y();
|
|
|
|
+ }
|
|
|
|
return browser_frame_view_->GetTopInset(false);
|
|
|
|
}
|
|
|
|
|
2023-11-14 18:16:43 +01:00
|
|
|
void BrowserFrame::UpdateThrobber(bool running) {
|
|
|
|
+ if (!browser_frame_view_) {
|
|
|
|
+ // Not supported with CEF Views-hosted DevTools windows.
|
|
|
|
+ return;
|
|
|
|
+ }
|
|
|
|
browser_frame_view_->UpdateThrobber(running);
|
|
|
|
}
|
|
|
|
|
|
|
|
@@ -240,6 +258,8 @@ BrowserNonClientFrameView* BrowserFrame::GetFrameView() const {
|
2022-08-09 21:42:18 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
bool BrowserFrame::UseCustomFrame() const {
|
|
|
|
+ if (!native_browser_frame_)
|
|
|
|
+ return true;
|
|
|
|
return native_browser_frame_->UseCustomFrame();
|
|
|
|
}
|
|
|
|
|
2023-11-14 18:16:43 +01:00
|
|
|
@@ -253,20 +273,30 @@ bool BrowserFrame::ShouldDrawFrameHeader() const {
|
2022-03-23 01:42:03 +01:00
|
|
|
|
|
|
|
void BrowserFrame::GetWindowPlacement(gfx::Rect* bounds,
|
|
|
|
ui::WindowShowState* show_state) const {
|
|
|
|
+ if (!native_browser_frame_) {
|
|
|
|
+ *show_state = ui::SHOW_STATE_DEFAULT;
|
|
|
|
+ return;
|
|
|
|
+ }
|
|
|
|
return native_browser_frame_->GetWindowPlacement(bounds, show_state);
|
|
|
|
}
|
2021-02-18 02:58:25 +01:00
|
|
|
|
|
|
|
content::KeyboardEventProcessingResult BrowserFrame::PreHandleKeyboardEvent(
|
|
|
|
const content::NativeWebKeyboardEvent& event) {
|
|
|
|
+ if (!native_browser_frame_)
|
|
|
|
+ return content::KeyboardEventProcessingResult::NOT_HANDLED;
|
|
|
|
return native_browser_frame_->PreHandleKeyboardEvent(event);
|
|
|
|
}
|
|
|
|
|
|
|
|
bool BrowserFrame::HandleKeyboardEvent(
|
|
|
|
const content::NativeWebKeyboardEvent& event) {
|
|
|
|
+ if (!native_browser_frame_)
|
|
|
|
+ return false;
|
|
|
|
return native_browser_frame_->HandleKeyboardEvent(event);
|
|
|
|
}
|
|
|
|
|
|
|
|
void BrowserFrame::OnBrowserViewInitViewsComplete() {
|
|
|
|
+ if (!browser_frame_view_)
|
|
|
|
+ return;
|
|
|
|
browser_frame_view_->OnBrowserViewInitViewsComplete();
|
|
|
|
}
|
|
|
|
|
2023-11-14 18:16:43 +01:00
|
|
|
@@ -367,6 +397,8 @@ ui::ColorProviderKey::ThemeInitializerSupplier* BrowserFrame::GetCustomTheme()
|
2022-04-08 22:48:56 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
void BrowserFrame::OnNativeWidgetWorkspaceChanged() {
|
|
|
|
+ if (!browser_view_)
|
|
|
|
+ return;
|
|
|
|
chrome::SaveWindowWorkspace(browser_view_->browser(), GetWorkspace());
|
|
|
|
chrome::SaveWindowVisibleOnAllWorkspaces(browser_view_->browser(),
|
|
|
|
IsVisibleOnAllWorkspaces());
|
2023-11-14 18:16:43 +01:00
|
|
|
@@ -478,6 +510,8 @@ void BrowserFrame::OnNativeThemeUpdated(ui::NativeTheme* observed_theme) {
|
2022-03-26 02:12:30 +01:00
|
|
|
|
2023-07-21 00:11:34 +02:00
|
|
|
ui::ColorProviderKey BrowserFrame::GetColorProviderKey() const {
|
2022-03-26 02:12:30 +01:00
|
|
|
auto key = Widget::GetColorProviderKey();
|
|
|
|
+ if (!browser_view_)
|
|
|
|
+ return key;
|
2023-04-04 20:00:13 +02:00
|
|
|
|
2023-08-09 23:17:17 +02:00
|
|
|
key.app_controller = browser_view_->browser()->app_controller();
|
|
|
|
|
2023-11-14 18:16:43 +01:00
|
|
|
@@ -632,5 +666,8 @@ bool BrowserFrame::RegenerateFrameOnThemeChange(
|
2022-11-15 18:50:53 +01:00
|
|
|
}
|
|
|
|
|
2023-05-30 10:55:32 +02:00
|
|
|
bool BrowserFrame::IsIncognitoBrowser() const {
|
|
|
|
+ if (!browser_view_) {
|
|
|
|
+ return true;
|
|
|
|
+ }
|
2023-01-30 18:43:54 +01:00
|
|
|
return browser_view_->browser()->profile()->IsIncognitoProfile();
|
2022-11-15 18:50:53 +01:00
|
|
|
}
|
2021-02-18 02:58:25 +01:00
|
|
|
diff --git chrome/browser/ui/views/frame/browser_frame.h chrome/browser/ui/views/frame/browser_frame.h
|
2023-09-15 21:51:43 +02:00
|
|
|
index 0c231b6ac5b01..6b5af98e18e42 100644
|
2021-02-18 02:58:25 +01:00
|
|
|
--- chrome/browser/ui/views/frame/browser_frame.h
|
|
|
|
+++ chrome/browser/ui/views/frame/browser_frame.h
|
2023-04-04 20:00:13 +02:00
|
|
|
@@ -61,7 +61,9 @@ enum class TabDragKind {
|
2021-02-18 02:58:25 +01:00
|
|
|
// This is a virtual interface that allows system specific browser frames.
|
|
|
|
class BrowserFrame : public views::Widget, public views::ContextMenuController {
|
|
|
|
public:
|
|
|
|
+ BrowserFrame();
|
|
|
|
explicit BrowserFrame(BrowserView* browser_view);
|
|
|
|
+ void InitBrowserView(BrowserView* browser_view);
|
|
|
|
|
2021-10-19 00:17:16 +02:00
|
|
|
BrowserFrame(const BrowserFrame&) = delete;
|
|
|
|
BrowserFrame& operator=(const BrowserFrame&) = delete;
|
2021-02-18 02:58:25 +01:00
|
|
|
diff --git chrome/browser/ui/views/frame/browser_view.cc chrome/browser/ui/views/frame/browser_view.cc
|
2024-01-04 15:42:19 +01:00
|
|
|
index b4cc05d5e3a1f..5971ab0111edf 100644
|
2021-02-18 02:58:25 +01:00
|
|
|
--- chrome/browser/ui/views/frame/browser_view.cc
|
|
|
|
+++ chrome/browser/ui/views/frame/browser_view.cc
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -343,11 +343,10 @@ using content::NativeWebKeyboardEvent;
|
2022-06-17 15:28:55 +02:00
|
|
|
using content::WebContents;
|
2021-08-28 03:55:15 +02:00
|
|
|
using web_modal::WebContentsModalDialogHost;
|
|
|
|
|
|
|
|
-namespace {
|
|
|
|
+// static
|
|
|
|
+const char BrowserView::kBrowserViewKey[] = "__BROWSER_VIEW__";
|
|
|
|
|
|
|
|
-// The name of a key to store on the window handle so that other code can
|
|
|
|
-// locate this object using just the handle.
|
|
|
|
-const char* const kBrowserViewKey = "__BROWSER_VIEW__";
|
|
|
|
+namespace {
|
|
|
|
|
|
|
|
#if BUILDFLAG(IS_CHROMEOS_ASH)
|
|
|
|
// UMA histograms that record animation smoothness for tab loading animation.
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -862,11 +861,23 @@ class BrowserView::AccessibilityModeObserver : public ui::AXModeObserver {
|
2021-03-04 23:36:57 +01:00
|
|
|
///////////////////////////////////////////////////////////////////////////////
|
|
|
|
// BrowserView, public:
|
2021-02-18 02:58:25 +01:00
|
|
|
|
|
|
|
+BrowserView::BrowserView() : BrowserView(nullptr) {}
|
|
|
|
+
|
|
|
|
BrowserView::BrowserView(std::unique_ptr<Browser> browser)
|
|
|
|
: views::ClientView(nullptr, nullptr),
|
|
|
|
- browser_(std::move(browser)),
|
|
|
|
accessibility_mode_observer_(
|
2023-11-21 20:17:55 +01:00
|
|
|
std::make_unique<AccessibilityModeObserver>(this)) {
|
|
|
|
+ if (browser) {
|
2021-02-18 02:58:25 +01:00
|
|
|
+ InitBrowser(std::move(browser));
|
2023-11-21 20:17:55 +01:00
|
|
|
+ }
|
2021-02-18 02:58:25 +01:00
|
|
|
+}
|
|
|
|
+
|
|
|
|
+void BrowserView::InitBrowser(std::unique_ptr<Browser> browser) {
|
|
|
|
+ DCHECK(!browser_);
|
|
|
|
+ browser_ = std::move(browser);
|
|
|
|
+
|
2022-09-26 21:30:45 +02:00
|
|
|
+ immersive_mode_controller_ = chrome::CreateImmersiveModeController(this);
|
2021-02-18 02:58:25 +01:00
|
|
|
+
|
2023-11-21 20:17:55 +01:00
|
|
|
// Store the actions so that the access is available for other classes.
|
|
|
|
if (base::FeatureList::IsEnabled(features::kSidePanelPinning)) {
|
|
|
|
browser_->SetUserData(BrowserActions::UserDataKey(),
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -926,7 +937,6 @@ BrowserView::BrowserView(std::unique_ptr<Browser> browser)
|
2021-03-04 23:36:57 +01:00
|
|
|
}
|
2021-02-18 02:58:25 +01:00
|
|
|
|
|
|
|
browser_->tab_strip_model()->AddObserver(this);
|
2022-09-26 21:30:45 +02:00
|
|
|
- immersive_mode_controller_ = chrome::CreateImmersiveModeController(this);
|
2021-02-18 02:58:25 +01:00
|
|
|
|
|
|
|
// Top container holds tab strip region and toolbar and lives at the front of
|
|
|
|
// the view hierarchy.
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -986,8 +996,15 @@ BrowserView::BrowserView(std::unique_ptr<Browser> browser)
|
2021-04-11 22:10:11 +02:00
|
|
|
contents_container->SetLayoutManager(std::make_unique<ContentsLayoutManager>(
|
|
|
|
devtools_web_view_, contents_web_view_));
|
|
|
|
|
|
|
|
- toolbar_ = top_container_->AddChildView(
|
|
|
|
- std::make_unique<ToolbarView>(browser_.get(), this));
|
2023-09-07 19:28:27 +02:00
|
|
|
+ toolbar_ = OverrideCreateToolbar();
|
2021-04-11 22:10:11 +02:00
|
|
|
+ if (!toolbar_) {
|
2021-07-23 18:40:13 +02:00
|
|
|
+ toolbar_ = new ToolbarView(browser_.get(), this, absl::nullopt);
|
2021-04-11 22:10:11 +02:00
|
|
|
+ } else {
|
|
|
|
+ browser_->set_toolbar_overridden(true);
|
|
|
|
+ // Update state that depends on the above flag.
|
|
|
|
+ browser_->command_controller()->FullscreenStateChanged();
|
|
|
|
+ }
|
2021-12-16 23:35:54 +01:00
|
|
|
+ top_container_->AddChildView(base::WrapUnique(toolbar_.get()));
|
2021-04-11 22:10:11 +02:00
|
|
|
|
|
|
|
contents_separator_ =
|
|
|
|
top_container_->AddChildView(std::make_unique<ContentsSeparator>());
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -1061,7 +1078,9 @@ BrowserView::~BrowserView() {
|
2023-10-13 19:48:01 +02:00
|
|
|
|
|
|
|
// All the tabs should have been destroyed already. If we were closed by the
|
|
|
|
// OS with some tabs than the NativeBrowserFrame should have destroyed them.
|
|
|
|
+ if (browser_) {
|
|
|
|
DCHECK_EQ(0, browser_->tab_strip_model()->count());
|
|
|
|
+ }
|
|
|
|
|
|
|
|
// Stop the animation timer explicitly here to avoid running it in a nested
|
|
|
|
// message loop, which may run by Browser destructor.
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -1075,12 +1094,14 @@ BrowserView::~BrowserView() {
|
2023-10-13 19:48:01 +02:00
|
|
|
// child views and it is an observer for avatar toolbar button if any.
|
|
|
|
autofill_bubble_handler_.reset();
|
|
|
|
|
|
|
|
+ if (browser_) {
|
|
|
|
auto* global_registry =
|
|
|
|
extensions::ExtensionCommandsGlobalRegistry::Get(browser_->profile());
|
|
|
|
if (global_registry->registry_for_active_window() ==
|
|
|
|
extension_keybinding_registry_.get()) {
|
|
|
|
global_registry->set_registry_for_active_window(nullptr);
|
|
|
|
}
|
|
|
|
+ }
|
|
|
|
|
|
|
|
// The TabStrip attaches a listener to the model. Make sure we shut down the
|
|
|
|
// TabStrip first so that it can cleanly remove the listener.
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -1098,7 +1119,9 @@ BrowserView::~BrowserView() {
|
2023-10-13 19:48:01 +02:00
|
|
|
|
|
|
|
// `SidePanelUI::RemoveSidePanelUIForBrowser()` deletes the
|
|
|
|
// SidePanelCoordinator.
|
|
|
|
+ if (browser()) {
|
|
|
|
SidePanelUI::RemoveSidePanelUIForBrowser(browser());
|
|
|
|
+ }
|
|
|
|
}
|
|
|
|
|
|
|
|
// static
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -1973,9 +1996,14 @@ void BrowserView::OnExclusiveAccessUserInput() {
|
2023-10-23 20:39:33 +02:00
|
|
|
|
|
|
|
bool BrowserView::ShouldHideUIForFullscreen() const {
|
|
|
|
// Immersive mode needs UI for the slide-down top panel.
|
|
|
|
- if (immersive_mode_controller_->IsEnabled())
|
|
|
|
+ // Avoid callback into |immersive_mode_controller_| during construction.
|
|
|
|
+ // See CEF issue #3527.
|
|
|
|
+ if (immersive_mode_controller_ &&
|
|
|
|
+ immersive_mode_controller_->IsEnabled())
|
2021-02-18 02:58:25 +01:00
|
|
|
return false;
|
|
|
|
|
|
|
|
+ if (!frame_->GetFrameView())
|
|
|
|
+ return false;
|
|
|
|
return frame_->GetFrameView()->ShouldHideTopUIForFullscreen();
|
|
|
|
}
|
|
|
|
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -3068,7 +3096,8 @@ DownloadShelf* BrowserView::GetDownloadShelf() {
|
2022-10-17 19:27:40 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
DownloadBubbleUIController* BrowserView::GetDownloadBubbleUIController() {
|
|
|
|
- DCHECK(toolbar_button_provider_);
|
|
|
|
+ if (!toolbar_button_provider_)
|
|
|
|
+ return nullptr;
|
|
|
|
if (auto* download_button = toolbar_button_provider_->GetDownloadButton())
|
|
|
|
return download_button->bubble_controller();
|
|
|
|
return nullptr;
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -3619,7 +3648,8 @@ void BrowserView::ReparentTopContainerForEndOfImmersive() {
|
2022-08-23 03:37:40 +02:00
|
|
|
if (top_container()->parent() == this)
|
|
|
|
return;
|
2021-02-18 02:58:25 +01:00
|
|
|
|
|
|
|
- overlay_view_->SetVisible(false);
|
|
|
|
+ if (overlay_view_)
|
|
|
|
+ overlay_view_->SetVisible(false);
|
|
|
|
top_container()->DestroyLayer();
|
|
|
|
AddChildViewAt(top_container(), 0);
|
|
|
|
EnsureFocusOrder();
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -4081,11 +4111,38 @@ void BrowserView::GetAccessiblePanes(std::vector<views::View*>* panes) {
|
2023-09-07 19:28:27 +02:00
|
|
|
bool BrowserView::ShouldDescendIntoChildForEventHandling(
|
|
|
|
gfx::NativeView child,
|
|
|
|
const gfx::Point& location) {
|
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+ const bool frameless_pip =
|
|
|
|
+ GetIsPictureInPictureType() &&
|
|
|
|
+ !browser_->SupportsWindowFeature(Browser::FEATURE_TITLEBAR);
|
|
|
|
+ if (frameless_pip) {
|
|
|
|
+ if (auto frame_view = frame()->GetFrameView()) {
|
|
|
|
+ int result = frame_view->NonClientHitTest(location);
|
|
|
|
+ if (result == HTTOP || result == HTTOPLEFT || result == HTTOPRIGHT) {
|
|
|
|
+ // Allow resize from the top of a frameless window.
|
|
|
|
+ return false;
|
|
|
|
+ }
|
|
|
|
+ }
|
|
|
|
+ }
|
|
|
|
+#endif
|
|
|
|
+
|
|
|
|
+ absl::optional<SkRegion> draggable_region;
|
|
|
|
+
|
|
|
|
// Window for PWAs with window-controls-overlay display override should claim
|
|
|
|
// mouse events that fall within the draggable region.
|
|
|
|
web_app::AppBrowserController* controller = browser()->app_controller();
|
|
|
|
- if (AreDraggableRegionsEnabled() && controller &&
|
|
|
|
- controller->draggable_region().has_value()) {
|
|
|
|
+ if (AreDraggableRegionsEnabled() && controller) {
|
|
|
|
+ draggable_region = controller->draggable_region();
|
|
|
|
+ }
|
|
|
|
+
|
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+ // Match logic in PictureInPictureBrowserFrameView::NonClientHitTest.
|
|
|
|
+ if (!draggable_region.has_value() && frameless_pip) {
|
|
|
|
+ draggable_region = browser_->cef_delegate()->GetDraggableRegion();
|
|
|
|
+ }
|
|
|
|
+#endif
|
|
|
|
+
|
|
|
|
+ if (draggable_region.has_value()) {
|
|
|
|
// Draggable regions are defined relative to the web contents.
|
|
|
|
gfx::Point point_in_contents_web_view_coords(location);
|
|
|
|
views::View::ConvertPointToTarget(GetWidget()->GetRootView(),
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -4094,7 +4151,7 @@ bool BrowserView::ShouldDescendIntoChildForEventHandling(
|
2023-09-07 19:28:27 +02:00
|
|
|
|
|
|
|
// Draggable regions should be ignored for clicks into any browser view's
|
|
|
|
// owned widgets, for example alerts, permission prompts or find bar.
|
|
|
|
- return !controller->draggable_region()->contains(
|
|
|
|
+ return !draggable_region->contains(
|
|
|
|
point_in_contents_web_view_coords.x(),
|
|
|
|
point_in_contents_web_view_coords.y()) ||
|
|
|
|
WidgetOwnedByAnchorContainsPoint(point_in_contents_web_view_coords);
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -4202,8 +4259,10 @@ void BrowserView::Layout() {
|
2021-04-11 22:10:11 +02:00
|
|
|
|
2021-02-18 02:58:25 +01:00
|
|
|
// TODO(jamescook): Why was this in the middle of layout code?
|
|
|
|
toolbar_->location_bar()->omnibox_view()->SetFocusBehavior(
|
2021-04-11 22:10:11 +02:00
|
|
|
- IsToolbarVisible() ? FocusBehavior::ALWAYS : FocusBehavior::NEVER);
|
2021-02-18 02:58:25 +01:00
|
|
|
- frame()->GetFrameView()->UpdateMinimumSize();
|
2021-04-11 22:10:11 +02:00
|
|
|
+ (IsToolbarVisible() || browser_->toolbar_overridden()) ?
|
|
|
|
+ FocusBehavior::ALWAYS : FocusBehavior::NEVER);
|
2021-02-18 02:58:25 +01:00
|
|
|
+ if (frame()->GetFrameView())
|
|
|
|
+ frame()->GetFrameView()->UpdateMinimumSize();
|
|
|
|
|
2023-08-09 23:17:17 +02:00
|
|
|
#if BUILDFLAG(IS_CHROMEOS_ASH)
|
|
|
|
// In chromeOS ash we round the bottom two corners of the browser frame by
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -4281,6 +4340,11 @@ void BrowserView::AddedToWidget() {
|
2021-04-11 22:10:11 +02:00
|
|
|
SetThemeProfileForWindow(GetNativeWindow(), browser_->profile());
|
|
|
|
#endif
|
|
|
|
|
|
|
|
+ // This browser view may already have a custom button provider set (e.g the
|
|
|
|
+ // hosted app frame).
|
|
|
|
+ if (!toolbar_button_provider_)
|
|
|
|
+ SetToolbarButtonProvider(toolbar_);
|
|
|
|
+
|
|
|
|
toolbar_->Init();
|
2021-02-18 02:58:25 +01:00
|
|
|
|
2023-04-04 20:00:13 +02:00
|
|
|
// TODO(pbos): Investigate whether the side panels should be creatable when
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -4329,13 +4393,9 @@ void BrowserView::AddedToWidget() {
|
2021-04-11 22:10:11 +02:00
|
|
|
|
|
|
|
EnsureFocusOrder();
|
|
|
|
|
|
|
|
- // This browser view may already have a custom button provider set (e.g the
|
|
|
|
- // hosted app frame).
|
|
|
|
- if (!toolbar_button_provider_)
|
|
|
|
- SetToolbarButtonProvider(toolbar_);
|
|
|
|
-
|
2021-02-18 02:58:25 +01:00
|
|
|
frame_->OnBrowserViewInitViewsComplete();
|
|
|
|
- frame_->GetFrameView()->UpdateMinimumSize();
|
|
|
|
+ if (frame_->GetFrameView())
|
|
|
|
+ frame_->GetFrameView()->UpdateMinimumSize();
|
|
|
|
using_native_frame_ = frame_->ShouldUseNativeFrame();
|
|
|
|
|
|
|
|
MaybeInitializeWebUITabStrip();
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -4755,7 +4815,8 @@ void BrowserView::ProcessFullscreen(bool fullscreen,
|
2022-01-13 00:11:27 +01:00
|
|
|
// Undo our anti-jankiness hacks and force a re-layout.
|
|
|
|
in_process_fullscreen_ = false;
|
|
|
|
ToolbarSizeChanged(false);
|
|
|
|
- frame_->GetFrameView()->OnFullscreenStateChanged();
|
|
|
|
+ if (frame_->GetFrameView())
|
|
|
|
+ frame_->GetFrameView()->OnFullscreenStateChanged();
|
|
|
|
}
|
|
|
|
|
|
|
|
bool BrowserView::ShouldUseImmersiveFullscreenForUrl(const GURL& url) const {
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -5144,6 +5205,8 @@ Profile* BrowserView::GetProfile() {
|
2022-01-13 00:11:27 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
void BrowserView::UpdateUIForTabFullscreen() {
|
|
|
|
+ if (!frame_->GetFrameView())
|
|
|
|
+ return;
|
|
|
|
frame()->GetFrameView()->UpdateFullscreenTopUI();
|
|
|
|
}
|
|
|
|
|
2024-01-04 15:42:19 +01:00
|
|
|
@@ -5166,6 +5229,8 @@ void BrowserView::HideDownloadShelf() {
|
2022-01-13 00:11:27 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
bool BrowserView::CanUserExitFullscreen() const {
|
|
|
|
+ if (!frame_->GetFrameView())
|
|
|
|
+ return true;
|
|
|
|
return frame_->GetFrameView()->CanUserExitFullscreen();
|
|
|
|
}
|
|
|
|
|
2021-02-18 02:58:25 +01:00
|
|
|
diff --git chrome/browser/ui/views/frame/browser_view.h chrome/browser/ui/views/frame/browser_view.h
|
2023-11-27 21:56:19 +01:00
|
|
|
index 7538090d415a7..25c7f80a5f4f4 100644
|
2021-02-18 02:58:25 +01:00
|
|
|
--- chrome/browser/ui/views/frame/browser_view.h
|
|
|
|
+++ chrome/browser/ui/views/frame/browser_view.h
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -137,11 +137,16 @@ class BrowserView : public BrowserWindow,
|
2023-08-09 23:17:17 +02:00
|
|
|
kUnknown
|
|
|
|
};
|
|
|
|
|
2021-02-18 02:58:25 +01:00
|
|
|
+ BrowserView();
|
|
|
|
explicit BrowserView(std::unique_ptr<Browser> browser);
|
|
|
|
+ void InitBrowser(std::unique_ptr<Browser> browser);
|
2021-03-04 23:36:57 +01:00
|
|
|
BrowserView(const BrowserView&) = delete;
|
|
|
|
BrowserView& operator=(const BrowserView&) = delete;
|
2021-02-18 02:58:25 +01:00
|
|
|
~BrowserView() override;
|
2021-08-28 03:55:15 +02:00
|
|
|
|
|
|
|
+ // Key used to bind BrowserView to the Widget with which it is associated.
|
|
|
|
+ static const char kBrowserViewKey[];
|
|
|
|
+
|
2023-01-30 18:43:54 +01:00
|
|
|
void set_frame(BrowserFrame* frame) {
|
|
|
|
frame_ = frame;
|
|
|
|
paint_as_active_subscription_ =
|
2023-11-27 21:56:19 +01:00
|
|
|
@@ -829,6 +834,9 @@ class BrowserView : public BrowserWindow,
|
2023-09-15 21:51:43 +02:00
|
|
|
// TopContainerBackground::PaintThemeCustomImage for details.
|
|
|
|
gfx::Point GetThemeOffsetFromBrowserView() const;
|
2021-04-11 22:10:11 +02:00
|
|
|
|
|
|
|
+ protected:
|
2023-09-07 19:28:27 +02:00
|
|
|
+ virtual ToolbarView* OverrideCreateToolbar() { return nullptr; }
|
2021-04-11 22:10:11 +02:00
|
|
|
+
|
|
|
|
private:
|
|
|
|
// Do not friend BrowserViewLayout. Use the BrowserViewLayoutDelegate
|
|
|
|
// interface to keep these two classes decoupled and testable.
|
|
|
|
diff --git chrome/browser/ui/views/frame/browser_view_layout.cc chrome/browser/ui/views/frame/browser_view_layout.cc
|
2023-11-21 20:17:55 +01:00
|
|
|
index b358a30764051..5c67244a43251 100644
|
2021-04-11 22:10:11 +02:00
|
|
|
--- chrome/browser/ui/views/frame/browser_view_layout.cc
|
|
|
|
+++ chrome/browser/ui/views/frame/browser_view_layout.cc
|
2023-09-15 21:51:43 +02:00
|
|
|
@@ -48,6 +48,10 @@
|
2021-08-28 03:55:15 +02:00
|
|
|
#include "ui/views/window/client_view.h"
|
2023-02-27 19:52:38 +01:00
|
|
|
#include "ui/views/window/hit_test_utils.h"
|
2021-08-28 03:55:15 +02:00
|
|
|
|
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+#include "cef/libcef/browser/chrome/views/chrome_views_util.h"
|
|
|
|
+#endif
|
|
|
|
+
|
|
|
|
using views::View;
|
|
|
|
using web_modal::ModalDialogHostObserver;
|
2023-01-03 00:34:43 +01:00
|
|
|
using web_modal::WebContentsModalDialogHost;
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -579,6 +583,13 @@ int BrowserViewLayout::LayoutWebUITabStrip(int top) {
|
2021-04-11 22:10:11 +02:00
|
|
|
|
|
|
|
int BrowserViewLayout::LayoutToolbar(int top) {
|
|
|
|
TRACE_EVENT0("ui", "BrowserViewLayout::LayoutToolbar");
|
2023-01-23 18:58:40 +01:00
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
2021-08-28 03:55:15 +02:00
|
|
|
+ if (cef::IsCefView(toolbar_)) {
|
2021-04-11 22:10:11 +02:00
|
|
|
+ // CEF may take ownership of the toolbar. Early exit to avoid the DCHECK
|
|
|
|
+ // in LayoutManager::SetViewVisibility().
|
|
|
|
+ return top;
|
|
|
|
+ }
|
2023-01-23 18:58:40 +01:00
|
|
|
+#endif
|
2021-04-11 22:10:11 +02:00
|
|
|
int browser_view_width = vertical_layout_rect_.width();
|
|
|
|
bool toolbar_visible = delegate_->IsToolbarVisible();
|
|
|
|
int height = toolbar_visible ? toolbar_->GetPreferredSize().height() : 0;
|
2022-04-14 21:12:23 +02:00
|
|
|
diff --git chrome/browser/ui/views/frame/contents_web_view.cc chrome/browser/ui/views/frame/contents_web_view.cc
|
2023-07-21 00:11:34 +02:00
|
|
|
index 8267a265a8e10..ee08f18e96a34 100644
|
2022-04-14 21:12:23 +02:00
|
|
|
--- chrome/browser/ui/views/frame/contents_web_view.cc
|
|
|
|
+++ chrome/browser/ui/views/frame/contents_web_view.cc
|
2023-07-21 00:11:34 +02:00
|
|
|
@@ -23,6 +23,11 @@
|
2022-04-14 21:12:23 +02:00
|
|
|
ContentsWebView::ContentsWebView(content::BrowserContext* browser_context)
|
|
|
|
: views::WebView(browser_context),
|
|
|
|
status_bubble_(nullptr) {
|
|
|
|
+ // Mouse events on draggable regions will not be handled by the WebView.
|
|
|
|
+ // Avoid the resulting DCHECK in NativeViewHost::OnMousePressed by
|
|
|
|
+ // configuring the NativeViewHost not to process events via the view
|
|
|
|
+ // hierarchy.
|
|
|
|
+ holder()->SetCanProcessEventsWithinSubtree(false);
|
|
|
|
}
|
|
|
|
|
|
|
|
ContentsWebView::~ContentsWebView() {
|
2023-09-07 19:28:27 +02:00
|
|
|
diff --git chrome/browser/ui/views/frame/picture_in_picture_browser_frame_view.cc chrome/browser/ui/views/frame/picture_in_picture_browser_frame_view.cc
|
2023-11-21 20:17:55 +01:00
|
|
|
index 4730d2459c324..714809151a0bd 100644
|
2023-09-07 19:28:27 +02:00
|
|
|
--- chrome/browser/ui/views/frame/picture_in_picture_browser_frame_view.cc
|
|
|
|
+++ chrome/browser/ui/views/frame/picture_in_picture_browser_frame_view.cc
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -575,6 +575,11 @@ PictureInPictureBrowserFrameView::PictureInPictureBrowserFrameView(
|
2023-09-07 19:28:27 +02:00
|
|
|
frame->GetNativeWindow()->SetEventTargeter(
|
|
|
|
std::make_unique<chromeos::InteriorResizeHandleTargeter>());
|
|
|
|
#endif
|
|
|
|
+
|
|
|
|
+ if (!browser_view->browser()->SupportsWindowFeature(
|
|
|
|
+ Browser::FEATURE_TITLEBAR)) {
|
|
|
|
+ top_bar_container_view_->SetVisible(false);
|
|
|
|
+ }
|
|
|
|
}
|
|
|
|
|
|
|
|
PictureInPictureBrowserFrameView::~PictureInPictureBrowserFrameView() {
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -671,18 +676,42 @@ gfx::Rect PictureInPictureBrowserFrameView::GetWindowBoundsForClientBounds(
|
2023-09-07 19:28:27 +02:00
|
|
|
|
|
|
|
int PictureInPictureBrowserFrameView::NonClientHitTest(
|
|
|
|
const gfx::Point& point) {
|
|
|
|
- // Allow interacting with the buttons.
|
|
|
|
- if (GetLocationIconViewBounds().Contains(point) ||
|
|
|
|
- GetBackToTabControlsBounds().Contains(point) ||
|
|
|
|
- GetCloseControlsBounds().Contains(point)) {
|
|
|
|
- return HTCLIENT;
|
|
|
|
+ const bool frameless = !top_bar_container_view_->GetVisible();
|
|
|
|
+ if (!frameless) {
|
|
|
|
+ // Allow interacting with the buttons.
|
|
|
|
+ if (GetLocationIconViewBounds().Contains(point) ||
|
|
|
|
+ GetBackToTabControlsBounds().Contains(point) ||
|
|
|
|
+ GetCloseControlsBounds().Contains(point)) {
|
2023-11-06 21:46:43 +01:00
|
|
|
+ return HTCLIENT;
|
|
|
|
+ }
|
2023-09-07 19:28:27 +02:00
|
|
|
+
|
|
|
|
+ for (size_t i = 0; i < content_setting_views_.size(); i++) {
|
|
|
|
+ if (GetContentSettingViewBounds(i).Contains(point)) {
|
|
|
|
+ return HTCLIENT;
|
|
|
|
+ }
|
|
|
|
+ }
|
|
|
|
}
|
|
|
|
|
2023-11-06 21:46:43 +01:00
|
|
|
- for (size_t i = 0; i < content_setting_views_.size(); i++) {
|
|
|
|
- if (GetContentSettingViewBounds(i).Contains(point)) {
|
|
|
|
- return HTCLIENT;
|
2023-09-07 19:28:27 +02:00
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+ if (frameless) {
|
|
|
|
+ // Match logic in BrowserView::ShouldDescendIntoChildForEventHandling.
|
|
|
|
+ const auto draggable_region =
|
|
|
|
+ browser_view()->browser()->cef_delegate()->GetDraggableRegion();
|
|
|
|
+ if (draggable_region.has_value()) {
|
|
|
|
+ // Draggable regions are defined relative to the web contents.
|
|
|
|
+ gfx::Point point_in_contents_web_view_coords(point);
|
|
|
|
+ views::View::ConvertPointToTarget(GetWidget()->GetRootView(),
|
|
|
|
+ browser_view()->contents_web_view(),
|
|
|
|
+ &point_in_contents_web_view_coords);
|
|
|
|
+
|
|
|
|
+ if (draggable_region->contains(
|
|
|
|
+ point_in_contents_web_view_coords.x(),
|
|
|
|
+ point_in_contents_web_view_coords.y())) {
|
|
|
|
+ return HTCAPTION;
|
|
|
|
+ }
|
2023-11-06 21:46:43 +01:00
|
|
|
}
|
|
|
|
}
|
2023-09-07 19:28:27 +02:00
|
|
|
+#endif // BUILDFLAG(ENABLE_CEF)
|
2023-11-06 21:46:43 +01:00
|
|
|
|
|
|
|
// Allow dragging and resizing the window.
|
|
|
|
int window_component = GetHTComponentForFrame(
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -749,7 +778,8 @@ void PictureInPictureBrowserFrameView::Layout() {
|
2023-09-07 19:28:27 +02:00
|
|
|
gfx::Rect content_area = GetLocalBounds();
|
|
|
|
content_area.Inset(FrameBorderInsets());
|
|
|
|
gfx::Rect top_bar = content_area;
|
|
|
|
- top_bar.set_height(kTopControlsHeight);
|
|
|
|
+ top_bar.set_height(
|
|
|
|
+ top_bar_container_view_->GetVisible() ? kTopControlsHeight : 0);
|
|
|
|
top_bar_container_view_->SetBoundsRect(top_bar);
|
|
|
|
#if !BUILDFLAG(IS_ANDROID)
|
|
|
|
if (auto_pip_setting_overlay_) {
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -1213,7 +1243,8 @@ gfx::Insets PictureInPictureBrowserFrameView::ResizeBorderInsets() const {
|
2023-09-07 19:28:27 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
int PictureInPictureBrowserFrameView::GetTopAreaHeight() const {
|
|
|
|
- return FrameBorderInsets().top() + kTopControlsHeight;
|
|
|
|
+ return FrameBorderInsets().top() +
|
|
|
|
+ (top_bar_container_view_->GetVisible() ? kTopControlsHeight : 0);
|
|
|
|
}
|
|
|
|
|
|
|
|
gfx::Size PictureInPictureBrowserFrameView::GetNonClientViewAreaSize() const {
|
2023-03-07 00:21:57 +01:00
|
|
|
diff --git chrome/browser/ui/views/page_action/page_action_icon_controller.cc chrome/browser/ui/views/page_action/page_action_icon_controller.cc
|
2023-10-19 20:08:48 +02:00
|
|
|
index ca602db58de3a..1db448a2526a4 100644
|
2023-03-07 00:21:57 +01:00
|
|
|
--- chrome/browser/ui/views/page_action/page_action_icon_controller.cc
|
|
|
|
+++ chrome/browser/ui/views/page_action/page_action_icon_controller.cc
|
2023-10-19 20:08:48 +02:00
|
|
|
@@ -97,6 +97,12 @@ void PageActionIconController::Init(const PageActionIconParams& params,
|
2023-03-07 00:21:57 +01:00
|
|
|
};
|
|
|
|
|
|
|
|
for (PageActionIconType type : params.types_enabled) {
|
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+ if (params.browser && params.browser->cef_delegate() &&
|
|
|
|
+ !params.browser->cef_delegate()->IsPageActionIconVisible(type)) {
|
|
|
|
+ continue;
|
|
|
|
+ }
|
|
|
|
+#endif
|
|
|
|
switch (type) {
|
|
|
|
case PageActionIconType::kPaymentsOfferNotification:
|
|
|
|
add_page_action_icon(
|
2021-02-18 02:58:25 +01:00
|
|
|
diff --git chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc
|
2023-11-27 21:56:19 +01:00
|
|
|
index 9536b049aeb9d..30f3091939675 100644
|
2021-02-18 02:58:25 +01:00
|
|
|
--- chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc
|
|
|
|
+++ chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -558,29 +558,41 @@ gfx::Range BrowserTabStripController::ListTabsInGroup(
|
2021-02-18 02:58:25 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
bool BrowserTabStripController::IsFrameCondensed() const {
|
|
|
|
+ if (!GetFrameView())
|
|
|
|
+ return false;
|
|
|
|
return GetFrameView()->IsFrameCondensed();
|
|
|
|
}
|
|
|
|
|
|
|
|
bool BrowserTabStripController::HasVisibleBackgroundTabShapes() const {
|
|
|
|
+ if (!GetFrameView())
|
|
|
|
+ return false;
|
|
|
|
return GetFrameView()->HasVisibleBackgroundTabShapes(
|
|
|
|
BrowserFrameActiveState::kUseCurrent);
|
|
|
|
}
|
|
|
|
|
|
|
|
bool BrowserTabStripController::EverHasVisibleBackgroundTabShapes() const {
|
|
|
|
+ if (!GetFrameView())
|
|
|
|
+ return false;
|
|
|
|
return GetFrameView()->EverHasVisibleBackgroundTabShapes();
|
|
|
|
}
|
|
|
|
|
|
|
|
bool BrowserTabStripController::CanDrawStrokes() const {
|
|
|
|
+ if (!GetFrameView())
|
|
|
|
+ return false;
|
|
|
|
return GetFrameView()->CanDrawStrokes();
|
|
|
|
}
|
|
|
|
|
|
|
|
SkColor BrowserTabStripController::GetFrameColor(
|
|
|
|
BrowserFrameActiveState active_state) const {
|
|
|
|
+ if (!GetFrameView())
|
|
|
|
+ return SK_ColorWHITE;
|
|
|
|
return GetFrameView()->GetFrameColor(active_state);
|
|
|
|
}
|
|
|
|
|
2021-06-04 03:34:56 +02:00
|
|
|
absl::optional<int> BrowserTabStripController::GetCustomBackgroundId(
|
2021-02-18 02:58:25 +01:00
|
|
|
BrowserFrameActiveState active_state) const {
|
|
|
|
+ if (!GetFrameView())
|
2021-06-04 03:34:56 +02:00
|
|
|
+ return absl::nullopt;
|
2021-02-18 02:58:25 +01:00
|
|
|
return GetFrameView()->GetCustomBackgroundId(active_state);
|
|
|
|
}
|
|
|
|
|
2021-04-11 22:10:11 +02:00
|
|
|
diff --git chrome/browser/ui/views/toolbar/toolbar_view.cc chrome/browser/ui/views/toolbar/toolbar_view.cc
|
2023-11-27 21:56:19 +01:00
|
|
|
index bd6ff6fce989a..7e36aa6df6655 100644
|
2021-04-11 22:10:11 +02:00
|
|
|
--- chrome/browser/ui/views/toolbar/toolbar_view.cc
|
|
|
|
+++ chrome/browser/ui/views/toolbar/toolbar_view.cc
|
2023-11-27 21:56:19 +01:00
|
|
|
@@ -194,7 +194,7 @@ class TabstripLikeBackground : public views::Background {
|
2023-08-09 23:17:17 +02:00
|
|
|
void Paint(gfx::Canvas* canvas, views::View* view) const override {
|
2023-09-15 21:51:43 +02:00
|
|
|
bool painted = TopContainerBackground::PaintThemeCustomImage(canvas, view,
|
|
|
|
browser_view_);
|
2023-08-09 23:17:17 +02:00
|
|
|
- if (!painted) {
|
|
|
|
+ if (!painted && browser_view_->frame()->GetFrameView()) {
|
|
|
|
SkColor frame_color =
|
|
|
|
browser_view_->frame()->GetFrameView()->GetFrameColor(
|
|
|
|
BrowserFrameActiveState::kUseCurrent);
|
2023-11-27 21:56:19 +01:00
|
|
|
@@ -223,12 +223,13 @@ END_METADATA
|
2021-04-11 22:10:11 +02:00
|
|
|
////////////////////////////////////////////////////////////////////////////////
|
|
|
|
// ToolbarView, public:
|
|
|
|
|
|
|
|
-ToolbarView::ToolbarView(Browser* browser, BrowserView* browser_view)
|
|
|
|
+ToolbarView::ToolbarView(Browser* browser, BrowserView* browser_view,
|
2021-07-23 18:40:13 +02:00
|
|
|
+ absl::optional<DisplayMode> display_mode)
|
2021-04-11 22:10:11 +02:00
|
|
|
: AnimationDelegateViews(this),
|
|
|
|
browser_(browser),
|
|
|
|
browser_view_(browser_view),
|
|
|
|
app_menu_icon_controller_(browser->profile(), this),
|
|
|
|
- display_mode_(GetDisplayMode(browser)) {
|
|
|
|
+ display_mode_(display_mode ? *display_mode : GetDisplayMode(browser)) {
|
|
|
|
SetID(VIEW_ID_TOOLBAR);
|
|
|
|
|
2023-06-26 12:13:38 +02:00
|
|
|
container_view_ = AddChildView(std::make_unique<ContainerView>());
|
2023-11-27 21:56:19 +01:00
|
|
|
@@ -253,6 +254,19 @@ ToolbarView::~ToolbarView() {
|
2023-04-26 21:55:59 +02:00
|
|
|
}
|
2023-03-07 00:21:57 +01:00
|
|
|
|
2023-04-26 21:55:59 +02:00
|
|
|
void ToolbarView::Init() {
|
2023-03-07 00:21:57 +01:00
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+ using ToolbarButtonType = cef::BrowserDelegate::ToolbarButtonType;
|
|
|
|
+ auto button_visible = [this](ToolbarButtonType type) {
|
|
|
|
+ if (this->browser_->cef_delegate()) {
|
|
|
|
+ return this->browser_->cef_delegate()->IsToolbarButtonVisible(type);
|
|
|
|
+ }
|
|
|
|
+ return true;
|
|
|
|
+ };
|
|
|
|
+ #define BUTTON_VISIBLE(type) button_visible(ToolbarButtonType::type)
|
|
|
|
+#else
|
|
|
|
+ #define BUTTON_VISIBLE(type) true
|
|
|
|
+#endif
|
|
|
|
+
|
2023-04-26 21:55:59 +02:00
|
|
|
#if defined(USE_AURA)
|
|
|
|
// Avoid generating too many occlusion tracking calculation events before this
|
|
|
|
// function returns. The occlusion status will be computed only once once this
|
2023-11-27 21:56:19 +01:00
|
|
|
@@ -277,12 +291,13 @@ void ToolbarView::Init() {
|
2023-06-26 12:13:38 +02:00
|
|
|
|
2023-04-26 21:55:59 +02:00
|
|
|
auto location_bar = std::make_unique<LocationBarView>(
|
|
|
|
browser_, browser_->profile(), browser_->command_controller(), this,
|
|
|
|
- display_mode_ != DisplayMode::NORMAL);
|
|
|
|
+ display_mode_ != DisplayMode::NORMAL && !browser_->toolbar_overridden());
|
|
|
|
// Make sure the toolbar shows by default.
|
|
|
|
size_animation_.Reset(1);
|
|
|
|
|
|
|
|
std::unique_ptr<DownloadToolbarButtonView> download_button;
|
|
|
|
- if (download::IsDownloadBubbleEnabled(browser_->profile())) {
|
|
|
|
+ if (download::IsDownloadBubbleEnabled(browser_->profile()) &&
|
|
|
|
+ BUTTON_VISIBLE(kDownload)) {
|
|
|
|
download_button =
|
|
|
|
std::make_unique<DownloadToolbarButtonView>(browser_view_);
|
|
|
|
}
|
2023-11-27 21:56:19 +01:00
|
|
|
@@ -364,8 +379,10 @@ void ToolbarView::Init() {
|
2023-04-04 20:00:13 +02:00
|
|
|
}
|
2023-03-07 00:21:57 +01:00
|
|
|
}
|
|
|
|
std::unique_ptr<media_router::CastToolbarButton> cast;
|
|
|
|
- if (media_router::MediaRouterEnabled(browser_->profile()))
|
|
|
|
+ if (media_router::MediaRouterEnabled(browser_->profile()) &&
|
|
|
|
+ BUTTON_VISIBLE(kCast)) {
|
|
|
|
cast = media_router::CastToolbarButton::Create(browser_);
|
|
|
|
+ }
|
|
|
|
|
|
|
|
std::unique_ptr<MediaToolbarButtonView> media_button;
|
|
|
|
if (base::FeatureList::IsEnabled(media::kGlobalMediaControls)) {
|
2023-11-27 21:56:19 +01:00
|
|
|
@@ -375,7 +392,8 @@ void ToolbarView::Init() {
|
2023-03-07 00:21:57 +01:00
|
|
|
|
|
|
|
std::unique_ptr<send_tab_to_self::SendTabToSelfToolbarIconView>
|
|
|
|
send_tab_to_self_button;
|
|
|
|
- if (!browser_->profile()->IsOffTheRecord()) {
|
|
|
|
+ if (!browser_->profile()->IsOffTheRecord() &&
|
|
|
|
+ BUTTON_VISIBLE(kSendTabToSelf)) {
|
|
|
|
send_tab_to_self_button =
|
|
|
|
std::make_unique<send_tab_to_self::SendTabToSelfToolbarIconView>(
|
|
|
|
browser_view_);
|
2023-11-27 21:56:19 +01:00
|
|
|
@@ -446,7 +464,8 @@ void ToolbarView::Init() {
|
2023-11-21 20:17:55 +01:00
|
|
|
send_tab_to_self_button_ =
|
|
|
|
container_view_->AddChildView(std::move(send_tab_to_self_button));
|
|
|
|
|
|
|
|
- if (!base::FeatureList::IsEnabled(features::kSidePanelPinning)) {
|
|
|
|
+ if (!base::FeatureList::IsEnabled(features::kSidePanelPinning) &&
|
|
|
|
+ BUTTON_VISIBLE(kSidePanel)) {
|
|
|
|
if (companion::IsCompanionFeatureEnabled()) {
|
|
|
|
side_panel_container_ = container_view_->AddChildView(
|
|
|
|
std::make_unique<SidePanelToolbarContainer>(browser_view_));
|
2021-04-11 22:10:11 +02:00
|
|
|
diff --git chrome/browser/ui/views/toolbar/toolbar_view.h chrome/browser/ui/views/toolbar/toolbar_view.h
|
2023-11-27 21:56:19 +01:00
|
|
|
index 3bfe52cff3511..11373de4c2a3a 100644
|
2021-04-11 22:10:11 +02:00
|
|
|
--- chrome/browser/ui/views/toolbar/toolbar_view.h
|
|
|
|
+++ chrome/browser/ui/views/toolbar/toolbar_view.h
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -92,7 +92,8 @@ class ToolbarView : public views::AccessiblePaneView,
|
2021-04-11 22:10:11 +02:00
|
|
|
// needs to be displayed.
|
|
|
|
};
|
|
|
|
|
|
|
|
- ToolbarView(Browser* browser, BrowserView* browser_view);
|
|
|
|
+ ToolbarView(Browser* browser, BrowserView* browser_view,
|
2021-07-23 18:40:13 +02:00
|
|
|
+ absl::optional<DisplayMode> display_mode);
|
2021-04-11 22:10:11 +02:00
|
|
|
ToolbarView(const ToolbarView&) = delete;
|
|
|
|
ToolbarView& operator=(const ToolbarView&) = delete;
|
|
|
|
~ToolbarView() override;
|
2023-09-07 19:28:27 +02:00
|
|
|
diff --git chrome/browser/ui/web_applications/draggable_region_host_impl.cc chrome/browser/ui/web_applications/draggable_region_host_impl.cc
|
2023-11-21 20:17:55 +01:00
|
|
|
index 8980b03f2c945..6e719e0abced7 100644
|
2023-09-07 19:28:27 +02:00
|
|
|
--- chrome/browser/ui/web_applications/draggable_region_host_impl.cc
|
|
|
|
+++ chrome/browser/ui/web_applications/draggable_region_host_impl.cc
|
|
|
|
@@ -11,6 +11,24 @@
|
|
|
|
#include "chrome/common/chrome_features.h"
|
|
|
|
#include "mojo/public/cpp/bindings/self_owned_receiver.h"
|
|
|
|
|
|
|
|
+namespace {
|
|
|
|
+
|
|
|
|
+bool IsSupported(Browser* browser) {
|
|
|
|
+ if (web_app::AppBrowserController::IsWebApp(browser))
|
|
|
|
+ return true;
|
|
|
|
+
|
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+ if (browser->cef_delegate() &&
|
|
|
|
+ browser->cef_delegate()->SupportsDraggableRegion()) {
|
|
|
|
+ return true;
|
|
|
|
+ }
|
|
|
|
+#endif
|
|
|
|
+
|
|
|
|
+ return false;
|
|
|
|
+}
|
|
|
|
+
|
|
|
|
+} // namespace
|
|
|
|
+
|
|
|
|
DraggableRegionsHostImpl::DraggableRegionsHostImpl(
|
|
|
|
content::RenderFrameHost& render_frame_host,
|
|
|
|
mojo::PendingReceiver<chrome::mojom::DraggableRegions> receiver)
|
|
|
|
@@ -28,7 +46,7 @@ void DraggableRegionsHostImpl::CreateIfAllowed(
|
2023-11-21 20:17:55 +01:00
|
|
|
auto* browser = chrome::FindBrowserWithTab(web_contents);
|
2023-09-07 19:28:27 +02:00
|
|
|
|
|
|
|
// We only want to bind the receiver for PWAs.
|
|
|
|
- if (!web_app::AppBrowserController::IsWebApp(browser))
|
|
|
|
+ if (!IsSupported(browser))
|
|
|
|
return;
|
|
|
|
|
|
|
|
// The object is bound to the lifetime of |render_frame_host| and the mojo
|
|
|
|
@@ -43,7 +61,7 @@ void DraggableRegionsHostImpl::UpdateDraggableRegions(
|
2023-11-21 20:17:55 +01:00
|
|
|
auto* browser = chrome::FindBrowserWithTab(web_contents);
|
2023-09-07 19:28:27 +02:00
|
|
|
// When a WebApp browser's WebContents is reparented to a tabbed browser, a
|
|
|
|
// draggable regions update may race with the reparenting logic.
|
|
|
|
- if (!web_app::AppBrowserController::IsWebApp(browser))
|
|
|
|
+ if (!IsSupported(browser))
|
|
|
|
return;
|
|
|
|
|
|
|
|
SkRegion sk_region;
|
|
|
|
@@ -56,5 +74,12 @@ void DraggableRegionsHostImpl::UpdateDraggableRegions(
|
|
|
|
}
|
|
|
|
|
|
|
|
auto* app_browser_controller = browser->app_controller();
|
|
|
|
- app_browser_controller->UpdateDraggableRegion(sk_region);
|
|
|
|
+ if (app_browser_controller) {
|
|
|
|
+ app_browser_controller->UpdateDraggableRegion(sk_region);
|
|
|
|
+ }
|
|
|
|
+#if BUILDFLAG(ENABLE_CEF)
|
|
|
|
+ else {
|
|
|
|
+ browser->cef_delegate()->UpdateDraggableRegion(sk_region);
|
|
|
|
+ }
|
|
|
|
+#endif
|
|
|
|
}
|