Introduce Platform::cacheMetadataInCacheStorage() to store V8 code cache in CacheStorage

content/browser/renderer_host/render_view_host_unittest.cc

Index: content/browser/renderer_host/render_view_host_unittest.cc
diff --git a/content/browser/renderer_host/render_view_host_unittest.cc b/content/browser/renderer_host/render_view_host_unittest.cc
index afe048cd24db894224efc2a755ba8122a6486ea4..7e3746cea992fbd9993c2886e1d16bb79e09a9df 100644
--- a/content/browser/renderer_host/render_view_host_unittest.cc
+++ b/content/browser/renderer_host/render_view_host_unittest.cc
@@ -244,17 +244,16 @@ TEST_F(RenderViewHostTest, RoutingIdSane) {
 
 class TestSaveImageFromDataURL : public RenderMessageFilter {
  public:
-  TestSaveImageFromDataURL(
-      BrowserContext* context)
-      : RenderMessageFilter(
-            0,
-            context,
-            BrowserContext::GetDefaultStoragePartition(context)->
-                GetURLRequestContext(),
-            nullptr,
-            nullptr,
-            nullptr,
-            nullptr) {
+  TestSaveImageFromDataURL(BrowserContext* context)
+      : RenderMessageFilter(0,
+                            context,
+                            BrowserContext::GetDefaultStoragePartition(context)
+                                ->GetURLRequestContext(),
+                            nullptr,
+                            nullptr,
+                            nullptr,
+                            nullptr,
+                            nullptr) {

[vogelheim, 2016/05/03 11:36:05]

The implementation will blindly deref this pointer rather than check for
nullptr. Not sure if this is ok for a test... 

[horo, 2016/05/09 04:00:04]

Sorry, I don't understand what is the problem here...

     Reset();
   }