Change CSSParserContext to either take explicit or implicit referrer

third_party/WebKit/Source/core/css/StyleRuleImport.cpp

Index: third_party/WebKit/Source/core/css/StyleRuleImport.cpp
diff --git a/third_party/WebKit/Source/core/css/StyleRuleImport.cpp b/third_party/WebKit/Source/core/css/StyleRuleImport.cpp
index 9693c45b9ab606f0abcef7d415df11deabc2bdbc..b677c8f597a7b3dc7ddee80cacb0ccc7e223dffe 100644
--- a/third_party/WebKit/Source/core/css/StyleRuleImport.cpp
+++ b/third_party/WebKit/Source/core/css/StyleRuleImport.cpp
@@ -78,13 +78,11 @@ void StyleRuleImport::setCSSStyleSheet(
     document = m_parentStyleSheet->singleOwnerDocument();
     context = m_parentStyleSheet->parserContext();
   }
-  context = CSSParserContext::create(
-      context, baseURL, charset,
-      (!baseURL.isNull() && document)
-          ? Referrer(baseURL.strippedForUseAsReferrer(),
-                     document->getReferrerPolicy())
-          : context->referrer(),
-      document);
+  context =
+      CSSParserContext::create(context, baseURL,
+                               document ? document->getReferrerPolicy()
+                                        : context->referrer().referrerPolicy,

[estark, 2017/03/28 11:02:34]

This looks like it will result in a different referrer in certain cases; for
example in the case where |m_parentStyleSheet| is null, we would previously end
up with CSSParserContext having referrer ("", ReferrerPolicyDefault), but now
it'll be (|baseURL|, ReferrerPolicyDefault), meaning we'll send non-empty
referrers in cases where we weren't previously. Is that correct and/or
intentional?

[jochen (gone - plz use gerrit), 2017/03/28 11:21:23]

good point.

I think it's correct(er) now. If we have a baseURL (here, the response().url()
from loading the stylesheet), we should use it as referrer (according to my
proposed spec change :) )

+                               charset, document);
 
   m_styleSheet = StyleSheetContents::create(this, href, context);