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

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

Issue 2508033004: Reduce unnecessary usage of TextCaseSensitivity::TextCaseInsensitive. (Closed)
Patch Set: Created 4 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 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 "platform/HTTPNames.h" 7 #include "platform/HTTPNames.h"
8 #include "platform/network/HTTPHeaderMap.h" 8 #include "platform/network/HTTPHeaderMap.h"
9 #include "platform/network/HTTPParsers.h" 9 #include "platform/network/HTTPParsers.h"
10 #include "wtf/HashSet.h" 10 #include "wtf/HashSet.h"
11 #include "wtf/Threading.h" 11 #include "wtf/Threading.h"
12 #include "wtf/text/AtomicString.h" 12 #include "wtf/text/AtomicString.h"
13 #include "wtf/text/WTFString.h" 13 #include "wtf/text/WTFString.h"
14 14
15 namespace blink { 15 namespace blink {
16 16
17 namespace { 17 namespace {
18 18
19 bool isHTTPWhitespace(UChar chr) { 19 bool isHTTPWhitespace(UChar chr) {
20 return chr == ' ' || chr == '\n' || chr == '\t' || chr == '\r'; 20 return chr == ' ' || chr == '\n' || chr == '\t' || chr == '\r';
21 } 21 }
22 22
23 class ForbiddenHeaderNames { 23 class ForbiddenHeaderNames {
24 WTF_MAKE_NONCOPYABLE(ForbiddenHeaderNames); 24 WTF_MAKE_NONCOPYABLE(ForbiddenHeaderNames);
25 USING_FAST_MALLOC(ForbiddenHeaderNames); 25 USING_FAST_MALLOC(ForbiddenHeaderNames);
26 26
27 public: 27 public:
28 bool has(const String& name) const { 28 bool has(const String& name) const {
29 return m_fixedNames.contains(name) || 29 return m_fixedNames.contains(name) ||
30 name.startsWith(m_proxyHeaderPrefix, TextCaseInsensitive) || 30 name.startsWith(m_proxyHeaderPrefix, TextCaseASCIIInsensitive) ||
31 name.startsWith(m_secHeaderPrefix, TextCaseInsensitive); 31 name.startsWith(m_secHeaderPrefix, TextCaseASCIIInsensitive);
32 } 32 }
33 33
34 static const ForbiddenHeaderNames& get(); 34 static const ForbiddenHeaderNames& get();
35 35
36 private: 36 private:
37 ForbiddenHeaderNames(); 37 ForbiddenHeaderNames();
38 38
39 String m_proxyHeaderPrefix; 39 String m_proxyHeaderPrefix;
40 String m_secHeaderPrefix; 40 String m_secHeaderPrefix;
41 HashSet<String, CaseFoldingHash> m_fixedNames; 41 HashSet<String, CaseFoldingHash> m_fixedNames;
(...skipping 156 matching lines...) Expand 10 before | Expand all | Expand 10 after
198 198
199 String FetchUtils::normalizeHeaderValue(const String& value) { 199 String FetchUtils::normalizeHeaderValue(const String& value) {
200 // https://fetch.spec.whatwg.org/#concept-header-value-normalize 200 // https://fetch.spec.whatwg.org/#concept-header-value-normalize
201 // Strip leading and trailing whitespace from header value. 201 // Strip leading and trailing whitespace from header value.
202 // HTTP whitespace bytes are 0x09, 0x0A, 0x0D, and 0x20. 202 // HTTP whitespace bytes are 0x09, 0x0A, 0x0D, and 0x20.
203 203
204 return value.stripWhiteSpace(isHTTPWhitespace); 204 return value.stripWhiteSpace(isHTTPWhitespace);
205 } 205 }
206 206
207 } // namespace blink 207 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/dom/Document.cpp ('k') | third_party/WebKit/Source/core/fetch/Resource.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698