[CSSTypedOM] Computed StylePropertyMap use ComputedStyle for Lengths

third_party/WebKit/Source/core/css/CSSComputedStyleDeclaration.h

Index: third_party/WebKit/Source/core/css/CSSComputedStyleDeclaration.h
diff --git a/third_party/WebKit/Source/core/css/CSSComputedStyleDeclaration.h b/third_party/WebKit/Source/core/css/CSSComputedStyleDeclaration.h
index 9a67d5d95471403920880f7c0d22751f968cdb5c..9684b45c5b2ae8456da9cb268e309e791214306d 100644
--- a/third_party/WebKit/Source/core/css/CSSComputedStyleDeclaration.h
+++ b/third_party/WebKit/Source/core/css/CSSComputedStyleDeclaration.h
@@ -66,6 +66,12 @@ class CORE_EXPORT CSSComputedStyleDeclaration final
   MutableStylePropertySet* copyPropertiesInSet(
       const Vector<CSSPropertyID>&) const;
 
+  // CSSOM functions.
+  unsigned length() const override;
+  String item(unsigned index) const override;
+  const CSSValue* getPropertyCSSValueInternal(

[meade_UTC10, 2017/01/05 03:43:45]

It doesn't seem like the getPropertyCSSValueInternal method should be made
public. AFAICT getPropertyCSSValueInternal actually just calls
getPropertyCSSValue(CSSPropertyID), could you just use that instead?

+      AtomicString customPropertyName) override;
+
   DECLARE_VIRTUAL_TRACE();
 
  private:
@@ -78,10 +84,8 @@ class CORE_EXPORT CSSComputedStyleDeclaration final
   // fix that.
   Node* styledNode() const;
 
-  // CSSOM functions. Don't make these public.
+  // CSSOM functions.
   CSSRule* parentRule() const override;
-  unsigned length() const override;
-  String item(unsigned index) const override;
   const ComputedStyle* computeComputedStyle() const;
   String getPropertyValue(const String& propertyName) override;
   String getPropertyPriority(const String& propertyName) override;
@@ -97,8 +101,6 @@ class CORE_EXPORT CSSComputedStyleDeclaration final
   String cssText() const override;
   void setCSSText(const String&, ExceptionState&) override;
   const CSSValue* getPropertyCSSValueInternal(CSSPropertyID) override;
-  const CSSValue* getPropertyCSSValueInternal(
-      AtomicString customPropertyName) override;
   String getPropertyValueInternal(CSSPropertyID) override;
   void setPropertyInternal(CSSPropertyID,
                            const String& customPropertyName,