picture-in-picture.patch 3.4 KB

1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374
  1. From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
  2. From: Heilig Benedek <[email protected]>
  3. Date: Sat, 10 Aug 2019 00:41:50 +0200
  4. Subject: feat: enable picture in picture mode for video players
  5. These files are needed to implement PiP, but the Electron build patches out
  6. chrome's generated resources for our own. This updates the #include so that we
  7. don't get errors for Chrome's generated resources, which are non-existent
  8. because we don't generate them in our build.
  9. diff --git a/chrome/browser/ui/views/overlay/close_image_button.cc b/chrome/browser/ui/views/overlay/close_image_button.cc
  10. index d236578cefc347e772305ac7ec54b9734e4aa20b..6bb89cea3b6b91ec8ea351ed60c4e4056a6340bb 100644
  11. --- a/chrome/browser/ui/views/overlay/close_image_button.cc
  12. +++ b/chrome/browser/ui/views/overlay/close_image_button.cc
  13. @@ -4,10 +4,13 @@
  14. #include "chrome/browser/ui/views/overlay/close_image_button.h"
  15. +#include "build/branding_buildflags.h"
  16. #include "build/chromeos_buildflags.h"
  17. #include "chrome/browser/ui/color/chrome_color_id.h"
  18. #include "chrome/grit/generated_resources.h"
  19. +#if BUILDFLAG(GOOGLE_CHROME_BRANDING)
  20. #include "components/vector_icons/vector_icons.h"
  21. +#endif // BUILDFLAG(GOOGLE_CHROME_BRANDING)
  22. #include "ui/base/l10n/l10n_util.h"
  23. #include "ui/base/metadata/metadata_impl_macros.h"
  24. #include "ui/base/models/image_model.h"
  25. @@ -27,7 +30,10 @@ CloseImageButton::CloseImageButton(PressedCallback callback)
  26. : OverlayWindowImageButton(std::move(callback)) {
  27. SetSize(gfx::Size(kCloseButtonSize, kCloseButtonSize));
  28. - auto* icon = &vector_icons::kCloseChromeRefreshIcon;
  29. + auto* icon = &views::kIcCloseIcon;
  30. +#if BUILDFLAG(GOOGLE_CHROME_BRANDING)
  31. + icon = &vector_icons::kCloseChromeRefreshIcon;
  32. +#endif // BUILDFLAG(GOOGLE_CHROME_BRANDING)
  33. SetImageModel(views::Button::STATE_NORMAL,
  34. ui::ImageModel::FromVectorIcon(*icon, kColorPipWindowForeground,
  35. kCloseButtonIconSize));
  36. diff --git a/chrome/browser/ui/views/overlay/video_overlay_window_views.cc b/chrome/browser/ui/views/overlay/video_overlay_window_views.cc
  37. index 2ff9c6dbcb1f6010b39389d0d35662408dd53aaa..8e96d85a16ce8b5e679df52860db6773c76b49f8 100644
  38. --- a/chrome/browser/ui/views/overlay/video_overlay_window_views.cc
  39. +++ b/chrome/browser/ui/views/overlay/video_overlay_window_views.cc
  40. @@ -17,9 +17,11 @@
  41. #include "build/build_config.h"
  42. #include "chrome/browser/picture_in_picture/picture_in_picture_occlusion_tracker.h"
  43. #include "chrome/browser/picture_in_picture/picture_in_picture_window_manager.h"
  44. +#if 0
  45. #include "chrome/browser/profiles/profile.h"
  46. #include "chrome/browser/ui/browser.h"
  47. #include "chrome/browser/ui/browser_finder.h"
  48. +#endif
  49. #include "chrome/browser/ui/color/chrome_color_id.h"
  50. #include "chrome/browser/ui/views/overlay/back_to_tab_label_button.h"
  51. #include "chrome/browser/ui/views/overlay/close_image_button.h"
  52. @@ -61,7 +63,7 @@
  53. #include "ui/aura/window.h"
  54. #endif
  55. -#if BUILDFLAG(IS_WIN)
  56. +#if 0
  57. #include "chrome/browser/shell_integration_win.h"
  58. #include "content/public/browser/render_widget_host_view.h"
  59. #include "ui/aura/window.h"
  60. @@ -314,7 +316,7 @@ std::unique_ptr<VideoOverlayWindowViews> VideoOverlayWindowViews::Create(
  61. overlay_window->Init(std::move(params));
  62. overlay_window->OnRootViewReady();
  63. -#if BUILDFLAG(IS_WIN)
  64. +#if 0
  65. std::wstring app_user_model_id;
  66. Browser* browser = chrome::FindBrowserWithTab(controller->GetWebContents());
  67. if (browser) {