From 605d00bd6f83176c7d05f4107181eb95fb9a799c Mon Sep 17 00:00:00 2001 From: Markus Handell Date: Fri, 3 May 2024 11:28:15 +0200 Subject: [PATCH] VideoFrameBuffer: remove TODO. After some investigation, it's not worth updating all consumers of the interface in line with the TODO comment. It's better to just remove the TODO as the call provides value in Chrome. Fixed: b/328533258 Change-Id: I7b60616b81a6d03dac1b3856b4aef2ed4e69cd1f Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/349701 Reviewed-by: Stefan Holmer Commit-Queue: Markus Handell Commit-Queue: Stefan Holmer Auto-Submit: Markus Handell Cr-Commit-Position: refs/heads/main@{#42221} --- api/video/video_frame_buffer.h | 1 - 1 file changed, 1 deletion(-) diff --git a/api/video/video_frame_buffer.h b/api/video/video_frame_buffer.h index ebea7d9406..bf0e83e018 100644 --- a/api/video/video_frame_buffer.h +++ b/api/video/video_frame_buffer.h @@ -127,7 +127,6 @@ class RTC_EXPORT VideoFrameBuffer : public webrtc::RefCountInterface { rtc::ArrayView types); // For logging: returns a textual representation of the storage. - // TODO(b/328533258): Make pure virtual once downstream adapts. virtual std::string storage_representation() const; protected: