Remove OptionalColor
authordino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 12 Sep 2016 22:16:24 +0000 (22:16 +0000)
committerdino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 12 Sep 2016 22:16:24 +0000 (22:16 +0000)
https://bugs.webkit.org/show_bug.cgi?id=161853
<rdar://problem/28252385>

Reviewed by Alex Christensen.

OptionalColor isn't used. The concept will be implemented
in a different manner.

* platform/graphics/Color.h:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@205831 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/Color.h

index 2e84ccf..a383459 100644 (file)
@@ -1,3 +1,16 @@
+2016-09-12  Dean Jackson  <dino@apple.com>
+
+        Remove OptionalColor
+        https://bugs.webkit.org/show_bug.cgi?id=161853
+        <rdar://problem/28252385>
+
+        Reviewed by Alex Christensen.
+
+        OptionalColor isn't used. The concept will be implemented
+        in a different manner.
+
+        * platform/graphics/Color.h:
+
 2016-09-12  Simon Fraser  <simon.fraser@apple.com>
 
         Add Intersection Observer to the features.json file.
index dd26270..cc5d82a 100644 (file)
@@ -194,20 +194,6 @@ private:
     bool m_valid;
 };
 
-class OptionalColor : public Color {
-public:
-    OptionalColor();
-    OptionalColor(const Color&);
-
-    explicit operator bool() const;
-
-private:
-    // FIXME: Change to use Optional<Color>?
-    // FIXME: Convert all callers to use Optional<Color>?
-    bool m_isEngaged;
-    Color m_color;
-};
-
 bool operator==(const Color&, const Color&);
 bool operator!=(const Color&, const Color&);