Oilpan: move WorkerGlobalScope to oilpan's heap.

Source/modules/imagebitmap/ImageBitmapFactories.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 18 matching lines @@
  */
 
 #ifndef ImageBitmapFactories_h
 #define ImageBitmapFactories_h
 
 #include "bindings/v8/ScriptPromise.h"
 #include "bindings/v8/ScriptPromiseResolver.h"
 #include "bindings/v8/ScriptState.h"
 #include "core/fileapi/FileReaderLoader.h"
 #include "core/fileapi/FileReaderLoaderClient.h"
-#include "core/workers/WorkerSupplementable.h"
 #include "platform/Supplementable.h"
 #include "platform/geometry/IntRect.h"
 #include "wtf/Forward.h"
 #include "wtf/HashSet.h"
 
 namespace WebCore {
 
 class Blob;
 class CanvasRenderingContext2D;
 class EventTarget;
 class ExceptionState;
 class HTMLCanvasElement;
 class HTMLImageElement;
 class HTMLVideoElement;
 class ImageBitmap;
 class ImageData;
 class ExecutionContext;
 
-class ImageBitmapFactories FINAL : public Supplement<DOMWindow>, public WorkerSupplement {
+class ImageBitmapFactories : public Supplement<DOMWindow> {
 
 class ImageBitmapLoader;
 
 public:
     static ScriptPromise createImageBitmap(EventTarget&, HTMLImageElement*, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, HTMLImageElement*, int sx, int sy, int sw, int sh, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, HTMLVideoElement*, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, HTMLVideoElement*, int sx, int sy, int sw, int sh, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, CanvasRenderingContext2D*, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, CanvasRenderingContext2D*, int sx, int sy, int sw, int sh, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, HTMLCanvasElement*, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, HTMLCanvasElement*, int sx, int sy, int sw, int sh, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, Blob*, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, Blob*, int sx, int sy, int sw, int sh, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, ImageData*, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, ImageData*, int sx, int sy, int sw, int sh, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, ImageBitmap*, ExceptionState&);
     static ScriptPromise createImageBitmap(EventTarget&, ImageBitmap*, int sx, int sy, int sw, int sh, ExceptionState&);
 
     void didFinishLoading(ImageBitmapLoader*);
 
     virtual ~ImageBitmapFactories() { }
 
+protected:
+    static const char* supplementName();
+
 private:
     class ImageBitmapLoader FINAL : public RefCounted<ImageBitmapLoader>, public FileReaderLoaderClient {
     public:
         static PassRefPtr<ImageBitmapLoader> create(ImageBitmapFactories& factory, PassRefPtr<ScriptPromiseResolver> resolver, const IntRect& cropRect)
         {
             return adoptRef(new ImageBitmapLoader(factory, resolver, cropRect));
         }
 
         void loadBlobAsync(ExecutionContext*, Blob*);
 
@@ skipping 10 matching lines @@
         virtual void didFinishLoading() OVERRIDE;
         virtual void didFail(FileError::ErrorCode) OVERRIDE;
 
         ScriptState* m_scriptState;
         FileReaderLoader m_loader;
         ImageBitmapFactories* m_factory;
         RefPtr<ScriptPromiseResolver> m_resolver;
         IntRect m_cropRect;
     };
 
-    static const char* supplementName();
     static ImageBitmapFactories& from(EventTarget&);
 
-    template <class T>
-    static ImageBitmapFactories& fromInternal(T&);
+    static ImageBitmapFactories& fromInternal(DOMWindow&);
 
     void addLoader(PassRefPtr<ImageBitmapLoader>);
 
     HashSet<RefPtr<ImageBitmapLoader> > m_pendingLoaders;
 };
 
+// FIXME: oilpan: remove once DOMWindow and its Supplementable becomes heap allocated.
+class WorkerGlobalScopeImageBitmapFactories FINAL : public NoBaseWillBeGarbageCollectedFinalized<WorkerGlobalScopeImageBitmapFactories>, public ImageBitmapFactories, public WillBeHeapSupplement<WorkerGlobalScope> {
+    WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(WorkerGlobalScopeImageBitmapFactories);
+public:
+    virtual ~WorkerGlobalScopeImageBitmapFactories() { }
+
+    virtual void trace(Visitor*);

[haraken, 2014/03/03 12:55:50]

Add OVERRIDE.

[sof, 2014/03/04 08:18:00]

That wouldn't be quite right, this is a/the base GC class. Kept as is.

+
+private:
+    friend class ImageBitmapFactories;
+    static ImageBitmapFactories& fromInternal(WorkerGlobalScope&);
+};
+
 } // namespace WebCore
 
 #endif // ImageBitmapFactories_h