Commit 06afc981 authored by Leonard Grey's avatar Leonard Grey Committed by Commit Bot

Don't install focus rings on hover buttons

Since these focus on hover, we pretty much never want to do this (..right?)

Bug: 874913
Change-Id: I68c129184e0cf6e5895923696158b10cbefd3adc
Reviewed-on: https://chromium-review.googlesource.com/1177914Reviewed-by: default avatarElly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Leonard Grey <lgrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583762}
parent 8cdb7d65
...@@ -72,6 +72,7 @@ HoverButton::HoverButton(views::ButtonListener* button_listener, ...@@ -72,6 +72,7 @@ HoverButton::HoverButton(views::ButtonListener* button_listener,
icon_view_(nullptr), icon_view_(nullptr),
secondary_icon_view_(nullptr), secondary_icon_view_(nullptr),
listener_(button_listener) { listener_(button_listener) {
SetInstallFocusRingOnFocus(false);
SetFocusBehavior(FocusBehavior::ALWAYS); SetFocusBehavior(FocusBehavior::ALWAYS);
SetFocusPainter(nullptr); SetFocusPainter(nullptr);
......
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