Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(423)

Unified Diff: third_party/WebKit/Source/core/css/resolver/ScopedStyleResolver.cpp

Issue 1858753003: Remove RawPtr from core/css (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/css/resolver/ScopedStyleResolver.cpp
diff --git a/third_party/WebKit/Source/core/css/resolver/ScopedStyleResolver.cpp b/third_party/WebKit/Source/core/css/resolver/ScopedStyleResolver.cpp
index ba120ef34cac916f93041edabd3fcb09fcd7a552..389606949eee0dfc9292fda431c4bf491e8d45b6 100644
--- a/third_party/WebKit/Source/core/css/resolver/ScopedStyleResolver.cpp
+++ b/third_party/WebKit/Source/core/css/resolver/ScopedStyleResolver.cpp
@@ -71,7 +71,7 @@ void ScopedStyleResolver::addFontFaceRules(const RuleSet& ruleSet)
CSSFontSelector* cssFontSelector = document.styleEngine().fontSelector();
const HeapVector<Member<StyleRuleFontFace>> fontFaceRules = ruleSet.fontFaceRules();
for (auto& fontFaceRule : fontFaceRules) {
- if (RawPtr<FontFace> fontFace = FontFace::create(&document, fontFaceRule))
+ if (FontFace* fontFace = FontFace::create(&document, fontFaceRule))
cssFontSelector->fontFaceCache()->add(cssFontSelector, fontFaceRule, fontFace);
}
if (fontFaceRules.size())
@@ -129,7 +129,7 @@ StyleRuleKeyframes* ScopedStyleResolver::keyframeStylesForAnimation(const String
return it->value.get();
}
-void ScopedStyleResolver::addKeyframeStyle(RawPtr<StyleRuleKeyframes> rule)
+void ScopedStyleResolver::addKeyframeStyle(StyleRuleKeyframes* rule)
{
AtomicString s(rule->name());
@@ -217,12 +217,12 @@ void ScopedStyleResolver::addTreeBoundaryCrossingRules(const RuleSet& authorRule
if (!authorRules.deepCombinatorOrShadowPseudoRules().isEmpty())
m_hasDeepOrShadowSelector = true;
- RawPtr<RuleSet> ruleSetForScope = RuleSet::create();
- addRules(ruleSetForScope.get(), authorRules.deepCombinatorOrShadowPseudoRules());
+ RuleSet* ruleSetForScope = RuleSet::create();
+ addRules(ruleSetForScope, authorRules.deepCombinatorOrShadowPseudoRules());
if (!isDocumentScope) {
- addRules(ruleSetForScope.get(), authorRules.contentPseudoElementRules());
- addRules(ruleSetForScope.get(), authorRules.slottedPseudoElementRules());
+ addRules(ruleSetForScope, authorRules.contentPseudoElementRules());
+ addRules(ruleSetForScope, authorRules.slottedPseudoElementRules());
}
if (!m_treeBoundaryCrossingRuleSet) {
@@ -230,7 +230,7 @@ void ScopedStyleResolver::addTreeBoundaryCrossingRules(const RuleSet& authorRule
treeScope().document().styleResolver()->addTreeBoundaryCrossingScope(treeScope().rootNode());
}
- m_treeBoundaryCrossingRuleSet->append(RuleSubSet::create(parentStyleSheet, sheetIndex, ruleSetForScope.release()));
+ m_treeBoundaryCrossingRuleSet->append(RuleSubSet::create(parentStyleSheet, sheetIndex, ruleSetForScope));
}
DEFINE_TRACE(ScopedStyleResolver::RuleSubSet)

Powered by Google App Engine
This is Rietveld 408576698