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

Side by Side Diff: third_party/WebKit/Source/core/dom/StyleEngine.cpp

Issue 2829873002: Collect media query results for non-matching stylesheets. (Closed)
Patch Set: Created 3 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 unified diff | Download patch
« no previous file with comments | « third_party/WebKit/Source/core/css/CSSStyleSheet.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2001 Dirk Mueller (mueller@kde.org) 4 * (C) 2001 Dirk Mueller (mueller@kde.org)
5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org) 5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org)
6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All
7 * rights reserved. 7 * rights reserved.
8 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. 8 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
9 * (http://www.torchmobile.com/) 9 * (http://www.torchmobile.com/)
10 * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved. 10 * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
(...skipping 1081 matching lines...) Expand 10 before | Expand all | Expand 10 after
1092 return; 1092 return;
1093 1093
1094 if (!tree_scope.GetDocument().body() || 1094 if (!tree_scope.GetDocument().body() ||
1095 tree_scope.GetDocument().HasNodesWithPlaceholderStyle()) { 1095 tree_scope.GetDocument().HasNodesWithPlaceholderStyle()) {
1096 tree_scope.GetDocument().SetNeedsStyleRecalc( 1096 tree_scope.GetDocument().SetNeedsStyleRecalc(
1097 kSubtreeStyleChange, StyleChangeReasonForTracing::Create( 1097 kSubtreeStyleChange, StyleChangeReasonForTracing::Create(
1098 StyleChangeReason::kCleanupPlaceholderStyles)); 1098 StyleChangeReason::kCleanupPlaceholderStyles));
1099 return; 1099 return;
1100 } 1100 }
1101 1101
1102 if (changed_rule_sets.IsEmpty())
1103 return;
1104
1102 if (changed_rule_flags & kKeyframesRules) 1105 if (changed_rule_flags & kKeyframesRules)
1103 ScopedStyleResolver::KeyframesRulesAdded(tree_scope); 1106 ScopedStyleResolver::KeyframesRulesAdded(tree_scope);
1104 1107
1105 Node& invalidation_root = 1108 Node& invalidation_root =
1106 ScopedStyleResolver::InvalidationRootForTreeScope(tree_scope); 1109 ScopedStyleResolver::InvalidationRootForTreeScope(tree_scope);
1107 if (invalidation_root.GetStyleChangeType() >= kSubtreeStyleChange) 1110 if (invalidation_root.GetStyleChangeType() >= kSubtreeStyleChange)
1108 return; 1111 return;
1109 1112
1110 if (fonts_changed || (changed_rule_flags & kFullRecalcRules)) { 1113 if (fonts_changed || (changed_rule_flags & kFullRecalcRules)) {
1111 invalidation_root.SetNeedsStyleRecalc( 1114 invalidation_root.SetNeedsStyleRecalc(
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
1172 } 1175 }
1173 1176
1174 DEFINE_TRACE_WRAPPERS(StyleEngine) { 1177 DEFINE_TRACE_WRAPPERS(StyleEngine) {
1175 for (auto sheet : injected_author_style_sheets_) { 1178 for (auto sheet : injected_author_style_sheets_) {
1176 visitor->TraceWrappers(sheet); 1179 visitor->TraceWrappers(sheet);
1177 } 1180 }
1178 visitor->TraceWrappers(document_style_sheet_collection_); 1181 visitor->TraceWrappers(document_style_sheet_collection_);
1179 } 1182 }
1180 1183
1181 } // namespace blink 1184 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/css/CSSStyleSheet.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698