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

Side by Side Diff: Source/core/html/InputType.cpp

Issue 16194013: Mouse press should focus on any types of form controls. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2001 Dirk Mueller (mueller@kde.org) 4 * (C) 2001 Dirk Mueller (mueller@kde.org)
5 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All r ights reserved. 5 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All r ights reserved.
6 * (C) 2006 Alexey Proskuryakov (ap@nypop.com) 6 * (C) 2006 Alexey Proskuryakov (ap@nypop.com)
7 * Copyright (C) 2007 Samuel Weinig (sam@webkit.org) 7 * Copyright (C) 2007 Samuel Weinig (sam@webkit.org)
8 * Copyright (C) 2009, 2010, 2011, 2012 Google Inc. All rights reserved. 8 * Copyright (C) 2009, 2010, 2011, 2012 Google Inc. All rights reserved.
9 * Copyright (C) 2012 Samsung Electronics. All rights reserved. 9 * Copyright (C) 2012 Samsung Electronics. All rights reserved.
10 * 10 *
(...skipping 508 matching lines...) Expand 10 before | Expand all | Expand 10 after
519 bool InputType::hasCustomFocusLogic() const 519 bool InputType::hasCustomFocusLogic() const
520 { 520 {
521 return true; 521 return true;
522 } 522 }
523 523
524 bool InputType::isKeyboardFocusable(KeyboardEvent* event) const 524 bool InputType::isKeyboardFocusable(KeyboardEvent* event) const
525 { 525 {
526 return element()->isTextFormControlKeyboardFocusable(event); 526 return element()->isTextFormControlKeyboardFocusable(event);
527 } 527 }
528 528
529 bool InputType::isMouseFocusable() const 529 bool InputType::shouldShowFocusRingOnMouseFocus() const
530 { 530 {
531 return element()->isTextFormControlMouseFocusable(); 531 return false;
532 } 532 }
533 533
534 bool InputType::shouldUseInputMethod() const 534 bool InputType::shouldUseInputMethod() const
535 { 535 {
536 return false; 536 return false;
537 } 537 }
538 538
539 void InputType::handleFocusEvent(Node*, FocusDirection) 539 void InputType::handleFocusEvent(Node*, FocusDirection)
540 { 540 {
541 } 541 }
(...skipping 559 matching lines...) Expand 10 before | Expand all | Expand 10 after
1101 1101
1102 void InputType::observeFeatureIfVisible(UseCounter::Feature feature) const 1102 void InputType::observeFeatureIfVisible(UseCounter::Feature feature) const
1103 { 1103 {
1104 if (RenderStyle* style = element()->renderStyle()) { 1104 if (RenderStyle* style = element()->renderStyle()) {
1105 if (style->visibility() != HIDDEN) 1105 if (style->visibility() != HIDDEN)
1106 UseCounter::count(element()->document(), feature); 1106 UseCounter::count(element()->document(), feature);
1107 } 1107 }
1108 } 1108 }
1109 1109
1110 } // namespace WebCore 1110 } // namespace WebCore
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698