Home Explore 引导页 在线工具 Blog
Register Sign In
Pchen0
/
electron
1
0
Fork 0
Files Issues 0 Pull Requests 0 Wiki
Tree: 2e4e6f10de
Branches Tags
electron
/
patches
/
chromium
/
fix_media_key_usage_with_globalshortcuts.patch

fix_media_key_usage_with_globalshortcuts.patch 4.3 KB
History Raw

1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798 
  1. From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
    2. 

  2. From: Shelley Vohr <[email protected]>
    3. 

  3. Date: Mon, 16 Aug 2021 17:55:32 +0200
    4. 

  4. Subject: fix: media key usage with globalShortcuts
    5. 

  5. 

  6. This patch enables media keys to work properly with Electron's globalShortcut
    7. 

  7. module. Chromium's default usage of RemoteCommandCenterDelegate on macOS falls
    8. 

  8. down into MPRemoteCommandCenter, which makes it such that an app will not
    9. 

  9. receive remote control events until it begins playing audio. This runs
    10. 

  10. counter to the design of globalShortcuts, and so we need to instead
    11. 

  11. use `ui::MediaKeysListener`.
    12. 

  12. 

  13. diff --git a/chrome/browser/extensions/global_shortcut_listener.cc b/chrome/browser/extensions/global_shortcut_listener.cc
    14. 

  14. index fa5be7e4dbddd8333263c412b84a1d93b2d5046a..1b3c65359bff41a96ff27aef6e979ac7e6e8518d 100644
    15. 

  15. --- a/chrome/browser/extensions/global_shortcut_listener.cc
    16. 

  16. +++ b/chrome/browser/extensions/global_shortcut_listener.cc
    17. 

  17. @@ -7,6 +7,7 @@
    18. 

  18.  #include "base/check.h"
    19. 

  19.  #include "base/notreached.h"
    20. 

  20.  #include "content/public/browser/browser_thread.h"
    21. 

  21. +#include "content/public/browser/media_keys_listener_manager.h"
    22. 

  22.  #include "ui/base/accelerators/accelerator.h"
    23. 

  23.  
    24. 

  24.  using content::BrowserThread;
    25. 

  25. @@ -66,6 +67,22 @@ void GlobalShortcutListener::UnregisterAccelerator(
    26. 

  26.      StopListening();
    27. 

  27.  }
    28. 

  28.  
    29. 

  29. +// static
    30. 

  30. +void GlobalShortcutListener::SetShouldUseInternalMediaKeyHandling(bool should_use) {
    31. 

  31. +  if (content::MediaKeysListenerManager::
    32. 

  32. +            IsMediaKeysListenerManagerEnabled()) {
    33. 

  33. +    content::MediaKeysListenerManager* media_keys_listener_manager =
    34. 

  34. +        content::MediaKeysListenerManager::GetInstance();
    35. 

  35. +    DCHECK(media_keys_listener_manager);
    36. 

  36. +
    37. 

  37. +    if (should_use) {
    38. 

  38. +      media_keys_listener_manager->EnableInternalMediaKeyHandling();
    39. 

  39. +    } else {
    40. 

  40. +      media_keys_listener_manager->DisableInternalMediaKeyHandling();
    41. 

  41. +    }
    42. 

  42. +  }
    43. 

  43. +}
    44. 

  44. +
    45. 

  45.  void GlobalShortcutListener::UnregisterAccelerators(Observer* observer) {
    46. 

  46.    CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
    47. 

  47.    if (IsShortcutHandlingSuspended())
    48. 

  48. diff --git a/chrome/browser/extensions/global_shortcut_listener.h b/chrome/browser/extensions/global_shortcut_listener.h
    49. 

  49. index f96af14cb915c8ab92b314ac15b6dffcdd6ec607..2556abe433493251ebd48d4aeddaa6d4f553208f 100644
    50. 

  50. --- a/chrome/browser/extensions/global_shortcut_listener.h
    51. 

  51. +++ b/chrome/browser/extensions/global_shortcut_listener.h
    52. 

  52. @@ -33,6 +33,8 @@ class GlobalShortcutListener {
    53. 

  53.  
    54. 

  54.    static GlobalShortcutListener* GetInstance();
    55. 

  55.  
    56. 

  56. +  static void SetShouldUseInternalMediaKeyHandling(bool should_use);
    57. 

  57. +
    58. 

  58.    // Register an observer for when a certain |accelerator| is struck. Returns
    59. 

  59.    // true if register successfully, or false if 1) the specificied |accelerator|
    60. 

  60.    // has been registered by another caller or other native applications, or
    61. 

  61. diff --git a/content/browser/media/media_keys_listener_manager_impl.cc b/content/browser/media/media_keys_listener_manager_impl.cc
    62. 

  62. index 8bc58dd31b5822682fe13ad7ded1dc40892b296d..1811361f25fd8276c317e4fcd883e998af8aeab8 100644
    63. 

  63. --- a/content/browser/media/media_keys_listener_manager_impl.cc
    64. 

  64. +++ b/content/browser/media/media_keys_listener_manager_impl.cc
    65. 

  65. @@ -86,7 +86,11 @@ bool MediaKeysListenerManagerImpl::StartWatchingMediaKey(
    66. 

  66.        CanActiveMediaSessionControllerReceiveEvents();
    67. 

  67.  
    68. 

  68.    // Tell the underlying MediaKeysListener to listen for the key.
    69. 

  69. -  if (should_start_watching && media_keys_listener_ &&
    70. 

  70. +  if (
    71. 

  71. +#if BUILDFLAG(IS_MAC)
    72. 

  72. +      !media_key_handling_enabled_ &&
    73. 

  73. +#endif  // BUILDFLAG(IS_MAC)
    74. 

  74. +      should_start_watching && media_keys_listener_ &&
    75. 

  75.        !media_keys_listener_->StartWatchingMediaKey(key_code)) {
    76. 

  76.      return false;
    77. 

  77.    }
    78. 

  78. @@ -320,6 +324,20 @@ void MediaKeysListenerManagerImpl::StartListeningForMediaKeysIfNecessary() {
    79. 

  79.          this, ui::MediaKeysListener::Scope::kGlobal);
    80. 

  80.      DCHECK(media_keys_listener_);
    81. 

  81.    }
    82. 

  82. +
    83. 

  83. +#if BUILDFLAG(IS_MAC)
    84. 

  84. +  // Chromium's implementation of SystemMediaControls falls
    85. 

  85. +  // down into MPRemoteCommandCenter, which makes it such that an app will not
    86. 

  86. +  // will not receive remote control events until it begins playing audio.
    87. 

  87. +  // If there's not already a MediaKeysListener instance, create one so
    88. 

  88. +  // that globalShortcuts work correctly.
    89. 

  89. +  if (!media_keys_listener_) {
    90. 

  90. +    media_keys_listener_ = ui::MediaKeysListener::Create(
    91. 

  91. +        this, ui::MediaKeysListener::Scope::kGlobal);
    92. 

  92. +    DCHECK(media_keys_listener_);
    93. 

  93. +  }
    94. 

  94. +#endif
    95. 

  95. +
    96. 

  96.    EnsureAuxiliaryServices();
    97. 

  97.  }
    98. 

  98.  
    99.