Change the assertion of background color opaqueness in BubbleFrameView to DCHECK
This is a temporary fix to the attached crash. The invariance will be fixed in b:414655934. Bug: b:411442010 Change-Id: I3aaf50c11d473d124fdb89558b0b806cf6e52b0f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6499055 Reviewed-by: Elly FJ <ellyjones@chromium.org> Commit-Queue: Zoraiz Naeem <zoraiznaeem@chromium.org> Cr-Commit-Position: refs/heads/main@{#1453599}
This commit is contained in:

committed by
Chromium LUCI CQ

parent
5cd39fdf16
commit
06816475f6
@ -843,7 +843,8 @@ void BubbleFrameView::UpdateClientViewBackground() {
|
||||
// artifacts. Make sure this isn't the case.
|
||||
const SkColor color =
|
||||
background_color().ResolveToSkColor(GetWidget()->GetColorProvider());
|
||||
CHECK(SkColor4f::FromColor(color).isOpaque());
|
||||
// TODO(b:414655934): Remove this assertion.
|
||||
DCHECK(SkColor4f::FromColor(color).isOpaque());
|
||||
client_view->SetBackground(CreateSolidBackground(color));
|
||||
client_view->SchedulePaint();
|
||||
}
|
||||
|
Reference in New Issue
Block a user