Oilpan: Prepare to move IdTargetObserver and IdTargetObserverRegistry to Oilpan heap.

Source/core/dom/IdTargetObserver.h

Index: Source/core/dom/IdTargetObserver.h
diff --git a/Source/core/dom/IdTargetObserver.h b/Source/core/dom/IdTargetObserver.h
index 1fc4cd6f57520d79965e372712e91ede0dfd34f8..25c70217397676b5478e31dc0bff7084aad38848 100644
--- a/Source/core/dom/IdTargetObserver.h
+++ b/Source/core/dom/IdTargetObserver.h
@@ -26,15 +26,17 @@
 #ifndef IdTargetObserver_h
 #define IdTargetObserver_h
 
+#include "platform/heap/Handle.h"
 #include "wtf/text/AtomicString.h"
 
 namespace WebCore {
 
 class IdTargetObserverRegistry;
 
-class IdTargetObserver {
+class IdTargetObserver : public NoBaseWillBeGarbageCollectedFinalized<IdTargetObserver> {
 public:
     virtual ~IdTargetObserver();
+    virtual void trace(Visitor*);
     virtual void idTargetChanged() = 0;
     virtual void unregister();
 
@@ -42,7 +44,7 @@ protected:
     IdTargetObserver(IdTargetObserverRegistry&, const AtomicString& id);
 
 private:
-    IdTargetObserverRegistry& m_registry;
+    RawPtrWillBeMember<IdTargetObserverRegistry> m_registry;

[wibling-chromium, 2014/05/09 08:23:47]

NIT: Have you considered adding a IdTargetObserverRegistry& registry() accessor
to stay as close as possible to the current code. Mads described this pattern in
a recent email sent to blink-dev titled, C++ reference and Oilpan transition
types.

[tkent, 2014/05/09 09:37:20]

Added |IdTargetObserverRegistry& registry()| in the new patch.

     AtomicString m_id;
 };