Commit e4e881c1 authored by Ahmed Mehfooz's avatar Ahmed Mehfooz Committed by Commit Bot

Remove special bottom inset for notification focus

The special case for letting the focus ring for the bottom most
notification exceed it's bottom side bounds is not applicable
with the recent message center changes.

Bug: 1020546
Change-Id: Ib0242dddc6aeb922d50a8795ce16c150734a6f76
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947625Reviewed-by: default avatarEvan Stade <estade@chromium.org>
Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#720771}
parent 5194a1d9
......@@ -186,11 +186,9 @@ SkPath MessageView::GetHighlightPath() const {
// Shrink focus ring size by -kFocusHaloInset on each side to draw
// them on top of the notifications. We need to do this because TrayBubbleView
// has a layer that masks to bounds due to which the focus ring can not extend
// outside the view. This is not required on the bottom most notification's
// bottom side.
// outside the view.
int inset = -views::PlatformStyle::kFocusHaloInset;
int bottom_inset = bottom_radius_ == 0 ? inset : 0;
rect.Inset(gfx::Insets(inset, inset, bottom_inset, inset));
rect.Inset(gfx::Insets(inset));
int top_radius = std::max(0, top_radius_ - inset);
int bottom_radius = std::max(0, bottom_radius_ - inset);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment