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

Side by Side Diff: third_party/WebKit/Source/platform/weborigin/KURL.h

Issue 2463703002: Optimize KURL protocols (Closed)
Patch Set: 16 bit test 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 /* 1 /*
2 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2011, 2012 Apple Inc. 2 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2011, 2012 Apple Inc.
3 * All rights reserved. 3 * All rights reserved.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions 6 * modification, are permitted provided that the following conditions
7 * are met: 7 * are met:
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 190 matching lines...) Expand 10 before | Expand all | Expand 10 after
201 201
202 template <typename CHAR> 202 template <typename CHAR>
203 void replaceComponents(const url::Replacements<CHAR>&); 203 void replaceComponents(const url::Replacements<CHAR>&);
204 204
205 template <typename CHAR> 205 template <typename CHAR>
206 void init(const KURL& base, 206 void init(const KURL& base,
207 const CHAR* relative, 207 const CHAR* relative,
208 int relativeLength, 208 int relativeLength,
209 const WTF::TextEncoding* queryEncoding); 209 const WTF::TextEncoding* queryEncoding);
210 void initInnerURL(); 210 void initInnerURL();
211 void initProtocolIsInHTTPFamily(); 211 void initProtocolMetadata();
212 212
213 bool m_isValid; 213 bool m_isValid;
214 bool m_protocolIsInHTTPFamily; 214 bool m_protocolIsInHTTPFamily;
215
216 // Keep an AtomicString for the protocol, to avoid copious copies for
217 // KURL::protocol.
218 String m_protocol;
esprehn 2016/11/04 23:48:32 This isn't atomic
219
215 url::Parsed m_parsed; 220 url::Parsed m_parsed;
216 String m_string; 221 String m_string;
217 std::unique_ptr<KURL> m_innerURL; 222 std::unique_ptr<KURL> m_innerURL;
218 }; 223 };
219 224
220 PLATFORM_EXPORT bool operator==(const KURL&, const KURL&); 225 PLATFORM_EXPORT bool operator==(const KURL&, const KURL&);
221 PLATFORM_EXPORT bool operator==(const KURL&, const String&); 226 PLATFORM_EXPORT bool operator==(const KURL&, const String&);
222 PLATFORM_EXPORT bool operator==(const String&, const KURL&); 227 PLATFORM_EXPORT bool operator==(const String&, const KURL&);
223 PLATFORM_EXPORT bool operator!=(const KURL&, const KURL&); 228 PLATFORM_EXPORT bool operator!=(const KURL&, const KURL&);
224 PLATFORM_EXPORT bool operator!=(const KURL&, const String&); 229 PLATFORM_EXPORT bool operator!=(const KURL&, const String&);
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
286 291
287 // KURLHash is the default hash for String 292 // KURLHash is the default hash for String
288 template <> 293 template <>
289 struct DefaultHash<blink::KURL> { 294 struct DefaultHash<blink::KURL> {
290 typedef blink::KURLHash Hash; 295 typedef blink::KURLHash Hash;
291 }; 296 };
292 297
293 } // namespace WTF 298 } // namespace WTF
294 299
295 #endif // KURL_h 300 #endif // KURL_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698