Add stride for imported Dmabuf in ArcVideoAccelerator.

components/arc/common/video_accelerator.mojom

 // Copyright 2016 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.
 
 // This file defined the mojo interface between Android and Chromium for video
 // decoding and encoding. See comments of ArcVideoAccelerator for more info.
 
 module arc.mojom;
 
 [Extensible]
@@ skipping 43 matching lines @@
   uint32 num_input_buffers;
   uint32 input_pixel_format;
 };
 
 interface VideoAcceleratorService {
   Initialize@0(ArcVideoAcceleratorConfig config) => (bool result);
 
   BindSharedMemory@1(PortType port, uint32 index, handle ashmem_fd,
                    uint32 offset, uint32 length);
 
-  BindDmabuf@2(PortType port, uint32 index, handle dmabuf_fd);
+  BindDmabuf@2(PortType port, uint32 index, handle dmabuf_fd, int32 stride);

[dcheng, 2016/05/07 06:22:56]

Why int32 and not uint32?

[Owen Lin, 2016/05/09 07:30:16]

It is int32_t in where it's used:
https://code.google.com/p/chromium/codesearch#chromium/src/ui/gfx/native_pixm...

In addition, according to video_frame.h, stride could be negative:
https://code.google.com/p/chromium/codesearch#chromium/src/media/base/video_f...

 
   UseBuffer@3(PortType port, uint32 index, BufferMetadata metadata);
 
   SetNumberOfOutputBuffers@4(uint32 number);
 
   Reset@5();
 };
 
 interface VideoAcceleratorServiceClient {
   enum Error {
     NO_ERROR = 0,
     ILLEGAL_STATE = 1,
     INVALID_ARGUMENT = 2,
     UNREADABLE_INPUT = 3,
     PLATFORM_FAILURE = 4,
   };
 
   Init@0(VideoAcceleratorService service_ptr);
 
   OnError@1(Error error);
 
   OnBufferDone@2(PortType port, uint32 index, BufferMetadata metadata);
 
   OnResetDone@3();
 
   OnOutputFormatChanged@4(VideoFormat format);
 };