Initial RemoteCommandService

components/policy/core/common/remote_commands/remote_command_job.h

Index: components/policy/core/common/remote_commands/remote_command_job.h
diff --git a/components/policy/core/common/remote_commands/remote_command_job.h b/components/policy/core/common/remote_commands/remote_command_job.h
index dd23747ead94c29fdde8d9af732532c2669998d1..f03466af97e425b544854b53bd069c7602f70c4a 100644
--- a/components/policy/core/common/remote_commands/remote_command_job.h
+++ b/components/policy/core/common/remote_commands/remote_command_job.h
@@ -37,6 +37,16 @@ class POLICY_EXPORT RemoteCommandJob {
     TERMINATED,           // The job was terminated before finishing by itself.
   };
 
+  // XXX
+  class Factory {
+   public:
+    virtual ~Factory() {}
+
+    // XXX
+    virtual scoped_ptr<RemoteCommandJob> BuildJobForType(

[bartfab (slow), 2015/01/29 10:18:05]

We could just make this a static method of RemoteCommandJob. Alternatively, if
you want a Factory, I would put it in its own, independent class in a separate
header.

[binjin, 2015/01/29 10:44:10]

I expect chrome and chromeos have different implementation of this class, I will
move it into a separate header.

It's in component/ so I think make it static method will be bad for
extensibility.

[bartfab (slow), 2015/01/29 15:28:58]

Good thinking. Yes, the factory will need to be inside *chrome*, not the
component, because it provides the bridge between the generic framework and
Chrome-specific command types.

+        enterprise_management::RemoteCommand_Type type) = 0;
+  };
+
   using FinishedCallback = base::Closure;
 
   virtual ~RemoteCommandJob();