Add registered custom property wrapping support to CSSAnimatableValueFactory

third_party/WebKit/Source/core/animation/animatable/AnimatableUnknown.h

 /*
  * 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 23 matching lines @@
 #include "core/CSSValueKeywords.h"
 #include "core/animation/animatable/AnimatableValue.h"
 #include "core/css/CSSIdentifierValue.h"
 
 namespace blink {
 
 class AnimatableUnknown final : public AnimatableValue {
  public:
   ~AnimatableUnknown() override {}
 
-  static PassRefPtr<AnimatableUnknown> create(CSSValue* value) {
+  static PassRefPtr<AnimatableUnknown> create(const CSSValue* value) {
     return adoptRef(new AnimatableUnknown(value));
   }
   static PassRefPtr<AnimatableUnknown> create(CSSValueID value) {
     return adoptRef(new AnimatableUnknown(CSSIdentifierValue::create(value)));
   }
 
-  CSSValue* toCSSValue() const { return m_value; }
+  const CSSValue* toCSSValue() const { return m_value; }
   CSSValueID toCSSValueID() const {
     return toCSSIdentifierValue(m_value.get())->getValueID();
   }
 
  protected:
   PassRefPtr<AnimatableValue> interpolateTo(const AnimatableValue* value,
                                             double fraction) const override {
     return defaultInterpolateTo(this, value, fraction);
   }
 
   bool usesDefaultInterpolationWith(const AnimatableValue*) const override;
 
  private:
-  explicit AnimatableUnknown(CSSValue* value) : m_value(value) {
-    DCHECK(m_value);
-  }
+  explicit AnimatableUnknown(const CSSValue* value) : m_value(value) {}
   AnimatableType type() const override { return TypeUnknown; }
   bool equalTo(const AnimatableValue*) const override;
 
-  const Persistent<CSSValue> m_value;
+  const Persistent<const CSSValue> m_value;
 };
 
 DEFINE_ANIMATABLE_VALUE_TYPE_CASTS(AnimatableUnknown, isUnknown());
 
 inline bool AnimatableUnknown::equalTo(const AnimatableValue* value) const {
   const AnimatableUnknown* unknown = toAnimatableUnknown(value);
-  return m_value == unknown->m_value || m_value->equals(*unknown->m_value);
+  return compareCSSValuePtr(m_value.get(), unknown->m_value.get());
 }
 
 inline bool AnimatableUnknown::usesDefaultInterpolationWith(
     const AnimatableValue* value) const {
-  const AnimatableUnknown& unknown = toAnimatableUnknown(*value);
-  return !m_value->equals(*unknown.m_value);
+  return equalTo(value);

[Eric Willigers, 2017/03/04 04:16:31]

Why the change from not equals to equals?

[alancutter (OOO until 2018), 2017/03/05 23:23:57]

Whoops, fixed.

 }
 
 }  // namespace blink
 
 #endif  // AnimatableUnknown_h