Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(70)

Unified Diff: components/network_time/network_time_test_utils.cc

Issue 2448943004: Add experimental feature info to certificate reports (Closed)
Patch Set: battre comment Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/network_time/network_time_test_utils.h ('k') | components/network_time/network_time_tracker.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/network_time/network_time_test_utils.cc
diff --git a/components/network_time/network_time_test_utils.cc b/components/network_time/network_time_test_utils.cc
index 38267b2e767b06164367e1b8c904d5e1ba95d2a6..b8203a4e2968bbab100ee1cd359909dc5f4a6be7 100644
--- a/components/network_time/network_time_test_utils.cc
+++ b/components/network_time/network_time_test_utils.cc
@@ -66,7 +66,7 @@ FieldTrialTest* FieldTrialTest::CreateForBrowserTest() {
void FieldTrialTest::SetNetworkQueriesWithVariationsService(
bool enable,
float query_probability,
- FetchBehavior fetch_behavior) {
+ NetworkTimeTracker::FetchBehavior fetch_behavior) {
const std::string kTrialName = "Trial";
const std::string kGroupName = "group";
const base::Feature kFeature{"NetworkTimeServiceQuerying",
@@ -80,13 +80,17 @@ void FieldTrialTest::SetNetworkQueriesWithVariationsService(
params["CheckTimeIntervalSeconds"] = base::Int64ToString(360);
std::string fetch_behavior_param;
switch (fetch_behavior) {
- case FETCHES_IN_BACKGROUND_ONLY:
+ case NetworkTimeTracker::FETCH_BEHAVIOR_UNKNOWN:
+ NOTREACHED();
+ fetch_behavior_param = "unknown";
+ break;
+ case NetworkTimeTracker::FETCHES_IN_BACKGROUND_ONLY:
fetch_behavior_param = "background-only";
break;
- case FETCHES_ON_DEMAND_ONLY:
+ case NetworkTimeTracker::FETCHES_ON_DEMAND_ONLY:
fetch_behavior_param = "on-demand-only";
break;
- case FETCHES_IN_BACKGROUND_AND_ON_DEMAND:
+ case NetworkTimeTracker::FETCHES_IN_BACKGROUND_AND_ON_DEMAND:
fetch_behavior_param = "background-and-on-demand";
break;
}
« no previous file with comments | « components/network_time/network_time_test_utils.h ('k') | components/network_time/network_time_tracker.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698