Commit c00290d6 authored by sky@chromium.org's avatar sky@chromium.org

Nukes Link::OnPaint, it's not needed and causes double focus border

Label::OnPaint also draws a focus border, so Link doesn't need to do
it.

BUG=330675
TEST=none
R=ben@chromium.org

Review URL: https://codereview.chromium.org/124553006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@243351 0039d316-1c4b-4281-b951-d872f2087c98
parent 90c62f8f
...@@ -148,13 +148,6 @@ void Link::OnEnabledChanged() { ...@@ -148,13 +148,6 @@ void Link::OnEnabledChanged() {
View::OnEnabledChanged(); View::OnEnabledChanged();
} }
void Link::OnPaint(gfx::Canvas* canvas) {
Label::OnPaint(canvas);
if (HasFocus())
canvas->DrawFocusRect(GetLocalBounds());
}
void Link::OnFocus() { void Link::OnFocus() {
Label::OnFocus(); Label::OnFocus();
// We render differently focused. // We render differently focused.
......
...@@ -47,7 +47,6 @@ class VIEWS_EXPORT Link : public Label { ...@@ -47,7 +47,6 @@ class VIEWS_EXPORT Link : public Label {
const ui::KeyEvent& event) OVERRIDE; const ui::KeyEvent& event) OVERRIDE;
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE; virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE;
virtual void OnEnabledChanged() OVERRIDE; virtual void OnEnabledChanged() OVERRIDE;
virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE;
virtual void OnFocus() OVERRIDE; virtual void OnFocus() OVERRIDE;
virtual void OnBlur() OVERRIDE; virtual void OnBlur() OVERRIDE;
virtual void SetFontList(const gfx::FontList& font_list) OVERRIDE; virtual void SetFontList(const gfx::FontList& font_list) OVERRIDE;
......
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