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

Unified Diff: third_party/WebKit/Source/platform/network/ResourceTimingInfo.cpp

Issue 2316573002: PlzNavigate: Support ResourceTiming API (Closed)
Patch Set: Rebase Created 4 years, 2 months 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: third_party/WebKit/Source/platform/network/ResourceTimingInfo.cpp
diff --git a/third_party/WebKit/Source/platform/network/ResourceTimingInfo.cpp b/third_party/WebKit/Source/platform/network/ResourceTimingInfo.cpp
index b28e1bf0407889eba6aa6649588eee8067a84fcf..efc469322a0fcc22f7c45f030fa305ae640a9815 100644
--- a/third_party/WebKit/Source/platform/network/ResourceTimingInfo.cpp
+++ b/third_party/WebKit/Source/platform/network/ResourceTimingInfo.cpp
@@ -17,7 +17,7 @@ std::unique_ptr<ResourceTimingInfo> ResourceTimingInfo::adopt(
info->m_originalTimingAllowOrigin =
AtomicString(data->m_originalTimingAllowOrigin);
info->m_loadFinishTime = data->m_loadFinishTime;
- info->m_initialRequest = ResourceRequest(data->m_initialRequest.get());
+ info->m_initialURL = data->m_initialURL.copy();
info->m_finalResponse = ResourceResponse(data->m_finalResponse.get());
for (auto& responseData : data->m_redirectChain)
info->m_redirectChain.append(ResourceResponse(responseData.get()));
@@ -34,7 +34,7 @@ ResourceTimingInfo::copyData() const {
m_originalTimingAllowOrigin.getString().isolatedCopy();
data->m_initialTime = m_initialTime;
data->m_loadFinishTime = m_loadFinishTime;
- data->m_initialRequest = m_initialRequest.copyData();
+ data->m_initialURL = m_initialURL.copy();
data->m_finalResponse = m_finalResponse.copyData();
for (const auto& response : m_redirectChain)
data->m_redirectChain.append(response.copyData());
@@ -44,7 +44,6 @@ ResourceTimingInfo::copyData() const {
}
void ResourceTimingInfo::addRedirect(const ResourceResponse& redirectResponse,
- long long encodedDataLength,
bool crossOrigin) {
m_redirectChain.append(redirectResponse);
if (m_hasCrossOriginRedirect)
@@ -53,8 +52,8 @@ void ResourceTimingInfo::addRedirect(const ResourceResponse& redirectResponse,
m_hasCrossOriginRedirect = true;
m_transferSize = 0;
} else {
- DCHECK_GE(encodedDataLength, 0);
- m_transferSize += encodedDataLength;
+ DCHECK_GE(redirectResponse.encodedDataLength(), 0);
+ m_transferSize += redirectResponse.encodedDataLength();
}
}

Powered by Google App Engine
This is Rietveld 408576698