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

Side by Side Diff: Source/core/css/resolver/StyleResolver.cpp

Issue 262643002: Removed unused firstNew parameter for appendAuthorStyleSheets. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 7 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 | « Source/core/css/resolver/StyleResolver.h ('k') | Source/core/dom/StyleEngine.cpp » ('j') | 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) 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. All rights reserved. 5 * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved.
6 * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org> 6 * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
7 * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org> 7 * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
8 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.t orchmobile.com/) 8 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.t orchmobile.com/)
9 * Copyright (c) 2011, Code Aurora Forum. All rights reserved. 9 * Copyright (c) 2011, Code Aurora Forum. All rights reserved.
10 * Copyright (C) Research In Motion Limited 2011. All rights reserved. 10 * Copyright (C) Research In Motion Limited 2011. All rights reserved.
(...skipping 186 matching lines...) Expand 10 before | Expand all | Expand 10 after
197 void StyleResolver::appendPendingAuthorStyleSheets() 197 void StyleResolver::appendPendingAuthorStyleSheets()
198 { 198 {
199 setBuildScopedStyleTreeInDocumentOrder(false); 199 setBuildScopedStyleTreeInDocumentOrder(false);
200 for (ListHashSet<CSSStyleSheet*, 16>::iterator it = m_pendingStyleSheets.beg in(); it != m_pendingStyleSheets.end(); ++it) 200 for (ListHashSet<CSSStyleSheet*, 16>::iterator it = m_pendingStyleSheets.beg in(); it != m_pendingStyleSheets.end(); ++it)
201 appendCSSStyleSheet(*it); 201 appendCSSStyleSheet(*it);
202 202
203 m_pendingStyleSheets.clear(); 203 m_pendingStyleSheets.clear();
204 finishAppendAuthorStyleSheets(); 204 finishAppendAuthorStyleSheets();
205 } 205 }
206 206
207 void StyleResolver::appendAuthorStyleSheets(unsigned firstNew, const WillBeHeapV ector<RefPtrWillBeMember<CSSStyleSheet> >& styleSheets) 207 void StyleResolver::appendAuthorStyleSheets(const WillBeHeapVector<RefPtrWillBeM ember<CSSStyleSheet> >& styleSheets)
208 { 208 {
209 // This handles sheets added to the end of the stylesheet list only. In othe r cases the style resolver 209 // This handles sheets added to the end of the stylesheet list only. In othe r cases the style resolver
210 // needs to be reconstructed. To handle insertions too the rule order number s would need to be updated. 210 // needs to be reconstructed. To handle insertions too the rule order number s would need to be updated.
211 unsigned size = styleSheets.size(); 211 unsigned size = styleSheets.size();
212 for (unsigned i = firstNew; i < size; ++i) 212 for (unsigned i = 0; i < size; ++i)
213 appendCSSStyleSheet(styleSheets[i].get()); 213 appendCSSStyleSheet(styleSheets[i].get());
214 } 214 }
215 215
216 void StyleResolver::finishAppendAuthorStyleSheets() 216 void StyleResolver::finishAppendAuthorStyleSheets()
217 { 217 {
218 collectFeatures(); 218 collectFeatures();
219 219
220 if (document().renderer() && document().renderer()->style()) 220 if (document().renderer() && document().renderer()->style())
221 document().renderer()->style()->font().update(document().styleEngine()-> fontSelector()); 221 document().renderer()->style()->font().update(document().styleEngine()-> fontSelector());
222 222
(...skipping 1137 matching lines...) Expand 10 before | Expand all | Expand 10 after
1360 visitor->trace(m_keyframesRuleMap); 1360 visitor->trace(m_keyframesRuleMap);
1361 visitor->trace(m_viewportDependentMediaQueryResults); 1361 visitor->trace(m_viewportDependentMediaQueryResults);
1362 visitor->trace(m_viewportStyleResolver); 1362 visitor->trace(m_viewportStyleResolver);
1363 visitor->trace(m_siblingRuleSet); 1363 visitor->trace(m_siblingRuleSet);
1364 visitor->trace(m_uncommonAttributeRuleSet); 1364 visitor->trace(m_uncommonAttributeRuleSet);
1365 visitor->trace(m_watchedSelectorsRules); 1365 visitor->trace(m_watchedSelectorsRules);
1366 visitor->trace(m_treeBoundaryCrossingRules); 1366 visitor->trace(m_treeBoundaryCrossingRules);
1367 } 1367 }
1368 1368
1369 } // namespace WebCore 1369 } // namespace WebCore
OLDNEW
« no previous file with comments | « Source/core/css/resolver/StyleResolver.h ('k') | Source/core/dom/StyleEngine.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698