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

Side by Side Diff: src/gpu/GrTextureToYUVPlanes.cpp

Issue 2164363002: Add SkColorSpace to GrDrawContext (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Remove ':' from comment Created 4 years, 5 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 | « src/gpu/GrTextureParamsAdjuster.cpp ('k') | src/gpu/GrYUVProvider.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2016 Google Inc. 2 * Copyright 2016 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "GrTextureToYUVPlanes.h" 8 #include "GrTextureToYUVPlanes.h"
9 #include "effects/GrSimpleTextureEffect.h" 9 #include "effects/GrSimpleTextureEffect.h"
10 #include "effects/GrYUVEffect.h" 10 #include "effects/GrYUVEffect.h"
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 } else { 64 } else {
65 singleChannelPixelConfig = kRGBA_8888_GrPixelConfig; 65 singleChannelPixelConfig = kRGBA_8888_GrPixelConfig;
66 } 66 }
67 67
68 // We issue draw(s) to convert from RGBA to Y, U, and V. All three plane s may have different 68 // We issue draw(s) to convert from RGBA to Y, U, and V. All three plane s may have different
69 // sizes however we optimize for two other cases - all planes are the sa me (1 draw to YUV), 69 // sizes however we optimize for two other cases - all planes are the sa me (1 draw to YUV),
70 // and U and V are the same but Y differs (2 draws, one for Y, one for U V). 70 // and U and V are the same but Y differs (2 draws, one for Y, one for U V).
71 if (sizes[0] == sizes[1] && sizes[1] == sizes[2]) { 71 if (sizes[0] == sizes[1] && sizes[1] == sizes[2]) {
72 yuvDrawContext = context->newDrawContext(SkBackingFit::kApprox, 72 yuvDrawContext = context->newDrawContext(SkBackingFit::kApprox,
73 sizes[0].fWidth, sizes[0].f Height, 73 sizes[0].fWidth, sizes[0].f Height,
74 kRGBA_8888_GrPixelConfig); 74 kRGBA_8888_GrPixelConfig, n ullptr);
75 if (!yuvDrawContext) { 75 if (!yuvDrawContext) {
76 return false; 76 return false;
77 } 77 }
78 } else { 78 } else {
79 yDrawContext = context->newDrawContext(SkBackingFit::kApprox, 79 yDrawContext = context->newDrawContext(SkBackingFit::kApprox,
80 sizes[0].fWidth, sizes[0].fHe ight, 80 sizes[0].fWidth, sizes[0].fHe ight,
81 singleChannelPixelConfig); 81 singleChannelPixelConfig, nul lptr);
82 if (!yDrawContext) { 82 if (!yDrawContext) {
83 return false; 83 return false;
84 } 84 }
85 if (sizes[1] == sizes[2]) { 85 if (sizes[1] == sizes[2]) {
86 // TODO: Add support for GL_RG when available. 86 // TODO: Add support for GL_RG when available.
87 uvDrawContext = context->newDrawContext(SkBackingFit::kApprox, 87 uvDrawContext = context->newDrawContext(SkBackingFit::kApprox,
88 sizes[1].fWidth, sizes[1 ].fHeight, 88 sizes[1].fWidth, sizes[1 ].fHeight,
89 kRGBA_8888_GrPixelConfig ); 89 kRGBA_8888_GrPixelConfig , nullptr);
90 if (!uvDrawContext) { 90 if (!uvDrawContext) {
91 return false; 91 return false;
92 } 92 }
93 } else { 93 } else {
94 uDrawContext = context->newDrawContext(SkBackingFit::kApprox, 94 uDrawContext = context->newDrawContext(SkBackingFit::kApprox,
95 sizes[1].fWidth, sizes[1] .fHeight, 95 sizes[1].fWidth, sizes[1] .fHeight,
96 singleChannelPixelConfig) ; 96 singleChannelPixelConfig, nullptr);
97 vDrawContext = context->newDrawContext(SkBackingFit::kApprox, 97 vDrawContext = context->newDrawContext(SkBackingFit::kApprox,
98 sizes[2].fWidth, sizes[2] .fHeight, 98 sizes[2].fWidth, sizes[2] .fHeight,
99 singleChannelPixelConfig) ; 99 singleChannelPixelConfig, nullptr);
100 if (!uDrawContext || !vDrawContext) { 100 if (!uDrawContext || !vDrawContext) {
101 return false; 101 return false;
102 } 102 }
103 } 103 }
104 } 104 }
105 105
106 // Do all the draws before any readback. 106 // Do all the draws before any readback.
107 if (yuvDrawContext) { 107 if (yuvDrawContext) {
108 if (!convert_texture(texture, yuvDrawContext.get(), 108 if (!convert_texture(texture, yuvDrawContext.get(),
109 sizes[0].fWidth, sizes[0].fHeight, 109 sizes[0].fWidth, sizes[0].fHeight,
(...skipping 111 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 if (!tex->readPixels(0, 0, sizes[2].fWidth, sizes[2].fHeight, 221 if (!tex->readPixels(0, 0, sizes[2].fWidth, sizes[2].fHeight,
222 kAlpha_8_GrPixelConfig, planes[2], rowBytes [2])) { 222 kAlpha_8_GrPixelConfig, planes[2], rowBytes [2])) {
223 return false; 223 return false;
224 } 224 }
225 return true; 225 return true;
226 } 226 }
227 } 227 }
228 } 228 }
229 return false; 229 return false;
230 } 230 }
OLDNEW
« no previous file with comments | « src/gpu/GrTextureParamsAdjuster.cpp ('k') | src/gpu/GrYUVProvider.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698