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

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

Issue 2399633003: reflow comments in core/css/resolver (Closed)
Patch Set: Created 4 years, 2 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.
6 * All rights reserved.
6 * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org> 7 * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
7 * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org> 8 * 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/) 9 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
10 * (http://www.torchmobile.com/)
9 * Copyright (c) 2011, Code Aurora Forum. All rights reserved. 11 * Copyright (c) 2011, Code Aurora Forum. All rights reserved.
10 * Copyright (C) Research In Motion Limited 2011. All rights reserved. 12 * Copyright (C) Research In Motion Limited 2011. All rights reserved.
11 * Copyright (C) 2013 Google Inc. All rights reserved. 13 * Copyright (C) 2013 Google Inc. All rights reserved.
12 * 14 *
13 * This library is free software; you can redistribute it and/or 15 * This library is free software; you can redistribute it and/or
14 * modify it under the terms of the GNU Library General Public 16 * modify it under the terms of the GNU Library General Public
15 * License as published by the Free Software Foundation; either 17 * License as published by the Free Software Foundation; either
16 * version 2 of the License, or (at your option) any later version. 18 * version 2 of the License, or (at your option) any later version.
17 * 19 *
18 * This library is distributed in the hope that it will be useful, 20 * This library is distributed in the hope that it will be useful,
(...skipping 13 matching lines...) Expand all
32 #include "core/css/resolver/StyleResolverState.h" 34 #include "core/css/resolver/StyleResolverState.h"
33 #include "core/style/ComputedStyle.h" 35 #include "core/style/ComputedStyle.h"
34 36
35 namespace blink { 37 namespace blink {
36 38
37 void CachedMatchedProperties::set(const ComputedStyle& style, 39 void CachedMatchedProperties::set(const ComputedStyle& style,
38 const ComputedStyle& parentStyle, 40 const ComputedStyle& parentStyle,
39 const MatchedPropertiesVector& properties) { 41 const MatchedPropertiesVector& properties) {
40 matchedProperties.appendVector(properties); 42 matchedProperties.appendVector(properties);
41 43
42 // Note that we don't cache the original ComputedStyle instance. It may be fur ther modified. 44 // Note that we don't cache the original ComputedStyle instance. It may be
43 // The ComputedStyle in the cache is really just a holder for the substructure s and never used as-is. 45 // further modified. The ComputedStyle in the cache is really just a holder
46 // for the substructures and never used as-is.
44 this->computedStyle = ComputedStyle::clone(style); 47 this->computedStyle = ComputedStyle::clone(style);
45 this->parentComputedStyle = ComputedStyle::clone(parentStyle); 48 this->parentComputedStyle = ComputedStyle::clone(parentStyle);
46 } 49 }
47 50
48 void CachedMatchedProperties::clear() { 51 void CachedMatchedProperties::clear() {
49 matchedProperties.clear(); 52 matchedProperties.clear();
50 computedStyle = nullptr; 53 computedStyle = nullptr;
51 parentComputedStyle = nullptr; 54 parentComputedStyle = nullptr;
52 } 55 }
53 56
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
134 if (style.hasVariableReferenceFromNonInheritedProperty()) 137 if (style.hasVariableReferenceFromNonInheritedProperty())
135 return false; 138 return false;
136 return true; 139 return true;
137 } 140 }
138 141
139 DEFINE_TRACE(MatchedPropertiesCache) { 142 DEFINE_TRACE(MatchedPropertiesCache) {
140 visitor->trace(m_cache); 143 visitor->trace(m_cache);
141 } 144 }
142 145
143 } // namespace blink 146 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698