Revise InterfaceRegistry API to support filtering interfaces @ Bind() time.

Revise InterfaceRegistry API to support filtering interfaces @ Bind() time.

InterfaceRegistry is now constructed with local service info: the local service identity and the spec exposed via the IR.
Later, Bind() is called with the IPRequest & the remote service identity & spec.

This moves interface intersection logic of specs to the client library from the service manager, & removes the allowed_interfaces and allowed_capabilities parameters from service.mojom:Service::OnConnect() which are now redundant.

Hooks this up for connection-level specs in service manager & client library (service_context.cc

R=rockot@chromium.org,tsepez@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/c87e52bc73fe59a5d460cd2e60ff7838f79d8ebf
Cr-Commit-Position: refs/heads/master@{#427771}

[Ben Goodger (Google), 2016-10-25 20:15:53]

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

[commit-bot: I haz the power, 2016-10-25 20:16:22]

Dry run: 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-10-25 20:25:40]

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

[commit-bot: I haz the power, 2016-10-25 20:25:41]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[Ben Goodger (Google), 2016-10-25 21:37:49]

Description was changed from

==========
Revise InterfaceRegistry API to support filtering interfaces @ Bind() time.

BUG=
==========

to

==========
Revise InterfaceRegistry API to support filtering interfaces @ Bind() time.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[Ben Goodger (Google), 2016-10-25 21:37:53]

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

[commit-bot: I haz the power, 2016-10-25 21:38:32]

Dry run: 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-10-25 21:53:23]

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

[commit-bot: I haz the power, 2016-10-25 21:53:24]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Ben Goodger (Google), 2016-10-25 21:53:40]

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

[commit-bot: I haz the power, 2016-10-25 21:54:07]

Dry run: 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-10-25 22:09:40]

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

[commit-bot: I haz the power, 2016-10-25 22:09:41]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Ben Goodger (Google), 2016-10-25 23:24:51]

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

[commit-bot: I haz the power, 2016-10-25 23:25:20]

Dry run: 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-10-25 23:36:49]

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

[commit-bot: I haz the power, 2016-10-25 23:36:50]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)

[Ben Goodger (Google), 2016-10-25 23:37:05]

Description was changed from

==========
Revise InterfaceRegistry API to support filtering interfaces @ Bind() time.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Revise InterfaceRegistry API to support filtering interfaces @ Bind() time.

InterfaceRegistry is now constructed with local service info: the local service
identity and the spec exposed via the IR.
Later, Bind() is called with the IPRequest & the remote service identity & spec.

This moves interface intersection logic of specs to the client library from the
service manager, & removes the allowed_interfaces and allowed_capabilities
parameters from service.mojom:Service::OnConnect() which are now redundant.

Hooks this up for connection-level specs in service manager & client library
(service_context.cc

R=rockot@chromium.org,tsepez@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[Ben Goodger (Google), 2016-10-25 23:37:05]

ben@chromium.org changed reviewers:
+ rockot@chromium.org, tsepez@chromium.org

[Ben Goodger (Google), 2016-10-25 23:38:40]

ken: *
tom: service.mojom

[Ben Goodger (Google), 2016-10-25 23:39:09]

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

[commit-bot: I haz the power, 2016-10-25 23:39:32]

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

[Tom Sepez, 2016-10-25 23:48:37]

Mojom LGTM per conversation.

[commit-bot: I haz the power, 2016-10-25 23:52:30]

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

[commit-bot: I haz the power, 2016-10-25 23:52:30]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)

[Ken Rockot(use gerrit already), 2016-10-25 23:52:44]

LGTM - some red (iOS code creates an InterfaceRegistry, and looks like blimp is
busted)

[Ben Goodger (Google), 2016-10-26 00:05:17]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-10-26 00:05:18]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
rockot@chromium.org
Link to the patchset: https://codereview.chromium.org/2446313003/#ps100001
(title: ".")

[commit-bot: I haz the power, 2016-10-26 00:05:47]

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-10-26 00:21:53]

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

[commit-bot: I haz the power, 2016-10-26 00:21:54]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Ben Goodger (Google), 2016-10-26 02:36:10]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-10-26 02:36:10]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
rockot@chromium.org
Link to the patchset: https://codereview.chromium.org/2446313003/#ps120001
(title: ".")

