Commit 925f57c7 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=dougarnett@chromium.org

Bug: 1023677
Change-Id: I947641610097a19e4c4f1db6c2eca32045869f46
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1957492
Auto-Submit: Ilia Samsonov <isamsonov@google.com>
Reviewed-by: default avatarSophie Chang <sophiechang@chromium.org>
Commit-Queue: Ilia Samsonov <isamsonov@google.com>
Cr-Commit-Position: refs/heads/master@{#726556}
parent ab312c26
...@@ -921,7 +921,7 @@ class PreviewsLitePageRedirectServerBrowserTest ...@@ -921,7 +921,7 @@ class PreviewsLitePageRedirectServerBrowserTest
// Param is true if DRP holdback should be enabled. // Param is true if DRP holdback should be enabled.
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
PreviewsLitePageRedirectServerBrowserTest, PreviewsLitePageRedirectServerBrowserTest,
::testing::Bool()); ::testing::Bool());
...@@ -1637,7 +1637,7 @@ class PreviewsLitePageRedirectServerTimeoutBrowserTest ...@@ -1637,7 +1637,7 @@ class PreviewsLitePageRedirectServerTimeoutBrowserTest
// Param is true if DRP holdback should be enabled. // Param is true if DRP holdback should be enabled.
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
PreviewsLitePageRedirectServerTimeoutBrowserTest, PreviewsLitePageRedirectServerTimeoutBrowserTest,
::testing::Bool()); ::testing::Bool());
...@@ -1705,7 +1705,7 @@ class PreviewsLitePageRedirectServerBadServerBrowserTest ...@@ -1705,7 +1705,7 @@ class PreviewsLitePageRedirectServerBadServerBrowserTest
// Param is true if DRP holdback should be enabled. // Param is true if DRP holdback should be enabled.
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
PreviewsLitePageRedirectServerBadServerBrowserTest, PreviewsLitePageRedirectServerBadServerBrowserTest,
::testing::Bool()); ::testing::Bool());
...@@ -1751,7 +1751,7 @@ class PreviewsLitePageRedirectServerDataSaverBrowserTest ...@@ -1751,7 +1751,7 @@ class PreviewsLitePageRedirectServerDataSaverBrowserTest
// Param is true if DRP holdback should be enabled. // Param is true if DRP holdback should be enabled.
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
PreviewsLitePageRedirectServerDataSaverBrowserTest, PreviewsLitePageRedirectServerDataSaverBrowserTest,
::testing::Bool()); ::testing::Bool());
...@@ -1788,7 +1788,7 @@ class PreviewsLitePageRedirectServerNoDataSaverHeaderBrowserTest ...@@ -1788,7 +1788,7 @@ class PreviewsLitePageRedirectServerNoDataSaverHeaderBrowserTest
// Param is true if DRP holdback should be enabled. // Param is true if DRP holdback should be enabled.
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
PreviewsLitePageRedirectServerNoDataSaverHeaderBrowserTest, PreviewsLitePageRedirectServerNoDataSaverHeaderBrowserTest,
::testing::Bool()); ::testing::Bool());
...@@ -1829,7 +1829,7 @@ class PreviewsLitePageRedirectNotificationDSEnabledBrowserTest ...@@ -1829,7 +1829,7 @@ class PreviewsLitePageRedirectNotificationDSEnabledBrowserTest
// Pecond param is true if DRP holdback should be enabled. // Pecond param is true if DRP holdback should be enabled.
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
PreviewsLitePageRedirectNotificationDSEnabledBrowserTest, PreviewsLitePageRedirectNotificationDSEnabledBrowserTest,
::testing::Bool()); ::testing::Bool());
...@@ -1889,7 +1889,7 @@ class PreviewsLitePageRedirectDSDisabledBrowserTest ...@@ -1889,7 +1889,7 @@ class PreviewsLitePageRedirectDSDisabledBrowserTest
// Param is true if DRP holdback should be enabled. // Param is true if DRP holdback should be enabled.
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
PreviewsLitePageRedirectDSDisabledBrowserTest, PreviewsLitePageRedirectDSDisabledBrowserTest,
::testing::Bool()); ::testing::Bool());
...@@ -1921,7 +1921,7 @@ class PreviewsLitePageRedirectControlBrowserTest ...@@ -1921,7 +1921,7 @@ class PreviewsLitePageRedirectControlBrowserTest
// Param is true if DRP holdback should be enabled. // Param is true if DRP holdback should be enabled.
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
PreviewsLitePageRedirectControlBrowserTest, PreviewsLitePageRedirectControlBrowserTest,
::testing::Bool()); ::testing::Bool());
...@@ -2010,7 +2010,7 @@ IN_PROC_BROWSER_TEST_P( ...@@ -2010,7 +2010,7 @@ IN_PROC_BROWSER_TEST_P(
} }
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
PreviewsLitePageRedirectServerNetworkIsolationBrowserTest, PreviewsLitePageRedirectServerNetworkIsolationBrowserTest,
::testing::Values(NetworkIsolationKeyMode::kNone, ::testing::Values(NetworkIsolationKeyMode::kNone,
NetworkIsolationKeyMode::kTopFrameOrigin, NetworkIsolationKeyMode::kTopFrameOrigin,
...@@ -2053,7 +2053,7 @@ class PreviewsLitePageRedirectAndPageHintsBrowserTest ...@@ -2053,7 +2053,7 @@ class PreviewsLitePageRedirectAndPageHintsBrowserTest
// Param is true if DRP holdback should be enabled. // Param is true if DRP holdback should be enabled.
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
PreviewsLitePageRedirectAndPageHintsBrowserTest, PreviewsLitePageRedirectAndPageHintsBrowserTest,
::testing::Bool()); ::testing::Bool());
...@@ -2295,22 +2295,22 @@ class CoinFlipHoldbackExperimentBrowserTestWithCoinFlipHoldbackDisabled ...@@ -2295,22 +2295,22 @@ class CoinFlipHoldbackExperimentBrowserTestWithCoinFlipHoldbackDisabled
// Param is true if DRP holdback should be enabled. // Param is true if DRP holdback should be enabled.
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
CoinFlipHoldbackExperimentBrowserTest, CoinFlipHoldbackExperimentBrowserTest,
::testing::Bool()); ::testing::Bool());
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
CoinFlipHoldbackExperimentBrowserTestWithRandomNavigationCoinFlip, CoinFlipHoldbackExperimentBrowserTestWithRandomNavigationCoinFlip,
::testing::Bool()); ::testing::Bool());
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
CoinFlipHoldbackExperimentBrowserTestWithoutRandomNavigationCoinFlip, CoinFlipHoldbackExperimentBrowserTestWithoutRandomNavigationCoinFlip,
::testing::Bool()); ::testing::Bool());
INSTANTIATE_TEST_SUITE_P( INSTANTIATE_TEST_SUITE_P(
/* no prefix */, All,
CoinFlipHoldbackExperimentBrowserTestWithCoinFlipHoldbackDisabled, CoinFlipHoldbackExperimentBrowserTestWithCoinFlipHoldbackDisabled,
::testing::Bool()); ::testing::Bool());
......
...@@ -307,6 +307,6 @@ TEST_P(PreviewsLitePageRedirectPredictorUnitTest, ...@@ -307,6 +307,6 @@ TEST_P(PreviewsLitePageRedirectPredictorUnitTest,
} }
// True if preresolving, false for preconnecting. // True if preresolving, false for preconnecting.
INSTANTIATE_TEST_SUITE_P(/* empty prefix */, INSTANTIATE_TEST_SUITE_P(All,
PreviewsLitePageRedirectPredictorUnitTest, PreviewsLitePageRedirectPredictorUnitTest,
testing::Bool()); testing::Bool());
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