Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(308)

Side by Side Diff: remoting/protocol/webrtc_frame_scheduler.cc

Issue 1902593004: Remove webrtc::DesktopCapturer::Callback::CreateSharedMemory() overrides. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « remoting/protocol/webrtc_frame_scheduler.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "remoting/protocol/webrtc_frame_scheduler.h" 5 #include "remoting/protocol/webrtc_frame_scheduler.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "remoting/base/constants.h" 10 #include "remoting/base/constants.h"
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 key_frame_request_ = true; 77 key_frame_request_ = true;
78 } 78 }
79 79
80 bool WebRtcFrameScheduler::ClearAndGetKeyFrameRequest() { 80 bool WebRtcFrameScheduler::ClearAndGetKeyFrameRequest() {
81 base::AutoLock lock(lock_); 81 base::AutoLock lock(lock_);
82 bool key_frame_request = key_frame_request_; 82 bool key_frame_request = key_frame_request_;
83 key_frame_request_ = false; 83 key_frame_request_ = false;
84 return key_frame_request; 84 return key_frame_request;
85 } 85 }
86 86
87 webrtc::SharedMemory* WebRtcFrameScheduler::CreateSharedMemory(size_t size) {
88 DCHECK(thread_checker_.CalledOnValidThread());
89 return nullptr;
90 }
91
92 void WebRtcFrameScheduler::OnCaptureCompleted(webrtc::DesktopFrame* frame) { 87 void WebRtcFrameScheduler::OnCaptureCompleted(webrtc::DesktopFrame* frame) {
93 DCHECK(thread_checker_.CalledOnValidThread()); 88 DCHECK(thread_checker_.CalledOnValidThread());
94 89
95 VLOG(1) << "Capture overhead " 90 VLOG(1) << "Capture overhead "
96 << (base::TimeTicks::Now() - last_capture_ticks_).InMilliseconds(); 91 << (base::TimeTicks::Now() - last_capture_ticks_).InMilliseconds();
97 capture_pending_ = false; 92 capture_pending_ = false;
98 93
99 std::unique_ptr<webrtc::DesktopFrame> owned_frame(frame); 94 std::unique_ptr<webrtc::DesktopFrame> owned_frame(frame);
100 95
101 if (encode_pending_) { 96 if (encode_pending_) {
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
172 // This gets resolved through a PLI request. 167 // This gets resolved through a PLI request.
173 webrtc_transport_->video_encoder_factory()->SendEncodedFrame( 168 webrtc_transport_->video_encoder_factory()->SendEncodedFrame(
174 capture_timestamp_ms, std::move(packet)); 169 capture_timestamp_ms, std::move(packet));
175 170
176 VLOG(1) << "Send duration " 171 VLOG(1) << "Send duration "
177 << (base::TimeTicks::Now() - current).InMilliseconds(); 172 << (base::TimeTicks::Now() - current).InMilliseconds();
178 } 173 }
179 174
180 } // namespace protocol 175 } // namespace protocol
181 } // namespace remoting 176 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/protocol/webrtc_frame_scheduler.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698