Use int64 time stamp when storing metadata to the HTTP cache.

content/renderer/renderer_blink_platform_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/renderer_blink_platform_impl.h"
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
@@ skipping 324 matching lines @@
 }
 
 blink::WebPrescientNetworking*
 RendererBlinkPlatformImpl::prescientNetworking() {
   return GetContentClient()->renderer()->GetPrescientNetworking();
 }
 
 void RendererBlinkPlatformImpl::cacheMetadata(const blink::WebURL& url,
                                               double response_time,
                                               const char* data,
                                               size_t size) {

[davidben, 2015/02/18 18:50:45]

Why not have this function do the conversion from double to int64 and call the
int64 cacheMetadata overload? Then you don't need to duplicate the IPC and
everything else up the stack.

   // Let the browser know we generated cacheable metadata for this resource. The
   // browser may cache it and return it on subsequent responses to speed
   // the processing of this resource.
   std::vector<char> copy(data, data + size);
   RenderThread::Get()->Send(
       new ViewHostMsg_DidGenerateCacheableMetadata(url, response_time, copy));
 }
 
+void RendererBlinkPlatformImpl::cacheMetadata(const blink::WebURL& url,
+                                              int64 response_time,
+                                              const char* data,
+                                              size_t size) {
+  // Let the browser know we generated cacheable metadata for this resource. The
+  // browser may cache it and return it on subsequent responses to speed
+  // the processing of this resource.
+  std::vector<char> copy(data, data + size);
+  RenderThread::Get()->Send(
+      new ViewHostMsg_DidGenerateCacheableMetadataOriginalResponseTime(
+          url, response_time, copy));
+}
+
 WebString RendererBlinkPlatformImpl::defaultLocale() {
   return base::ASCIIToUTF16(RenderThread::Get()->GetLocale());
 }
 
 void RendererBlinkPlatformImpl::suddenTerminationChanged(bool enabled) {
   if (enabled) {
     // We should not get more enables than disables, but we want it to be a
     // non-fatal error if it does happen.
     DCHECK_GT(sudden_termination_disables_, 0);
     sudden_termination_disables_ = std::max(sudden_termination_disables_ - 1,
@@ skipping 849 matching lines @@
 //------------------------------------------------------------------------------
 
 void RendererBlinkPlatformImpl::MockBatteryStatusChangedForTesting(
     const blink::WebBatteryStatus& status) {
   if (!g_test_battery_status_listener)
     return;
   g_test_battery_status_listener->updateBatteryStatus(status);
 }
 
 }  // namespace content