Oilpan: make DataObject and SharedWorker heap Supplementables.

Source/core/workers/SharedWorker.h

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2010 Apple 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.
@@ skipping 22 matching lines @@
 #define SharedWorker_h
 
 #include "core/workers/AbstractWorker.h"
 #include "heap/Handle.h"
 #include "platform/Supplementable.h"
 
 namespace WebCore {
 
 class ExceptionState;
 
-class SharedWorker FINAL : public AbstractWorker, public ScriptWrappable, public Supplementable<SharedWorker> {
+class SharedWorker FINAL : public AbstractWorker, public ScriptWrappable, public WillBeHeapSupplementable<SharedWorker> {
+    WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(SharedWorker);
 public:
     static PassRefPtrWillBeRawPtr<SharedWorker> create(ExecutionContext*, const String& url, const String& name, ExceptionState&);
     virtual ~SharedWorker();
 
     MessagePort* port() const { return m_port.get(); }
 
     virtual const AtomicString& interfaceName() const OVERRIDE;
 
     // Prevents this SharedWorker + JS wrapper from being garbage collected.
     void setPreventGC();
     // Allows this SharedWorker + JS wrapper to be garbage collected.
     void unsetPreventGC();
 
     virtual void trace(Visitor*) OVERRIDE;

[haraken, 2014/03/21 10:35:13]

Don't you need to call WillBeHeapSupplementable<SharedWorker>::trace() ?

[sof, 2014/03/21 10:42:20]

Definitely; overlooked.

 
 private:
     explicit SharedWorker(ExecutionContext*);
 
     RefPtr<MessagePort> m_port;
 };
 
 } // namespace WebCore
 
 #endif // SharedWorker_h