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

Unified Diff: Source/core/dom/Document.cpp

Issue 142653003: Retry "Replace RenderFullScreen with top layer" (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: new revised patch Created 6 years, 11 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: Source/core/dom/Document.cpp
diff --git a/Source/core/dom/Document.cpp b/Source/core/dom/Document.cpp
index 19d77afcb8e6d2c02e924007454be93d07f6d5fe..bad04404d15cd6edde4902d5e65b57500d5d254b 100644
--- a/Source/core/dom/Document.cpp
+++ b/Source/core/dom/Document.cpp
@@ -4846,9 +4846,15 @@ void Document::removeFromTopLayer(Element* element)
HTMLDialogElement* Document::activeModalDialog() const
{
- if (m_topLayerElements.isEmpty())
- return 0;
- return toHTMLDialogElement(m_topLayerElements.last().get());
+ Vector<RefPtr<Element> >::const_reverse_iterator end = m_topLayerElements.rend();
+ for (Vector<RefPtr<Element> >::const_reverse_iterator it = m_topLayerElements.rbegin(); it != end; ++it) {
+ if (!it->get()->hasTagName(dialogTag))
+ continue;
+ HTMLDialogElement* dialog = toHTMLDialogElement(it->get());
+ if (dialog->isModal())
+ return dialog;
+ }
+ return 0;
}
void Document::webkitExitPointerLock()

Powered by Google App Engine
This is Rietveld 408576698