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

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

Issue 1433103002: Use FocusParams in FocusController::setFocusedElement and Document::setFocusedElement arguments. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 1 month 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 ff7212a3bdf054348ee7cc6bb8a600ca1b482ea5..2351061beffa6fe31b4f88515be7642791eec300 100644
--- a/third_party/WebKit/Source/core/dom/Element.cpp
+++ b/third_party/WebKit/Source/core/dom/Element.cpp
@@ -2376,7 +2376,7 @@ void Element::focus(const FocusParams& params)
}
RefPtrWillBeRawPtr<Node> protect(this);
- if (!document().page()->focusController().setFocusedElement(this, document().frame(), params.type, params.sourceCapabilities))
+ if (!document().page()->focusController().setFocusedElement(this, document().frame(), params))
return;
// Setting the focused node above might have invalidated the layout due to scripts.
@@ -2427,7 +2427,7 @@ void Element::blur()
if (doc.page())
doc.page()->focusController().setFocusedElement(0, doc.frame());
else
- doc.setFocusedElement(nullptr);
+ doc.clearFocusedElement();
}
}
« no previous file with comments | « third_party/WebKit/Source/core/dom/Element.h ('k') | third_party/WebKit/Source/core/editing/FrameSelection.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698