Oilpan: tidy up NavigationScheduler.

Source/core/loader/NavigationScheduler.h

 /*
  * Copyright (C) 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2009 Adam Barth. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
@@ skipping 14 matching lines @@
  * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
  * ON ANY 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 NavigationScheduler_h
 #define NavigationScheduler_h
 
 #include "core/CoreExport.h"
-#include "platform/Timer.h"
 #include "platform/heap/Handle.h"
-#include "platform/scheduler/CancellableTaskFactory.h"
 #include "wtf/Forward.h"
 #include "wtf/HashMap.h"
 #include "wtf/Noncopyable.h"
 #include "wtf/OwnPtr.h"
 #include "wtf/PassOwnPtr.h"
 #include "wtf/PassRefPtr.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
 
+class CancellableTaskFactory;
 class Document;
 class FormSubmission;
 class LocalFrame;
-class NavigationScheduler;
 class ScheduledNavigation;
 
-class NavigationDisablerForBeforeUnload {
-    WTF_MAKE_NONCOPYABLE(NavigationDisablerForBeforeUnload);
-    STACK_ALLOCATED();
+class CORE_EXPORT NavigationScheduler final : public NoBaseWillBeGarbageCollectedFinalized<NavigationScheduler> {
+    WTF_MAKE_NONCOPYABLE(NavigationScheduler);
+    WTF_MAKE_FAST_ALLOCATED_WILL_BE_REMOVED(NavigationScheduler);
 public:
-    NavigationDisablerForBeforeUnload()
+    static PassOwnPtrWillBeRawPtr<NavigationScheduler> create(LocalFrame* frame)
     {
-        s_navigationDisableCount++;
+        return adoptPtrWillBeNoop(new NavigationScheduler(frame));
     }
-    ~NavigationDisablerForBeforeUnload()
-    {
-        ASSERT(s_navigationDisableCount);
-        s_navigationDisableCount--;
-    }
-    static bool isNavigationAllowed() { return !s_navigationDisableCount; }
 
-private:
-    static unsigned s_navigationDisableCount;
-};
-
-class FrameNavigationDisabler {
-    WTF_MAKE_NONCOPYABLE(FrameNavigationDisabler);
-    STACK_ALLOCATED();
-public:
-    explicit FrameNavigationDisabler(LocalFrame*);
-    ~FrameNavigationDisabler();
-
-private:
-    FrameNavigationDisabler() = delete;
-
-    NavigationScheduler& m_navigationScheduler;
-};
-
-class CORE_EXPORT NavigationScheduler final {
-    WTF_MAKE_NONCOPYABLE(NavigationScheduler);
-    DISALLOW_ALLOCATION();
-public:
-    explicit NavigationScheduler(LocalFrame*);
     ~NavigationScheduler();
 
     bool locationChangePending();
     bool isNavigationScheduled() const;
 
     void scheduleRedirect(double delay, const String& url);
     void scheduleLocationChange(Document*, const String& url, bool replacesCurrentItem = true);
     void schedulePageBlock(Document*);
     void scheduleFormSubmission(Document*, PassRefPtrWillBeRawPtr<FormSubmission>);
     void scheduleReload();
 
     void startTimer();
     void cancel();
 
     DECLARE_TRACE();
 
 private:
     friend class FrameNavigationDisabler;
+    explicit NavigationScheduler(LocalFrame*);
 
     void disableFrameNavigation() { ++m_navigationDisableCount; }
     void enableFrameNavigation() { --m_navigationDisableCount; }
     bool isFrameNavigationAllowed() const { return !m_navigationDisableCount; }
 
     bool shouldScheduleReload() const;
     bool shouldScheduleNavigation(const String& url) const;
 
     void navigateTask();
     void schedule(PassOwnPtrWillBeRawPtr<ScheduledNavigation>);
 
     static bool mustReplaceCurrentItem(LocalFrame* targetFrame);
 
     RawPtrWillBeMember<LocalFrame> m_frame;
-    CancellableTaskFactory m_navigateTaskFactory;
+    OwnPtr<CancellableTaskFactory> m_navigateTaskFactory;
     OwnPtrWillBeMember<ScheduledNavigation> m_redirect;
     int m_navigationDisableCount;
 };
 
+class NavigationDisablerForBeforeUnload {
+    WTF_MAKE_NONCOPYABLE(NavigationDisablerForBeforeUnload);
+    STACK_ALLOCATED();
+public:
+    NavigationDisablerForBeforeUnload()
+    {
+        s_navigationDisableCount++;
+    }
+    ~NavigationDisablerForBeforeUnload()
+    {
+        ASSERT(s_navigationDisableCount);
+        s_navigationDisableCount--;
+    }
+    static bool isNavigationAllowed() { return !s_navigationDisableCount; }
+
+private:
+    static unsigned s_navigationDisableCount;
+};
+
+class FrameNavigationDisabler {
+    WTF_MAKE_NONCOPYABLE(FrameNavigationDisabler);
+    STACK_ALLOCATED();
+public:
+    explicit FrameNavigationDisabler(LocalFrame*);
+    ~FrameNavigationDisabler();
+
+private:
+    RawPtrWillBeMember<NavigationScheduler> m_navigationScheduler;
+};
+
 } // namespace blink
 
 #endif // NavigationScheduler_h