Implement GpuArcVideoService for arc video accelerator

components/arc/common/arc_bridge.mojom

 // Copyright 2015 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.
 
 module arc;
 
 // Describes the boot phase of the ARC instance, as defined by AOSP in
 // com.android.server.SystemService
 enum InstanceBootPhase {
   // Boot phase indicating that the instance is not running
@@ skipping 205 matching lines @@
   // Notifies that a notification is removed on Android.
   // |key| is the identifier of the notification.
   OnNotificationRemoved(string key);
 
   // Acquire and release wake locks on the host side.
   OnAcquireDisplayWakeLock(DisplayWakeLockType type);
   OnReleaseDisplayWakeLock(DisplayWakeLockType type);
 
   // 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 {
   // Establishes full-duplex communication with the host.
   // |pipe| is the MessagePipe endpoint that is bound to the ArcBridgeHostPtr
   // binding.
   Init(ArcBridgeHost host_ptr);
 
   // Registers a virtual input device on the container side.
   // |name| is the device name, like "Chrome OS Keyboard".
@@ skipping 20 matching lines @@
   // package and activity, which cannot be empty.
   RequestAppIcon(string package, string activity,
                  ScaleFactor scale_factor);
 
   // Sends an event from Chrome notification UI to Android.
   // |event| is a type of occured event.
   SendNotificationEventToAndroid(string key, ArcNotificationEvent event);
 
   // Requests ARC instance to return the current process list.
   RequestProcessList();
+
+  // Notify the ipc handle for video accelerator is created.
+  NotifyVideoAcceleratorChannelCreated(handle channel_handle);
 };