Change StyleResolver::document() to be private

Source/core/css/resolver/ScopedStyleResolver.cpp

Index: Source/core/css/resolver/ScopedStyleResolver.cpp
diff --git a/Source/core/css/resolver/ScopedStyleResolver.cpp b/Source/core/css/resolver/ScopedStyleResolver.cpp
index 684a888226b43b602bc48d34737a34672b19e846..2b013924cee29eac68af43189820da48dbebfc52 100644
--- a/Source/core/css/resolver/ScopedStyleResolver.cpp
+++ b/Source/core/css/resolver/ScopedStyleResolver.cpp
@@ -74,7 +74,7 @@ void ScopedStyleResolver::addRulesFromSheet(CSSStyleSheet* cssSheet, const Media
     unsigned index = m_authorStyleSheets.size() - 1;
     StyleSheetContents* sheet = cssSheet->contents();
 
-    AddRuleFlags addRuleFlags = resolver->document().securityOrigin()->canRequest(sheet->baseURL()) ? RuleHasDocumentSecurityOrigin : RuleHasNoSpecialState;
+    AddRuleFlags addRuleFlags = m_scope->document().securityOrigin()->canRequest(sheet->baseURL()) ? RuleHasDocumentSecurityOrigin : RuleHasNoSpecialState;

[rune, 2014/09/24 08:16:20]

Why are we having StyleResolver call into ScopedStyleResolver and then back into
StyleResolver?

There are two statements here that needs the context of the ScopedStyleResolver
afaict:

m_authorStyleSheets.append(cssSheet)
resolver->processScopedRules(...)

We have the treeScope necessary for processScoped rules retrieved in
StyleResolver::appendCSSStyleSheet. We could make this method just add the
stylesheet to m_authorStyleSheets and return the index?

     const RuleSet& ruleSet = sheet->ensureRuleSet(medium, addRuleFlags);
     resolver->addMediaQueryResults(ruleSet.viewportDependentMediaQueryResults());
     resolver->processScopedRules(ruleSet, cssSheet, index, treeScope().rootNode());