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

Source/core/loader/PrerendererClient.h

Index: Source/core/loader/PrerendererClient.h
diff --git a/Source/core/loader/PrerendererClient.h b/Source/core/loader/PrerendererClient.h
index bf82b72ea3c0e5003fbcbbb37f058d0e3ab116ff..4d595f759d47804181c4bc30618cfc8c5f77458a 100644
--- a/Source/core/loader/PrerendererClient.h
+++ b/Source/core/loader/PrerendererClient.h
@@ -40,10 +40,8 @@ class Document;
 class Page;
 class Prerender;
 
-class PrerendererClient : public Supplement<Page> {
+class PrerendererClient : public WillBeHeapSupplement<Page> {
 public:
-    virtual ~PrerendererClient() { }

[haraken, 2014/04/30 11:18:08]

You can use DECLARE_EMPTY_DESTRUCTOR_WILL_BE_REMOVED.

[zerny-chromium, 2014/04/30 11:46:53]

Should we not rather remove it since it is not needed in either build?

[haraken, 2014/04/30 12:02:08]

If you omit an empty destructor, the destructor is implicitly generated in
header files, which can cause #include cycles. Also we sometimes need to define
an empty, virtual destructor. For these situations, we introduced
DECLARE_EMPTY_VIRTUAL_DESTRUCTOR_WILL_BE_REMOVED.

Either way this wouldn't be a big deal. You can just remove it if it doesn't
matter :)

-
     virtual void willAddPrerender(Prerender*) = 0;
 
     static const char* supplementName();