Oilpan: Move all supplements of Page, Document, and WorkerClients to the managed heap.

Source/core/html/imports/HTMLImportsController.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 31 matching lines @@
 
 namespace WebCore {
 
 class FetchRequest;
 class ExecutionContext;
 class ResourceFetcher;
 class HTMLImportChild;
 class HTMLImportChildClient;
 class HTMLImportLoader;
 
-class HTMLImportsController FINAL : public HTMLImport, public DocumentSupplement {
+class HTMLImportsController FINAL : public NoBaseWillBeGarbageCollectedFinalized<HTMLImportsController>, public HTMLImport, public DocumentSupplement {
+    WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(HTMLImportsController);
     WTF_MAKE_FAST_ALLOCATED;

[haraken, 2014/04/30 02:41:22]

Shouldn't this be WTF_MAKE_FAST_ALLOCATED_WILL_BE_REMOVED? I'm curious why this
CL compiles without adding _WILL_BE_REMOVED.

[zerny-chromium, 2014/04/30 08:29:34]

Thanks! Nothing at compile-time disallows defining new/delete for a GC'ed class.
I'll look into checking against redeclaration in the plugin. (I actually thought
that was added already, but that was on the branch plugin).

 public:
     static void provideTo(Document&);
 
     explicit HTMLImportsController(Document&);
     virtual ~HTMLImportsController();
 
     bool isMaster(const Document& document) const { return m_master == &document; }
     bool shouldBlockScriptExecution(const Document&) const;
     void wasDetachedFrom(const Document&);
 
@@ skipping 37 matching lines @@
 
     typedef Vector<RefPtr<HTMLImportLoader> > LoaderList;
     LoaderList m_loaders;
 };
 
 DEFINE_TYPE_CASTS(HTMLImportsController, HTMLImport, import, import->isRoot(), import.isRoot());
 
 } // namespace WebCore
 
 #endif // HTMLImportsController_h