Commit f987c517 authored by Sebastien Marchand's avatar Sebastien Marchand Committed by Commit Bot

Remove launched features from fieldtrial_testing_config.json

This removes several entries from fieldtrial_testing_config.json for
features that are in the FEATURE_ENABLED_BY_DEFAULT state.

Methodology:
- Make a script that list all the features listed in
  fieldtrial_testing_config.json
- Run git grep for all these features in the chrome codebase, save the
  first line of the output in a file (this could be improved for
  features that wrap over multiple lines).
- Find the line containing FEATURE_ENABLED_BY_DEFAULT, check if the
  entry is trivial (no param changes etc) and if the feature isn't
  behind a #ifdef block, remove it if it sounds safe.

A follow up CL will remove all entries that don't have a corresponding
feature in the codebase.

Bug: 1053702
Change-Id: Ifb526b74d988bdf18f8997f536463956a4d9578e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2062697Reviewed-by: default avatarAlexei Svitkine <asvitkine@chromium.org>
Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744052}
parent 64e434bf
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