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

Unified Diff: third_party/WebKit/Source/core/html/parser/HTMLPreloadScanner.cpp

Issue 2668903003: Replace WTF::emptyString{16Bit}() with a static global (Closed)
Patch Set: Replace WTF::emptyString{16Bit}() with a static global Created 3 years, 11 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: third_party/WebKit/Source/core/html/parser/HTMLPreloadScanner.cpp
diff --git a/third_party/WebKit/Source/core/html/parser/HTMLPreloadScanner.cpp b/third_party/WebKit/Source/core/html/parser/HTMLPreloadScanner.cpp
index 2b5e1553a7596536bfdd0a629646bfd35c9b186a..7ea5576e8dd2e8f9f01da5ecdb21a5561fc01cec 100644
--- a/third_party/WebKit/Source/core/html/parser/HTMLPreloadScanner.cpp
+++ b/third_party/WebKit/Source/core/html/parser/HTMLPreloadScanner.cpp
@@ -122,7 +122,7 @@ static String initiatorFor(const StringImpl* tagImpl) {
if (match(tagImpl, videoTag))
return videoTag.localName();
NOTREACHED();
- return emptyString();
+ return emptyString;
}
static bool mediaAttributeMatches(const MediaValuesCached& mediaValues,
@@ -443,7 +443,7 @@ class TokenPreloadScanner::StartTagScanner {
// FIXME: Its not clear that this if is needed, the loader probably ignores
// charset for image requests anyway.
if (match(m_tagImpl, imgTag) || match(m_tagImpl, videoTag))
- return emptyString();
+ return emptyString;
return m_charset;
}

Powered by Google App Engine
This is Rietveld 408576698