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

Side by Side Diff: remoting/host/client_session.cc

Issue 836183002: Rename sequence_number to timestamp. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 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/host/client_session.h ('k') | remoting/host/video_scheduler.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/host/client_session.h" 5 #include "remoting/host/client_session.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 8
9 #include "base/message_loop/message_loop_proxy.h" 9 #include "base/message_loop/message_loop_proxy.h"
10 #include "remoting/base/capabilities.h" 10 #include "remoting/base/capabilities.h"
(...skipping 365 matching lines...) Expand 10 before | Expand all | Expand 10 after
376 client_clipboard_factory_.InvalidateWeakPtrs(); 376 client_clipboard_factory_.InvalidateWeakPtrs();
377 input_injector_.reset(); 377 input_injector_.reset();
378 screen_controls_.reset(); 378 screen_controls_.reset();
379 desktop_environment_.reset(); 379 desktop_environment_.reset();
380 380
381 // Notify the ChromotingHost that this client is disconnected. 381 // Notify the ChromotingHost that this client is disconnected.
382 // TODO(sergeyu): Log failure reason? 382 // TODO(sergeyu): Log failure reason?
383 event_handler_->OnSessionClosed(this); 383 event_handler_->OnSessionClosed(this);
384 } 384 }
385 385
386 void ClientSession::OnSequenceNumberUpdated( 386 void ClientSession::OnEventTimestamp(protocol::ConnectionToClient* connection,
387 protocol::ConnectionToClient* connection, int64 sequence_number) { 387 int64 timestamp) {
388 DCHECK(CalledOnValidThread()); 388 DCHECK(CalledOnValidThread());
389 DCHECK_EQ(connection_.get(), connection); 389 DCHECK_EQ(connection_.get(), connection);
390 390
391 if (video_scheduler_.get()) 391 if (video_scheduler_.get())
392 video_scheduler_->UpdateSequenceNumber(sequence_number); 392 video_scheduler_->SetLatestEventTimestamp(timestamp);
393 } 393 }
394 394
395 void ClientSession::OnRouteChange( 395 void ClientSession::OnRouteChange(
396 protocol::ConnectionToClient* connection, 396 protocol::ConnectionToClient* connection,
397 const std::string& channel_name, 397 const std::string& channel_name,
398 const protocol::TransportRoute& route) { 398 const protocol::TransportRoute& route) {
399 DCHECK(CalledOnValidThread()); 399 DCHECK(CalledOnValidThread());
400 DCHECK_EQ(connection_.get(), connection); 400 DCHECK_EQ(connection_.get(), connection);
401 event_handler_->OnSessionRouteChange(this, channel_name, route); 401 event_handler_->OnSessionRouteChange(this, channel_name, route);
402 } 402 }
(...skipping 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
513 return make_scoped_ptr(new AudioEncoderVerbatim()); 513 return make_scoped_ptr(new AudioEncoderVerbatim());
514 } else if (audio_config.codec == protocol::ChannelConfig::CODEC_OPUS) { 514 } else if (audio_config.codec == protocol::ChannelConfig::CODEC_OPUS) {
515 return make_scoped_ptr(new AudioEncoderOpus()); 515 return make_scoped_ptr(new AudioEncoderOpus());
516 } 516 }
517 517
518 NOTREACHED(); 518 NOTREACHED();
519 return nullptr; 519 return nullptr;
520 } 520 }
521 521
522 } // namespace remoting 522 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/host/client_session.h ('k') | remoting/host/video_scheduler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698