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

Side by Side Diff: remoting/test/protocol_perftest.cc

Issue 2113523007: More cleanups in FrameStats (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix android Created 4 years, 5 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 <utility> 5 #include <utility>
6 6
7 #include "base/base64.h" 7 #include "base/base64.h"
8 #include "base/files/file_util.h" 8 #include "base/files/file_util.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "base/memory/ptr_util.h" 10 #include "base/memory/ptr_util.h"
(...skipping 132 matching lines...) Expand 10 before | Expand all | Expand 10 after
143 void DeliverHostMessage(const protocol::ExtensionMessage& message) override {} 143 void DeliverHostMessage(const protocol::ExtensionMessage& message) override {}
144 void SetDesktopSize(const webrtc::DesktopSize& size, 144 void SetDesktopSize(const webrtc::DesktopSize& size,
145 const webrtc::DesktopVector& dpi) override {} 145 const webrtc::DesktopVector& dpi) override {}
146 protocol::ClipboardStub* GetClipboardStub() override { return nullptr; } 146 protocol::ClipboardStub* GetClipboardStub() override { return nullptr; }
147 protocol::CursorShapeStub* GetCursorShapeStub() override { 147 protocol::CursorShapeStub* GetCursorShapeStub() override {
148 return &cursor_shape_stub_; 148 return &cursor_shape_stub_;
149 } 149 }
150 150
151 // VideoRenderer interface. 151 // VideoRenderer interface.
152 bool Initialize(const ClientContext& client_context, 152 bool Initialize(const ClientContext& client_context,
153 protocol::PerformanceTracker* perf_tracker) override { 153 protocol::FrameStatsConsumer* stats_consumer) override {
154 return true; 154 return true;
155 } 155 }
156 void OnSessionConfig(const protocol::SessionConfig& config) override {} 156 void OnSessionConfig(const protocol::SessionConfig& config) override {}
157 protocol::VideoStub* GetVideoStub() override { return this; } 157 protocol::VideoStub* GetVideoStub() override { return this; }
158 protocol::FrameConsumer* GetFrameConsumer() override { return this; } 158 protocol::FrameConsumer* GetFrameConsumer() override { return this; }
159 protocol::FrameStatsConsumer* GetFrameStatsConsumer() override {
160 return nullptr;
161 }
159 162
160 // protocol::VideoStub interface. 163 // protocol::VideoStub interface.
161 void ProcessVideoPacket(std::unique_ptr<VideoPacket> packet, 164 void ProcessVideoPacket(std::unique_ptr<VideoPacket> packet,
162 const base::Closure& done) override { 165 const base::Closure& done) override {
163 if (packet->data().empty()) { 166 if (packet->data().empty()) {
164 // Ignore keep-alive packets 167 // Ignore keep-alive packets
165 done.Run(); 168 done.Run();
166 return; 169 return;
167 } 170 }
168 171
(...skipping 467 matching lines...) Expand 10 before | Expand all | Expand 10 after
636 639
637 TEST_P(ProtocolPerfTest, TotalLatencyIce) { 640 TEST_P(ProtocolPerfTest, TotalLatencyIce) {
638 MeasureTotalLatency(false); 641 MeasureTotalLatency(false);
639 } 642 }
640 643
641 TEST_P(ProtocolPerfTest, TotalLatencyWebrtc) { 644 TEST_P(ProtocolPerfTest, TotalLatencyWebrtc) {
642 MeasureTotalLatency(true); 645 MeasureTotalLatency(true);
643 } 646 }
644 647
645 } // namespace remoting 648 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/protocol/webrtc_video_renderer_adapter.cc ('k') | remoting/test/test_chromoting_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698