Remove V8 serialize options from fieldtrial config.

Remove V8 serialize options from fieldtrial config.

This experiment isn't meant to launch to stable, and we want to use the
fieldtrial configs to test what's going to launch to stable.

BUG=601894
Committed: https://crrev.com/dbb9b030f66920877033bcca14bb9d9ca046be4e
Cr-Commit-Position: refs/heads/master@{#389165}

[sullivan, 2016-04-21 22:19:19]

sullivan@chromium.org changed reviewers:
+ asvitkine@chromium.org, vogelheim@chromium.org

[chromium-reviews, 2016-04-22 08:36:26]

lgtm

Am 22.04.2016 12:19 vorm. schrieb <sullivan@chromium.org>:
>
> Reviewers: Alexei Svitkine, vogelheim
> CL: https://codereview.chromium.org/1907093002/
>
> Description:
> Remove V8 serialize options from fieldtrial config.
>
> This experiment isn't meant to launch to stable, and we want to use the
> fieldtrial configs to test what's going to launch to stable.
>
> BUG=601894
>
> Base URL: https://chromium.googlesource.com/chromium/src.git@master
>
> Affected files (+1, -45 lines):
> M testing/variations/fieldtrial_testing_config_android.json
> M testing/variations/fieldtrial_testing_config_chromeos.json
> M testing/variations/fieldtrial_testing_config_linux.json
> M testing/variations/fieldtrial_testing_config_mac.json
> M testing/variations/fieldtrial_testing_config_win.json
> M tools/perf/docs/perf_bot_sheriffing.md
>
>
> Index: testing/variations/fieldtrial_testing_config_android.json
> diff --git a/testing/variations/fieldtrial_testing_config_android.json
b/testing/variations/fieldtrial_testing_config_android.json
> index
538486e76db60fd8a3ed5765bab1064d268d2727..e4e39278409e4d6e54e65da0f28fbf182bf4a92d
100644
> --- a/testing/variations/fieldtrial_testing_config_android.json
> +++ b/testing/variations/fieldtrial_testing_config_android.json
> @@ -437,15 +437,6 @@
> }
> }
> ],
> - "V8SerializeOptions": [
> - {
> - "enable_features": [
> - "V8_Serialize_Eager",
> - "V8_Serialize_Age_Code"
> - ],
> - "group_name": "SerializeEagerAndAgeCode"
> - }
> - ],
> "VarationsServiceControl": [
> {
> "group_name": "Interval_30min",
> Index: testing/variations/fieldtrial_testing_config_chromeos.json
> diff --git a/testing/variations/fieldtrial_testing_config_chromeos.json
b/testing/variations/fieldtrial_testing_config_chromeos.json
> index
07ad33b5e01b606643fe42b2066be042d2cffe57..d7e33bac422709e72bab9ce926106525e8643e7a
100644
> --- a/testing/variations/fieldtrial_testing_config_chromeos.json
> +++ b/testing/variations/fieldtrial_testing_config_chromeos.json
> @@ -213,15 +213,6 @@
> "group_name": "TokenBinding"
> }
> ],
> - "V8SerializeOptions": [
> - {
> - "enable_features": [
> - "V8_Serialize_Eager",
> - "V8_Serialize_Age_Code"
> - ],
> - "group_name": "SerializeEagerAndAgeCode"
> - }
> - ],
> "VarationsServiceControl": [
> {
> "group_name": "Interval_30min",
> Index: testing/variations/fieldtrial_testing_config_linux.json
> diff --git a/testing/variations/fieldtrial_testing_config_linux.json
b/testing/variations/fieldtrial_testing_config_linux.json
> index
96b6baa10cedfacb3f95d3363470abe2bb552c91..42c899a17735d48cca9877fe9d7e4ef6a5046efd
100644
> --- a/testing/variations/fieldtrial_testing_config_linux.json
> +++ b/testing/variations/fieldtrial_testing_config_linux.json
> @@ -268,15 +268,6 @@
> "group_name": "TokenBinding"
> }
> ],
> - "V8SerializeOptions": [
> - {
> - "enable_features": [
> - "V8_Serialize_Eager",
> - "V8_Serialize_Age_Code"
> - ],
> - "group_name": "SerializeEagerAndAgeCode"
> - }
> - ],
> "VarationsServiceControl": [
> {
> "group_name": "Interval_30min",
> Index: testing/variations/fieldtrial_testing_config_mac.json
> diff --git a/testing/variations/fieldtrial_testing_config_mac.json
b/testing/variations/fieldtrial_testing_config_mac.json
> index
b04d535db70e32cb003c91d8afdf77f23f329457..acc07ea463517d6094cda1257c383e3282f287d2
100644
> --- a/testing/variations/fieldtrial_testing_config_mac.json
> +++ b/testing/variations/fieldtrial_testing_config_mac.json
> @@ -297,15 +297,6 @@
> "group_name": "EnabledTranslateLanguage"
> }
> ],
> - "V8SerializeOptions": [
> - {
> - "enable_features": [
> - "V8_Serialize_Eager",
> - "V8_Serialize_Age_Code"
> - ],
> - "group_name": "SerializeEagerAndAgeCode"
> - }
> - ],
> "VarationsServiceControl": [
> {
> "group_name": "Interval_30min",
> Index: testing/variations/fieldtrial_testing_config_win.json
> diff --git a/testing/variations/fieldtrial_testing_config_win.json
b/testing/variations/fieldtrial_testing_config_win.json
> index
709c46932b5a83fc49f4554287aa1e187d62e2c4..1f2ed924495a8cf894e261e051aefa85691adee6
100644
> --- a/testing/variations/fieldtrial_testing_config_win.json
> +++ b/testing/variations/fieldtrial_testing_config_win.json
> @@ -366,15 +366,6 @@
> "group_name": "On"
> }
> ],
> - "V8SerializeOptions": [
> - {
> - "enable_features": [
> - "V8_Serialize_Eager",
> - "V8_Serialize_Age_Code"
> - ],
> - "group_name": "SerializeEagerAndAgeCode"
> - }
> - ],
> "VarationsServiceControl": [
> {
> "group_name": "Interval_30min",
> Index: tools/perf/docs/perf_bot_sheriffing.md
> diff --git a/tools/perf/docs/perf_bot_sheriffing.md b/tools/perf/docs/
perf_bot_sheriffing.md
> index
e1ea36ea4ac7f32cb90cf936fade65ae168ecf8d..fd4bea9b488b3dc74881420d7960eb0f596d0720
100644
> --- a/tools/perf/docs/perf_bot_sheriffing.md
> +++ b/tools/perf/docs/perf_bot_sheriffing.md
> @@ -243,6 +243,7 @@ is the [list of Pri-1 bugs that have not been pinged
today](https://bugs.chromiu
> **[Pri-2 bugs](
https://bugs.chromium.org/p/chromium/issues/list?can=2&q=label%3APerformance-...
> are for disabled tests. These should be pinged weekly, and work towards
fixing
> should be ongoing when the sheriff is not working on a Pri-1 issue. Here
is the
> +
> [list of Pri-2 bugs that have not been pinged in a week](
https://bugs.chromium.org/p/chromium/issues/list?can=2&q=label:Performance-Bo...
).
>
> <!-- Unresolved issues:
>
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Alexei Svitkine (slow), 2016-04-22 14:39:59]

lgtm

[sullivan, 2016-04-22 15:07:42]

The CQ bit was checked by sullivan@chromium.org

[commit-bot: I haz the power, 2016-04-22 15:07:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1907093002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1907093002/1

[commit-bot: I haz the power, 2016-04-22 18:13:24]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:49:47]

Description was changed from

==========
Remove V8 serialize options from fieldtrial config.

This experiment isn't meant to launch to stable, and we want to use the
fieldtrial configs to test what's going to launch to stable.

BUG=601894
==========

to

==========
Remove V8 serialize options from fieldtrial config.

This experiment isn't meant to launch to stable, and we want to use the
fieldtrial configs to test what's going to launch to stable.

BUG=601894

Committed: https://crrev.com/dbb9b030f66920877033bcca14bb9d9ca046be4e
Cr-Commit-Position: refs/heads/master@{#389165}
==========

[commit-bot: I haz the power, 2016-04-22 19:49:48]

Patchset 1 (id:??) landed as
https://crrev.com/dbb9b030f66920877033bcca14bb9d9ca046be4e
Cr-Commit-Position: refs/heads/master@{#389165}