[macOS] API test DragAndDropTests.DropColor is failing on macOS 10.12
authorwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 9 Oct 2018 03:28:31 +0000 (03:28 +0000)
committerwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 9 Oct 2018 03:28:31 +0000 (03:28 +0000)
https://bugs.webkit.org/show_bug.cgi?id=190386

Reviewed by Ryosuke Niwa.

Unlike macOS 10.13 and 10.14, -[NSColor redColor] is not equivalent to a fully red color in sRGB on macOS 10.12.
Fix the test by dragging and dropping an NSColor that is equivalent to rgb(255, 0, 0) on all relevant versions
of macOS.

* TestWebKitAPI/Tests/mac/DragAndDropTestsMac.mm:
(TEST):

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

Tools/ChangeLog
Tools/TestWebKitAPI/Tests/mac/DragAndDropTestsMac.mm

index 58a6d92..33f1f12 100644 (file)
@@ -1,5 +1,19 @@
 2018-10-08  Wenson Hsieh  <wenson_hsieh@apple.com>
 
+        [macOS] API test DragAndDropTests.DropColor is failing on macOS 10.12
+        https://bugs.webkit.org/show_bug.cgi?id=190386
+
+        Reviewed by Ryosuke Niwa.
+
+        Unlike macOS 10.13 and 10.14, -[NSColor redColor] is not equivalent to a fully red color in sRGB on macOS 10.12.
+        Fix the test by dragging and dropping an NSColor that is equivalent to rgb(255, 0, 0) on all relevant versions
+        of macOS.
+
+        * TestWebKitAPI/Tests/mac/DragAndDropTestsMac.mm:
+        (TEST):
+
+2018-10-08  Wenson Hsieh  <wenson_hsieh@apple.com>
+
         [Cocoa] [WK2] Add support for text alignment and text lists in font attributes
         https://bugs.webkit.org/show_bug.cgi?id=190342
         <rdar://problem/44767118>
index e335b6d..a8cc282 100644 (file)
@@ -78,7 +78,7 @@ TEST(DragAndDropTests, DropColor)
 {
     NSPasteboard *pasteboard = [NSPasteboard pasteboardWithUniqueName];
     [pasteboard declareTypes:@[NSColorPboardType] owner:nil];
-    [[NSColor redColor] writeToPasteboard:pasteboard];
+    [[NSColor colorWithRed:1 green:0 blue:0 alpha:1] writeToPasteboard:pasteboard];
 
     auto simulator = adoptNS([[DragAndDropSimulator alloc] initWithWebViewFrame:NSMakeRect(0, 0, 400, 400)]);
     TestWKWebView *webView = [simulator webView];