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

Unified Diff: Source/core/loader/FormSubmission.cpp

Issue 123003002: Make calls to AtomicString(const String&) explicit in loader/ and fetch/ (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 12 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: Source/core/loader/FormSubmission.cpp
diff --git a/Source/core/loader/FormSubmission.cpp b/Source/core/loader/FormSubmission.cpp
index 178a996bc763acb3abfd5f022db45241dbde6af4..4be3a6b4f279cda1ab97634b1a5d92d50cb3c99a 100644
--- a/Source/core/loader/FormSubmission.cpp
+++ b/Source/core/loader/FormSubmission.cpp
@@ -259,7 +259,7 @@ void FormSubmission::populateFrameLoadRequest(FrameLoadRequest& frameRequest)
frameRequest.setFrameName(m_target);
if (!m_referrer.isEmpty())
- frameRequest.resourceRequest().setHTTPReferrer(m_referrer);
+ frameRequest.resourceRequest().setHTTPReferrer(AtomicString(m_referrer));
if (m_method == FormSubmission::PostMethod) {
frameRequest.resourceRequest().setHTTPMethod("POST");
@@ -267,13 +267,13 @@ void FormSubmission::populateFrameLoadRequest(FrameLoadRequest& frameRequest)
// construct some user headers if necessary
if (m_boundary.isEmpty())
- frameRequest.resourceRequest().setHTTPContentType(m_contentType);
+ frameRequest.resourceRequest().setHTTPContentType(AtomicString(m_contentType));
eseidel 2013/12/31 22:05:03 content types should probably be stored as AtomicS
Inactive 2013/12/31 22:47:25 Done.
else
frameRequest.resourceRequest().setHTTPContentType(m_contentType + "; boundary=" + m_boundary);
}
frameRequest.resourceRequest().setURL(requestURL());
- FrameLoader::addHTTPOriginIfNeeded(frameRequest.resourceRequest(), m_origin);
+ FrameLoader::addHTTPOriginIfNeeded(frameRequest.resourceRequest(), AtomicString(m_origin));
}
}

Powered by Google App Engine
This is Rietveld 408576698