Move the suborigin experiment to test-only mode.

Move the suborigin experiment to test-only mode.

We're seeing crashes in the wild for folks with the experimental features
flag turned on, as our suborigins implementation breaks origins that
natively use an underscore in their DNS name (renderer crash with
bad_message_reason 114 (RFH_INVALID_ORIGIN_ON_COMMIT)).

This patch moves suborigins behind the test flag, so that it isn't
enabled for users who have opted into experimental web platform
features. We expect to re-enable it for those users once we've reworked
serialization in https://github.com/w3c/webappsec-suborigins/issues/38

BUG=617588
TBR=jww@chromium.org

Committed: https://crrev.com/66ab9f9074c3d1082a7ab1062914fece3810c104
Cr-Commit-Position: refs/heads/master@{#401840}

[Mike West, 2016-06-24 08:04:52]

mkwst@chromium.org changed reviewers:
+ jochen@chromium.org

[Mike West, 2016-06-24 08:04:53]

TBRing Joel for when he gets back from parental leave.

Jochen, would you mind evaluating this approach in the meantime?

[jochen (gone - plz use gerrit), 2016-06-24 08:06:39]

lgtm

[Mike West, 2016-06-24 08:22:47]

The CQ bit was checked by mkwst@chromium.org

[commit-bot: I haz the power, 2016-06-24 08:22:56]

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

[commit-bot: I haz the power, 2016-06-24 09:21:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-24 09:23:24]

Description was changed from

==========
Move the suborigin experiment to test-only mode.

We're seeing crashes in the wild for folks with the experimental features
flag turned on, as our suborigins implementation breaks origins that
natively use an underscore in their DNS name (renderer crash with
bad_message_reason 114 (RFH_INVALID_ORIGIN_ON_COMMIT)).

This patch moves suborigins behind the test flag, so that it isn't
enabled for users who have opted into experimental web platform
features. We expect to re-enable it for those users once we've reworked
serialization in https://github.com/w3c/webappsec-suborigins/issues/38

BUG=617588
TBR=jww@chromium.org
==========

to

==========
Move the suborigin experiment to test-only mode.

We're seeing crashes in the wild for folks with the experimental features
flag turned on, as our suborigins implementation breaks origins that
natively use an underscore in their DNS name (renderer crash with
bad_message_reason 114 (RFH_INVALID_ORIGIN_ON_COMMIT)).

This patch moves suborigins behind the test flag, so that it isn't
enabled for users who have opted into experimental web platform
features. We expect to re-enable it for those users once we've reworked
serialization in https://github.com/w3c/webappsec-suborigins/issues/38

BUG=617588
TBR=jww@chromium.org

Committed: https://crrev.com/66ab9f9074c3d1082a7ab1062914fece3810c104
Cr-Commit-Position: refs/heads/master@{#401840}
==========

[commit-bot: I haz the power, 2016-06-24 09:23:25]

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