Partial implementation of inline StylePropertyMap.

third_party/WebKit/Source/core/css/cssom/StylePropertyMap.cpp

 // Copyright 2016 the chromium authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/css/cssom/StylePropertyMap.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "core/css/cssom/SimpleLength.h"
 #include "core/css/cssom/StyleValue.h"
 
 namespace blink {
 
 StyleValue* StylePropertyMap::get(const String& propertyName)
 {
     return get(cssPropertyID(propertyName));
 }
 
 HeapVector<Member<StyleValue>> StylePropertyMap::getAll(const String& propertyName)
 {
-    return getAll(cssPropertyID(propertyName));
+    CSSPropertyID propertyID = cssPropertyID(propertyName);
+    if (propertyID != CSSPropertyInvalid) {
+        return getAll(propertyID);
+    }
+    // TODO(meade): Handle custom properties here.
+    return HeapVector<Member<StyleValue>>();
 }
 
 bool StylePropertyMap::has(const String& propertyName)
 {
-    return has(cssPropertyID(propertyName));
+    CSSPropertyID propertyID = cssPropertyID(propertyName);
+    if (propertyID != CSSPropertyInvalid) {
+        return has(cssPropertyID(propertyName));

[Timothy Loh, 2016/02/09 07:29:32]

You probably meant to use propertyID here and all the other functions

[meade_UTC10, 2016/02/10 05:55:39]

Done.

+    }
+    // TODO(meade): Handle custom properties here.
+    return false;
 }
 
 void StylePropertyMap::set(const String& propertyName, StyleValueOrStyleValueSequenceOrString& item, ExceptionState& exceptionState)
 {
-    set(cssPropertyID(propertyName), item, exceptionState);
+    CSSPropertyID propertyID = cssPropertyID(propertyName);
+    if (propertyID != CSSPropertyInvalid) {
+        set(cssPropertyID(propertyName), item, exceptionState);
+        return;
+    }
+    // TODO(meade): Handle custom properties here.
+    exceptionState.throwTypeError("Invalid propertyName: " + propertyName);
 }
 
 void StylePropertyMap::append(const String& propertyName, StyleValueOrStyleValueSequenceOrString& item, ExceptionState& exceptionState)
 {
-    append(cssPropertyID(propertyName), item, exceptionState);
+    CSSPropertyID propertyID = cssPropertyID(propertyName);
+    if (propertyID != CSSPropertyInvalid) {
+        append(cssPropertyID(propertyName), item, exceptionState);
+        return;
+    }
+    // TODO(meade): Handle custom properties here.
+    exceptionState.throwTypeError("Invalid propertyName: " + propertyName);
 }
 
 void StylePropertyMap::remove(const String& propertyName, ExceptionState& exceptionState)
 {
-    remove(cssPropertyID(propertyName), exceptionState);
+    CSSPropertyID propertyID = cssPropertyID(propertyName);
+    if (propertyID != CSSPropertyInvalid) {
+        remove(cssPropertyID(propertyName), exceptionState);
+        return;
+    }
+    // TODO(meade): Handle custom properties here.
+    exceptionState.throwTypeError("Invalid propertyName: " + propertyName);
 }
 
 } // namespace blink