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

Unified Diff: webkit/tools/test_shell/simple_resource_loader_bridge.cc

Issue 7602023: Use a monotonic clock (TimeTicks) to report network times to WebCore. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Use strong typing Created 9 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: webkit/tools/test_shell/simple_resource_loader_bridge.cc
diff --git a/webkit/tools/test_shell/simple_resource_loader_bridge.cc b/webkit/tools/test_shell/simple_resource_loader_bridge.cc
index a3e0f64aa7eec11086e9e4dffbe3332760b42a02..ad96500bc7d6dc21c0c6a1b394ca837b80097e3a 100644
--- a/webkit/tools/test_shell/simple_resource_loader_bridge.cc
+++ b/webkit/tools/test_shell/simple_resource_loader_bridge.cc
@@ -235,6 +235,7 @@ class RequestProxy : public net::URLRequest::Delegate,
bool has_new_first_party_for_cookies = false;
GURL new_first_party_for_cookies;
if (peer_ && peer_->OnReceivedRedirect(new_url, info,
+ base::TimeTicks(), base::TimeTicks(),
&has_new_first_party_for_cookies,
&new_first_party_for_cookies)) {
g_io_thread->message_loop()->PostTask(
@@ -249,7 +250,7 @@ class RequestProxy : public net::URLRequest::Delegate,
void NotifyReceivedResponse(const ResourceResponseInfo& info) {
if (peer_)
- peer_->OnReceivedResponse(info);
+ peer_->OnReceivedResponse(info, base::TimeTicks(), base::TimeTicks());
}
void NotifyReceivedData(int bytes_read) {
@@ -288,7 +289,7 @@ class RequestProxy : public net::URLRequest::Delegate,
void NotifyCompletedRequest(const net::URLRequestStatus& status,
const std::string& security_info,
- const base::Time& complete_time) {
+ const base::TimeTicks& complete_time) {
if (peer_) {
peer_->OnCompletedRequest(status, security_info, complete_time);
DropPeer(); // ensure no further notifications
@@ -423,7 +424,7 @@ class RequestProxy : public net::URLRequest::Delegate,
virtual void OnCompletedRequest(const net::URLRequestStatus& status,
const std::string& security_info,
- const base::Time& complete_time) {
+ const base::TimeTicks& complete_time) {
if (download_to_file_)
file_stream_.Close();
owner_loop_->PostTask(
@@ -519,7 +520,7 @@ class RequestProxy : public net::URLRequest::Delegate,
// |failed_file_request_status_|. Otherwise use request_'s status.
OnCompletedRequest(failed_file_request_status_.get() ?
*failed_file_request_status_ : request_->status(),
- std::string(), base::Time());
+ std::string(), base::TimeTicks());
request_.reset(); // destroy on the io thread
}
@@ -731,7 +732,7 @@ class SyncRequestProxy : public RequestProxy {
virtual void OnCompletedRequest(const net::URLRequestStatus& status,
const std::string& security_info,
- const base::Time& complete_time) {
+ const base::TimeTicks& complete_time) {
if (download_to_file_)
file_stream_.Close();
result_->status = status;

Powered by Google App Engine
This is Rietveld 408576698