Add forEach() to CSSVariablesMap

Source/core/css/CSSVariablesMap.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 56 matching lines @@
         return m_styleDeclaration->removeVariable(name);
     return false;
 }
 
 void CSSVariablesMap::clear(ExceptionState& es) const
 {
     if (m_styleDeclaration)
         return m_styleDeclaration->clearVariables(es);
 }
 
+void CSSVariablesMap::forEach(PassRefPtr<CSSVariablesMapForEachCallback> callback, ScriptValue* thisArg) const
+{
+    if (!m_styleDeclaration)
+        return;
+    RefPtr<CSSVariablesIterator> iterator = m_styleDeclaration->variablesIterator();

[esprehn, 2013/09/12 01:13:42]

Is the iterator really necessary? It'd be much nicer if you used a normal C++
iterator.

[alancutter (OOO until 2018), 2013/09/13 03:06:14]

After the changes to remove the static map of iterators from this patch I needed
to add some methods to the iterator class. I think this prevents me from using a
regular C++ iterator interface.

+    while (!iterator->atEnd()) {
+        String name = iterator->name();
+        String value = iterator->value();
+        if (thisArg)
+            callback->handleItem(*thisArg, value, name, const_cast<CSSVariablesMap*>(this));
+        else
+            callback->handleItem(value, name, const_cast<CSSVariablesMap*>(this));
+        iterator->advance();
+    }
+}
+
+void CSSVariablesMap::forEach(PassRefPtr<CSSVariablesMapForEachCallback> callback, ScriptValue thisArg) const
+{
+    forEach(callback, &thisArg);
+}
+
+void CSSVariablesMap::forEach(PassRefPtr<CSSVariablesMapForEachCallback> callback) const
+{
+    forEach(callback, 0);
+}
+
 } // namespace WebCore