[Origin Trials] Only flag feature bindings installed after all install methods have returned

[Origin Trials] Only flag feature bindings installed after all install methods have returned

This fixes an issue which we haven't run into yet, but will as soon as
an origin trial feature tries to install bindings onto core *and*
modules at the same time. We need to ensure that we only flag the
feature as having had its bindings installed once *all* installation
functions have had a chance to install them.

Committed: https://crrev.com/7e54fd8b37da95028478b77c1aa524ba20bba393
Cr-Commit-Position: refs/heads/master@{#401888}

[iclelland, 2016-06-23 14:49:29]

iclelland@chromium.org changed reviewers:
+ haraken@chromium.org

[iclelland, 2016-06-23 14:49:29]

+r haraken, PTAL, thanks!

[haraken, 2016-06-24 01:01:58]

LGTM but I'd like to consider a way to auto-generate the code to flip the flag.

[iclelland, 2016-06-24 03:38:44]

On 2016/06/24 01:01:58, haraken wrote:
> LGTM but I'd like to consider a way to auto-generate the code to flip the
flag.

Thanks -- and I completely agree; generating this code is one of my goals for
Q3.

[iclelland, 2016-06-24 16:06:38]

The CQ bit was checked by iclelland@chromium.org

[commit-bot: I haz the power, 2016-06-24 16:07:17]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-24 17:17:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-24 17:20:37]

Description was changed from

==========
[Origin Trials] Only flag feature bindings installed after all install methods
have returned

This fixes an issue which we haven't run into yet, but will as soon as
an origin trial feature tries to install bindings onto core *and*
modules at the same time. We need to ensure that we only flag the
feature as having had its bindings installed once *all* installation
functions have had a chance to install them.
==========

to

==========
[Origin Trials] Only flag feature bindings installed after all install methods
have returned

This fixes an issue which we haven't run into yet, but will as soon as
an origin trial feature tries to install bindings onto core *and*
modules at the same time. We need to ensure that we only flag the
feature as having had its bindings installed once *all* installation
functions have had a chance to install them.

Committed: https://crrev.com/7e54fd8b37da95028478b77c1aa524ba20bba393
Cr-Commit-Position: refs/heads/master@{#401888}
==========

[commit-bot: I haz the power, 2016-06-24 17:20:38]

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