Added system log uploader.

chrome/browser/chromeos/policy/device_cloud_policy_manager_chromeos.cc

Index: chrome/browser/chromeos/policy/device_cloud_policy_manager_chromeos.cc
diff --git a/chrome/browser/chromeos/policy/device_cloud_policy_manager_chromeos.cc b/chrome/browser/chromeos/policy/device_cloud_policy_manager_chromeos.cc
index f7c2aa57c6fed7300141ecb1fef3244764852733..100d1d9ed3d00390733c94e6b1f0460451bfe546 100644
--- a/chrome/browser/chromeos/policy/device_cloud_policy_manager_chromeos.cc
+++ b/chrome/browser/chromeos/policy/device_cloud_policy_manager_chromeos.cc
@@ -24,6 +24,7 @@
 #include "chrome/browser/chromeos/policy/remote_commands/device_commands_factory_chromeos.h"
 #include "chrome/browser/chromeos/policy/server_backed_state_keys_broker.h"
 #include "chrome/browser/chromeos/policy/status_uploader.h"
+#include "chrome/browser/chromeos/policy/system_log_uploader.h"
 #include "chrome/common/pref_names.h"
 #include "chromeos/chromeos_constants.h"
 #include "chromeos/chromeos_switches.h"
@@ -253,6 +254,7 @@ void DeviceCloudPolicyManagerChromeOS::StartConnection(
   // the monitoring settings and only perform monitoring if it is active.
   if (install_attributes->IsEnterpriseDevice()) {
     CreateStatusUploader();
+    CreateSystemLogUploader();
     heartbeat_scheduler_.reset(
         new HeartbeatScheduler(g_browser_process->gcm_driver(),
                                install_attributes->GetDomain(),
@@ -277,6 +279,7 @@ void DeviceCloudPolicyManagerChromeOS::Unregister(
 
 void DeviceCloudPolicyManagerChromeOS::Disconnect() {
   status_uploader_.reset();
+  syslog_uploader_.reset();
   heartbeat_scheduler_.reset();
   core()->Disconnect();
 
@@ -341,4 +344,8 @@ void DeviceCloudPolicyManagerChromeOS::CreateStatusUploader() {
       task_runner_));
 }
 
+void DeviceCloudPolicyManagerChromeOS::CreateSystemLogUploader() {
+  syslog_uploader_.reset(new SystemLogUploader(task_runner_));

[Andrew T Wilson (Slow), 2015/07/03 15:40:47]

This is OK to leave as-is, but in general I don't usually create factory methods
that are one line since they aren't really saving lines of code, especially if
they are called only once.

[Polina Bondarenko, 2015/07/08 10:07:23]

Removed the factory method.

+}
+
 }  // namespace policy