Reduce use of deprecatedCharacters in WebCore
[WebKit-https.git] / Source / WebCore / page / ContentSecurityPolicy.cpp
index 5a5a7f7..57488b1 100644 (file)
@@ -47,7 +47,6 @@
 #include <inspector/ScriptCallStackFactory.h>
 #include <wtf/HashSet.h>
 #include <wtf/text/TextPosition.h>
-#include <wtf/text/WTFString.h>
 
 using namespace Inspector;
 
@@ -189,8 +188,9 @@ static void skipWhile(const UChar*& position, const UChar* end)
 
 static bool isSourceListNone(const String& value)
 {
-    const UChar* begin = value.deprecatedCharacters();
-    const UChar* end = value.deprecatedCharacters() + value.length();
+    auto characters = StringView(value).upconvertedCharacters();
+    const UChar* begin = characters;
+    const UChar* end = characters + value.length();
     skipWhile<isASCIISpace>(begin, end);
 
     const UChar* position = begin;
@@ -339,7 +339,8 @@ void CSPSourceList::parse(const String& value)
     // We represent 'none' as an empty m_list.
     if (isSourceListNone(value))
         return;
-    parse(value.deprecatedCharacters(), value.deprecatedCharacters() + value.length());
+    auto characters = StringView(value).upconvertedCharacters();
+    parse(characters, characters + value.length());
 }
 
 bool CSPSourceList::matches(const URL& url)
@@ -680,7 +681,8 @@ private:
     void parse(const String& value)
     {
         String nonce;
-        const UChar* position = value.deprecatedCharacters();
+        auto characters = StringView(value).upconvertedCharacters();
+        const UChar* position = characters;
         const UChar* end = position + value.length();
 
         skipWhile<isASCIISpace>(position, end);
@@ -723,7 +725,8 @@ public:
 private:
     void parse(const String& value)
     {
-        const UChar* begin = value.deprecatedCharacters();
+        auto characters = StringView(value).upconvertedCharacters();
+        const UChar* begin = characters;
         const UChar* position = begin;
         const UChar* end = begin + value.length();
 
@@ -1211,7 +1214,8 @@ void CSPDirectiveList::parse(const String& policy)
     if (policy.isEmpty())
         return;
 
-    const UChar* position = policy.deprecatedCharacters();
+    auto characters = StringView(policy).upconvertedCharacters();
+    const UChar* position = characters;
     const UChar* end = position + policy.length();
 
     while (position < end) {
@@ -1290,7 +1294,9 @@ void CSPDirectiveList::parseReportURI(const String& name, const String& value)
         m_policy->reportDuplicateDirective(name);
         return;
     }
-    const UChar* position = value.deprecatedCharacters();
+
+    auto characters = StringView(value).upconvertedCharacters();
+    const UChar* position = characters;
     const UChar* end = position + value.length();
 
     while (position < end) {
@@ -1344,7 +1350,8 @@ void CSPDirectiveList::parseReflectedXSS(const String& name, const String& value
         return;
     }
 
-    const UChar* position = value.deprecatedCharacters();
+    auto characters = StringView(value).upconvertedCharacters();
+    const UChar* position = characters;
     const UChar* end = position + value.length();
 
     skipWhile<isASCIISpace>(position, end);
@@ -1443,7 +1450,8 @@ void ContentSecurityPolicy::didReceiveHeader(const String& header, HeaderType ty
     // RFC2616, section 4.2 specifies that headers appearing multiple times can
     // be combined with a comma. Walk the header string, and parse each comma
     // separated chunk as a separate header.
-    const UChar* begin = header.deprecatedCharacters();
+    auto characters = StringView(header).upconvertedCharacters();
+    const UChar* begin = characters;
     const UChar* position = begin;
     const UChar* end = begin + header.length();
     while (position < end) {