Filter out data urls in the preload scanner

third_party/WebKit/Source/core/html/parser/HTMLResourcePreloaderTest.cpp

Index: third_party/WebKit/Source/core/html/parser/HTMLResourcePreloaderTest.cpp
diff --git a/third_party/WebKit/Source/core/html/parser/HTMLResourcePreloaderTest.cpp b/third_party/WebKit/Source/core/html/parser/HTMLResourcePreloaderTest.cpp
index c28df17d67dd1c3a10c6f8d4fe8b4d677a552e88..8326807b93f94012f7398c57190e01dfb1aff5dc 100644
--- a/third_party/WebKit/Source/core/html/parser/HTMLResourcePreloaderTest.cpp
+++ b/third_party/WebKit/Source/core/html/parser/HTMLResourcePreloaderTest.cpp
@@ -49,11 +49,12 @@ class HTMLResourcePreloaderTest : public testing::Test {
     // TODO(yoav): Need a mock loader here to verify things are happenning
     // beyond preconnect.
     PreloaderNetworkHintsMock networkHints;
-    std::unique_ptr<PreloadRequest> preloadRequest = PreloadRequest::create(
-        String(), TextPosition(), testCase.url,
-        KURL(ParsedURLStringTag(), testCase.baseURL), Resource::Image,
-        ReferrerPolicy(), FetchRequest::ResourceWidth(),
-        ClientHintsPreferences(), PreloadRequest::RequestTypePreconnect);
+    std::unique_ptr<PreloadRequest> preloadRequest =

[Yoav Weiss, 2016/10/25 04:34:14]

ditto

[Charlie Harrison, 2016/10/25 13:32:48]

Done.

+        PreloadRequest::createIfNeeded(
+            String(), TextPosition(), testCase.url,
+            KURL(ParsedURLStringTag(), testCase.baseURL), Resource::Image,
+            ReferrerPolicy(), FetchRequest::ResourceWidth(),
+            ClientHintsPreferences(), PreloadRequest::RequestTypePreconnect);
     if (testCase.isCORS)

[Yoav Weiss, 2016/10/25 04:34:14]

Assuming we're certain here that request was created (which I think we are), can
you DCHECK it?

[Charlie Harrison, 2016/10/25 13:32:48]

Done.

       preloadRequest->setCrossOrigin(CrossOriginAttributeAnonymous);
     HTMLResourcePreloader* preloader =