From 6ef36e708f47eb3a42c42091ab975a2186700604 Mon Sep 17 00:00:00 2001 From: sergeyu Date: Tue, 21 Jun 2016 16:50:00 -0700 Subject: [PATCH] Remove DesktopCapturer::Callback::OnCaptureCompleted() The method was deprecated and shouldn't be used anywhere now. BUG=webrtc:5950 Review-Url: https://codereview.webrtc.org/2080573004 Cr-Commit-Position: refs/heads/master@{#13248} --- webrtc/modules/desktop_capture/desktop_capturer.h | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/webrtc/modules/desktop_capture/desktop_capturer.h b/webrtc/modules/desktop_capture/desktop_capturer.h index d6da81d9fe..778c93c63a 100644 --- a/webrtc/modules/desktop_capture/desktop_capturer.h +++ b/webrtc/modules/desktop_capture/desktop_capturer.h @@ -47,15 +47,7 @@ class DesktopCapturer { // Called after a frame has been captured. |frame| is not nullptr if and // only if |result| is SUCCESS. virtual void OnCaptureResult(Result result, - std::unique_ptr frame) { - OnCaptureCompleted(frame.release()); - } - - // Deprecated version of the method above that uses raw pointer instead of - // std::unique_ptr<>. - // TODO(sergeyu): Remove this method and make OnCaptureResult() pure - // virtual. crbug.com/webrtc/5950 - virtual void OnCaptureCompleted(DesktopFrame* frame) { delete frame; }; + std::unique_ptr frame) = 0; protected: virtual ~Callback() {}