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

Unified Diff: Source/core/html/forms/RadioInputType.cpp

Issue 24246011: Move form-related 78 files to core/html/forms/. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 3 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
« no previous file with comments | « Source/core/html/forms/RadioInputType.h ('k') | Source/core/html/forms/RangeInputType.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/html/forms/RadioInputType.cpp
diff --git a/Source/core/html/RadioInputType.cpp b/Source/core/html/forms/RadioInputType.cpp
similarity index 93%
rename from Source/core/html/RadioInputType.cpp
rename to Source/core/html/forms/RadioInputType.cpp
index 8b3dac2064909aa08cf59f2785fbde2718b4b0a3..d18e856d89ac48b9eab211f639c80e50c27bcdaf 100644
--- a/Source/core/html/RadioInputType.cpp
+++ b/Source/core/html/forms/RadioInputType.cpp
@@ -20,14 +20,14 @@
*/
#include "config.h"
-#include "core/html/RadioInputType.h"
+#include "core/html/forms/RadioInputType.h"
#include "HTMLNames.h"
#include "core/dom/KeyboardEvent.h"
#include "core/dom/MouseEvent.h"
#include "core/dom/NodeTraversal.h"
#include "core/html/HTMLInputElement.h"
-#include "core/html/InputTypeNames.h"
+#include "core/html/forms/InputTypeNames.h"
#include "core/page/SpatialNavigation.h"
#include "core/platform/LocalizedStrings.h"
#include "wtf/PassOwnPtr.h"
@@ -72,9 +72,10 @@ void RadioInputType::handleKeydownEvent(KeyboardEvent* event)
// Left and up mean "previous radio button".
// Right and down mean "next radio button".
- // Tested in WinIE, and even for RTL, left still means previous radio button (and so moves
- // to the right). Seems strange, but we'll match it.
- // However, when using Spatial Navigation, we need to be able to navigate without changing the selection.
+ // Tested in WinIE, and even for RTL, left still means previous radio button
+ // (and so moves to the right). Seems strange, but we'll match it. However,
+ // when using Spatial Navigation, we need to be able to navigate without
+ // changing the selection.
Document& document = element()->document();
if (isSpatialNavigationEnabled(document.frame()))
return;
@@ -124,7 +125,7 @@ bool RadioInputType::isKeyboardFocusable() const
if (isSpatialNavigationEnabled(element()->document().frame()))
return true;
- // Never allow keyboard tabbing to leave you in the same radio group. Always
+ // Never allow keyboard tabbing to leave you in the same radio group. Always
// skip any other elements in the group.
Element* currentFocusedElement = element()->document().focusedElement();
if (currentFocusedElement && currentFocusedElement->hasTagName(inputTag)) {
@@ -169,9 +170,9 @@ void RadioInputType::didDispatchClick(Event* event, const ClickHandlingState& st
// Make sure it is still a radio button and only do the restoration if it still belongs to our group.
HTMLInputElement* checkedRadioButton = state.checkedRadioButton.get();
if (checkedRadioButton
- && checkedRadioButton->isRadioButton()
- && checkedRadioButton->form() == element()->form()
- && checkedRadioButton->name() == element()->name()) {
+ && checkedRadioButton->isRadioButton()
+ && checkedRadioButton->form() == element()->form()
+ && checkedRadioButton->name() == element()->name()) {
checkedRadioButton->setChecked(true);
}
}
« no previous file with comments | « Source/core/html/forms/RadioInputType.h ('k') | Source/core/html/forms/RangeInputType.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698