Allow user to manually fetch remote commands

chrome/browser/ui/webui/policy_ui.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/policy_ui.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/compiler_specific.h"
@@ skipping 18 matching lines @@
 #include "components/policy/core/common/cloud/cloud_policy_client.h"
 #include "components/policy/core/common/cloud/cloud_policy_constants.h"
 #include "components/policy/core/common/cloud/cloud_policy_core.h"
 #include "components/policy/core/common/cloud/cloud_policy_refresh_scheduler.h"
 #include "components/policy/core/common/cloud/cloud_policy_store.h"
 #include "components/policy/core/common/cloud/cloud_policy_validator.h"
 #include "components/policy/core/common/policy_map.h"
 #include "components/policy/core/common/policy_namespace.h"
 #include "components/policy/core/common/policy_service.h"
 #include "components/policy/core/common/policy_types.h"
+#include "components/policy/core/common/remote_commands/remote_commands_service.h"
 #include "components/policy/core/common/schema.h"
 #include "components/policy/core/common/schema_map.h"
 #include "components/policy/core/common/schema_registry.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_ui.h"
 #include "content/public/browser/web_ui_data_source.h"
 #include "content/public/browser/web_ui_message_handler.h"
 #include "google_apis/gaia/gaia_auth_util.h"
 #include "grit/browser_resources.h"
 #include "grit/components_strings.h"
@@ skipping 751 matching lines @@
   web_ui()->CallJavascriptFunction("policy.Page.setStatus", status);
 }
 
 void PolicyUIHandler::HandleInitialized(const base::ListValue* args) {
   SendPolicyNames();
   SendPolicyValues();
   SendStatus();
 }
 
 void PolicyUIHandler::HandleReloadPolicies(const base::ListValue* args) {
+#if defined(OS_CHROMEOS)
+  // Allow user to manually fetch remote commands, in case invalidation
+  // service is not working properly.
+  // TODO(binjin): evaluate and possibly remove this after invalidation

[bartfab (slow), 2015/04/29 15:21:38]

Nit: Please reference a bug number.

[binjin, 2015/04/29 17:24:42]

Done.

+  // service is landed and tested.
+  policy::BrowserPolicyConnectorChromeOS* connector =

[bartfab (slow), 2015/04/29 15:21:38]

Nit: #include
"chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"

[binjin, 2015/04/29 17:24:42]

It's already included, inside defined(OS_CHROMEOS) guard.

+      g_browser_process->platform_part()->browser_policy_connector_chromeos();

[bartfab (slow), 2015/04/29 15:21:38]

Nit: #include "chrome/browser/browser_process_platform_part.h"

[binjin, 2015/04/29 17:24:42]

Done.

+  connector->GetDeviceCloudPolicyManager()
+      ->core()
+      ->remote_commands_service()
+      ->FetchRemoteCommands();
+#endif
   GetPolicyService()->RefreshPolicies(
       base::Bind(&PolicyUIHandler::OnRefreshPoliciesDone,
                  weak_factory_.GetWeakPtr()));
 }
 
 void PolicyUIHandler::OnRefreshPoliciesDone() const {
   web_ui()->CallJavascriptFunction("policy.Page.reloadPoliciesDone");
 }
 
 policy::PolicyService* PolicyUIHandler::GetPolicyService() const {
   return policy::ProfilePolicyConnectorFactory::GetForBrowserContext(
              web_ui()->GetWebContents()->GetBrowserContext())->policy_service();
 }
 
 PolicyUI::PolicyUI(content::WebUI* web_ui) : WebUIController(web_ui) {
   web_ui->AddMessageHandler(new PolicyUIHandler);
   content::WebUIDataSource::Add(Profile::FromWebUI(web_ui),
                                 CreatePolicyUIHTMLSource());
 }
 
 PolicyUI::~PolicyUI() {
 }