Implement GpuArcVideoService for arc video accelerator

components/arc/common/arc_bridge.mojom

Index: components/arc/common/arc_bridge.mojom
diff --git a/components/arc/common/arc_bridge.mojom b/components/arc/common/arc_bridge.mojom
index f555fbcc3d02d17bdacfe36548f9717bcc01f326..d21afae6328bdd0e80d13ab0e5155a426ef35e13 100644
--- a/components/arc/common/arc_bridge.mojom
+++ b/components/arc/common/arc_bridge.mojom
@@ -223,6 +223,9 @@ interface ArcBridgeHost {
 
   // Notifies Chrome that the ARC process list has been updated.
   OnUpdateProcessList(array<RunningAppProcessInfo> processes);
+
+  // Notifies Chrome a request for ipc channel of video acceleration.
+  OnRequestArcVideoAcceleratorChannel();

[Luis Héctor Chávez, 2015/12/16 16:37:22]

Instead of adding methods directly to ArcBridge, can you create a pair of
interfaces in video.mojom?

that should contain something like:

interface VideoHost {
  RequestArcVideoAcceleratorChannel() => (handle channel_handle);
};

interface VideoInstance {
  Init(VideoHost host_ptr);
};

And then only add OnVideoInstanceReady(VideoInstance instance_ptr); here?

That way ArcBridgeService does not have to know anything about video or how to
send the IPC channel.

[kcwu, 2015/12/22 11:58:25]

Done.
Since your refactoring was reverted, I try to do the similar thing and need to
rebase with your refactoring.

 };
 
 interface ArcBridgeInstance {
@@ -263,4 +266,7 @@ interface ArcBridgeInstance {
 
   // Requests ARC instance to return the current process list.
   RequestProcessList();
+
+  // Notify the ipc handle for video accelerator is created.
+  NotifyVideoAcceleratorChannelCreated(handle channel_handle);
 };