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

Unified Diff: remoting/host/plugin/host_script_object.cc

Issue 8725016: Refactor IT2Me-specific functions into a HostObserver subclass. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove unused header Created 9 years, 1 month 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
Index: remoting/host/plugin/host_script_object.cc
diff --git a/remoting/host/plugin/host_script_object.cc b/remoting/host/plugin/host_script_object.cc
index 779aae348144f4e786505bdf83b71675bfac93f8..8959445511607f58e2d7991d896a1385dbfa0073 100644
--- a/remoting/host/plugin/host_script_object.cc
+++ b/remoting/host/plugin/host_script_object.cc
@@ -17,6 +17,7 @@
#include "remoting/host/host_config.h"
#include "remoting/host/host_key_pair.h"
#include "remoting/host/in_memory_host_config.h"
+#include "remoting/host/it2me_observer.h"
#include "remoting/host/plugin/host_log_handler.h"
#include "remoting/host/plugin/policy_hack/nat_policy.h"
#include "remoting/host/register_support_host_request.h"
@@ -99,10 +100,10 @@ HostNPScriptObject::HostNPScriptObject(
HostNPScriptObject::~HostNPScriptObject() {
CHECK_EQ(base::PlatformThread::CurrentId(), np_thread_id_);
- // Shutdown DesktopEnvironment first so that it doesn't try to post
+ // Shutdown It2MeObserver first so that it doesn't try to post
// tasks on the UI thread while we are stopping the host.
- if (desktop_environment_.get()) {
- desktop_environment_->Shutdown();
+ if (it2me_observer_.get()) {
+ it2me_observer_->Shutdown();
}
HostLogHandler::UnregisterLoggingScriptObject(this);
@@ -525,6 +526,9 @@ void HostNPScriptObject::FinishConnect(
host_->AddStatusObserver(log_to_server_.get());
}
host_->set_it2me(true);
+ it2me_observer_.reset(new It2MeObserver(host_.get(), &host_context_));
+ it2me_observer_->Init();
+ host_->AddStatusObserver(it2me_observer_.get());
{
base::AutoLock auto_lock(ui_strings_lock_);

Powered by Google App Engine
This is Rietveld 408576698