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

Unified Diff: Source/core/css/StylePropertySet.h

Issue 188693002: Revert of Move all RefPtr's to CSSValue to oilpan transition types, except for the one in CSSCursorImageValue. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/css/StylePropertySerializer.cpp ('k') | Source/core/css/StylePropertySet.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/css/StylePropertySet.h
diff --git a/Source/core/css/StylePropertySet.h b/Source/core/css/StylePropertySet.h
index 549cc9888c47fba3738fb999e6691c980e266210..50087aa3b2e7c5ba6c934eadbee7723791eb94db 100644
--- a/Source/core/css/StylePropertySet.h
+++ b/Source/core/css/StylePropertySet.h
@@ -41,20 +41,12 @@
class StylePropertyShorthand;
class StyleSheetContents;
-class StylePropertySet : public RefCountedWillBeRefCountedGarbageCollected<StylePropertySet> {
+class StylePropertySet : public RefCounted<StylePropertySet> {
friend class PropertyReference;
public:
-
-#if ENABLE(OILPAN)
- // When oilpan is enabled override the finalize method to dispatch to the subclasses'
- // destructor. This can be removed once the MutableStylePropertySet's OwnPtr is moved
- // to the heap.
- void finalize();
-#else
// Override RefCounted's deref() to ensure operator delete is called on
// the appropriate subclass type.
void deref();
-#endif
class PropertyReference {
public:
@@ -125,9 +117,6 @@
bool propertyMatches(CSSPropertyID, const CSSValue*) const;
- void trace(Visitor*);
- void traceAfterDispatch(Visitor*) { }
-
protected:
enum { MaxArraySize = (1 << 28) - 1 };
@@ -158,16 +147,9 @@
unsigned propertyCount() const { return m_arraySize; }
- const RawPtrWillBeMember<CSSValue>* valueArray() const;
+ const CSSValue** valueArray() const;
const StylePropertyMetadata* metadataArray() const;
int findPropertyIndex(CSSPropertyID) const;
-
- void traceAfterDispatch(Visitor*);
-
- void* operator new(std::size_t, void* location)
- {
- return location;
- }
void* m_storage;
@@ -175,14 +157,14 @@
ImmutableStylePropertySet(const CSSProperty*, unsigned count, CSSParserMode);
};
-inline const RawPtrWillBeMember<CSSValue>* ImmutableStylePropertySet::valueArray() const
-{
- return reinterpret_cast<const RawPtrWillBeMember<CSSValue>*>(const_cast<const void**>(&(this->m_storage)));
+inline const CSSValue** ImmutableStylePropertySet::valueArray() const
+{
+ return reinterpret_cast<const CSSValue**>(const_cast<const void**>(&(this->m_storage)));
}
inline const StylePropertyMetadata* ImmutableStylePropertySet::metadataArray() const
{
- return reinterpret_cast<const StylePropertyMetadata*>(&reinterpret_cast<const char*>(&(this->m_storage))[m_arraySize * sizeof(RawPtrWillBeMember<CSSValue>)]);
+ return reinterpret_cast<const StylePropertyMetadata*>(&reinterpret_cast<const char*>(&(this->m_storage))[m_arraySize * sizeof(CSSValue*)]);
}
DEFINE_TYPE_CASTS(ImmutableStylePropertySet, StylePropertySet, set, !set->isMutable(), !set.isMutable());
@@ -200,7 +182,7 @@
unsigned propertyCount() const { return m_propertyVector.size(); }
- void addParsedProperties(const WillBeHeapVector<CSSProperty, 256>&);
+ void addParsedProperties(const Vector<CSSProperty, 256>&);
void addParsedProperty(const CSSProperty&);
// These expand shorthand properties into multiple properties.
@@ -229,8 +211,6 @@
CSSStyleDeclaration* ensureCSSStyleDeclaration();
int findPropertyIndex(CSSPropertyID) const;
- void traceAfterDispatch(Visitor*);
-
private:
explicit MutableStylePropertySet(CSSParserMode);
explicit MutableStylePropertySet(const StylePropertySet&);
@@ -242,7 +222,7 @@
friend class StylePropertySet;
- WillBeHeapVector<CSSProperty, 4> m_propertyVector;
+ Vector<CSSProperty, 4> m_propertyVector;
};
DEFINE_TYPE_CASTS(MutableStylePropertySet, StylePropertySet, set, set->isMutable(), set.isMutable());
@@ -278,7 +258,6 @@
return !propertyCount();
}
-#if !ENABLE(OILPAN)
inline void StylePropertySet::deref()
{
if (!derefBase())
@@ -289,7 +268,6 @@
else
delete toImmutableStylePropertySet(this);
}
-#endif // !ENABLE(OILPAN)
inline int StylePropertySet::findPropertyIndex(CSSPropertyID propertyID) const
{
« no previous file with comments | « Source/core/css/StylePropertySerializer.cpp ('k') | Source/core/css/StylePropertySet.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698