Oilpan: add [WillBeGarbageCollected] for Node.

Source/core/frame/DOMWindow.cpp

Index: Source/core/frame/DOMWindow.cpp
diff --git a/Source/core/frame/DOMWindow.cpp b/Source/core/frame/DOMWindow.cpp
index 3b2c2b9012b322e5ed2606ea54dbd332fe7d9d8f..1ec3bbfaad6d2338ffce05c2574fe50dbc8f788e 100644
--- a/Source/core/frame/DOMWindow.cpp
+++ b/Source/core/frame/DOMWindow.cpp
@@ -489,15 +489,20 @@ DOMWindow::~DOMWindow()
     ASSERT(m_hasBeenReset);
     reset();
 
+#if ENABLE(OILPAN)
+    // Stub out early to prevent Document access in removeAllEventListeners().
+    m_document = nullptr;
+#endif
+
     removeAllEventListeners();

[haraken, 2014/05/23 15:02:45]

Instead of moving only didClearTouchEventHandlers() into the weak processing of
Document::m_domWindow, how about just moving the entire
removeAllEventListeners() into the weak processing?

Then you can completely comment out line 497 in oilpan builds and remove line
492 - 496.

[sof, 2014/05/23 15:30:31]

Hmm, don't understand what you are suggesting :) The DOMWindow specific parts of
removeAllEventListeners() needs to remain under DOMWindow's control, not
Document and its weak processing. But perhaps you mean something else?

 
 #if ENABLE(OILPAN)
-    ASSERT(m_document->isDisposed());
+    // FIXME: This should be part of ActiveDOM Object shutdown

[haraken, 2014/05/23 15:02:45]

Nit: ActiveDOMObject

+    clearEventQueue();
 #else
     ASSERT(m_document->isStopped());
-#endif
-
     clearDocument();
+#endif
 }
 
 const AtomicString& DOMWindow::interfaceName() const
@@ -1875,6 +1880,7 @@ PassOwnPtr<LifecycleNotifier<DOMWindow> > DOMWindow::createLifecycleNotifier()
 
 void DOMWindow::trace(Visitor* visitor)
 {
+    visitor->trace(m_document);
     visitor->trace(m_screen);
     visitor->trace(m_history);
     visitor->trace(m_locationbar);