Reporting of policy errors via host-offline-reason: part 1

remoting/host/it2me/it2me_host.cc

Index: remoting/host/it2me/it2me_host.cc
diff --git a/remoting/host/it2me/it2me_host.cc b/remoting/host/it2me/it2me_host.cc
index b9a17f8c9431b159fddbcd15a586eb438b315995..d34d7e889fe40e27904052384ce334e563d1f0e1 100644
--- a/remoting/host/it2me/it2me_host.cc
+++ b/remoting/host/it2me/it2me_host.cc
@@ -70,7 +70,8 @@ void It2MeHost::Connect() {
   policy_watcher_.reset(
       policy_hack::PolicyWatcher::Create(host_context_->network_task_runner()));
   policy_watcher_->StartWatching(
-      base::Bind(&It2MeHost::OnPolicyUpdate, this));
+      base::Bind(&It2MeHost::OnPolicyUpdate, this),
+      base::Bind(&It2MeHost::OnPolicyError, this));
 
   // Switch to the network thread to start the actual connection.
   host_context_->network_task_runner()->PostTask(
@@ -332,6 +333,11 @@ void It2MeHost::OnPolicyUpdate(scoped_ptr<base::DictionaryValue> policies) {
   }
 }
 
+void It2MeHost::OnPolicyError() {
+  // TODO(lukasza): Report the policy error to the user.

[kelvinp, 2014/11/13 19:40:57]

Can you add a bug number for the TODO item?
// TODO(lukasza): Report the policy error to the user (crbug.com/<bug_number>)

[Łukasz Anforowicz, 2014/11/13 19:57:22]

Done.

+  NOTIMPLEMENTED();
+}
+
 void It2MeHost::UpdateNatPolicy(bool nat_traversal_enabled) {
   DCHECK(host_context_->network_task_runner()->BelongsToCurrentThread());