Revert "Simplify ElementVisibilityObserver implementation."

third_party/WebKit/Source/core/dom/ElementVisibilityObserver.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef ElementVisibilityObserver_h
 #define ElementVisibilityObserver_h
 
-#include "core/CoreExport.h"
-#include "core/dom/IntersectionObserverCallback.h"
-#include "platform/heap/Handle.h"
+#include "core/dom/IntersectionObserver.h"
+#include "platform/heap/Heap.h"
+#include "platform/heap/Member.h"
 
 namespace blink {
 
 class Element;
-class IntersectionObserver;
-class IntersectionObserverEntry;
 
 // ElementVisibilityObserver is a helper class to be used to track the
-// visibility of an Element in the viewport; it is implemented on top of
-// IntersectionObserver.
-//
-// When creating an ElementVisibilityObserver instance, alongside the element
-// reference, the caller will have to supply an object reference implementing
-// the |Client| interface and its |onVisibilityChanged| method. The callback
-// method will be invoked when the element changes visibility state,
-// the boolean argument indicating which.
-class ElementVisibilityObserver final : public IntersectionObserverCallback {
+// visibility of an Element in the viewport. Creating an
+// ElementVisibilityObserver is a no-op with regards to CPU cycle. The observing
+// has be started by calling |start()| and can be stopped with |stop()|.
+// When creating an instance, the caller will have to pass a callback taking
+// a boolean as an argument. The boolean will be the new visibility state.
+// The ElementVisibilityObserver is implemented on top of IntersectionObserver.
+// It is a layer meant to simplify the usage for C++ Blink code checking for the
+// visibility of an element.
+class ElementVisibilityObserver final : public GarbageCollectedFinalized<ElementVisibilityObserver> {
     WTF_MAKE_NONCOPYABLE(ElementVisibilityObserver);
 public:
-    class CORE_EXPORT Client : public GarbageCollectedMixin {
-    public:
-        virtual void onVisibilityChanged(bool isVisible) = 0;
-        virtual ExecutionContext* getElementVisibilityExecutionContext() const = 0;
-    };
+    using VisibilityCallback = Function<void(bool), WTF::SameThreadAffinity>;
 
-    static ElementVisibilityObserver* create(Element*, Client*);
-    ~ElementVisibilityObserver();
+    ElementVisibilityObserver(Element*, std::unique_ptr<VisibilityCallback>);
+    virtual ~ElementVisibilityObserver();
+
+    void start();
+    void stop();
+
     DECLARE_VIRTUAL_TRACE();
 
-    void stop();
+private:
+    class ElementVisibilityCallback;
 
-    // IntersectionObserverCallback implementation:
-    void handleEvent(const HeapVector<Member<IntersectionObserverEntry>>&, IntersectionObserver&) override;
-    ExecutionContext* getExecutionContext() const override;
+    void onVisibilityChanged(const HeapVector<Member<IntersectionObserverEntry>>&);
 
-private:
-    explicit ElementVisibilityObserver(Client*);
-
-    void start(Element*);
-
-    Member<Client> m_client;
+    Member<Element> m_element;
     Member<IntersectionObserver> m_intersectionObserver;
+    std::unique_ptr<VisibilityCallback> m_callback;
 };
 
 } // namespace blink
 
 #endif // ElementVisibilityObserver_h