Add support for android in gpu test expections parser

So the parser can know 'Android' platform. Also the failed case
'GPUTestExpectationsParserTest.WebGLTestExpectationsValidation'
is passed.

BUG=118542
TEST=content_unittests


Review URL: https://chromiumcodereview.appspot.com/10833003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148719 0039d316-1c4b-4281-b951-d872f2087c98
parent 484dcdf5
......@@ -36,8 +36,6 @@ AudioRendererHostTest.CreatePlayAndShutdown
AudioRendererHostTest.CreatePlayPauseAndShutdown
AudioRendererHostTest.SimulateError
AudioRendererHostTest.SimulateErrorAndClose
# crbug.com/118542
GPUTestExpectationsParserTest.WebGLTestExpectationsValidation
# crbug.com/104950
DeviceOrientationProviderTest.ObserverNotRemoved
DeviceOrientationProviderTest.StartFailing
......
......@@ -37,6 +37,7 @@ enum Token {
kConfigMac,
kConfigLinux,
kConfigChromeOS,
kConfigAndroid,
// gpu vendor
kConfigNVidia,
kConfigAMD,
......@@ -79,6 +80,7 @@ const TokenInfo kTokenData[] = {
{ "mac", GPUTestConfig::kOsMac },
{ "linux", GPUTestConfig::kOsLinux },
{ "chromeos", GPUTestConfig::kOsChromeOS },
{ "android", GPUTestConfig::kOsAndroid },
{ "nvidia", 0x10DE },
{ "amd", 0x1002 },
{ "intel", 0x8086 },
......@@ -234,6 +236,7 @@ bool GPUTestExpectationsParser::ParseConfig(
case kConfigMac:
case kConfigLinux:
case kConfigChromeOS:
case kConfigAndroid:
case kConfigNVidia:
case kConfigAMD:
case kConfigIntel:
......@@ -281,6 +284,7 @@ bool GPUTestExpectationsParser::ParseLine(
case kConfigMac:
case kConfigLinux:
case kConfigChromeOS:
case kConfigAndroid:
case kConfigNVidia:
case kConfigAMD:
case kConfigIntel:
......@@ -391,6 +395,7 @@ bool GPUTestExpectationsParser::UpdateTestConfig(
case kConfigMac:
case kConfigLinux:
case kConfigChromeOS:
case kConfigAndroid:
if ((config->os() & kTokenData[token].flag) != 0) {
PushErrorMessage(kErrorMessage[kErrorEntryWithOsConflicts],
line_number);
......
......@@ -89,7 +89,7 @@ TEST_F(GPUTestExpectationsParserTest, ValidUnrelatedTestEntry) {
TEST_F(GPUTestExpectationsParserTest, AllModifiers) {
const std::string text =
"BUG12345 XP VISTA WIN7 LEOPARD SNOWLEOPARD LION LINUX CHROMEOS "
"BUG12345 XP VISTA WIN7 LEOPARD SNOWLEOPARD LION LINUX CHROMEOS ANDROID "
"NVIDIA INTEL AMD VMWARE RELEASE DEBUG : MyTest = "
"PASS FAIL FLAKY TIMEOUT SKIP";
......@@ -115,7 +115,7 @@ TEST_F(GPUTestExpectationsParserTest, DuplicateModifiers) {
TEST_F(GPUTestExpectationsParserTest, AllModifiersLowerCase) {
const std::string text =
"BUG12345 xp vista win7 leopard snowleopard lion linux chromeos "
"BUG12345 xp vista win7 leopard snowleopard lion linux chromeos android "
"nvidia intel amd vmware release debug : MyTest = "
"pass fail flaky timeout skip";
......
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