Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(558)

Unified Diff: Source/core/html/HTMLFormControlElement.h

Issue 16194013: Mouse press should focus on any types of form controls. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rebase Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/html/HTMLFormControlElement.h
diff --git a/Source/core/html/HTMLFormControlElement.h b/Source/core/html/HTMLFormControlElement.h
index afdafdeb1f6d53062ab552dceb3f515488f203d0..6d0a64465e20ef353144367d3829343e9ae86b9e 100644
--- a/Source/core/html/HTMLFormControlElement.h
+++ b/Source/core/html/HTMLFormControlElement.h
@@ -121,7 +121,10 @@ protected:
virtual bool supportsFocus() const OVERRIDE;
virtual bool rendererIsFocusable() const OVERRIDE;
virtual bool isKeyboardFocusable(KeyboardEvent*) const;
- virtual bool isMouseFocusable() const;
+ virtual bool shouldShowFocusRingOnMouseFocus() const;
+ virtual bool shouldHaveFocusAppearance() const OVERRIDE;
+ virtual void dispatchFocusEvent(PassRefPtr<Node> oldFocusedNode, FocusDirection) OVERRIDE;
+ virtual void willCallDefaultEventHandler(const Event&) OVERRIDE;
virtual void didRecalcStyle(StyleChange) OVERRIDE;
@@ -167,7 +170,7 @@ private:
bool m_isValid : 1;
bool m_wasChangedSinceLastFormControlChangeEvent : 1;
-
+ bool m_wasFocusedByMouse : 1;
bool m_hasAutofocused : 1;
};
« no previous file with comments | « Source/core/html/BaseMultipleFieldsDateAndTimeInputType.cpp ('k') | Source/core/html/HTMLFormControlElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698