Commit f933929e authored by clamy@chromium.org's avatar clamy@chromium.org

Remove the report load timing field in ResourceRequest

Since this field is always set to true before being used only in WebURLLoaderImpl, this CL removes it entirely.

BUG=376025

Review URL: https://codereview.chromium.org/316353004

git-svn-id: svn://svn.chromium.org/blink/trunk@176109 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent d50e233e
......@@ -126,7 +126,6 @@ void ResourceLoader::init(const ResourceRequest& passedRequest)
{
ResourceRequest request(passedRequest);
m_host->willSendRequest(m_resource->identifier(), request, ResourceResponse(), m_options.initiatorInfo);
request.setReportLoadTiming(true);
ASSERT(m_state != Terminated);
ASSERT(!request.isNull());
m_originalRequest = m_request = applyOptions(request);
......@@ -298,7 +297,6 @@ void ResourceLoader::willSendRequest(blink::WebURLLoader*, blink::WebURLRequest&
return;
m_host->willSendRequest(m_resource->identifier(), request, redirectResponse, m_options.initiatorInfo);
request.setReportLoadTiming(true);
ASSERT(!request.isNull());
m_resource->updateRequest(request);
m_request = request;
......
......@@ -312,7 +312,6 @@ void InspectorResourceAgent::willSendRequest(unsigned long identifier, DocumentL
}
}
request.setReportLoadTiming(true);
request.setReportRawHeaders(true);
if (m_state->getBoolean(ResourceAgentState::cacheDisabled))
......
......@@ -214,11 +214,6 @@ void WebURLRequest::setReportUploadProgress(bool reportUploadProgress)
m_private->m_resourceRequest->setReportUploadProgress(reportUploadProgress);
}
bool WebURLRequest::reportLoadTiming() const
{
return m_private->m_resourceRequest->reportLoadTiming();
}
void WebURLRequest::setReportRawHeaders(bool reportRawHeaders)
{
m_private->m_resourceRequest->setReportRawHeaders(reportRawHeaders);
......@@ -229,11 +224,6 @@ bool WebURLRequest::reportRawHeaders() const
return m_private->m_resourceRequest->reportRawHeaders();
}
void WebURLRequest::setReportLoadTiming(bool reportLoadTiming)
{
m_private->m_resourceRequest->setReportLoadTiming(reportLoadTiming);
}
WebURLRequest::TargetType WebURLRequest::targetType() const
{
// FIXME: Temporary special case until downstream chromium.org knows of the new TargetTypes.
......
......@@ -359,7 +359,6 @@ void ResourceRequest::initialize(const KURL& url, ResourceRequestCachePolicy cac
m_httpMethod = "GET";
m_allowStoredCredentials = true;
m_reportUploadProgress = false;
m_reportLoadTiming = false;
m_reportRawHeaders = false;
m_hasUserGesture = false;
m_downloadToFile = false;
......
......@@ -170,10 +170,6 @@ public:
bool reportUploadProgress() const { return m_reportUploadProgress; }
void setReportUploadProgress(bool reportUploadProgress) { m_reportUploadProgress = reportUploadProgress; }
// Whether the timing information should be collected for the request.
bool reportLoadTiming() const { return m_reportLoadTiming; }
void setReportLoadTiming(bool reportLoadTiming) { m_reportLoadTiming = reportLoadTiming; }
// Whether actual headers being sent/received should be collected and reported for the request.
bool reportRawHeaders() const { return m_reportRawHeaders; }
void setReportRawHeaders(bool reportRawHeaders) { m_reportRawHeaders = reportRawHeaders; }
......@@ -230,7 +226,6 @@ private:
RefPtr<FormData> m_httpBody;
bool m_allowStoredCredentials : 1;
bool m_reportUploadProgress : 1;
bool m_reportLoadTiming : 1;
bool m_reportRawHeaders : 1;
bool m_hasUserGesture : 1;
bool m_downloadToFile : 1;
......
......@@ -149,10 +149,6 @@ public:
BLINK_PLATFORM_EXPORT bool reportUploadProgress() const;
BLINK_PLATFORM_EXPORT void setReportUploadProgress(bool);
// Controls whether load timing info is collected for the request.
BLINK_PLATFORM_EXPORT bool reportLoadTiming() const;
BLINK_PLATFORM_EXPORT void setReportLoadTiming(bool);
// Controls whether actual headers sent and received for request are
// collected and reported.
BLINK_PLATFORM_EXPORT bool reportRawHeaders() const;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment