Commit 0d109814 authored by Henrik Boström's avatar Henrik Boström Committed by Commit Bot

[macOS Capture] Restore SampleBufferTransformer's pool size to 10.

When the M88 cut was made, the kDefaultBufferPoolSize was 10. While
investigating https://crbug.com/1148964 we disabled and then re-enabled
the InCaptureConvertTnoNv12 feature and along the way changed the pool
size to 5 (trying to see if the issue was related to running out of
handles, but that wasn't the case).

To ensure M88 and Canary uses the same pool size, this CL restores the
old pool value to 10. If we decide to change the pool size we should do
that in a separate easily back-mergeable CL, but there have been no
signs yet that 10 is too many.

Bug: chromium:1148964
Change-Id: Ib122ca1ed9ca061951ee5d77dfa3ddbd84d90f05
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2553548
Commit-Queue: Henrik Boström <hbos@chromium.org>
Commit-Queue: Markus Handell <handellm@google.com>
Auto-Submit: Henrik Boström <hbos@chromium.org>
Reviewed-by: default avatarMarkus Handell <handellm@google.com>
Cr-Commit-Position: refs/heads/master@{#830143}
parent 0b3578b8
...@@ -27,7 +27,7 @@ const base::Feature kInCaptureConvertToNv12WithLibyuv{ ...@@ -27,7 +27,7 @@ const base::Feature kInCaptureConvertToNv12WithLibyuv{
namespace { namespace {
constexpr size_t kDefaultBufferPoolSize = 5; constexpr size_t kDefaultBufferPoolSize = 10;
// NV12 a.k.a. 420v // NV12 a.k.a. 420v
constexpr OSType kPixelFormatNv12 = constexpr OSType kPixelFormatNv12 =
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment