Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(722)

Unified Diff: third_party/WebKit/Source/core/animation/animatable/AnimatableColor.h

Issue 2750293003: Delete unused AnimatableValue code (Closed)
Patch Set: Fix unit tests Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/animation/animatable/AnimatableColor.h
diff --git a/third_party/WebKit/Source/core/animation/animatable/AnimatableColor.h b/third_party/WebKit/Source/core/animation/animatable/AnimatableColor.h
index 83b5540a79e5225c520b17fefd7d5a9028299fc5..32e55c6a90948d8442df1b0eaf9686fc9a71d909 100644
--- a/third_party/WebKit/Source/core/animation/animatable/AnimatableColor.h
+++ b/third_party/WebKit/Source/core/animation/animatable/AnimatableColor.h
@@ -38,15 +38,12 @@
namespace blink {
-class CORE_EXPORT AnimatableColorImpl {
+class AnimatableColorImpl {
DISALLOW_NEW();
public:
AnimatableColorImpl(float red, float green, float blue, float alpha);
AnimatableColorImpl(Color);
- Color toColor() const;
- AnimatableColorImpl interpolateTo(const AnimatableColorImpl&,
- double fraction) const;
bool operator==(const AnimatableColorImpl&) const;
private:
@@ -60,17 +57,11 @@ class CORE_EXPORT AnimatableColorImpl {
// property. Currently it is used for all properties, even those which do not
// support a separate 'visited link' color (eg SVG properties). This is correct
// but inefficient.
-class CORE_EXPORT AnimatableColor final : public AnimatableValue {
+class AnimatableColor final : public AnimatableValue {
public:
static PassRefPtr<AnimatableColor> create(
const AnimatableColorImpl&,
const AnimatableColorImpl& visitedLinkColor);
- Color getColor() const { return m_color.toColor(); }
- Color visitedLinkColor() const { return m_visitedLinkColor.toColor(); }
-
- protected:
- PassRefPtr<AnimatableValue> interpolateTo(const AnimatableValue*,
- double fraction) const override;
private:
AnimatableColor(const AnimatableColorImpl& color,

Powered by Google App Engine
This is Rietveld 408576698