Commit 392e5c79 authored by philipj@opera.com's avatar philipj@opera.com

Tweak the documentation of "fullscreen element ready check"

Use |element| for the variable (italic in spec) and add missing word:
https://www.w3.org/Bugs/Public/show_bug.cgi?id=26580

NOTRY=true

Review URL: https://codereview.chromium.org/473803002

git-svn-id: svn://svn.chromium.org/blink/trunk@180249 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent c3cf4e0c
...@@ -171,14 +171,14 @@ void FullscreenElementStack::documentWasDisposed() ...@@ -171,14 +171,14 @@ void FullscreenElementStack::documentWasDisposed()
bool FullscreenElementStack::elementReady(Element& element, RequestType requestType) bool FullscreenElementStack::elementReady(Element& element, RequestType requestType)
{ {
// A fullscreen element ready check for an element returns true if all of the following are // A fullscreen element ready check for an element |element| returns true if all of the
// true, and false otherwise: // following are true, and false otherwise:
// element is in a document. // |element| is in a document.
if (!element.inDocument()) if (!element.inDocument())
return false; return false;
// element's node document's fullscreen enabled flag is set. // |element|'s node document's fullscreen enabled flag is set.
if (!fullscreenIsAllowedForAllOwners(element.document())) { if (!fullscreenIsAllowedForAllOwners(element.document())) {
if (requestType == PrefixedVideoRequest) if (requestType == PrefixedVideoRequest)
UseCounter::count(element.document(), UseCounter::VideoFullscreenAllowedExemption); UseCounter::count(element.document(), UseCounter::VideoFullscreenAllowedExemption);
...@@ -186,14 +186,14 @@ bool FullscreenElementStack::elementReady(Element& element, RequestType requestT ...@@ -186,14 +186,14 @@ bool FullscreenElementStack::elementReady(Element& element, RequestType requestT
return false; return false;
} }
// element's node document's fullscreen element stack is either empty or its top element is an // |element|'s node document's fullscreen element stack is either empty or its top element is an
// ancestor of element. // ancestor of |element|.
if (Element* topElement = fullscreenElementFrom(element.document())) { if (Element* topElement = fullscreenElementFrom(element.document())) {
if (!element.isDescendantOf(topElement)) if (!element.isDescendantOf(topElement))
return false; return false;
} }
// element has no ancestor element whose local name is iframe and namespace is the HTML // |element| has no ancestor element whose local name is iframe and namespace is the HTML
// namespace. // namespace.
if (Traversal<HTMLIFrameElement>::firstAncestor(element)) if (Traversal<HTMLIFrameElement>::firstAncestor(element))
return false; return false;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment