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

Side by Side Diff: third_party/WebKit/Source/core/fetch/FetchUtils.cpp

Issue 1860693004: Allow custom inspector header for CORS preflight. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "core/fetch/FetchUtils.h" 5 #include "core/fetch/FetchUtils.h"
6 6
7 #include "core/inspector/InspectorInstrumentation.h"
7 #include "platform/network/HTTPHeaderMap.h" 8 #include "platform/network/HTTPHeaderMap.h"
8 #include "platform/network/HTTPParsers.h" 9 #include "platform/network/HTTPParsers.h"
9 #include "wtf/HashSet.h" 10 #include "wtf/HashSet.h"
10 #include "wtf/Threading.h" 11 #include "wtf/Threading.h"
11 #include "wtf/text/AtomicString.h" 12 #include "wtf/text/AtomicString.h"
12 #include "wtf/text/WTFString.h" 13 #include "wtf/text/WTFString.h"
13 14
14 namespace blink { 15 namespace blink {
15 16
16 namespace { 17 namespace {
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
85 bool FetchUtils::isSimpleHeader(const AtomicString& name, const AtomicString& va lue) 86 bool FetchUtils::isSimpleHeader(const AtomicString& name, const AtomicString& va lue)
86 { 87 {
87 // http://fetch.spec.whatwg.org/#simple-header 88 // http://fetch.spec.whatwg.org/#simple-header
88 // "A simple header is a header whose name is either one of `Accept`, 89 // "A simple header is a header whose name is either one of `Accept`,
89 // `Accept-Language`, and `Content-Language`, or whose name is 90 // `Accept-Language`, and `Content-Language`, or whose name is
90 // `Content-Type` and value, once parsed, is one of 91 // `Content-Type` and value, once parsed, is one of
91 // `application/x-www-form-urlencoded`, `multipart/form-data`, and 92 // `application/x-www-form-urlencoded`, `multipart/form-data`, and
92 // `text/plain`." 93 // `text/plain`."
93 // Treat 'Save-Data' as a simple header, since it is added by Chrome when 94 // Treat 'Save-Data' as a simple header, since it is added by Chrome when
94 // Data Saver feature is enabled. 95 // Data Saver feature is enabled.
96 // Treat inspector header as a simple header, since it is added by blink whe n
97 // inspector is open.
95 98
96 if (equalIgnoringCase(name, "accept") 99 if (equalIgnoringCase(name, "accept")
97 || equalIgnoringCase(name, "accept-language") 100 || equalIgnoringCase(name, "accept-language")
98 || equalIgnoringCase(name, "content-language") 101 || equalIgnoringCase(name, "content-language")
102 || equalIgnoringCase(name, InspectorInstrumentation::kInspectorEmulateNe tworkConditionsClientId)
99 || equalIgnoringCase(name, "save-data")) 103 || equalIgnoringCase(name, "save-data"))
100 return true; 104 return true;
101 105
102 if (equalIgnoringCase(name, "content-type")) { 106 if (equalIgnoringCase(name, "content-type")) {
103 AtomicString mimeType = extractMIMETypeFromMediaType(value); 107 AtomicString mimeType = extractMIMETypeFromMediaType(value);
104 return equalIgnoringCase(mimeType, "application/x-www-form-urlencoded") 108 return equalIgnoringCase(mimeType, "application/x-www-form-urlencoded")
105 || equalIgnoringCase(mimeType, "multipart/form-data") 109 || equalIgnoringCase(mimeType, "multipart/form-data")
106 || equalIgnoringCase(mimeType, "text/plain"); 110 || equalIgnoringCase(mimeType, "text/plain");
107 } 111 }
108 112
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
199 String FetchUtils::normalizeHeaderValue(const String& value) 203 String FetchUtils::normalizeHeaderValue(const String& value)
200 { 204 {
201 // https://fetch.spec.whatwg.org/#concept-header-value-normalize 205 // https://fetch.spec.whatwg.org/#concept-header-value-normalize
202 // Strip leading and trailing whitespace from header value. 206 // Strip leading and trailing whitespace from header value.
203 // HTTP whitespace bytes are 0x09, 0x0A, 0x0D, and 0x20. 207 // HTTP whitespace bytes are 0x09, 0x0A, 0x0D, and 0x20.
204 208
205 return value.stripWhiteSpace(isHTTPWhitespace); 209 return value.stripWhiteSpace(isHTTPWhitespace);
206 } 210 }
207 211
208 } // namespace blink 212 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698