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

Unified Diff: third_party/WebKit/Source/core/page/FocusController.cpp

Issue 2938123002: Revert of [Android] Adding Smart GO/NEXT feature in Chrome (Closed)
Patch Set: Created 3 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: third_party/WebKit/Source/core/page/FocusController.cpp
diff --git a/third_party/WebKit/Source/core/page/FocusController.cpp b/third_party/WebKit/Source/core/page/FocusController.cpp
index 621ff3e9dc89ad45ebe469fa25c8fb7cd8a42a3d..2aeabe8ca422443303126e1ebd70b72187553b8c 100644
--- a/third_party/WebKit/Source/core/page/FocusController.cpp
+++ b/third_party/WebKit/Source/core/page/FocusController.cpp
@@ -47,7 +47,6 @@
#include "core/frame/RemoteFrame.h"
#include "core/frame/Settings.h"
#include "core/html/HTMLAreaElement.h"
-#include "core/html/HTMLFormElement.h"
#include "core/html/HTMLImageElement.h"
#include "core/html/HTMLPlugInElement.h"
#include "core/html/HTMLShadowElement.h"
@@ -55,7 +54,6 @@
#include "core/html/TextControlElement.h"
#include "core/input/EventHandler.h"
#include "core/layout/HitTestResult.h"
-#include "core/layout/LayoutObject.h"
#include "core/page/ChromeClient.h"
#include "core/page/FocusChangedObserver.h"
#include "core/page/FrameTree.h"
@@ -1066,49 +1064,6 @@
DCHECK(type == kWebFocusTypeForward || type == kWebFocusTypeBackward);
ScopedFocusNavigation scope = ScopedFocusNavigation::CreateFor(element);
return FindFocusableElementAcrossFocusScopes(type, scope);
-}
-
-Element* FocusController::NextFocusableElementInForm(Element* element,
- WebFocusType focus_type) {
- element->GetDocument().UpdateStyleAndLayoutIgnorePendingStylesheets();
- if (!element->IsHTMLElement())
- return nullptr;
-
- if (!element->IsFormControlElement() &&
- !ToHTMLElement(element)->isContentEditableForBinding())
- return nullptr;
-
- HTMLFormElement* form_owner = nullptr;
- if (ToHTMLElement(element)->isContentEditableForBinding())
- form_owner = Traversal<HTMLFormElement>::FirstAncestor(*element);
- else
- form_owner = ToHTMLFormControlElement(element)->formOwner();
-
- if (!form_owner)
- return nullptr;
-
- Element* next_element = element;
- for (next_element = FindFocusableElement(focus_type, *next_element);
- next_element;
- next_element = FindFocusableElement(focus_type, *next_element)) {
- if (ToHTMLElement(next_element)->isContentEditableForBinding() &&
- next_element->IsDescendantOf(form_owner))
- return next_element;
- if (!next_element->IsFormControlElement())
- continue;
- HTMLFormControlElement* form_element =
- ToHTMLFormControlElement(next_element);
- if (form_element->formOwner() != form_owner ||
- form_element->IsDisabledOrReadOnly())
- continue;
- LayoutObject* layout = next_element->GetLayoutObject();
- if (layout && layout->IsTextControl()) {
- // TODO(ajith.v) Extend it for select elements, radio buttons and check
- // boxes
- return next_element;
- }
- }
- return nullptr;
}
Element* FocusController::FindFocusableElementInShadowHost(
« no previous file with comments | « third_party/WebKit/Source/core/page/FocusController.h ('k') | third_party/WebKit/Source/web/WebLocalFrameImpl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698