mirror of
https://github.com/electron/electron.git
synced 2026-04-10 03:01:51 -04:00
chore: cherry-pick 8c346e3cd9 from chromium. (#26830)
* chore: cherry-pick 8c346e3cd9 from chromium. * update patches Co-authored-by: John Kleinschmidt <jkleinsc@electronjs.org> Co-authored-by: Electron Bot <electron@github.com>
This commit is contained in:
@@ -117,3 +117,4 @@ ignore_renderframehostimpl_detach_for_speculative_rfhs.patch
|
||||
cherry-pick-eec5025668f8.patch
|
||||
cherry-pick-bbc6ab5bb49c.patch
|
||||
cherry-pick-5ffbb7ed173a.patch
|
||||
ui_check_that_unpremultiply_is_passed_a_32bpp_image.patch
|
||||
|
||||
@@ -0,0 +1,36 @@
|
||||
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
||||
From: Scott Violet <sky@chromium.org>
|
||||
Date: Fri, 13 Nov 2020 20:47:38 +0000
|
||||
Subject: ui: CHECK that UnPremultiply is passed a 32bpp image
|
||||
|
||||
To do otherwise results in accessing random data.
|
||||
|
||||
BUG=1147430
|
||||
TEST=none
|
||||
|
||||
(cherry picked from commit 1f673896837ab8c687d93fec604c96c78c7f679b)
|
||||
|
||||
Change-Id: Icedacbaac64cad3fc903e6423c6f9aad8c1e8cb5
|
||||
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2531118
|
||||
Commit-Queue: danakj <danakj@chromium.org>
|
||||
Reviewed-by: danakj <danakj@chromium.org>
|
||||
Cr-Original-Commit-Position: refs/heads/master@{#826300}
|
||||
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2538047
|
||||
Reviewed-by: Scott Violet <sky@chromium.org>
|
||||
Commit-Queue: Scott Violet <sky@chromium.org>
|
||||
Cr-Commit-Position: refs/branch-heads/4240@{#1452}
|
||||
Cr-Branched-From: f297677702651916bbf65e59c0d4bbd4ce57d1ee-refs/heads/master@{#800218}
|
||||
|
||||
diff --git a/ui/gfx/skbitmap_operations.cc b/ui/gfx/skbitmap_operations.cc
|
||||
index c08079f585b033849c744838c1db7a05ef729450..30372e849213d0446ac7edd72452782606bf938f 100644
|
||||
--- a/ui/gfx/skbitmap_operations.cc
|
||||
+++ b/ui/gfx/skbitmap_operations.cc
|
||||
@@ -630,6 +630,8 @@ SkBitmap SkBitmapOperations::UnPreMultiply(const SkBitmap& bitmap) {
|
||||
return bitmap;
|
||||
if (bitmap.isOpaque())
|
||||
return bitmap;
|
||||
+ // It's expected this code is called with a 32bpp image.
|
||||
+ CHECK_EQ(kN32_SkColorType, bitmap.colorType());
|
||||
|
||||
const SkImageInfo& opaque_info =
|
||||
bitmap.info().makeAlphaType(kOpaque_SkAlphaType);
|
||||
Reference in New Issue
Block a user