Home Explore 引导页 在线工具 Blog
Register Sign In
Pchen0
/
electron
1
0
Fork 0
Files Issues 0 Pull Requests 0 Wiki
Tree: c3036d4557
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

12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879 
  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/renderer/platform/media/multi_buffer_data_source.cc b/third_party/blink/renderer/platform/media/multi_buffer_data_source.cc
    16. 

  16. index 2b5c5f059159865cb84a7efcefe47cd788d5f5ee..1627649fb5085c637cde556074b242c4939c81de 100644
    17. 

  17. --- a/third_party/blink/renderer/platform/media/multi_buffer_data_source.cc
    18. 

  18. +++ b/third_party/blink/renderer/platform/media/multi_buffer_data_source.cc
    19. 

  19. @@ -11,8 +11,10 @@
    20. 

  20.  #include "base/functional/bind.h"
    21. 

  21.  #include "base/functional/callback_helpers.h"
    22. 

  22.  #include "base/location.h"
    23. 

  23. +#include "base/no_destructor.h"
    24. 

  24.  #include "base/numerics/safe_conversions.h"
    25. 

  25.  #include "base/task/single_thread_task_runner.h"
    26. 

  26. +#include "base/strings/string_util.h"
    27. 

  27.  #include "media/base/media_log.h"
    28. 

  28.  #include "net/base/net_errors.h"
    29. 

  29.  #include "third_party/blink/renderer/platform/media/buffered_data_source_host_impl.h"
    30. 

  30. @@ -61,8 +63,20 @@ const int kUpdateBufferSizeFrequency = 32;
    31. 

  31.  // How long to we delay a seek after a read?
    32. 

  32.  constexpr base::TimeDelta kSeekDelay = base::Milliseconds(20);
    33. 

  33.  
    34. 

  34. +std::vector<std::string>* GetStreamingSchemes() {
    35. 

  35. +  static base::NoDestructor<std::vector<std::string>> streaming_schemes({
    36. 

  36. +    url::kHttpsScheme,
    37. 

  37. +    url::kHttpScheme
    38. 

  38. +  });
    39. 

  39. +  return streaming_schemes.get();
    40. 

  40. +}
    41. 

  41. +
    42. 

  42.  }  // namespace
    43. 

  43.  
    44. 

  44. +void AddStreamingScheme(const char* new_scheme) {
    45. 

  45. +  GetStreamingSchemes()->push_back(new_scheme);
    46. 

  46. +}
    47. 

  47. +
    48. 

  48.  class MultiBufferDataSource::ReadOperation {
    49. 

  49.   public:
    50. 

  50.    ReadOperation() = delete;
    51. 

  51. @@ -154,7 +168,14 @@ bool MultiBufferDataSource::media_has_played() const {
    52. 

  52.  
    53. 

  53.  bool MultiBufferDataSource::AssumeFullyBuffered() const {
    54. 

  54.    DCHECK(url_data_);
    55. 

  55. -  return !url_data_->url().SchemeIsHTTPOrHTTPS();
    56. 

  56. +
    57. 

  57. +  const std::string scheme = url_data_->url().scheme();
    58. 

  58. +  for (const std::string& streaming_scheme : *GetStreamingSchemes()) {
    59. 

  59. +    if (base::EqualsCaseInsensitiveASCII(scheme, streaming_scheme)) {
    60. 

  60. +      return false;
    61. 

  61. +    }
    62. 

  62. +  }
    63. 

  63. +  return true;
    64. 

  64.  }
    65. 

  65.  
    66. 

  66.  void MultiBufferDataSource::SetReader(MultiBufferReader* reader) {
    67. 

  67. diff --git a/third_party/blink/renderer/platform/media/multi_buffer_data_source.h b/third_party/blink/renderer/platform/media/multi_buffer_data_source.h
    68. 

  68. index 29ca8efb5994ccb5b65d2762088dd53e0583cb72..c533149f567731e6bb6ed2d94dcffd03346f7f87 100644
    69. 

  69. --- a/third_party/blink/renderer/platform/media/multi_buffer_data_source.h
    70. 

  70. +++ b/third_party/blink/renderer/platform/media/multi_buffer_data_source.h
    71. 

  71. @@ -33,6 +33,8 @@ namespace blink {
    72. 

  72.  class BufferedDataSourceHost;
    73. 

  73.  class MultiBufferReader;
    74. 

  74.  
    75. 

  75. +void BLINK_PLATFORM_EXPORT AddStreamingScheme(const char* new_scheme);
    76. 

  76. +
    77. 

  77.  // A data source capable of loading URLs and buffering the data using an
    78. 

  78.  // in-memory sliding window.
    79. 

  79.  //
    80.