Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(296)

Unified Diff: headless/public/util/generic_url_request_job.cc

Issue 2511143002: Implement GenericURLRequestJob::GetLoadTimingInfo (Closed)
Patch Set: Added a TODO Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: headless/public/util/generic_url_request_job.cc
diff --git a/headless/public/util/generic_url_request_job.cc b/headless/public/util/generic_url_request_job.cc
index 9c7245bc43093991dbe604ac089ce41ac9fae205..a82925d9623c5a77dbcb681a2b00c95c162aecfc 100644
--- a/headless/public/util/generic_url_request_job.cc
+++ b/headless/public/util/generic_url_request_job.cc
@@ -140,6 +140,7 @@ void GenericURLRequestJob::OnFetchComplete(
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;
@@ -186,4 +187,10 @@ bool GenericURLRequestJob::GetCharset(std::string* charset) {
return response_headers_->GetCharset(charset);
}
+void GenericURLRequestJob::GetLoadTimingInfo(
+ net::LoadTimingInfo* load_timing_info) const {
+ // TODO(alexclarke): Investigate setting the other members too where possible.
+ load_timing_info->receive_headers_end = response_time_;
+}
+
} // namespace headless
« no previous file with comments | « headless/public/util/generic_url_request_job.h ('k') | headless/public/util/generic_url_request_job_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698