Changeset 190755 in webkit

Timestamp:

Oct 8, 2015 3:59:15 PM (9 years ago)

Author:

Chris Dumez

Message:

data: URLs should not be preloaded
​https://bugs.webkit.org/show_bug.cgi?id=149829

Reviewed by Darin Adler.

Fix review comments after r190605:
Use protocolIs() instead of String::startsWith().

• html/parser/HTMLPreloadScanner.cpp:

(WebCore::TokenPreloadScanner::StartTagScanner::shouldPreload):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/parser/HTMLPreloadScanner.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2015-10-08  Chris Dumez  <cdumez@apple.com>
+
+        data: URLs should not be preloaded
+        https://bugs.webkit.org/show_bug.cgi?id=149829
+
+        Reviewed by Darin Adler.
+
+        Fix review comments after r190605:
+        Use protocolIs() instead of String::startsWith().
+
+        * html/parser/HTMLPreloadScanner.cpp:
+        (WebCore::TokenPreloadScanner::StartTagScanner::shouldPreload):
+
 2015-10-08  Chris Dumez  <cdumez@apple.com>
 

trunk/Source/WebCore/html/parser/HTMLPreloadScanner.cpp

@@ -238 +238 @@
             return false;
 
-        if (m_urlToLoad.startsWith("data:", false))
+        if (protocolIs(m_urlToLoad, "data"))
             return false;