net: Return LoadTiming information in the case of a cache hit

net/url_request/url_request.cc

Index: net/url_request/url_request.cc
===================================================================
--- net/url_request/url_request.cc	(revision 199488)
+++ net/url_request/url_request.cc	(working copy)
@@ -1034,7 +1034,18 @@
   // socket is closed and the ClientSocketHandle is Reset, which will happen
   // once the body is complete.  The start times should already be populated.
   if (job_) {
+    // Keep a copy of the two times the URLRequest sets.
+    base::TimeTicks request_start = load_timing_info_.request_start;
+    base::Time request_start_time = load_timing_info_.request_start_time;

[mmenke, 2013/05/10 18:32:26]

I'm not a big fan of this, but other solutions seem worse a little worse:

1)  Have HttpCache::Transaction::GetLoadTimingInfo back up the values it
overwrites (Seems more regression prone than this, though this isn't great in
that respect, either).

2)  Keep the times in separate variables, and use them in
URLRequest::GetLoadTimingInfo (Can't just use them here, since I believe some
request types skip this notification).  Seems a little weird to pluck them from
a different location in GetLoadTimingInfo().

+
+    // Clear load times.  Shouldn't be neded, but gives the GetLoadTimingInfo a
+    // consistent place to start from.
+    load_timing_info_ = LoadTimingInfo();
     job_->GetLoadTimingInfo(&load_timing_info_);
+
+    load_timing_info_.request_start = request_start;
+    load_timing_info_.request_start_time = request_start_time;
+
     ConvertRealLoadTimesToBlockingTimes(&load_timing_info_);
   }
 }