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));
+    CSSPropertyID propertyID = cssPropertyID(propertyName);
+    if (propertyID != CSSPropertyInvalid) {
+        return get(propertyID);
+    }
+    // TODO(meade): Handle custom properties here.
+    return nullptr;
 }
 
 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(propertyID);
+    }
+    // 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(propertyID, item, exceptionState);
+        return;
+    }
+    // TODO(meade): Handle custom properties here.
+    exceptionState.throwTypeError("Invalid propertyName: " + propertyName);

[Timothy Loh, 2016/03/23 03:45:09]

Why do only some of these have exceptions? Are we going to add them to the other
functions later?

[meade_UTC10, 2016/03/29 06:27:37]

Dunno. Probably assumed that setting something that doesn't exist is an error,
but you can just return null for getting something that doesn't exist. Added
exceptions.

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