Commit cba058dd authored by Ilia Samsonov's avatar Ilia Samsonov Committed by Commit Bot

Add prefix to empty param tests.

googletest will no longer support empty prefix
for parameterized INSTANTIATE_TEST_SUITE_P.
Adding "All" prefix to existing empty prefix test suites.

This CL was uploaded by git cl split.

R=thakis@chromium.org

Bug: 1023677
Change-Id: I754c80d5f281438539432343841ff3a251b90283
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1926922
Auto-Submit: Ilia Samsonov <isamsonov@google.com>
Reviewed-by: default avatarNico Weber <thakis@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721733}
parent b863d471
...@@ -64,7 +64,7 @@ TEST_P(CursorUtilTest, ScaleAndRotate) { ...@@ -64,7 +64,7 @@ TEST_P(CursorUtilTest, ScaleAndRotate) {
EXPECT_EQ(pixel_color, bitmap.pixmap().getColor(0, 0)); EXPECT_EQ(pixel_color, bitmap.pixmap().getColor(0, 0));
} }
INSTANTIATE_TEST_SUITE_P(, CursorUtilTest, testing::Bool()); INSTANTIATE_TEST_SUITE_P(All, CursorUtilTest, testing::Bool());
} // namespace } // namespace
} // namespace ui } // namespace ui
...@@ -142,7 +142,7 @@ TEST_P(BiDiLineIteratorTest, RTLPunctuationNoCustomBehavior) { ...@@ -142,7 +142,7 @@ TEST_P(BiDiLineIteratorTest, RTLPunctuationNoCustomBehavior) {
} }
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
, All,
BiDiLineIteratorTest, BiDiLineIteratorTest,
::testing::Values(base::i18n::TextDirection::LEFT_TO_RIGHT, ::testing::Values(base::i18n::TextDirection::LEFT_TO_RIGHT,
base::i18n::TextDirection::RIGHT_TO_LEFT)); base::i18n::TextDirection::RIGHT_TO_LEFT));
......
...@@ -77,7 +77,7 @@ const RemoveAcceleratorCharData RemoveAcceleratorCharTest::kCases[] = { ...@@ -77,7 +77,7 @@ const RemoveAcceleratorCharData RemoveAcceleratorCharTest::kCases[] = {
}; };
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
, All,
RemoveAcceleratorCharTest, RemoveAcceleratorCharTest,
testing::ValuesIn(RemoveAcceleratorCharTest::kCases), testing::ValuesIn(RemoveAcceleratorCharTest::kCases),
[](const testing::TestParamInfo<RemoveAcceleratorCharData>& param_info) { [](const testing::TestParamInfo<RemoveAcceleratorCharData>& param_info) {
...@@ -158,7 +158,7 @@ const FindValidBoundaryData FindValidBoundaryBeforeTest::kCases[] = { ...@@ -158,7 +158,7 @@ const FindValidBoundaryData FindValidBoundaryBeforeTest::kCases[] = {
}; };
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
, All,
FindValidBoundaryBeforeTest, FindValidBoundaryBeforeTest,
testing::ValuesIn(FindValidBoundaryBeforeTest::kCases), testing::ValuesIn(FindValidBoundaryBeforeTest::kCases),
[](const testing::TestParamInfo<FindValidBoundaryData>& param_info) { [](const testing::TestParamInfo<FindValidBoundaryData>& param_info) {
...@@ -231,7 +231,7 @@ const FindValidBoundaryData FindValidBoundaryAfterTest::kCases[] = { ...@@ -231,7 +231,7 @@ const FindValidBoundaryData FindValidBoundaryAfterTest::kCases[] = {
}; };
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
, All,
FindValidBoundaryAfterTest, FindValidBoundaryAfterTest,
testing::ValuesIn(FindValidBoundaryAfterTest::kCases), testing::ValuesIn(FindValidBoundaryAfterTest::kCases),
[](const testing::TestParamInfo<FindValidBoundaryData>& param_info) { [](const testing::TestParamInfo<FindValidBoundaryData>& param_info) {
......
...@@ -194,7 +194,7 @@ class SnapshotAuraTest : public testing::TestWithParam<bool> { ...@@ -194,7 +194,7 @@ class SnapshotAuraTest : public testing::TestWithParam<bool> {
DISALLOW_COPY_AND_ASSIGN(SnapshotAuraTest); DISALLOW_COPY_AND_ASSIGN(SnapshotAuraTest);
}; };
INSTANTIATE_TEST_SUITE_P(, SnapshotAuraTest, ::testing::Bool()); INSTANTIATE_TEST_SUITE_P(All, SnapshotAuraTest, ::testing::Bool());
#if defined(OS_WIN) && !defined(NDEBUG) #if defined(OS_WIN) && !defined(NDEBUG)
// https://crbug.com/852512 // https://crbug.com/852512
......
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