Commit b70df62d authored by Alison Maher's avatar Alison Maher Committed by Commit Bot

Update WebThemeEngineImplDefaultBrowserTest.GetSystemColor

WebThemeEngineImplDefaultBrowserTest.GetSystemColor fails
non-deterministically on Windows2008Server. To fix this, the test is
updated to only run on Win7 and above.

Bug: 1010924
Change-Id: Ibda98ca611c27176ff831d0c85462579ae2c37dd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1856578
Commit-Queue: Alison Maher <almaher@microsoft.com>
Reviewed-by: default avatarAvi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#705220}
parent 17998a4e
...@@ -19,6 +19,11 @@ class WebThemeEngineImplDefaultBrowserTest : public ContentBrowserTest { ...@@ -19,6 +19,11 @@ class WebThemeEngineImplDefaultBrowserTest : public ContentBrowserTest {
#if defined(OS_WIN) #if defined(OS_WIN)
IN_PROC_BROWSER_TEST_F(WebThemeEngineImplDefaultBrowserTest, GetSystemColor) { IN_PROC_BROWSER_TEST_F(WebThemeEngineImplDefaultBrowserTest, GetSystemColor) {
// The test non-deterministically fails on Windows Server 2008 builders due to
// a difference in the default theme. As a result, only run the test on
// Windows version 7 and above.
if (base::win::GetVersion() < base::win::Version::WIN7)
return;
GURL url( GURL url(
"data:text/html," "data:text/html,"
"<!doctype html><html>" "<!doctype html><html>"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment