Remove InstanceID field trial, since it is always enabled

Remove InstanceID field trial, since it is always enabled

This will allow other parts of Chrome to depend on the InstanceID API.

BUG=477084, 589461

[johnme, 2016-02-24 13:34:49]

johnme@chromium.org changed reviewers:
+ holte@chromium.org, zea@chromium.org

[johnme, 2016-02-24 13:34:49]

zea@chromium.org: Please review changes in components/gcm_driver/. Also, is
there some other kill switch we should be checking here? I can't find any
overall kill switch for Chrome GCM.

holte@chromium.org: Please review changes in testing/variations/

[Nicolas Zea, 2016-02-24 19:50:42]

zea@chromium.org changed reviewers:
+ jianli@chromium.org

[Nicolas Zea, 2016-02-24 19:50:43]

+Jian to review, since he added that field trial.

[jianli, 2016-02-24 20:11:06]

This indeed is used as a temporary kill switch solution, required by the launch.
We will switch to kill switch provided by feature API when it is ready.

[johnme, 2016-02-25 12:32:09]

On 2016/02/24 20:11:06, jianli wrote:
> This indeed is used as a temporary kill switch solution, required by the
launch.
> We will switch to kill switch provided by feature API when it is ready.

Is the kill switch meant to apply to all uses of InstanceID, or just for the
extensions API? I'm about to move Web Push over to InstanceID - does it make
sense for us to share this kill switch?

[jianli, 2016-02-29 22:43:53]

On 2016/02/25 12:32:09, johnme wrote:
> On 2016/02/24 20:11:06, jianli wrote:
> > This indeed is used as a temporary kill switch solution, required by the
> launch.
> > We will switch to kill switch provided by feature API when it is ready.
> 
> Is the kill switch meant to apply to all uses of InstanceID, or just for the
> extensions API? I'm about to move Web Push over to InstanceID - does it make
> sense for us to share this kill switch?

Yes, it applies to all uses. This will only be deployed when something goes
terribly wrong that would affect all the uses of InstanceID. So I think we
should keep as it is. You can add a kill switch specially for web push if you
feel it is needed.

[johnme, 2016-03-08 13:29:46]

Description was changed from

==========
Remove InstanceID field trial, since it is always enabled

This will allow other parts of Chrome to depend on the InstanceID API.

BUG=477084,589461
==========

to

==========
Remove InstanceID field trial, since it is always enabled

This will allow other parts of Chrome to depend on the InstanceID API.

BUG=477084,589461
==========

[johnme, 2016-03-08 13:30:42]

On 2016/02/29 22:43:53, jianli wrote:
> On 2016/02/25 12:32:09, johnme wrote:
> > Is the kill switch meant to apply to all uses of InstanceID, or just for the
> > extensions API? I'm about to move Web Push over to InstanceID - does it make
> > sense for us to share this kill switch?
> 
> Yes, it applies to all uses. This will only be deployed when something goes
> terribly wrong that would affect all the uses of InstanceID. So I think we
> should keep as it is. You can add a kill switch specially for web push if you
> feel it is needed.

Sounds reasonable to share it for now, thanks. Abandoning this CL.