Oilpan: Move [WillBeGarbageCollected] from Node to EventTarget.

Source/modules/serviceworkers/ServiceWorkerContainer.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 35 matching lines @@
 class WebServiceWorker;
 }
 
 namespace WebCore {
 
 class Dictionary;
 class ExecutionContext;
 class ServiceWorker;
 
 class ServiceWorkerContainer FINAL :
-    public RefCounted<ServiceWorkerContainer>,
+    public RefCountedWillBeRefCountedGarbageCollected<ServiceWorkerContainer>,

[haraken, 2014/06/02 07:25:06]

I'm curious why ServiceWorkerContainer needs to become
RefCountedGarbageCollected. Does it derive EventTarget?

[zerny-chromium, 2014/06/02 07:47:33]

Good point. The IDL implied change is that controller must return a RawPtr in
the oilpan build. I'll do just that and leave the class itself (including the
RefPtr<EventTarget>) as is. That will change in a follow-up CL, but I guess we
could use a Persistent<EventTarget> in that case.

[haraken, 2014/06/02 07:54:32]

Probably we want to just use a Persistent for m_controller (instead of making
ServiceWorkerContainer RefCountedWillBeRefCountedGarbageCollected), since
serviceworker/ is actively being developed and we don't want to mess it up at
the moment.

     public ScriptWrappable,
     public ContextLifecycleObserver,
     public DOMWindowLifecycleObserver,
     public blink::WebServiceWorkerProviderClient {
 public:
-    static PassRefPtr<ServiceWorkerContainer> create(ExecutionContext*);
+    static PassRefPtrWillBeRawPtr<ServiceWorkerContainer> create(ExecutionContext*);
     ~ServiceWorkerContainer();
 
     void detachClient();
 
     ScriptPromise registerServiceWorker(ExecutionContext*, const String& pattern, const Dictionary&);
     ScriptPromise unregisterServiceWorker(ExecutionContext*, const String& scope = String());
 
-    PassRefPtr<ServiceWorker> controller() { return m_controller; }
+    PassRefPtrWillBeRawPtr<ServiceWorker> controller() { return m_controller; }
 
     // WebServiceWorkerProviderClient overrides.
     virtual void setController(blink::WebServiceWorker*) OVERRIDE;
     virtual void dispatchMessageEvent(const blink::WebString& message, const blink::WebMessagePortChannelArray&) OVERRIDE;
 
     // FIXME: Delete this when the embedder switches to setController.
     virtual void setCurrentServiceWorker(blink::WebServiceWorker*) OVERRIDE;
 
+    void trace(Visitor* visitor) { visitor->trace(m_controller); }
+
 private:
     explicit ServiceWorkerContainer(ExecutionContext*);
 
     blink::WebServiceWorkerProvider* m_provider;
-    RefPtr<ServiceWorker> m_controller;
+    RefPtrWillBeMember<ServiceWorker> m_controller;
 };
 
 } // namespace WebCore
 
 #endif // ServiceWorkerContainer_h