Commit 23ea4334 authored by Tommy C. Li's avatar Tommy C. Li Committed by Commit Bot

Omnibox Refresh: Fix focus ring hide behavior when popup opens

This CL ensures that the focus ring is shown/hidden properly when the
popup is opened/closed by scheduling a paint operation when the popup
opens and closes.

Bug: 823535
Change-Id: I19303bebc5b64a91f3497d3898e805a81dbacf31
Reviewed-on: https://chromium-review.googlesource.com/1091187
Commit-Queue: Tommy Li <tommycli@chromium.org>
Reviewed-by: default avatarBret Sepulveda <bsep@chromium.org>
Reviewed-by: default avatarJustin Donnelly <jdonnelly@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565776}
parent aa2249a7
...@@ -1230,6 +1230,10 @@ void LocationBarView::OnChanged() { ...@@ -1230,6 +1230,10 @@ void LocationBarView::OnChanged() {
void LocationBarView::OnPopupVisibilityChanged() { void LocationBarView::OnPopupVisibilityChanged() {
RefreshBackground(); RefreshBackground();
// The focus ring may be hidden or shown when the popup visibility changes.
if (focus_ring_)
focus_ring_->SchedulePaint();
} }
const ToolbarModel* LocationBarView::GetToolbarModel() const { const ToolbarModel* LocationBarView::GetToolbarModel() const {
......
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