fixed equality

Author: gaaclarke

lib/ui/painting.dart

@@ -455,8 +455,9 @@ class Color {
     if (other.runtimeType != runtimeType) {
       return false;
     }
-    return other is Color
-        && other.value == value;
+    return other is Color &&
+        other.value == value &&
+        other.colorSpace == colorSpace;
   }
 
   @override

lib/web_ui/lib/painting.dart

@@ -247,8 +247,9 @@ class Color {
     if (other.runtimeType != runtimeType) {
       return false;
     }
-    return other is Color
-        && other.value == value;
+    return other is Color &&
+        other.value == value &&
+        other.colorSpace == colorSpace;
   }
 
   @override

testing/dart/color_test.dart

@@ -301,6 +301,14 @@ void main() {
     expect(srgb.hashCode, notEquals(p3.hashCode));
   });
 
+  test('equality considers colorspace', () {
+    const Color srgb = Color.from(
+        alpha: 1, red: 1, green: 0, blue: 0);
+    const Color p3 = Color.from(
+        alpha: 1, red: 1, green: 0, blue: 0, colorSpace: ColorSpace.displayP3);
+    expect(srgb, notEquals(p3));
+  });
+
   // Regression test for https://github.com/flutter/flutter/issues/41257
   // CupertinoDynamicColor was overriding base class and calling super(0).
   test('subclass of Color can override value', () {