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

Unified Diff: remoting/host/client_session.cc

Issue 502123003: Remove implicit conversions from scoped_refptr to T* in remoting/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/host/cast_extension_session.cc ('k') | remoting/host/oauth_token_getter.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/client_session.cc
diff --git a/remoting/host/client_session.cc b/remoting/host/client_session.cc
index 31c3bf145d35707d8486ff753c36fe523ed43830..d741e1cdda881d42aeeab43bcd79a6d64fb81840 100644
--- a/remoting/host/client_session.cc
+++ b/remoting/host/client_session.cc
@@ -142,21 +142,21 @@ void ClientSession::ControlVideo(const protocol::VideoControl& video_control) {
VLOG(1) << "Received VideoControl (enable="
<< video_control.enable() << ")";
pause_video_ = !video_control.enable();
- if (video_scheduler_)
+ if (video_scheduler_.get())
video_scheduler_->Pause(pause_video_);
}
if (video_control.has_lossless_encode()) {
VLOG(1) << "Received VideoControl (lossless_encode="
<< video_control.lossless_encode() << ")";
lossless_video_encode_ = video_control.lossless_encode();
- if (video_scheduler_)
+ if (video_scheduler_.get())
video_scheduler_->SetLosslessEncode(lossless_video_encode_);
}
if (video_control.has_lossless_color()) {
VLOG(1) << "Received VideoControl (lossless_color="
<< video_control.lossless_color() << ")";
lossless_video_color_ = video_control.lossless_color();
- if (video_scheduler_)
+ if (video_scheduler_.get())
video_scheduler_->SetLosslessColor(lossless_video_color_);
}
}
@@ -207,7 +207,7 @@ void ClientSession::SetCapabilities(
void ClientSession::RequestPairing(
const protocol::PairingRequest& pairing_request) {
- if (pairing_registry_ && pairing_request.has_client_name()) {
+ if (pairing_registry_.get() && pairing_request.has_client_name()) {
protocol::PairingRegistry::Pairing pairing =
pairing_registry_->CreatePairing(pairing_request.client_name());
protocol::PairingResponse pairing_response;
« no previous file with comments | « remoting/host/cast_extension_session.cc ('k') | remoting/host/oauth_token_getter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698