Remove unused Color::getHSV function
authorweinig@apple.com <weinig@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 May 2020 11:17:06 +0000 (11:17 +0000)
committerweinig@apple.com <weinig@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 May 2020 11:17:06 +0000 (11:17 +0000)
https://bugs.webkit.org/show_bug.cgi?id=212119

Reviewed by Simon Fraser.

Source/WebCore:

* platform/graphics/Color.cpp:
(WebCore::Color::getHSV const): Deleted.
* platform/graphics/Color.h:
Remove Color::getHSV(). It was unused outside of the API test for it.

Tools:

* TestWebKitAPI/Tests/WebCore/ColorTests.cpp:
Remove tests for Color::getHSV().

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

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/Color.cpp
Source/WebCore/platform/graphics/Color.h
Tools/ChangeLog
Tools/TestWebKitAPI/Tests/WebCore/ColorTests.cpp

index 69a9cac..d9c1b87 100644 (file)
@@ -1,3 +1,15 @@
+2020-05-20  Sam Weinig  <weinig@apple.com>
+
+        Remove unused Color::getHSV function
+        https://bugs.webkit.org/show_bug.cgi?id=212119
+
+        Reviewed by Simon Fraser.
+
+        * platform/graphics/Color.cpp:
+        (WebCore::Color::getHSV const): Deleted.
+        * platform/graphics/Color.h:
+        Remove Color::getHSV(). It was unused outside of the API test for it.    
+
 2020-05-20  Youenn Fablet  <youenn@apple.com>
 
         Allow calling VideoSampleObserver::videoSampleAvailable from a background thread
index d1087b1..324edf0 100644 (file)
@@ -504,37 +504,6 @@ void Color::getHSL(double& hue, double& saturation, double& lightness) const
         saturation = (chroma / (2.0 - (max + min)));
 }
 
-void Color::getHSV(double& hue, double& saturation, double& value) const
-{
-    double r = static_cast<double>(red()) / 255.0;
-    double g = static_cast<double>(green()) / 255.0;
-    double b = static_cast<double>(blue()) / 255.0;
-    double max = std::max(std::max(r, g), b);
-    double min = std::min(std::min(r, g), b);
-    double chroma = max - min;
-
-    if (!chroma)
-        hue = 0.0;
-    else if (max == r)
-        hue = (60.0 * ((g - b) / chroma)) + 360.0;
-    else if (max == g)
-        hue = (60.0 * ((b - r) / chroma)) + 120.0;
-    else
-        hue = (60.0 * ((r - g) / chroma)) + 240.0;
-
-    if (hue >= 360.0)
-        hue -= 360.0;
-
-    hue /= 360.0;
-
-    if (!max)
-        saturation = 0;
-    else
-        saturation = chroma / max;
-
-    value = max;
-}
-
 std::pair<ColorSpace, FloatComponents> Color::colorSpaceAndComponents() const
 {
     if (isExtended()) {
index c29eece..37f42f4 100644 (file)
@@ -211,7 +211,6 @@ public:
     // replaced with alternative accessors.
 
     WEBCORE_EXPORT void getHSL(double& h, double& s, double& l) const;
-    WEBCORE_EXPORT void getHSV(double& h, double& s, double& v) const;
 
     WEBCORE_EXPORT std::pair<ColorSpace, FloatComponents> colorSpaceAndComponents() const;
 
index a017a37..49505ca 100644 (file)
@@ -1,3 +1,13 @@
+2020-05-20  Sam Weinig  <weinig@apple.com>
+
+        Remove unused Color::getHSV function
+        https://bugs.webkit.org/show_bug.cgi?id=212119
+
+        Reviewed by Simon Fraser.
+
+        * TestWebKitAPI/Tests/WebCore/ColorTests.cpp:
+        Remove tests for Color::getHSV().
+
 2020-05-20  Philippe Normand  <pnormand@igalia.com>
 
         [Flatpak SDK] Add bubblewrap ... wrapper
index c1105e0..76645df 100644 (file)
@@ -32,118 +32,6 @@ using namespace WebCore;
 
 namespace TestWebKitAPI {
 
-TEST(Color, RGBToHSV_White)
-{
-    Color color = Color::white;
-    
-    double h = 0;
-    double s = 0;
-    double v = 0;
-    color.getHSV(h, s, v);
-
-    EXPECT_DOUBLE_EQ(0, h);
-    EXPECT_DOUBLE_EQ(0, s);
-    EXPECT_DOUBLE_EQ(1, v);
-}
-
-TEST(Color, RGBToHSV_Black)
-{
-    Color color = Color::black;
-    
-    double h = 0;
-    double s = 0;
-    double v = 0;
-    color.getHSV(h, s, v);
-
-    EXPECT_DOUBLE_EQ(0, h);
-    EXPECT_DOUBLE_EQ(0, s);
-    EXPECT_DOUBLE_EQ(0, v);
-}
-
-TEST(Color, RGBToHSV_Red)
-{
-    Color color(255, 0, 0);
-    
-    double h = 0;
-    double s = 0;
-    double v = 0;
-    color.getHSV(h, s, v);
-
-    EXPECT_DOUBLE_EQ(0, h);
-    EXPECT_DOUBLE_EQ(1, s);
-    EXPECT_DOUBLE_EQ(1, v);
-}
-
-TEST(Color, RGBToHSV_Green)
-{
-    Color color(0, 255, 0);
-    
-    double h = 0;
-    double s = 0;
-    double v = 0;
-    color.getHSV(h, s, v);
-
-    EXPECT_DOUBLE_EQ(0.33333333333333331, h);
-    EXPECT_DOUBLE_EQ(1, s);
-    EXPECT_DOUBLE_EQ(1, v);
-}
-
-TEST(Color, RGBToHSV_Blue)
-{
-    Color color(0, 0, 255);
-    
-    double h = 0;
-    double s = 0;
-    double v = 0;
-    color.getHSV(h, s, v);
-
-    EXPECT_DOUBLE_EQ(0.66666666666666663, h);
-    EXPECT_DOUBLE_EQ(1, s);
-    EXPECT_DOUBLE_EQ(1, v);
-}
-
-TEST(Color, RGBToHSV_DarkGray)
-{
-    Color color = Color::darkGray;
-    
-    double h = 0;
-    double s = 0;
-    double v = 0;
-    color.getHSV(h, s, v);
-
-    EXPECT_DOUBLE_EQ(0, h);
-    EXPECT_DOUBLE_EQ(0, s);
-    EXPECT_DOUBLE_EQ(0.50196078431372548, v);
-}
-
-TEST(Color, RGBToHSV_Gray)
-{
-    Color color = Color::gray;
-    
-    double h = 0;
-    double s = 0;
-    double v = 0;
-    color.getHSV(h, s, v);
-
-    EXPECT_DOUBLE_EQ(0, h);
-    EXPECT_DOUBLE_EQ(0, s);
-    EXPECT_DOUBLE_EQ(0.62745098039215685, v);
-}
-
-TEST(Color, RGBToHSV_LightGray)
-{
-    Color color = Color::lightGray;
-    
-    double h = 0;
-    double s = 0;
-    double v = 0;
-    color.getHSV(h, s, v);
-
-    EXPECT_DOUBLE_EQ(0, h);
-    EXPECT_DOUBLE_EQ(0, s);
-    EXPECT_DOUBLE_EQ(0.75294117647058822, v);
-}
-
 TEST(Color, RGBToHSL_White)
 {
     Color color = Color::white;