Clear mutable inline style when it is empty.

Source/core/css/StylePropertySet.cpp

Index: Source/core/css/StylePropertySet.cpp
diff --git a/Source/core/css/StylePropertySet.cpp b/Source/core/css/StylePropertySet.cpp
index eb340403236e94d191a43671e6788b1f4073537a..c250710f15a2388a15537fa7f4c4f8a41c2d9f5f 100644
--- a/Source/core/css/StylePropertySet.cpp
+++ b/Source/core/css/StylePropertySet.cpp
@@ -118,7 +118,7 @@ String StylePropertySet::getPropertyValue(CSSPropertyID propertyID) const
 
 PassRefPtr<CSSVariablesIterator> MutableStylePropertySet::variablesIterator()
 {
-    return VariablesIterator::create(this);
+    return new VariablesIterator(this);

[eseidel, 2013/12/23 23:26:28]

Oh. The ::create() pattern is still perfered (since we're not correctly calling
adoptRef here!)  It's just create methods are generally mostly empty:

static PassRefPtr<Foo> Foo::create(Bar*)
{
   return adoptRef(new Foo(bar));
}

the adoptRef is important, because RefCounted subclasses start with a single
ref, which is adopted by adoptRef() and placed into a PassRefPtr.

[chrishtr, 2013/12/23 23:54:51]

Done.

 }
 
 PassRefPtr<CSSValue> StylePropertySet::getPropertyCSSValue(CSSPropertyID propertyID) const