From b01de0bfb6c684ac57d72902af961c28545035e4 Mon Sep 17 00:00:00 2001 From: Aaron Vaage Date: Wed, 23 May 2018 15:02:07 -0700 Subject: [PATCH] Disallow Copy and Assign WebVtt File Buffer We should never need to copy or assign the webvtt file buffer so this change enforces that. Change-Id: I119865e96c0188781adbe115beb2486b1b56542a --- packager/media/formats/webvtt/webvtt_file_buffer.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/packager/media/formats/webvtt/webvtt_file_buffer.h b/packager/media/formats/webvtt/webvtt_file_buffer.h index 954309d859..be168dd4b7 100644 --- a/packager/media/formats/webvtt/webvtt_file_buffer.h +++ b/packager/media/formats/webvtt/webvtt_file_buffer.h @@ -32,7 +32,9 @@ class WebVttFileBuffer { size_t sample_count() const { return sample_count_; } private: - // TODO(vaage): Add missing deleted copy and assign constructors. + WebVttFileBuffer(const WebVttFileBuffer&) = delete; + WebVttFileBuffer& operator=(const WebVttFileBuffer&) = delete; + std::string buffer_; size_t sample_count_ = 0; };