Commit 32e81772 authored by Chris Lu's avatar Chris Lu Committed by Chromium LUCI CQ

[ios] Fix default browser fullscreen promo experiment retrieval

GetFieldTrial() is the correct API to call for finch config experiments.

Bug: 1165875
Change-Id: Ic3a0657c1ee2c6e1bb815c91607ec12b0ed41afa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2626104Reviewed-by: default avatarEugene But <eugenebut@chromium.org>
Commit-Queue: Chris Lu <thegreenfrog@chromium.org>
Cr-Commit-Position: refs/heads/master@{#843187}
parent 9e29f51e
...@@ -97,16 +97,18 @@ bool ShouldShowRemindMeLaterDefaultBrowserFullscreenPromo() { ...@@ -97,16 +97,18 @@ bool ShouldShowRemindMeLaterDefaultBrowserFullscreenPromo() {
bool IsInRemindMeLaterGroup() { bool IsInRemindMeLaterGroup() {
return base::FeatureList::IsEnabled( return base::FeatureList::IsEnabled(
kDefaultBrowserFullscreenPromoExperiment) && kDefaultBrowserFullscreenPromoExperiment) &&
base::FieldTrialList::FindFullName( base::FeatureList::GetFieldTrial(
kDefaultBrowserFullscreenPromoExperiment.name) == kDefaultBrowserFullscreenPromoExperiment)
->group_name() ==
kDefaultBrowserFullscreenPromoExperimentRemindMeGroupName; kDefaultBrowserFullscreenPromoExperimentRemindMeGroupName;
} }
bool IsInModifiedStringsGroup() { bool IsInModifiedStringsGroup() {
return base::FeatureList::IsEnabled( return base::FeatureList::IsEnabled(
kDefaultBrowserFullscreenPromoExperiment) && kDefaultBrowserFullscreenPromoExperiment) &&
base::FieldTrialList::FindFullName( base::FeatureList::GetFieldTrial(
kDefaultBrowserFullscreenPromoExperiment.name) == kDefaultBrowserFullscreenPromoExperiment)
->group_name() ==
kDefaultBrowserFullscreenPromoExperimentChangeStringsGroupName; kDefaultBrowserFullscreenPromoExperimentChangeStringsGroupName;
} }
......
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