Clear mutable inline style when it is empty.

Source/core/css/StylePropertySet.cpp

 /*
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc. All rights reserved.
  * Copyright (C) 2011 Research In Motion Limited. All rights reserved.
  * Copyright (C) 2013 Intel Corporation. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
@@ skipping 100 matching lines @@
 {
     RefPtr<CSSValue> value = getPropertyCSSValue(propertyID);
     if (value)
         return value->cssText();
 
     return StylePropertySerializer(*this).getPropertyValue(propertyID);
 }
 
 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
 {
     int foundPropertyIndex = findPropertyIndex(propertyID);
     if (foundPropertyIndex == -1)
         return 0;
     return propertyAt(foundPropertyIndex).value();
 }
 
@@ skipping 521 matching lines @@
     result.appendLiteral(": ");
     result.append(propertyValue()->cssText());
     if (isImportant())
         result.appendLiteral(" !important");
     result.append(';');
     return result.toString();
 }
 
 
 } // namespace WebCore