Block execution of failed 'crossorigin' <script>s.

Source/core/css/resolver/StyleResourceLoader.cpp

Index: Source/core/css/resolver/StyleResourceLoader.cpp
diff --git a/Source/core/css/resolver/StyleResourceLoader.cpp b/Source/core/css/resolver/StyleResourceLoader.cpp
index 3d2977384207059fef31e6fbcc225a1094a0da52..bc65caedac0399694117c7726bc9f2e873087ff0 100644
--- a/Source/core/css/resolver/StyleResourceLoader.cpp
+++ b/Source/core/css/resolver/StyleResourceLoader.cpp
@@ -115,10 +115,9 @@ void StyleResourceLoader::loadPendingShapeImage(RenderStyle* renderStyle, ShapeV
     CSSImageValue* cssImageValue =  pendingImage->cssImageValue();
 
     ResourceLoaderOptions options = ResourceFetcher::defaultResourceOptions();
-    options.requestOriginPolicy = PotentiallyCrossOriginEnabled;
     options.allowCredentials = DoNotAllowStoredCredentials;
 
-    shapeValue->setImage(cssImageValue->cachedImage(m_fetcher, options));
+    shapeValue->setImage(cssImageValue->cachedImage(m_fetcher, options, FetchRequest::UseDefaultOriginRestrictionForType));

[abarth-chromium, 2013/11/20 05:12:31]

Should setImage take a FetchRequest?

[sof, 2013/11/20 10:10:31]

I could well be misunderstanding, but do you mean cachedImage() ? It currently
constructs a FetchRequest instead, adding some local CSSImageValue state, but it
appears possible to have the caller do that construction here & supply it. 

More than happy to tidy this up in a followup.

 }
 
 void StyleResourceLoader::loadPendingImages(RenderStyle* style, const ElementStyleResources& elementStyleResources)