Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(106)

Unified Diff: cc/ipc/quads_struct_traits.cc

Issue 2773103002: cc: Unify all sources of color space for YUVVideoDrawQuad (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/ipc/quads_struct_traits.h ('k') | cc/ipc/struct_traits_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/ipc/quads_struct_traits.cc
diff --git a/cc/ipc/quads_struct_traits.cc b/cc/ipc/quads_struct_traits.cc
index 011f62771f9543cd5d55ff03200bcf2226e6655b..b5d52e3a4dd792afceb67a4276132c7eab111401 100644
--- a/cc/ipc/quads_struct_traits.cc
+++ b/cc/ipc/quads_struct_traits.cc
@@ -188,39 +188,6 @@ bool StructTraits<cc::mojom::TileQuadStateDataView, cc::DrawQuad>::Read(
return true;
}
-cc::mojom::YUVColorSpace
-EnumTraits<cc::mojom::YUVColorSpace, cc::YUVVideoDrawQuad::ColorSpace>::ToMojom(
- cc::YUVVideoDrawQuad::ColorSpace color_space) {
- switch (color_space) {
- case cc::YUVVideoDrawQuad::REC_601:
- return cc::mojom::YUVColorSpace::REC_601;
- case cc::YUVVideoDrawQuad::REC_709:
- return cc::mojom::YUVColorSpace::REC_709;
- case cc::YUVVideoDrawQuad::JPEG:
- return cc::mojom::YUVColorSpace::JPEG;
- }
- NOTREACHED();
- return cc::mojom::YUVColorSpace::JPEG;
-}
-
-// static
-bool EnumTraits<cc::mojom::YUVColorSpace, cc::YUVVideoDrawQuad::ColorSpace>::
- FromMojom(cc::mojom::YUVColorSpace input,
- cc::YUVVideoDrawQuad::ColorSpace* out) {
- switch (input) {
- case cc::mojom::YUVColorSpace::REC_601:
- *out = cc::YUVVideoDrawQuad::REC_601;
- return true;
- case cc::mojom::YUVColorSpace::REC_709:
- *out = cc::YUVVideoDrawQuad::REC_709;
- return true;
- case cc::mojom::YUVColorSpace::JPEG:
- *out = cc::YUVVideoDrawQuad::JPEG;
- return true;
- }
- return false;
-}
-
// static
bool StructTraits<cc::mojom::YUVVideoQuadStateDataView, cc::DrawQuad>::Read(
cc::mojom::YUVVideoQuadStateDataView data,
@@ -245,8 +212,6 @@ bool StructTraits<cc::mojom::YUVVideoQuadStateDataView, cc::DrawQuad>::Read(
"The A plane resource should be the last resource ID.");
quad->resources.count = data.a_plane_resource_id() ? 4 : 3;
- if (!data.ReadColorSpace(&quad->color_space))
- return false;
quad->resource_offset = data.resource_offset();
quad->resource_multiplier = data.resource_multiplier();
quad->bits_per_channel = data.bits_per_channel();
« no previous file with comments | « cc/ipc/quads_struct_traits.h ('k') | cc/ipc/struct_traits_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698