Commit 95845389 authored by Kevin Bailey's avatar Kevin Bailey Committed by Commit Bot

[omnibox] Add tab switch button tooltip (and thus accessibility label)

Bug: 780835, 853929
Change-Id: I9f5018fa6522d76ab7a8c57b635b084764f482db
Reviewed-on: https://chromium-review.googlesource.com/1142129Reviewed-by: default avatarJustin Donnelly <jdonnelly@chromium.org>
Commit-Queue: Kevin Bailey <krb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#576120}
parent 53a08b86
......@@ -31,7 +31,6 @@ OmniboxTabSwitchButton::OmniboxTabSwitchButton(OmniboxPopupContentsView* model,
result_view_(result_view),
initialized_(false),
animation_(new gfx::SlideAnimation(this)) {
// TODO(krb): SetTooltipText(text);
SetBgColorOverride(GetBackgroundColor());
SetImage(STATE_NORMAL,
gfx::CreateVectorIcon(omnibox::kSwitchIcon,
......@@ -48,6 +47,7 @@ OmniboxTabSwitchButton::OmniboxTabSwitchButton(OmniboxPopupContentsView* model,
} else {
SetText(l10n_util::GetStringUTF16(IDS_OMNIBOX_TAB_SUGGEST_HINT));
}
SetTooltipText(l10n_util::GetStringUTF16(IDS_OMNIBOX_TAB_SUGGEST_HINT));
set_corner_radius(CalculatePreferredSize().height() / 2.f);
animation_->SetSlideDuration(500);
SetElideBehavior(gfx::FADE_TAIL);
......
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