Skip to content

Commit

Permalink
[IdleTimeout] Adding Feature Flag
Browse files Browse the repository at this point in the history
Putting the IdleTimeout and IdleTimeout policies behind a flag.

Bug: b/301676922
Change-Id: Ifab65ae67e1af2b04ec3f2be6df844821b69ee62
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5192409
Commit-Queue: Salma Elmahallawy <[email protected]>
Reviewed-by: Fabio Tirelo <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1247270}
  • Loading branch information
Salma Elmahallawy authored and Chromium LUCI CQ committed Jan 15, 2024
1 parent b59b95d commit e8a5df1
Show file tree
Hide file tree
Showing 5 changed files with 20 additions and 0 deletions.
5 changes: 5 additions & 0 deletions chrome/browser/flag-metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -4736,6 +4736,11 @@
],
"expiry_milestone": 125
},
{
"name": "idle-timeout-policies",
"owners": ["[email protected]", "[email protected]"],
"expiry_milestone": 125
},
{
"name": "ignore-gpu-blocklist",
"owners": [ "[email protected]", "[email protected]" ],
Expand Down
1 change: 1 addition & 0 deletions ios/chrome/browser/flags/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ source_set("flags") {
"//components/dom_distiller/core",
"//components/download/public/background_service:public",
"//components/enterprise",
"//components/enterprise/idle",
"//components/feature_engagement/public",
"//components/feed:feature_list",
"//components/flags_ui",
Expand Down
4 changes: 4 additions & 0 deletions ios/chrome/browser/flags/about_flags.mm
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@
#import "components/dom_distiller/core/dom_distiller_switches.h"
#import "components/download/public/background_service/features.h"
#import "components/enterprise/browser/enterprise_switches.h"
#import "components/enterprise/idle/idle_features.h"
#import "components/feature_engagement/public/feature_constants.h"
#import "components/feature_engagement/public/feature_list.h"
#import "components/feed/feed_feature_list.h"
Expand Down Expand Up @@ -1555,6 +1556,9 @@
flag_descriptions::kTabGridCompositionalLayoutName,
flag_descriptions::kTabGridCompositionalLayoutDescription,
flags_ui::kOsIos, FEATURE_VALUE_TYPE(kTabGridCompositionalLayout)},
{"idle-timeout-policies", flag_descriptions::kIdleTimeoutPoliciesName,
flag_descriptions::kIdleTimeoutPoliciesDescription, flags_ui::kOsIos,
FEATURE_VALUE_TYPE(enterprise_idle::kIdleTimeout)},
{"bottom-omnibox-promo-fre", flag_descriptions::kBottomOmniboxPromoFREName,
flag_descriptions::kBottomOmniboxPromoFREDescription, flags_ui::kOsIos,
FEATURE_WITH_PARAMS_VALUE_TYPE(kBottomOmniboxPromoFRE,
Expand Down
5 changes: 5 additions & 0 deletions ios/chrome/browser/flags/ios_chrome_flag_descriptions.cc
Original file line number Diff line number Diff line change
Expand Up @@ -802,6 +802,11 @@ const char kPromosManagerUsesFETDescription[] =
"Migrates the Promos Manager to use the Feature Engagement Tracker as its "
"impression tracking system";

const char kIdleTimeoutPoliciesName[] =
"IdleTimeout and IdleTimeoutActions Policies";
const char kIdleTimeoutPoliciesDescription[] =
"Enable IdleTimeout and IdleTimeoutActions enterprise policies.";

const char kIPHPriceNotificationsWhileBrowsingName[] =
"Price Tracking IPH Display";
const char kIPHPriceNotificationsWhileBrowsingDescription[] =
Expand Down
5 changes: 5 additions & 0 deletions ios/chrome/browser/flags/ios_chrome_flag_descriptions.h
Original file line number Diff line number Diff line change
Expand Up @@ -381,6 +381,11 @@ extern const char kFullScreenPromoOnOmniboxCopyPasteDescription[];
extern const char kFullscreenPromosManagerSkipInternalLimitsName[];
extern const char kFullscreenPromosManagerSkipInternalLimitsDescription[];

// Title and description for the flag to enable IdleTimeout and
// IdleTimeoutActions policies.
extern const char kIdleTimeoutPoliciesName[];
extern const char kIdleTimeoutPoliciesDescription[];

// Title and description for the command line switch used to determine the
// active fullscreen viewport adjustment mode.
extern const char kFullscreenSmoothScrollingName[];
Expand Down

0 comments on commit e8a5df1

Please sign in to comment.