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

Unified Diff: ui/gfx/color_space_win.cc

Issue 2697863003: color: Clarify default behaviors (Closed)
Patch Set: color: Remove redundant PrimaryID/TransferID/MatrixID values Created 3 years, 10 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
Index: ui/gfx/color_space_win.cc
diff --git a/ui/gfx/color_space_win.cc b/ui/gfx/color_space_win.cc
index d3ab700939a53301a461f7a1d54a891b20fab156..8bad14fe729b192358f3465f4da0853829655ed4 100644
--- a/ui/gfx/color_space_win.cc
+++ b/ui/gfx/color_space_win.cc
@@ -24,8 +24,6 @@ DXVA2_ExtendedFormat ColorSpaceWin::GetExtendedFormat(
case gfx::ColorSpace::RangeID::FULL:
format.NominalRange = DXVA2_NominalRange_0_255;
break;
-
- case gfx::ColorSpace::RangeID::UNSPECIFIED:
case gfx::ColorSpace::RangeID::DERIVED:
// Not handled
break;
@@ -44,14 +42,11 @@ DXVA2_ExtendedFormat ColorSpaceWin::GetExtendedFormat(
break;
case gfx::ColorSpace::MatrixID::RGB:
- case gfx::ColorSpace::MatrixID::UNSPECIFIED:
- case gfx::ColorSpace::MatrixID::RESERVED:
case gfx::ColorSpace::MatrixID::FCC:
case gfx::ColorSpace::MatrixID::YCOCG:
case gfx::ColorSpace::MatrixID::BT2020_NCL:
case gfx::ColorSpace::MatrixID::BT2020_CL:
case gfx::ColorSpace::MatrixID::YDZDX:
- case gfx::ColorSpace::MatrixID::UNKNOWN:
// Not handled
break;
}
@@ -73,15 +68,12 @@ DXVA2_ExtendedFormat ColorSpaceWin::GetExtendedFormat(
format.VideoPrimaries = DXVA2_VideoPrimaries_SMPTE240M;
break;
- case gfx::ColorSpace::PrimaryID::RESERVED0:
- case gfx::ColorSpace::PrimaryID::UNSPECIFIED:
- case gfx::ColorSpace::PrimaryID::RESERVED:
case gfx::ColorSpace::PrimaryID::FILM:
case gfx::ColorSpace::PrimaryID::BT2020:
case gfx::ColorSpace::PrimaryID::SMPTEST428_1:
case gfx::ColorSpace::PrimaryID::SMPTEST431_2:
case gfx::ColorSpace::PrimaryID::SMPTEST432_1:
- case gfx::ColorSpace::PrimaryID::UNKNOWN:
+ case gfx::ColorSpace::PrimaryID::INVALID:
case gfx::ColorSpace::PrimaryID::XYZ_D50:
case gfx::ColorSpace::PrimaryID::ADOBE_RGB:
case gfx::ColorSpace::PrimaryID::CUSTOM:
@@ -111,9 +103,6 @@ DXVA2_ExtendedFormat ColorSpaceWin::GetExtendedFormat(
format.VideoTransferFunction = DXVA2_VideoTransFunc_sRGB;
break;
- case gfx::ColorSpace::TransferID::RESERVED0:
- case gfx::ColorSpace::TransferID::UNSPECIFIED:
- case gfx::ColorSpace::TransferID::RESERVED:
case gfx::ColorSpace::TransferID::LOG:
case gfx::ColorSpace::TransferID::LOG_SQRT:
case gfx::ColorSpace::TransferID::IEC61966_2_4:
@@ -123,7 +112,7 @@ DXVA2_ExtendedFormat ColorSpaceWin::GetExtendedFormat(
case gfx::ColorSpace::TransferID::SMPTEST2084:
case gfx::ColorSpace::TransferID::SMPTEST428_1:
case gfx::ColorSpace::TransferID::ARIB_STD_B67:
- case gfx::ColorSpace::TransferID::UNKNOWN:
+ case gfx::ColorSpace::TransferID::INVALID:
case gfx::ColorSpace::TransferID::GAMMA24:
case gfx::ColorSpace::TransferID::SMPTEST2084_NON_HDR:
case gfx::ColorSpace::TransferID::CUSTOM:
« ui/gfx/color_space.cc ('K') | « ui/gfx/color_space.cc ('k') | ui/gfx/color_transform.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698