Implement FullScreen using top layer.

third_party/WebKit/Source/core/dom/Fullscreen.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
@@ skipping 10 matching lines @@
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  *
  */
 
 #include "config.h"
 #include "core/dom/Fullscreen.h"
 
+#include "bindings/core/v8/ExceptionMessages.h"
 #include "core/HTMLNames.h"
 #include "core/dom/Document.h"
 #include "core/dom/ElementTraversal.h"
 #include "core/events/Event.h"
 #include "core/frame/FrameHost.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/OriginsUsingFeatures.h"
 #include "core/frame/Settings.h"
 #include "core/frame/UseCounter.h"
 #include "core/html/HTMLIFrameElement.h"
 #include "core/html/HTMLMediaElement.h"
 #include "core/input/EventHandler.h"
 #include "core/inspector/ConsoleMessage.h"
-#include "core/layout/LayoutFullScreen.h"
 #include "core/page/ChromeClient.h"
+#include "core/page/Page.h"
+#include "core/style/ComputedStyle.h"
 #include "platform/UserGestureIndicator.h"
 
 namespace blink {
 
 using namespace HTMLNames;
 
 static bool fullscreenIsAllowedForAllOwners(const Document& document)
 {
     for (const Element* owner = document.ownerElement(); owner; owner = owner->document().ownerElement()) {
         if (!isHTMLIFrameElement(owner))
@@ skipping 96 matching lines @@
     return 0;
 }
 
 bool Fullscreen::isFullScreen(Document& document)
 {
     return currentFullScreenElementFrom(document);
 }
 
 Fullscreen::Fullscreen(Document& document)
     : DocumentLifecycleObserver(&document)
-    , m_fullScreenLayoutObject(nullptr)
     , m_eventQueueTimer(this, &Fullscreen::eventQueueTimerFired)
 {
     document.setHasFullscreenSupplement();
 }
 
 Fullscreen::~Fullscreen()
 {
 }
 
 inline Document* Fullscreen::document()
 {
     return lifecycleContext();
 }
 
 void Fullscreen::documentWasDetached()
 {
     m_eventQueue.clear();
 
-    if (m_fullScreenLayoutObject)
-        m_fullScreenLayoutObject->destroy();
-
 #if ENABLE(OILPAN)
     m_fullScreenElement = nullptr;
     m_fullScreenElementStack.clear();
 #endif
 
 }
 
 #if !ENABLE(OILPAN)
 void Fullscreen::documentWasDisposed()
 {
@@ skipping 224 matching lines @@
     // 4. The fullscreenEnabled attribute must return true if the context object has its
     //    fullscreen enabled flag set and fullscreen is supported, and false otherwise.
 
     // Top-level browsing contexts are implied to have their allowFullScreen attribute set.
     return fullscreenIsAllowedForAllOwners(document) && fullscreenIsSupported(document);
 }
 
 void Fullscreen::didEnterFullScreenForElement(Element* element)
 {
     ASSERT(element);
+    ASSERT(element->isInTopLayer());

[dsinclair, 2015/09/28 18:19:36]

[ RUN      ] WebViewTest.EnterFullscreenResetScrollAndScaleState
ASSERTION FAILED: element->isInTopLayer()
../../third_party/WebKit/Source/core/dom/Fullscreen.cpp(426) : void
blink::Fullscreen::didEnterFullScreenForElement(blink::Element *)
1   0xc33e1a
2   0x73d39c
3   0x2a7c987
4   0x2a7d6ba
5   0x2a7db53
6   0x2a84ef9
7   0x2a84b9e
8   0xcdc2bd
9   0x71f1ac
10  0xce8c03
11  0xce8ada
12  0x71f154
13  0x7ff7e9a4376d __libc_start_main
14  0x4d87c1
Received signal 11 SEGV_MAPERR 0000fbadbeef
#0 0x000000faa4bb base::debug::(anonymous namespace)::StackDumpSignalHandler()
#1 0x7ff7ea559cb0 <unknown>
#2 0x000001bd587a blink::Fullscreen::didEnterFullScreenForElement()
#3 0x000000c33e1a blink::FullscreenController::didEnterFullScreen()
#4 0x00000073d39c
blink::WebViewTest_EnterFullscreenResetScrollAndScaleState_Test::TestBody()


Managed to get this to crash, only happened in one unit test, all the layout
tests seem file. Will see if I can figure out how it happens.

http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...

+
     if (!document()->isActive())
         return;
 
-    if (m_fullScreenLayoutObject)
-        m_fullScreenLayoutObject->unwrapLayoutObject();
-
     m_fullScreenElement = element;
-
-    // Create a placeholder block for a the full-screen element, to keep the page from reflowing
-    // when the element is removed from the normal flow. Only do this for a LayoutBox, as only
-    // a box will have a frameRect. The placeholder will be created in setFullScreenLayoutObject()
-    // during layout.
-    LayoutObject* layoutObject = m_fullScreenElement->layoutObject();
-    bool shouldCreatePlaceholder = layoutObject && layoutObject->isBox();
-    if (shouldCreatePlaceholder) {
-        m_savedPlaceholderFrameRect = toLayoutBox(layoutObject)->frameRect();
-        m_savedPlaceholderComputedStyle = ComputedStyle::clone(layoutObject->styleRef());
-    }
-
-    if (m_fullScreenElement != document()->documentElement())
-        LayoutFullScreen::wrapLayoutObject(layoutObject, layoutObject ? layoutObject->parent() : 0, document());
-
     m_fullScreenElement->setContainsFullScreenElementOnAncestorsCrossingFrameBoundaries(true);
 
     // FIXME: This should not call updateStyleIfNeeded.
     document()->setNeedsStyleRecalc(SubtreeStyleChange, StyleChangeReasonForTracing::create(StyleChangeReason::FullScreen));
     document()->updateLayoutTreeIfNeeded();
 
     m_fullScreenElement->didBecomeFullscreenElement();
 
     if (document()->frame())
         document()->frame()->eventHandler().scheduleHoverStateUpdate();
 
     m_eventQueueTimer.startOneShot(0, FROM_HERE);
 }
 
 void Fullscreen::didExitFullScreenForElement(Element*)
 {
     if (!m_fullScreenElement)
         return;
 
     if (!document()->isActive())
         return;
 
+    document()->removeFromTopLayer(m_fullScreenElement.get());
     m_fullScreenElement->willStopBeingFullscreenElement();
 
     m_fullScreenElement->setContainsFullScreenElementOnAncestorsCrossingFrameBoundaries(false);
 
-    if (m_fullScreenLayoutObject)
-        m_fullScreenLayoutObject->unwrapLayoutObject();
-
     m_fullScreenElement = nullptr;
     document()->setNeedsStyleRecalc(SubtreeStyleChange, StyleChangeReasonForTracing::create(StyleChangeReason::FullScreen));
 
     if (document()->frame())
         document()->frame()->eventHandler().scheduleHoverStateUpdate();
 
     // When fullyExitFullscreen is called, we call exitFullscreen on the topDocument(). That means
     // that the events will be queued there. So if we have no events here, start the timer on the
     // exiting document.
     Document* exitingDocument = document();
     if (m_eventQueue.isEmpty())
         exitingDocument = &document()->topDocument();
     ASSERT(exitingDocument);
     from(*exitingDocument).m_eventQueueTimer.startOneShot(0, FROM_HERE);
 }
 
-void Fullscreen::setFullScreenLayoutObject(LayoutFullScreen* layoutObject)
+void Fullscreen::updatedSize(Element* element)
 {
-    if (layoutObject == m_fullScreenLayoutObject)
-        return;
-
-    if (layoutObject && m_savedPlaceholderComputedStyle) {
-        layoutObject->createPlaceholder(m_savedPlaceholderComputedStyle.release(), m_savedPlaceholderFrameRect);
-    } else if (layoutObject && m_fullScreenLayoutObject && m_fullScreenLayoutObject->placeholder()) {
-        LayoutBlock* placeholder = m_fullScreenLayoutObject->placeholder();
-        layoutObject->createPlaceholder(ComputedStyle::clone(placeholder->styleRef()), placeholder->frameRect());
-    }
-
-    if (m_fullScreenLayoutObject)
-        m_fullScreenLayoutObject->unwrapLayoutObject();
-    ASSERT(!m_fullScreenLayoutObject);
-
-    m_fullScreenLayoutObject = layoutObject;
-}
-
-void Fullscreen::fullScreenLayoutObjectDestroyed()
-{
-    m_fullScreenLayoutObject = nullptr;
+    element->setNeedsStyleRecalc(LocalStyleChange, StyleChangeReasonForTracing::create(StyleChangeReason::FullScreen));
 }
 
 void Fullscreen::enqueueChangeEvent(Document& document, RequestType requestType)
 {
     RefPtrWillBeRawPtr<Event> event;
     if (requestType == UnprefixedRequest) {
         event = createEvent(EventTypeNames::fullscreenchange, document);
     } else {
         ASSERT(document.hasFullscreenSupplement());
         Fullscreen& fullscreen = from(document);
@@ skipping 68 matching lines @@
 void Fullscreen::clearFullscreenElementStack()
 {
     m_fullScreenElementStack.clear();
 }
 
 void Fullscreen::popFullscreenElementStack()
 {
     if (m_fullScreenElementStack.isEmpty())
         return;
 
+    document()->removeFromTopLayer(m_fullScreenElementStack.last().first.get());
     m_fullScreenElementStack.removeLast();
 }
 
 void Fullscreen::pushFullscreenElementStack(Element& element, RequestType requestType)
 {
     m_fullScreenElementStack.append(std::make_pair(&element, requestType));
+    document()->addToTopLayer(&element);
 }
 
 DEFINE_TRACE(Fullscreen)
 {
 #if ENABLE(OILPAN)
     visitor->trace(m_fullScreenElement);
     visitor->trace(m_fullScreenElementStack);
     visitor->trace(m_eventQueue);
 #endif
     WillBeHeapSupplement<Document>::trace(visitor);
     DocumentLifecycleObserver::trace(visitor);
 }
 
 } // namespace blink