[commit-bot: I haz the power, 2016-10-26 02:36:34]

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-10-26 02:52:10]

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

[commit-bot: I haz the power, 2016-10-26 02:52:11]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Ben Goodger (Google), 2016-10-26 05:40:51]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-10-26 05:40:51]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
rockot@chromium.org
Link to the patchset: https://codereview.chromium.org/2446313003/#ps140001
(title: ".")

[commit-bot: I haz the power, 2016-10-26 05:41:12]

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-10-26 06:18:16]

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

[commit-bot: I haz the power, 2016-10-26 06:18:17]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Ben Goodger (Google), 2016-10-26 06:37:42]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-10-26 06:37:42]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
rockot@chromium.org
Link to the patchset: https://codereview.chromium.org/2446313003/#ps160001
(title: ".")

[commit-bot: I haz the power, 2016-10-26 06:37:54]

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-10-26 08:53:49]

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

[commit-bot: I haz the power, 2016-10-26 08:53:50]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Ben Goodger (Google), 2016-10-26 17:20:24]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-10-26 17:20:25]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
rockot@chromium.org
Link to the patchset: https://codereview.chromium.org/2446313003/#ps180001
(title: ".")

[commit-bot: I haz the power, 2016-10-26 17:20:41]

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-10-26 17:29:06]

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

[commit-bot: I haz the power, 2016-10-26 17:29:07]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)

[Ben Goodger (Google), 2016-10-26 17:39:10]

The CQ bit was checked by ben@chromium.org

[commit-bot: I haz the power, 2016-10-26 17:39:54]

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-10-26 19:07:15]

Description was changed from

==========
Revise InterfaceRegistry API to support filtering interfaces @ Bind() time.

InterfaceRegistry is now constructed with local service info: the local service
identity and the spec exposed via the IR.
Later, Bind() is called with the IPRequest & the remote service identity & spec.

This moves interface intersection logic of specs to the client library from the
service manager, & removes the allowed_interfaces and allowed_capabilities
parameters from service.mojom:Service::OnConnect() which are now redundant.

Hooks this up for connection-level specs in service manager & client library
(service_context.cc

R=rockot@chromium.org,tsepez@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Revise InterfaceRegistry API to support filtering interfaces @ Bind() time.

InterfaceRegistry is now constructed with local service info: the local service
identity and the spec exposed via the IR.
Later, Bind() is called with the IPRequest & the remote service identity & spec.

This moves interface intersection logic of specs to the client library from the
service manager, & removes the allowed_interfaces and allowed_capabilities
parameters from service.mojom:Service::OnConnect() which are now redundant.

Hooks this up for connection-level specs in service manager & client library
(service_context.cc

R=rockot@chromium.org,tsepez@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[commit-bot: I haz the power, 2016-10-26 19:07:16]

Committed patchset #10 (id:180001)

[commit-bot: I haz the power, 2016-10-26 19:22:25]

Description was changed from

==========
Revise InterfaceRegistry API to support filtering interfaces @ Bind() time.

InterfaceRegistry is now constructed with local service info: the local service
identity and the spec exposed via the IR.
Later, Bind() is called with the IPRequest & the remote service identity & spec.

This moves interface intersection logic of specs to the client library from the
service manager, & removes the allowed_interfaces and allowed_capabilities
parameters from service.mojom:Service::OnConnect() which are now redundant.

Hooks this up for connection-level specs in service manager & client library
(service_context.cc

R=rockot@chromium.org,tsepez@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Revise InterfaceRegistry API to support filtering interfaces @ Bind() time.

InterfaceRegistry is now constructed with local service info: the local service
identity and the spec exposed via the IR.
Later, Bind() is called with the IPRequest & the remote service identity & spec.

This moves interface intersection logic of specs to the client library from the
service manager, & removes the allowed_interfaces and allowed_capabilities
parameters from service.mojom:Service::OnConnect() which are now redundant.

Hooks this up for connection-level specs in service manager & client library
(service_context.cc

R=rockot@chromium.org,tsepez@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

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

[commit-bot: I haz the power, 2016-10-26 19:22:26]

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