Fix a load timing bug in the case of SPDY session reuse

net/url_request/url_request_test_job.h

Index: net/url_request/url_request_test_job.h
===================================================================
--- net/url_request/url_request_test_job.h	(revision 191718)
+++ net/url_request/url_request_test_job.h	(working copy)
@@ -8,6 +8,7 @@
 #include <string>
 
 #include "base/memory/weak_ptr.h"
+#include "net/base/load_timing_info.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_job.h"
 
@@ -90,6 +91,10 @@
   bool auto_advance() { return auto_advance_; }
   void set_auto_advance(bool auto_advance) { auto_advance_ = auto_advance; }
 
+  void set_load_timing_info(const LoadTimingInfo& load_timing_info) {

[eroman, 2013/04/06 01:02:02]

Is this needed (since the member is protected)

[mmenke, 2013/04/08 15:31:24]

It is if we don't want to create another subclass - the test creates
URLRequestTestJobs directly, rather than a subclass.

We could remove all the added load timing stuff here, and add it to a subclass,
but I don't think that gets us much.  Other tests may find it useful in the
future, and it doesn't break any old behavior or add significant complexity.

+    load_timing_info_ = load_timing_info;
+  }
+
   RequestPriority priority() const { return priority_; }
 
   // Factory method for protocol factory registration if callers don't subclass
@@ -104,6 +109,8 @@
   virtual void Kill() OVERRIDE;
   virtual bool GetMimeType(std::string* mime_type) const OVERRIDE;
   virtual void GetResponseInfo(HttpResponseInfo* info) OVERRIDE;
+  virtual void GetLoadTimingInfo(
+      LoadTimingInfo* load_timing_info) const OVERRIDE;
   virtual int GetResponseCode() const OVERRIDE;
   virtual bool IsRedirectResponse(GURL* location,
                                   int* http_status_code) OVERRIDE;
@@ -152,6 +159,8 @@
   IOBuffer* async_buf_;
   int async_buf_size_;
 
+  LoadTimingInfo load_timing_info_;
+
   base::WeakPtrFactory<URLRequestTestJob> weak_factory_;
 };