1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374 |
- From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
- From: Heilig Benedek <[email protected]>
- Date: Sat, 10 Aug 2019 00:41:50 +0200
- Subject: feat: enable picture in picture mode for video players
- These files are needed to implement PiP, but the Electron build patches out
- chrome's generated resources for our own. This updates the #include so that we
- don't get errors for Chrome's generated resources, which are non-existent
- because we don't generate them in our build.
- diff --git a/chrome/browser/ui/views/overlay/close_image_button.cc b/chrome/browser/ui/views/overlay/close_image_button.cc
- index d236578cefc347e772305ac7ec54b9734e4aa20b..6bb89cea3b6b91ec8ea351ed60c4e4056a6340bb 100644
- --- a/chrome/browser/ui/views/overlay/close_image_button.cc
- +++ b/chrome/browser/ui/views/overlay/close_image_button.cc
- @@ -4,10 +4,13 @@
-
- #include "chrome/browser/ui/views/overlay/close_image_button.h"
-
- +#include "build/branding_buildflags.h"
- #include "build/chromeos_buildflags.h"
- #include "chrome/browser/ui/color/chrome_color_id.h"
- #include "chrome/grit/generated_resources.h"
- +#if BUILDFLAG(GOOGLE_CHROME_BRANDING)
- #include "components/vector_icons/vector_icons.h"
- +#endif // BUILDFLAG(GOOGLE_CHROME_BRANDING)
- #include "ui/base/l10n/l10n_util.h"
- #include "ui/base/metadata/metadata_impl_macros.h"
- #include "ui/base/models/image_model.h"
- @@ -27,7 +30,10 @@ CloseImageButton::CloseImageButton(PressedCallback callback)
- : OverlayWindowImageButton(std::move(callback)) {
- SetSize(gfx::Size(kCloseButtonSize, kCloseButtonSize));
-
- - auto* icon = &vector_icons::kCloseChromeRefreshIcon;
- + auto* icon = &views::kIcCloseIcon;
- +#if BUILDFLAG(GOOGLE_CHROME_BRANDING)
- + icon = &vector_icons::kCloseChromeRefreshIcon;
- +#endif // BUILDFLAG(GOOGLE_CHROME_BRANDING)
- SetImageModel(views::Button::STATE_NORMAL,
- ui::ImageModel::FromVectorIcon(*icon, kColorPipWindowForeground,
- kCloseButtonIconSize));
- diff --git a/chrome/browser/ui/views/overlay/video_overlay_window_views.cc b/chrome/browser/ui/views/overlay/video_overlay_window_views.cc
- index 2ff9c6dbcb1f6010b39389d0d35662408dd53aaa..8e96d85a16ce8b5e679df52860db6773c76b49f8 100644
- --- a/chrome/browser/ui/views/overlay/video_overlay_window_views.cc
- +++ b/chrome/browser/ui/views/overlay/video_overlay_window_views.cc
- @@ -17,9 +17,11 @@
- #include "build/build_config.h"
- #include "chrome/browser/picture_in_picture/picture_in_picture_occlusion_tracker.h"
- #include "chrome/browser/picture_in_picture/picture_in_picture_window_manager.h"
- +#if 0
- #include "chrome/browser/profiles/profile.h"
- #include "chrome/browser/ui/browser.h"
- #include "chrome/browser/ui/browser_finder.h"
- +#endif
- #include "chrome/browser/ui/color/chrome_color_id.h"
- #include "chrome/browser/ui/views/overlay/back_to_tab_label_button.h"
- #include "chrome/browser/ui/views/overlay/close_image_button.h"
- @@ -61,7 +63,7 @@
- #include "ui/aura/window.h"
- #endif
-
- -#if BUILDFLAG(IS_WIN)
- +#if 0
- #include "chrome/browser/shell_integration_win.h"
- #include "content/public/browser/render_widget_host_view.h"
- #include "ui/aura/window.h"
- @@ -314,7 +316,7 @@ std::unique_ptr<VideoOverlayWindowViews> VideoOverlayWindowViews::Create(
- overlay_window->Init(std::move(params));
- overlay_window->OnRootViewReady();
-
- -#if BUILDFLAG(IS_WIN)
- +#if 0
- std::wstring app_user_model_id;
- Browser* browser = chrome::FindBrowserWithTab(controller->GetWebContents());
- if (browser) {
|