ResizeObserver implementation, part 2: lifetime

third_party/WebKit/Source/core/observer/ResizeObserver.cpp

 // 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.
 
 #include "core/observer/ResizeObserver.h"
 
 #include "core/dom/Element.h"
+#include "core/observer/ResizeObservation.h"
 #include "core/observer/ResizeObserverCallback.h"
-#include "core/observer/ResizeObserverEntry.h"
+#include "core/observer/ResizeObserverController.h"
 
 namespace blink {
 
 ResizeObserver* ResizeObserver::create(Document& document, ResizeObserverCallback* callback)
 {
     return new ResizeObserver(callback, document);
 }
 
 ResizeObserver::ResizeObserver(ResizeObserverCallback* callback, Document& document)
+    : m_callback(callback)
 {
+    document.ensureResizeObserverController();
+    m_controller = document.resizeObserverController();

[esprehn, 2016/07/21 17:52:28]

don't call it twice here, use the return value of ensureResizeObserverController

[atotic1, 2016/07/21 19:15:00]

Done.

+    m_controller->addObserver(*this);
 }
 
 void ResizeObserver::observe(Element* target)
 {
 }
 
 void ResizeObserver::unobserve(Element* target)
 {
 }
 
 void ResizeObserver::disconnect()
 {
 }
 
 DEFINE_TRACE(ResizeObserver)
 {
+    visitor->trace(m_callback);
+    visitor->trace(m_observations);
+    visitor->trace(m_controller);
 }
 
 } // namespace blink