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

Unified Diff: third_party/WebKit/Source/core/dom/Element.cpp

Issue 2839993002: [Android] Adding Smart GO/NEXT feature in Chrome (Closed)
Patch Set: Fixed ImeTest#testSelectActionBarClearedOnTappingOutsideInput flaky failure 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/dom/Element.cpp
diff --git a/third_party/WebKit/Source/core/dom/Element.cpp b/third_party/WebKit/Source/core/dom/Element.cpp
index 66314b51feffba0ded95ee0fa42fe022d7a49ec8..a8f184ebf5070ef58554b678387778f1715de618 100644
--- a/third_party/WebKit/Source/core/dom/Element.cpp
+++ b/third_party/WebKit/Source/core/dom/Element.cpp
@@ -2891,10 +2891,6 @@ bool Element::SupportsSpatialNavigationFocus() const {
}
bool Element::IsFocusable() const {
- // Style cannot be cleared out for non-active documents, so in that case the
- // needsLayoutTreeUpdateForNode check is invalid.
- DCHECK(!GetDocument().IsActive() ||
- !GetDocument().NeedsLayoutTreeUpdateForNode(*this));
Changwan Ryu 2017/06/09 15:40:04 yosin@ / kochi@ may have more knowledge, but I don
AKVT 2017/06/12 12:21:24 Done.
return isConnected() && SupportsFocus() && !IsInert() &&
LayoutObjectIsFocusable();
}

Powered by Google App Engine
This is Rietveld 408576698