Move pending state from FullscreenController to Fullscreen

third_party/WebKit/Source/web/FullscreenController.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 15 matching lines @@
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef FullscreenController_h
 #define FullscreenController_h
 
 #include "platform/geometry/FloatPoint.h"
 #include "platform/geometry/IntSize.h"
-#include "platform/heap/Handle.h"
-#include "wtf/RefPtr.h"
+#include <memory>
 
 namespace blink {
 
 class Element;
 class LocalFrame;
 class WebViewImpl;
 
-class FullscreenController final
-    : public GarbageCollected<FullscreenController> {
+class FullscreenController {
  public:
-  static FullscreenController* create(WebViewImpl*);
+  static std::unique_ptr<FullscreenController> create(WebViewImpl*);
 
   // Called by Fullscreen (via ChromeClient) to request entering or exiting
   // fullscreen.
-  void enterFullscreenForElement(Element*);
-  void exitFullscreen(LocalFrame*);
+  void enterFullscreen(LocalFrame&);
+  void exitFullscreen(LocalFrame&);
 
   // Called by content::RenderWidget (via WebWidget) to notify that we've
   // entered or exited fullscreen. This can be because we requested it, or it
   // can be initiated by the browser directly.
   void didEnterFullscreen();
   void didExitFullscreen();
 
   // Called by Fullscreen (via ChromeClient) to notify that the fullscreen
   // element has changed.
   void fullscreenElementChanged(Element*, Element*);
 
-  bool isFullscreen() { return m_fullscreenFrame; }
+  bool isFullscreen() { return m_state == State::Fullscreen; }
 
   void updateSize();
 
   void didUpdateLayout();
 
-  DECLARE_TRACE();
-
  protected:
   explicit FullscreenController(WebViewImpl*);
 
  private:
   void updatePageScaleConstraints(bool removeConstraints);
 
   WebViewImpl* m_webViewImpl;
 
-  bool m_haveEnteredFullscreen;
-  float m_exitFullscreenPageScaleFactor;
-  IntSize m_exitFullscreenScrollOffset;
-  FloatPoint m_exitFullscreenVisualViewportOffset;
-  bool m_needsScrollAndScaleRestore;
+  enum class State {
+    Initial,

[mlamouri (slow - plz ping), 2016/12/05 13:54:54]

Is `Initial` used after exiting fullscreen scroll and scale has been restored?

[foolip, 2016/12/06 11:31:39]

I have added a documentation block, can you take a look?

+    EnteringFullscreen,
+    Fullscreen,
+    ExitingFullscreen,
+    NeedsScrollAndScaleRestore

[mlamouri (slow - plz ping), 2016/12/05 13:54:54]

Add a comment explaining what that means?

+  };
+  State m_state = State::Initial;
 
-  // If set, the WebView is transitioning to fullscreen for this element.
-  Member<Element> m_provisionalFullscreenElement;
-
-  // If set, the WebView is in fullscreen mode for an element in this frame.
-  Member<LocalFrame> m_fullscreenFrame;
-
-  bool m_isCancelingFullscreen;
+  float m_initialPageScaleFactor = 0.0f;
+  IntSize m_initialScrollOffset;
+  FloatPoint m_initialVisualViewportOffset;
 };
 
 }  // namespace blink
 
 #endif