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

Unified Diff: chrome/common/extensions/extension_localization_peer.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: chrome/common/extensions/extension_localization_peer.cc
diff --git a/chrome/common/extensions/extension_localization_peer.cc b/chrome/common/extensions/extension_localization_peer.cc
index dfb4f1ddf8d417b39c01a91227cc738f21c6882a..cf70dcf61c6937ca6f038611639187c7b208a8de 100644
--- a/chrome/common/extensions/extension_localization_peer.cc
+++ b/chrome/common/extensions/extension_localization_peer.cc
@@ -48,6 +48,8 @@ void ExtensionLocalizationPeer::OnUploadProgress(
bool ExtensionLocalizationPeer::OnReceivedRedirect(
const GURL& new_url,
const webkit_glue::ResourceResponseInfo& info,
+ const base::TimeTicks& start_time,
+ const base::TimeTicks& end_time,
bool* has_new_first_party_for_cookies,
GURL* new_first_party_for_cookies) {
NOTREACHED();
@@ -55,8 +57,12 @@ bool ExtensionLocalizationPeer::OnReceivedRedirect(
}
void ExtensionLocalizationPeer::OnReceivedResponse(
- const webkit_glue::ResourceResponseInfo& info) {
+ const webkit_glue::ResourceResponseInfo& info,
+ const base::TimeTicks& start_time,
+ const base::TimeTicks& end_time) {
response_info_ = info;
+ start_time_ = start_time;
+ end_time_ = end_time;
}
void ExtensionLocalizationPeer::OnReceivedData(const char* data,
@@ -68,14 +74,14 @@ void ExtensionLocalizationPeer::OnReceivedData(const char* data,
void ExtensionLocalizationPeer::OnCompletedRequest(
const net::URLRequestStatus& status,
const std::string& security_info,
- const base::Time& completion_time) {
+ const base::TimeTicks& completion_time) {
// Make sure we delete ourselves at the end of this call.
scoped_ptr<ExtensionLocalizationPeer> this_deleter(this);
// Give sub-classes a chance at altering the data.
if (status.status() != net::URLRequestStatus::SUCCESS) {
// We failed to load the resource.
- original_peer_->OnReceivedResponse(response_info_);
+ original_peer_->OnReceivedResponse(response_info_, start_time_, end_time_);
net::URLRequestStatus status(net::URLRequestStatus::CANCELED,
net::ERR_ABORTED);
original_peer_->OnCompletedRequest(status, security_info, completion_time);
@@ -84,7 +90,7 @@ void ExtensionLocalizationPeer::OnCompletedRequest(
ReplaceMessages();
- original_peer_->OnReceivedResponse(response_info_);
+ original_peer_->OnReceivedResponse(response_info_, start_time_, end_time_);
if (!data_.empty())
original_peer_->OnReceivedData(data_.data(),
static_cast<int>(data_.size()),

Powered by Google App Engine
This is Rietveld 408576698