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

Unified Diff: sky/engine/core/css/resolver/StyleResolver.cpp

Issue 848483003: Simplify media query handling for stylesheets. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 11 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
« no previous file with comments | « sky/engine/core/css/resolver/StyleResolver.h ('k') | sky/engine/core/dom/StyleEngine.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/core/css/resolver/StyleResolver.cpp
diff --git a/sky/engine/core/css/resolver/StyleResolver.cpp b/sky/engine/core/css/resolver/StyleResolver.cpp
index 20fb48afa186f7ab34e71361a7ff75cec5edd0e6..ae0a105825120f6a1ccf66f29e78ab31cb27fe4c 100644
--- a/sky/engine/core/css/resolver/StyleResolver.cpp
+++ b/sky/engine/core/css/resolver/StyleResolver.cpp
@@ -55,7 +55,6 @@
#include "sky/engine/core/css/parser/BisonCSSParser.h"
#include "sky/engine/core/css/resolver/AnimatedStyleBuilder.h"
#include "sky/engine/core/css/resolver/MatchResult.h"
-#include "sky/engine/core/css/resolver/MediaQueryResult.h"
#include "sky/engine/core/css/resolver/SharedStyleFinder.h"
#include "sky/engine/core/css/resolver/StyleAdjuster.h"
#include "sky/engine/core/css/resolver/StyleBuilder.h"
@@ -118,18 +117,10 @@ static RuleSet& defaultStyles()
StyleResolver::StyleResolver(Document& document)
: m_document(document)
- , m_printMediaType(false)
, m_styleResourceLoader(document.fetcher())
, m_styleResolverStatsSequence(0)
, m_accessCount(0)
{
- FrameView* view = document.view();
- if (view) {
- m_medium = adoptPtr(new MediaQueryEvaluator(&view->frame()));
- m_printMediaType = equalIgnoringCase(view->mediaType(), MediaTypeNames::print);
- } else {
- m_medium = adoptPtr(new MediaQueryEvaluator("all"));
- }
}
void StyleResolver::addToStyleSharingList(Element& element)
@@ -704,23 +695,4 @@ void StyleResolver::applyPropertiesToStyle(const CSSPropertyValue* properties, s
}
}
-void StyleResolver::resetMediaQueryAffectedByViewportChange()
-{
- m_viewportDependentMediaQueryResults.clear();
-}
-
-void StyleResolver::addMediaQueryAffectedByViewportChange(const MediaQueryResultList& list)
-{
- m_viewportDependentMediaQueryResults.appendVector(list);
-}
-
-bool StyleResolver::mediaQueryAffectedByViewportChange() const
-{
- for (unsigned i = 0; i < m_viewportDependentMediaQueryResults.size(); ++i) {
- if (m_medium->eval(m_viewportDependentMediaQueryResults[i]->expression()) != m_viewportDependentMediaQueryResults[i]->result())
- return true;
- }
- return false;
-}
-
} // namespace blink
« no previous file with comments | « sky/engine/core/css/resolver/StyleResolver.h ('k') | sky/engine/core/dom/StyleEngine.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698