Commit e5f8f85c authored by Chih-Yu Huang's avatar Chih-Yu Huang Committed by Commit Bot

component/arc: Rename mojo struct ColorPlaneLayout.

In CL:1824717, we separated media::VideoFrameLayout::Plane to
media::ColorPlaneLayout. This CL also rename the correlated mojo
struct to "ColorPlaneLayout".

BUG=b:136716638
TEST=component_unittests --gtest_filter=VideoAcceleratorStructTraitsTest.*

Change-Id: I8073594a342af294f942b47eb1982303284f7261
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1837315Reviewed-by: default avatarHirokazu Honda <hiroh@chromium.org>
Reviewed-by: default avatarDominick Ng <dominickn@chromium.org>
Commit-Queue: Chih-Yu Huang <akahuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#702766}
parent 48e9460f
...@@ -238,9 +238,9 @@ bool StructTraits<arc::mojom::SizeDataView, gfx::Size>::Read( ...@@ -238,9 +238,9 @@ bool StructTraits<arc::mojom::SizeDataView, gfx::Size>::Read(
} }
// static // static
bool StructTraits<arc::mojom::MediaVideoFramePlaneDataView, bool StructTraits<
media::ColorPlaneLayout>:: arc::mojom::ColorPlaneLayoutDataView,
Read(arc::mojom::MediaVideoFramePlaneDataView data, media::ColorPlaneLayout>::Read(arc::mojom::ColorPlaneLayoutDataView data,
media::ColorPlaneLayout* out) { media::ColorPlaneLayout* out) {
out->offset = data.offset(); out->offset = data.offset();
out->stride = data.stride(); out->stride = data.stride();
......
...@@ -73,7 +73,7 @@ struct EnumTraits<arc::mojom::VideoPixelFormat, media::VideoPixelFormat> { ...@@ -73,7 +73,7 @@ struct EnumTraits<arc::mojom::VideoPixelFormat, media::VideoPixelFormat> {
}; };
template <> template <>
struct StructTraits<arc::mojom::MediaVideoFramePlaneDataView, struct StructTraits<arc::mojom::ColorPlaneLayoutDataView,
media::ColorPlaneLayout> { media::ColorPlaneLayout> {
static int32_t stride(const media::ColorPlaneLayout& r) { return r.stride; } static int32_t stride(const media::ColorPlaneLayout& r) { return r.stride; }
...@@ -81,7 +81,7 @@ struct StructTraits<arc::mojom::MediaVideoFramePlaneDataView, ...@@ -81,7 +81,7 @@ struct StructTraits<arc::mojom::MediaVideoFramePlaneDataView,
static uint32_t size(const media::ColorPlaneLayout& r) { return r.size; } static uint32_t size(const media::ColorPlaneLayout& r) { return r.size; }
static bool Read(arc::mojom::MediaVideoFramePlaneDataView data, static bool Read(arc::mojom::ColorPlaneLayoutDataView data,
media::ColorPlaneLayout* out); media::ColorPlaneLayout* out);
}; };
......
...@@ -106,7 +106,7 @@ struct Size { ...@@ -106,7 +106,7 @@ struct Size {
// The information of a plane of a video frame that describes how physical // The information of a plane of a video frame that describes how physical
// buffers are allocated. // buffers are allocated.
struct MediaVideoFramePlane { struct ColorPlaneLayout {
int32 stride; int32 stride;
uint32 offset; uint32 offset;
uint32 size; uint32 size;
...@@ -117,7 +117,7 @@ struct MediaVideoFramePlane { ...@@ -117,7 +117,7 @@ struct MediaVideoFramePlane {
struct VideoFrameLayout { struct VideoFrameLayout {
VideoPixelFormat format; VideoPixelFormat format;
Size coded_size; Size coded_size;
array<MediaVideoFramePlane> planes; array<ColorPlaneLayout> planes;
bool is_multi_planar; bool is_multi_planar;
uint32 buffer_addr_align; uint32 buffer_addr_align;
uint64 modifier; uint64 modifier;
......
...@@ -33,7 +33,7 @@ deps = [ ...@@ -33,7 +33,7 @@ deps = [
type_mappings = [ type_mappings = [
"arc.mojom.DecoderBuffer=arc::DecoderBuffer[move_only]", "arc.mojom.DecoderBuffer=arc::DecoderBuffer[move_only]",
"arc.mojom.DecodeStatus=media::DecodeStatus", "arc.mojom.DecodeStatus=media::DecodeStatus",
"arc.mojom.MediaVideoFramePlane=media::ColorPlaneLayout", "arc.mojom.ColorPlaneLayout=media::ColorPlaneLayout",
"arc.mojom.Size=gfx::Size", "arc.mojom.Size=gfx::Size",
"arc.mojom.VideoCodecProfile=media::VideoCodecProfile", "arc.mojom.VideoCodecProfile=media::VideoCodecProfile",
"arc.mojom.VideoFrame=scoped_refptr<media::VideoFrame>[nullable_is_same_type]", "arc.mojom.VideoFrame=scoped_refptr<media::VideoFrame>[nullable_is_same_type]",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment