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

Side by Side Diff: sky/engine/core/css/resolver/SharedStyleFinder.cpp

Issue 1229273004: Remove Animations and Transitions. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years, 5 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
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 135 matching lines...) Expand 10 before | Expand all | Expand 10 after
146 } 146 }
147 return false; 147 return false;
148 } 148 }
149 149
150 inline Element* SharedStyleFinder::findElementForStyleSharing() const 150 inline Element* SharedStyleFinder::findElementForStyleSharing() const
151 { 151 {
152 StyleSharingList& styleSharingList = m_styleResolver.styleSharingList(); 152 StyleSharingList& styleSharingList = m_styleResolver.styleSharingList();
153 for (StyleSharingList::iterator it = styleSharingList.begin(); it != styleSh aringList.end(); ++it) { 153 for (StyleSharingList::iterator it = styleSharingList.begin(); it != styleSh aringList.end(); ++it) {
154 Element& candidate = **it; 154 Element& candidate = **it;
155 // We shouldn't have elements in the style sharing list that don't 155 // We shouldn't have elements in the style sharing list that don't
156 // support style sharing but we can end up with one currently if it gets 156 // support style sharing.
157 // added to the list and then it starts an animation. 157 if (!canShareStyleWithElement(candidate))
158 if (candidate.hasActiveAnimations() || !canShareStyleWithElement(candida te))
159 continue; 158 continue;
160 if (it != styleSharingList.begin()) { 159 if (it != styleSharingList.begin()) {
161 // Move the element to the front of the LRU 160 // Move the element to the front of the LRU
162 styleSharingList.remove(it); 161 styleSharingList.remove(it);
163 styleSharingList.prepend(&candidate); 162 styleSharingList.prepend(&candidate);
164 } 163 }
165 return &candidate; 164 return &candidate;
166 } 165 }
167 m_styleResolver.addToStyleSharingList(element()); 166 m_styleResolver.addToStyleSharingList(element());
168 return 0; 167 return 0;
(...skipping 22 matching lines...) Expand all
191 INCREMENT_STYLE_STATS_COUNTER(m_styleResolver, sharedStyleMissed); 190 INCREMENT_STYLE_STATS_COUNTER(m_styleResolver, sharedStyleMissed);
192 return 0; 191 return 0;
193 } 192 }
194 193
195 INCREMENT_STYLE_STATS_COUNTER(m_styleResolver, sharedStyleFound); 194 INCREMENT_STYLE_STATS_COUNTER(m_styleResolver, sharedStyleFound);
196 195
197 return shareElement->renderStyle(); 196 return shareElement->renderStyle();
198 } 197 }
199 198
200 } 199 }
OLDNEW
« no previous file with comments | « sky/engine/core/css/resolver/CSSToStyleMap.cpp ('k') | sky/engine/core/css/resolver/StyleAdjuster.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698