Bug 195958 – REGRESSION (r239904): Update dark mode defines in a few places that got missed

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Timothy Hatcher]

The default text color on non-dark mode systems was broken by r239904.

<rdar://problem/47678852>

[Timothy Hatcher]

Created attachment 365192 [details]
Patch

[EWS Watchlist]

Comment on attachment 365192 [details]
Patch

Attachment 365192 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/11568971

New failing tests:
inspector/css/shadow-scoped-style.html
printing/width-overflow.html

[EWS Watchlist]

Created attachment 365211 [details]
Archive of layout-test-results from ews103 for mac-highsierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-highsierra  Platform: Mac OS X 10.13.6

[EWS Watchlist]

Comment on attachment 365192 [details]
Patch

Attachment 365192 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/11568944

New failing tests:
inspector/css/shadow-scoped-style.html
printing/width-overflow.html

[EWS Watchlist]

Created attachment 365212 [details]
Archive of layout-test-results from ews107 for mac-highsierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-highsierra-wk2  Platform: Mac OS X 10.13.6

[Timothy Hatcher]

Created attachment 365219 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 365219 [details]
Patch

Clearing flags on attachment: 365219

Committed r243169: <https://trac.webkit.org/changeset/243169>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Shawn Roberts]

This revision is causing the following test to be a 100% text diff failure.

css-dark-mode/default-colors.html

Reproduced locally, fails in r243169, passes in r243168.

Reproduced with :

run-webkit-tests css-dark-mode/default-colors.html --iterations 10 -f

Flakiness Dashboard:

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=css-dark-mode%2Fdefault-colors.html

Diff: 

--- /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/css-dark-mode/default-colors-expected.txt
+++ /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/css-dark-mode/default-colors-actual.txt
@@ -5,6 +5,6 @@
 PASS Body text color is black, since the page hasn't set a supported color scheme 
 PASS View base background color is white, since the page hasn't set a supported color scheme 
 PASS Set prefers-color-schemes: light dark on the body 
-PASS Body text color is white 
+FAIL Body text color is white assert_equals: expected "rgb(255, 255, 255)" but got "rgb(0, 0, 0)"
 PASS View base background color is a dark grey

[Shawn Roberts]

This revision is also causing a 100% text diff failure in the following test as well :

inspector/css/force-page-appearance.html

Same reproduction steps. Fails in 243169, passes in 243168

Flakiness Dashboard:

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=inspector%2Fcss%2Fforce-page-appearance.html

Diff:

--- /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/inspector/css/force-page-appearance-expected.txt
+++ /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/inspector/css/force-page-appearance-actual.txt
@@ -13,14 +13,18 @@
 PASS: WI.cssManager.forcedAppearance should be Dark.
 PASS: DOMNodeStyles should need refresh.
 PASS: expectEqual("200px", "200px")
-PASS: expectEqual("rgb(255, 255, 255)", "rgb(255, 255, 255)")
+FAIL: expectEqual("rgb(0, 0, 0)", "rgb(255, 255, 255)")
+    Expected: "rgb(255, 255, 255)"
+    Actual: "rgb(0, 0, 0)"
 
 -- Running test case: Switch to Dark appearance by default
 PASS: WI.cssManager.defaultAppearance should be Dark.
 PASS: WI.cssManager.forcedAppearance should be Dark.
 PASS: DOMNodeStyles should need refresh.
 PASS: expectEqual("200px", "200px")
-PASS: expectEqual("rgb(255, 255, 255)", "rgb(255, 255, 255)")
+FAIL: expectEqual("rgb(0, 0, 0)", "rgb(255, 255, 255)")
+    Expected: "rgb(255, 255, 255)"
+    Actual: "rgb(0, 0, 0)"
 
 -- Running test case: Force appearance to Light
 PASS: WI.cssManager.defaultAppearance should be Dark.
@@ -34,5 +38,7 @@
 PASS: WI.cssManager.forcedAppearance should be null.
 PASS: DOMNodeStyles should need refresh.
 PASS: expectEqual("200px", "200px")
-PASS: expectEqual("rgb(255, 255, 255)", "rgb(255, 255, 255)")
+FAIL: expectEqual("rgb(0, 0, 0)", "rgb(255, 255, 255)")
+    Expected: "rgb(255, 255, 255)"
+    Actual: "rgb(0, 0, 0)"

[Timothy Hatcher]

Created attachment 365364 [details]
Test Fix

[Timothy Hatcher]

Reopened to land fix.

[WebKit Commit Bot]

Comment on attachment 365364 [details]
Test Fix

Clearing flags on attachment: 365364

Committed r243229: <https://trac.webkit.org/changeset/243229>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Timothy Hatcher]

Reopened for test gardening.

[Timothy Hatcher]

Created attachment 365392 [details]
Test Gardening

[WebKit Commit Bot]

Comment on attachment 365392 [details]
Test Gardening

Rejecting attachment 365392 [details] from commit-queue.

New failing tests:
inspector/css/shadow-scoped-style.html
printing/width-overflow.html
Full output: https://webkit-queues.webkit.org/results/11584601

[WebKit Commit Bot]

Created attachment 365403 [details]
Archive of layout-test-results from webkit-cq-01 for mac-highsierra

The attached test failures were seen while running run-webkit-tests on the commit-queue.
Bot: webkit-cq-01  Port: mac-highsierra  Platform: Mac OS X 10.13.6

[Timothy Hatcher]

Created attachment 365420 [details]
Test Gardening

[WebKit Commit Bot]

Comment on attachment 365420 [details]
Test Gardening

Clearing flags on attachment: 365420

Committed r243257: <https://trac.webkit.org/changeset/243257>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.