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

Side by Side Diff: ui/gfx/color_space_win.cc

Issue 2742613002: color: Always use parametric color spaces for raster (Closed)
Patch Set: Incorporate review feedback 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 unified diff | Download patch
« no previous file with comments | « ui/gfx/color_space.cc ('k') | ui/gfx/color_transform.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ui/gfx/color_space_win.h" 5 #include "ui/gfx/color_space_win.h"
6 6
7 namespace gfx { 7 namespace gfx {
8 8
9 DXVA2_ExtendedFormat ColorSpaceWin::GetExtendedFormat( 9 DXVA2_ExtendedFormat ColorSpaceWin::GetExtendedFormat(
10 const ColorSpace& color_space) { 10 const ColorSpace& color_space) {
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
71 format.VideoPrimaries = DXVA2_VideoPrimaries_SMPTE240M; 71 format.VideoPrimaries = DXVA2_VideoPrimaries_SMPTE240M;
72 break; 72 break;
73 73
74 case gfx::ColorSpace::PrimaryID::FILM: 74 case gfx::ColorSpace::PrimaryID::FILM:
75 case gfx::ColorSpace::PrimaryID::BT2020: 75 case gfx::ColorSpace::PrimaryID::BT2020:
76 case gfx::ColorSpace::PrimaryID::SMPTEST428_1: 76 case gfx::ColorSpace::PrimaryID::SMPTEST428_1:
77 case gfx::ColorSpace::PrimaryID::SMPTEST431_2: 77 case gfx::ColorSpace::PrimaryID::SMPTEST431_2:
78 case gfx::ColorSpace::PrimaryID::SMPTEST432_1: 78 case gfx::ColorSpace::PrimaryID::SMPTEST432_1:
79 case gfx::ColorSpace::PrimaryID::XYZ_D50: 79 case gfx::ColorSpace::PrimaryID::XYZ_D50:
80 case gfx::ColorSpace::PrimaryID::ADOBE_RGB: 80 case gfx::ColorSpace::PrimaryID::ADOBE_RGB:
81 case gfx::ColorSpace::PrimaryID::ICC_BASED:
81 case gfx::ColorSpace::PrimaryID::CUSTOM: 82 case gfx::ColorSpace::PrimaryID::CUSTOM:
82 case gfx::ColorSpace::PrimaryID::INVALID: 83 case gfx::ColorSpace::PrimaryID::INVALID:
83 // Not handled 84 // Not handled
84 break; 85 break;
85 } 86 }
86 87
87 switch (color_space.transfer_) { 88 switch (color_space.transfer_) {
88 case gfx::ColorSpace::TransferID::BT709: 89 case gfx::ColorSpace::TransferID::BT709:
89 case gfx::ColorSpace::TransferID::SMPTE170M: 90 case gfx::ColorSpace::TransferID::SMPTE170M:
90 format.VideoTransferFunction = DXVA2_VideoTransFunc_709; 91 format.VideoTransferFunction = DXVA2_VideoTransFunc_709;
(...skipping 20 matching lines...) Expand all
111 case gfx::ColorSpace::TransferID::LOG_SQRT: 112 case gfx::ColorSpace::TransferID::LOG_SQRT:
112 case gfx::ColorSpace::TransferID::IEC61966_2_4: 113 case gfx::ColorSpace::TransferID::IEC61966_2_4:
113 case gfx::ColorSpace::TransferID::BT1361_ECG: 114 case gfx::ColorSpace::TransferID::BT1361_ECG:
114 case gfx::ColorSpace::TransferID::BT2020_10: 115 case gfx::ColorSpace::TransferID::BT2020_10:
115 case gfx::ColorSpace::TransferID::BT2020_12: 116 case gfx::ColorSpace::TransferID::BT2020_12:
116 case gfx::ColorSpace::TransferID::SMPTEST2084: 117 case gfx::ColorSpace::TransferID::SMPTEST2084:
117 case gfx::ColorSpace::TransferID::SMPTEST428_1: 118 case gfx::ColorSpace::TransferID::SMPTEST428_1:
118 case gfx::ColorSpace::TransferID::ARIB_STD_B67: 119 case gfx::ColorSpace::TransferID::ARIB_STD_B67:
119 case gfx::ColorSpace::TransferID::GAMMA24: 120 case gfx::ColorSpace::TransferID::GAMMA24:
120 case gfx::ColorSpace::TransferID::SMPTEST2084_NON_HDR: 121 case gfx::ColorSpace::TransferID::SMPTEST2084_NON_HDR:
122 case gfx::ColorSpace::TransferID::ICC_BASED:
121 case gfx::ColorSpace::TransferID::CUSTOM: 123 case gfx::ColorSpace::TransferID::CUSTOM:
122 case gfx::ColorSpace::TransferID::INVALID: 124 case gfx::ColorSpace::TransferID::INVALID:
123 // Not handled 125 // Not handled
124 break; 126 break;
125 } 127 }
126 128
127 return format; 129 return format;
128 } 130 }
129 131
130 DXGI_COLOR_SPACE_TYPE ColorSpaceWin::GetDXGIColorSpace( 132 DXGI_COLOR_SPACE_TYPE ColorSpaceWin::GetDXGIColorSpace(
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
212 case gfx::ColorSpace::TransferID::SMPTE170M: 214 case gfx::ColorSpace::TransferID::SMPTE170M:
213 ret.YCbCr_Matrix = 1; 215 ret.YCbCr_Matrix = 1;
214 216
215 default: 217 default:
216 break; 218 break;
217 } 219 }
218 return ret; 220 return ret;
219 } 221 }
220 222
221 } // namespace gfx 223 } // namespace gfx
OLDNEW
« no previous file with comments | « 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