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

Side by Side Diff: third_party/WebKit/Source/core/css/resolver/MatchedPropertiesCache.cpp

Issue 2566403003: Migrate WTF::Vector::append() to ::push_back() [part 3 of N] (Closed)
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com) 3 * (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
4 * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com) 4 * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
5 * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. 5 * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc.
6 * All rights reserved. 6 * All rights reserved.
7 * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org> 7 * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
8 * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org> 8 * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
9 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. 9 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
10 * (http://www.torchmobile.com/) 10 * (http://www.torchmobile.com/)
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 cacheEntry.value->clear(); 105 cacheEntry.value->clear();
106 } 106 }
107 m_cache.clear(); 107 m_cache.clear();
108 } 108 }
109 109
110 void MatchedPropertiesCache::clearViewportDependent() { 110 void MatchedPropertiesCache::clearViewportDependent() {
111 Vector<unsigned, 16> toRemove; 111 Vector<unsigned, 16> toRemove;
112 for (const auto& cacheEntry : m_cache) { 112 for (const auto& cacheEntry : m_cache) {
113 CachedMatchedProperties* cacheItem = cacheEntry.value.get(); 113 CachedMatchedProperties* cacheItem = cacheEntry.value.get();
114 if (cacheItem->computedStyle->hasViewportUnits()) 114 if (cacheItem->computedStyle->hasViewportUnits())
115 toRemove.append(cacheEntry.key); 115 toRemove.push_back(cacheEntry.key);
116 } 116 }
117 m_cache.removeAll(toRemove); 117 m_cache.removeAll(toRemove);
118 } 118 }
119 119
120 bool MatchedPropertiesCache::isCacheable(const StyleResolverState& state) { 120 bool MatchedPropertiesCache::isCacheable(const StyleResolverState& state) {
121 const ComputedStyle& style = *state.style(); 121 const ComputedStyle& style = *state.style();
122 const ComputedStyle& parentStyle = *state.parentStyle(); 122 const ComputedStyle& parentStyle = *state.parentStyle();
123 123
124 if (style.unique() || 124 if (style.unique() ||
125 (style.styleType() != PseudoIdNone && parentStyle.unique())) 125 (style.styleType() != PseudoIdNone && parentStyle.unique()))
(...skipping 11 matching lines...) Expand all
137 if (style.hasVariableReferenceFromNonInheritedProperty()) 137 if (style.hasVariableReferenceFromNonInheritedProperty())
138 return false; 138 return false;
139 return true; 139 return true;
140 } 140 }
141 141
142 DEFINE_TRACE(MatchedPropertiesCache) { 142 DEFINE_TRACE(MatchedPropertiesCache) {
143 visitor->trace(m_cache); 143 visitor->trace(m_cache);
144 } 144 }
145 145
146 } // namespace blink 146 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698