Commit 29729314 authored by Christopher Cameron's avatar Christopher Cameron Committed by Commit Bot

MacOS Zero Copy Capture: Fix CMSampleBufferScopedAccessPermission

This should release the CMSampleBuffer in the destructor, not the
constructor.

Bug: 1125879
Change-Id: Idb266a891fdadaf16a0469e821644c2a1b78e63f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2482245Reviewed-by: default avatarMarkus Handell <handellm@google.com>
Commit-Queue: Markus Handell <handellm@google.com>
Cr-Commit-Position: refs/heads/master@{#818320}
parent 1726c2a7
......@@ -58,10 +58,8 @@ class CMSampleBufferScopedAccessPermission
: public media::VideoCaptureDevice::Client::Buffer::ScopedAccessPermission {
public:
CMSampleBufferScopedAccessPermission(CMSampleBufferRef buffer)
: buffer_(buffer, base::scoped_policy::RETAIN) {
buffer_.reset();
}
~CMSampleBufferScopedAccessPermission() override {}
: buffer_(buffer, base::scoped_policy::RETAIN) {}
~CMSampleBufferScopedAccessPermission() override { buffer_.reset(); }
private:
base::ScopedCFTypeRef<CMSampleBufferRef> buffer_;
......
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