Implement GenericURLRequestJob::GetLoadTimingInfo

headless/public/util/generic_url_request_job.cc

 // Copyright 2016 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 "headless/public/util/generic_url_request_job.h"
 
 #include <string.h>
 #include <algorithm>
 
 #include "base/logging.h"
@@ skipping 122 matching lines @@
 void GenericURLRequestJob::OnFetchStartError(net::Error error) {
   DispatchStartError(error);
 }
 
 void GenericURLRequestJob::OnFetchComplete(
     const GURL& final_url,
     int http_response_code,
     scoped_refptr<net::HttpResponseHeaders> response_headers,
     const char* body,
     size_t body_size) {
+  response_time_ = base::TimeTicks::Now();
   http_response_code_ = http_response_code;
   response_headers_ = response_headers;
   body_ = body;
   body_size_ = body_size;
 
   DispatchHeadersComplete();
 
   std::string mime_type;
   GetMimeType(&mime_type);
 
@@ skipping 26 matching lines @@
     return false;
   return response_headers_->GetMimeType(mime_type);
 }
 
 bool GenericURLRequestJob::GetCharset(std::string* charset) {
   if (!response_headers_)
     return false;
   return response_headers_->GetCharset(charset);
 }
 
+void GenericURLRequestJob::GetLoadTimingInfo(
+    net::LoadTimingInfo* load_timing_info) const {
+  load_timing_info->receive_headers_end = response_time_;

[altimin, 2016/11/17 15:45:24]

Should we also fill other fields in net::LoadTimingInfo?

[alex clarke (OOO till 29th), 2016/11/17 16:05:59]

Possibly :)

I'm not really sure how they are used (and how important they are).  Currently
some of them we simply can't get with our current fetch interface.  I'd like to
understand this better before I rush off and implement something.

[altimin, 2016/11/17 16:08:39]

Let's add a TODO! :)

+}
+
 }  // namespace headless