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

feat_add_streaming-protocol_registry_to_multibuffer_data_source.patch 3.4 KB
History Raw

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

  2. From: Paul Frazee <[email protected]>
    3. 

  3. Date: Sat, 6 Jun 2020 10:30:45 -0500
    4. 

  4. Subject: feat: add streaming-protocol registry to multibuffer_data_source
    5. 

  5. 

  6. blink::WebMediaPlayerImpl - which provides the <video> and <audio> behaviors - needs to know
    7. 

  7. whether a data source will stream or fully buffer the response. It determines this behavior
    8. 

  8. with MultibufferDataSource::AssumeFullyBuffered() which has http/s hardwired. An incorrect
    9. 

  9. determination will cause the video/audio to fail playing.
    10. 

  10. 

  11. This patch adds a list of "streaming protocols" to the MultibufferDataSource in order to allow
    12. 

  12. other protocols to register their streaming behavior. MultibufferDataSource::AssumeFullyBuffered()
    13. 

  13. then refers to the list so that it can correctly determine the data source's settings.
    14. 

  14. 

  15. diff --git a/third_party/blink/public/platform/media/multi_buffer_data_source.h b/third_party/blink/public/platform/media/multi_buffer_data_source.h
    16. 

  16. index 3a116703a77ef03715c8e3afca77b90205b32d63..e2bfe736c76d2c35ab5a0533b7569d86b22422f6 100644
    17. 

  17. --- a/third_party/blink/public/platform/media/multi_buffer_data_source.h
    18. 

  18. +++ b/third_party/blink/public/platform/media/multi_buffer_data_source.h
    19. 

  19. @@ -33,6 +33,8 @@ namespace blink {
    20. 

  20.  class BufferedDataSourceHost;
    21. 

  21.  class MultiBufferReader;
    22. 

  22.  
    23. 

  23. +void BLINK_PLATFORM_EXPORT AddStreamingScheme(const char* new_scheme);
    24. 

  24. +
    25. 

  25.  // A data source capable of loading URLs and buffering the data using an
    26. 

  26.  // in-memory sliding window.
    27. 

  27.  //
    28. 

  28. diff --git a/third_party/blink/renderer/platform/media/multi_buffer_data_source.cc b/third_party/blink/renderer/platform/media/multi_buffer_data_source.cc
    29. 

  29. index 4b37d8de877ad9ff94f11152eefb01117d1afe66..7db4a9c407d18279603c16b3b58f2456cc1e86fa 100644
    30. 

  30. --- a/third_party/blink/renderer/platform/media/multi_buffer_data_source.cc
    31. 

  31. +++ b/third_party/blink/renderer/platform/media/multi_buffer_data_source.cc
    32. 

  32. @@ -9,9 +9,11 @@
    33. 

  33.  #include "base/bind.h"
    34. 

  34.  #include "base/callback_helpers.h"
    35. 

  35.  #include "base/location.h"
    36. 

  36. +#include "base/no_destructor.h"
    37. 

  37.  #include "base/numerics/ranges.h"
    38. 

  38.  #include "base/numerics/safe_conversions.h"
    39. 

  39.  #include "base/single_thread_task_runner.h"
    40. 

  40. +#include "base/strings/string_util.h"
    41. 

  41.  #include "media/base/media_log.h"
    42. 

  42.  #include "net/base/net_errors.h"
    43. 

  43.  #include "third_party/blink/public/platform/media/buffered_data_source_host_impl.h"
    44. 

  44. @@ -60,8 +62,20 @@ const int kUpdateBufferSizeFrequency = 32;
    45. 

  45.  // How long to we delay a seek after a read?
    46. 

  46.  constexpr base::TimeDelta kSeekDelay = base::TimeDelta::FromMilliseconds(20);
    47. 

  47.  
    48. 

  48. +std::vector<std::string>* GetStreamingSchemes() {
    49. 

  49. +  static base::NoDestructor<std::vector<std::string>> streaming_schemes({
    50. 

  50. +    url::kHttpsScheme,
    51. 

  51. +    url::kHttpScheme
    52. 

  52. +  });
    53. 

  53. +  return streaming_schemes.get();
    54. 

  54. +}
    55. 

  55. +
    56. 

  56.  }  // namespace
    57. 

  57.  
    58. 

  58. +void AddStreamingScheme(const char* new_scheme) {
    59. 

  59. +  GetStreamingSchemes()->push_back(new_scheme);
    60. 

  60. +}
    61. 

  61. +
    62. 

  62.  class MultiBufferDataSource::ReadOperation {
    63. 

  63.   public:
    64. 

  64.    ReadOperation() = delete;
    65. 

  65. @@ -153,7 +167,14 @@ bool MultiBufferDataSource::media_has_played() const {
    66. 

  66.  
    67. 

  67.  bool MultiBufferDataSource::AssumeFullyBuffered() const {
    68. 

  68.    DCHECK(url_data_);
    69. 

  69. -  return !url_data_->url().SchemeIsHTTPOrHTTPS();
    70. 

  70. +
    71. 

  71. +  const std::string scheme = url_data_->url().scheme();
    72. 

  72. +  for (const std::string& streaming_scheme : *GetStreamingSchemes()) {
    73. 

  73. +    if (base::LowerCaseEqualsASCII(scheme, streaming_scheme)) {
    74. 

  74. +      return false;
    75. 

  75. +    }
    76. 

  76. +  }
    77. 

  77. +  return true;
    78. 

  78.  }
    79. 

  79.  
    80. 

  80.  void MultiBufferDataSource::SetReader(MultiBufferReader* reader) {
    81.