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

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

Issue 2086293004: Document/ShadowRoot split of pointerLockElement (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update comments Created 4 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
« no previous file with comments | « third_party/WebKit/Source/core/dom/TreeScope.h ('k') | third_party/WebKit/Source/core/frame/UseCounter.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/dom/TreeScope.cpp
diff --git a/third_party/WebKit/Source/core/dom/TreeScope.cpp b/third_party/WebKit/Source/core/dom/TreeScope.cpp
index fdea83b380fc9ff57a3dc8783132d9474952136d..ff7396764d943d7314394e226cec2ecc21e08a39 100644
--- a/third_party/WebKit/Source/core/dom/TreeScope.cpp
+++ b/third_party/WebKit/Source/core/dom/TreeScope.cpp
@@ -388,6 +388,23 @@ Element* TreeScope::adjustedFocusedElement() const
return nullptr;
}
+Element* TreeScope::adjustedPointerLockElement(const Element& target) const
+{
+ const Element* adjustedTarget = ⌖
+ // Unless the target is in the same TreeScope as |scope|, traverse up shadow trees to
+ // find a shadow host that is in the same TreeScope as |scope|.
+ for (const Element* ancestor = ⌖ ancestor; ancestor = ancestor->shadowHost()) {
+ // Exception is that if the host has V0 or UA shadow, skip the adjustment because
+ // .pointerLockElement is not available for non-V1 shadows.
+ // TODO(kochi): Once V0 code is removed, use the same logic as .activeElement for
+ // Shadow DOM V1.
+ if (ancestor->shadowRootIfV1())
+ adjustedTarget = ancestor;
+ if (this == ancestor->treeScope())
+ return const_cast<Element*>(adjustedTarget);
+ }
+ return nullptr;
+}
unsigned short TreeScope::comparePosition(const TreeScope& otherScope) const
{
if (otherScope == this)
« no previous file with comments | « third_party/WebKit/Source/core/dom/TreeScope.h ('k') | third_party/WebKit/Source/core/frame/UseCounter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698