drgreen/android_frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Null check color_space_ptr in draw functor v2

Browse Source 
Missed toXYZD50 usage last time. Still assume sRGB if null.

Bug: 187798471
Test: Manually tested setting color_space_ptr to null in
      GL/VKFunctorDrawable
Change-Id: Idee2660d368dd55e45f5d07d52839105ba951ff3
This commit is contained in:
Bo Liu 2021-07-19 10:14:12 -04:00
parent 9028f4992c
commit fe71933655
1 changed files with 16 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
native/webview/plat_support/draw_functor.cpp
View File

@ -116,8 +116,14 @@ void draw_gl(int functor, void* data,
}
COMPILE_ASSERT(sizeof(params.color_space_toXYZD50) == sizeof(skcms_Matrix3x3),
gamut_transform_size_mismatch);
draw_gl_params.color_space_ptr->toXYZD50(
reinterpret_cast<skcms_Matrix3x3*>(&params.color_space_toXYZD50));
if (draw_gl_params.color_space_ptr) {
draw_gl_params.color_space_ptr->toXYZD50(
reinterpret_cast<skcms_Matrix3x3*>(&params.color_space_toXYZD50));
} else {
// Assume sRGB.
memcpy(&params.color_space_toXYZD50, &SkNamedGamut::kSRGB,
sizeof(params.color_space_toXYZD50));
}
SupportData* support = static_cast<SupportData*>(data);
support->callbacks.draw_gl(functor, support->data, &params);
@ -196,8 +202,14 @@ void drawVk(int functor, void* data,
};
COMPILE_ASSERT(sizeof(params.color_space_toXYZD50) == sizeof(skcms_Matrix3x3),
gamut_transform_size_mismatch);
draw_vk_params.color_space_ptr->toXYZD50(
reinterpret_cast<skcms_Matrix3x3*>(&params.color_space_toXYZD50));
if (draw_vk_params.color_space_ptr) {
draw_vk_params.color_space_ptr->toXYZD50(
reinterpret_cast<skcms_Matrix3x3*>(&params.color_space_toXYZD50));
} else {
// Assume sRGB.
memcpy(&params.color_space_toXYZD50, &SkNamedGamut::kSRGB,
sizeof(params.color_space_toXYZD50));
}
COMPILE_ASSERT(NELEM(params.transform) == NELEM(draw_vk_params.transform),
mismatched_transform_matrix_sizes);
for (int i = 0; i < NELEM(params.transform); ++i) {