📰 Throw when unregistered feature is queried in tests

[NTP Client] Throw when unregistered feature is queried in tests

Allows to register features and explicitly specifying if they should
be enabled or not, and now throws an exception when the status of
an unregistered feature is queried.

Syntax changes from
  @EnableFeatures({"Foo", "Bar"})
to
  @Features({
    @Features.Register("Foo"),
    @Features.Register(value = "Bar", enabled = true),
    @Features.Register(value = "Baz", enabled = false)})

BUG=None

Review-Url: https://codereview.chromium.org/2723003010
Cr-Commit-Position: refs/heads/master@{#454908}
Committed: https://chromium.googlesource.com/chromium/src/+/715f098c8b479c868c9541dab6cb9e33781de43f

[dgn, 2017-03-03 17:17:20]

The CQ bit was checked by dgn@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-03 17:17:40]

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

[dgn, 2017-03-03 17:30:29]

dgn@chromium.org changed reviewers:
+ bauerb@chromium.org

[dgn, 2017-03-03 17:30:29]

PTAL

[Bernhard Bauer, 2017-03-03 17:58:31]

Neat! LGTM.

[dgn, 2017-03-03 17:59:57]

The CQ bit was unchecked by dgn@chromium.org

[dgn, 2017-03-03 17:59:57]

The CQ bit was checked by dgn@chromium.org

[commit-bot: I haz the power, 2017-03-03 18:00:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-03 18:00:19]

This CL has an open dependency (Issue 2722243002 Patch 60001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[dgn, 2017-03-06 18:02:13]

The CQ bit was checked by dgn@chromium.org

[commit-bot: I haz the power, 2017-03-06 18:02:53]

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, 2017-03-06 19:09:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488823333971230, "parent_rev":
"7950721f084767700b62bb6e1c90ea155efae980", "commit_rev":
"715f098c8b479c868c9541dab6cb9e33781de43f"}

[commit-bot: I haz the power, 2017-03-06 19:09:56]

Description was changed from

==========
[NTP Client] Throw when unregistered feature is queried in tests

Allows to register features and explicitly specifying if they should
be enabled or not, and now throws an exception when the status of
an unregistered feature is queried.

Syntax changes from
  @EnableFeatures({"Foo", "Bar"})
to
  @Features({
    @Features.Register("Foo"),
    @Features.Register(value = "Bar", enabled = true),
    @Features.Register(value = "Baz", enabled = false)})

BUG=None
==========

to

==========
[NTP Client] Throw when unregistered feature is queried in tests

Allows to register features and explicitly specifying if they should
be enabled or not, and now throws an exception when the status of
an unregistered feature is queried.

Syntax changes from
  @EnableFeatures({"Foo", "Bar"})
to
  @Features({
    @Features.Register("Foo"),
    @Features.Register(value = "Bar", enabled = true),
    @Features.Register(value = "Baz", enabled = false)})

BUG=None

Review-Url: https://codereview.chromium.org/2723003010
Cr-Commit-Position: refs/heads/master@{#454908}
Committed:
https://chromium.googlesource.com/chromium/src/+/715f098c8b479c868c9541dab6cb...
==========

[commit-bot: I haz the power, 2017-03-06 19:09:57]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/715f098c8b479c868c9541dab6cb...