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

Unified Diff: remoting/client/plugin/chromoting_instance.cc

Issue 10800077: Create and destroy Pepper audio player on connect and disconnect. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/client/plugin/chromoting_instance.cc
diff --git a/remoting/client/plugin/chromoting_instance.cc b/remoting/client/plugin/chromoting_instance.cc
index 873586c80ef7ec9eacc7763581c381e3d2d04a4a..71ccd2a74633dacbc02aa7caa23ff27fd42f053f 100644
--- a/remoting/client/plugin/chromoting_instance.cc
+++ b/remoting/client/plugin/chromoting_instance.cc
@@ -228,7 +228,6 @@ bool ChromotingInstance::Init(uint32_t argc,
context_.decode_task_runner(), consumer_proxy);
view_.reset(new PepperView(this, &context_, rectangle_decoder_.get()));
consumer_proxy->Attach(view_->AsWeakPtr());
- audio_player_.reset(new PepperAudioPlayer(this));
return true;
}
@@ -476,6 +475,7 @@ void ChromotingInstance::Connect(const ClientConfig& config) {
jingle_glue::JingleThreadWrapper::EnsureForCurrentThread();
host_connection_.reset(new protocol::ConnectionToHost(true));
+ audio_player_.reset(new PepperAudioPlayer(this));
client_.reset(new ChromotingClient(config, context_.main_task_runner(),
host_connection_.get(), this,
rectangle_decoder_.get(),
@@ -541,6 +541,7 @@ void ChromotingInstance::Disconnect() {
input_handler_.reset();
input_tracker_.reset();
mouse_input_filter_.reset();
+ audio_player_.reset();
host_connection_.reset();
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698