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

Unified Diff: sky/engine/core/dom/StyleEngine.cpp

Issue 775783002: Remove dead code from StyleEngine. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 6 years 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
« no previous file with comments | « sky/engine/core/dom/StyleEngine.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/core/dom/StyleEngine.cpp
diff --git a/sky/engine/core/dom/StyleEngine.cpp b/sky/engine/core/dom/StyleEngine.cpp
index 80de97a7caae6d810b9d2e1bcd9f3b9c148970e9..f38c4f42d5c8e3205b20d8922b0f2c626f5858cb 100644
--- a/sky/engine/core/dom/StyleEngine.cpp
+++ b/sky/engine/core/dom/StyleEngine.cpp
@@ -153,11 +153,6 @@ const Vector<RefPtr<StyleSheet> >& StyleEngine::styleSheetsForStyleSheetList(Tre
return ensureStyleSheetCollectionFor(treeScope)->styleSheetsForStyleSheetList();
}
-const Vector<RefPtr<CSSStyleSheet> >& StyleEngine::activeAuthorStyleSheets() const
-{
- return documentStyleSheetCollection()->activeAuthorStyleSheets();
-}
-
void StyleEngine::modifiedStyleSheet(StyleSheet* sheet)
{
if (!sheet)
@@ -189,11 +184,6 @@ void StyleEngine::addStyleSheetCandidateNode(Node* node, bool createdByParser)
insertTreeScopeInDocumentOrder(m_activeTreeScopes, &treeScope);
}
-void StyleEngine::removeStyleSheetCandidateNode(Node* node)
-{
- removeStyleSheetCandidateNode(node, 0, *m_document);
-}
-
void StyleEngine::removeStyleSheetCandidateNode(Node* node, ContainerNode* scopingNode, TreeScope& treeScope)
{
ASSERT(isHTMLStyleElement(node) || treeScope == m_document);
@@ -206,16 +196,6 @@ void StyleEngine::removeStyleSheetCandidateNode(Node* node, ContainerNode* scopi
m_activeTreeScopes.remove(&treeScope);
}
-void StyleEngine::modifiedStyleSheetCandidateNode(Node* node)
-{
- if (!node->inDocument())
- return;
-
- TreeScope& treeScope = isHTMLStyleElement(*node) ? node->treeScope() : *m_document;
- ASSERT(isHTMLStyleElement(node) || treeScope == m_document);
- markTreeScopeDirty(treeScope);
-}
-
bool StyleEngine::shouldUpdateDocumentStyleSheetCollection(StyleResolverUpdateMode updateMode) const
{
return m_documentScopeDirty || updateMode == FullStyleUpdate;
@@ -244,15 +224,6 @@ void StyleEngine::clearMediaQueryRuleSetStyleSheets()
clearMediaQueryRuleSetOnTreeScopeStyleSheets(m_dirtyTreeScopes);
}
-void StyleEngine::updateStyleSheetsInImport(DocumentStyleSheetCollector& parentCollector)
-{
- ASSERT(!isMaster());
- Vector<RefPtr<StyleSheet> > sheetsForList;
- ImportedDocumentStyleSheetCollector subcollector(parentCollector, sheetsForList);
- documentStyleSheetCollection()->collectStyleSheets(this, subcollector);
- documentStyleSheetCollection()->swapSheetsForSheetList(sheetsForList);
-}
-
void StyleEngine::updateActiveStyleSheets(StyleResolverUpdateMode updateMode)
{
ASSERT(isMaster());
@@ -286,28 +257,6 @@ void StyleEngine::updateActiveStyleSheets(StyleResolverUpdateMode updateMode)
m_documentScopeDirty = false;
}
-const Vector<RefPtr<CSSStyleSheet> > StyleEngine::activeStyleSheetsForInspector() const
-{
- if (m_activeTreeScopes.isEmpty())
- return documentStyleSheetCollection()->activeAuthorStyleSheets();
-
- Vector<RefPtr<CSSStyleSheet> > activeStyleSheets;
-
- activeStyleSheets.appendVector(documentStyleSheetCollection()->activeAuthorStyleSheets());
-
- TreeScopeSet::const_iterator begin = m_activeTreeScopes.begin();
- TreeScopeSet::const_iterator end = m_activeTreeScopes.end();
- for (TreeScopeSet::const_iterator it = begin; it != end; ++it) {
- if (TreeScopeStyleSheetCollection* collection = m_styleSheetCollectionMap.get(*it))
- activeStyleSheets.appendVector(collection->activeAuthorStyleSheets());
- }
-
- // FIXME: Inspector needs a vector which has all active stylesheets.
- // However, creating such a large vector might cause performance regression.
- // Need to implement some smarter solution.
- return activeStyleSheets;
-}
-
void StyleEngine::didRemoveShadowRoot(ShadowRoot* shadowRoot)
{
if (shadowRoot->scopedStyleResolver())
« no previous file with comments | « sky/engine/core/dom/StyleEngine.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698