Oilpan: Move CachedMetadataHandler to Oilpan heap on Oilpan-enabled world.

Source/modules/serviceworkers/ServiceWorkerScriptCachedMetadataHandler.cpp

Index: Source/modules/serviceworkers/ServiceWorkerScriptCachedMetadataHandler.cpp
diff --git a/Source/modules/serviceworkers/ServiceWorkerScriptCachedMetadataHandler.cpp b/Source/modules/serviceworkers/ServiceWorkerScriptCachedMetadataHandler.cpp
index d53961c1989c57b3af22c5865d6273334e3ad0e1..813b2ad8e4f1dcc6db9714bba9d563a9f0536443 100644
--- a/Source/modules/serviceworkers/ServiceWorkerScriptCachedMetadataHandler.cpp
+++ b/Source/modules/serviceworkers/ServiceWorkerScriptCachedMetadataHandler.cpp
@@ -23,6 +23,14 @@ ServiceWorkerScriptCachedMetadataHandler::~ServiceWorkerScriptCachedMetadataHand
 {
 }
 
+#if ENABLE(OILPAN)

[haraken, 2015/07/15 07:48:02]

We normally add #if ENABLE(OILPAN) inside the trace function.

[Yuta Kitamura, 2015/07/15 08:05:34]

Do you mean I should define the trace function even if it's
not used in the non-Oilpan world? If enable_oilpan is 0,
nothing in the class hierarchy is Oilpan-managed.

[haraken, 2015/07/15 08:06:50]

I don't know if it makes sense, but we're doing that in other classes.

[Yuta Kitamura, 2015/07/15 08:11:03]

OK, will address in the next patchset.

[Yuta Kitamura, 2015/07/15 08:20:58]

Done, landing.

+DEFINE_TRACE(ServiceWorkerScriptCachedMetadataHandler)
+{
+    visitor->trace(m_workerGlobalScope);
+    CachedMetadataHandler::trace(visitor);
+}
+#endif
+
 void ServiceWorkerScriptCachedMetadataHandler::setCachedMetadata(unsigned dataTypeID, const char* data, size_t size, CacheType type)
 {
     if (type != SendToPlatform)