Clear mutable inline style when it is empty.

Source/core/css/VariablesIterator.cpp

 /*
  * Copyright (C) 2013 Google Inc. 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.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  */
 
 #include "config.h"
 #include "core/css/VariablesIterator.h"
 
 #include "RuntimeEnabledFeatures.h"
 #include "core/css/CSSVariableValue.h"
 #include "core/css/StylePropertySet.h"
+#include "core/dom/Element.h"
 
 namespace WebCore {
 
-PassRefPtr<VariablesIterator> VariablesIterator::create(MutableStylePropertySet* propertySet)
+void AbstractVariablesIterator::initRemainingNames(
+    const StylePropertySet* propertySet)
 {
-    ASSERT(RuntimeEnabledFeatures::cssVariablesEnabled());
     const size_t propertyCount = propertySet->propertyCount();
     size_t variableCount = 0;
     Vector<AtomicString> remainingNames(propertyCount);
     for (int i = propertyCount; i--; ) {
         const StylePropertySet::PropertyReference& property = propertySet->propertyAt(i);
         if (property.id() == CSSPropertyVariable)
             remainingNames[variableCount++] = toCSSVariableValue(property.value())->name();
     }
+    // FIXME: Make use of the Vector move constructor when rvalues are supported on all platforms.
     remainingNames.shrink(variableCount);
 
-    RefPtr<VariablesIterator> iterator = adoptRef(new VariablesIterator(propertySet));
-    // FIXME: Make use of the Vector move constructor when rvalues are supported on all platforms.
-    iterator->takeRemainingNames(remainingNames);
-    return iterator.release();
+    takeRemainingNames(remainingNames);
+}
+
+VariablesIterator::VariablesIterator(MutableStylePropertySet* propertySet) :
+    m_propertySet(propertySet)
+{
+    ASSERT(RuntimeEnabledFeatures::cssVariablesEnabled());
+    initRemainingNames(propertySet);
+}
+
+PassRefPtr<VariablesIterator> VariablesIterator::create(MutableStylePropertySet* propertySet)
+{
+    return adoptRef(new VariablesIterator(propertySet));
 }
 
 String AbstractVariablesIterator::value() const
 {
     return propertySet()->variableValue(name());
 }
 
 void AbstractVariablesIterator::addedVariable(const AtomicString& name)
 {
     ASSERT(!m_remainingNames.contains(name));
@@ skipping 21 matching lines @@
 {
     if (!atEnd())
         m_remainingNames.removeLast();
     if (!m_newNames.isEmpty()) {
         m_remainingNames.appendVector(m_newNames);
         m_newNames.clear();
     }
 }
 
 } // namespace WebCore