Remove shell::Connection* parameter to InterfaceFactory<T>::Create()

Remove shell::Connection* parameter to InterfaceFactory<T>::Create()

Replace with shell::Identity, since that's all anyone used.

R=rockot@chromium.org
BUG=

Committed: https://crrev.com/0b326f1b91f8f70f94ddb11e51c15beb6cb2e286
Cr-Commit-Position: refs/heads/master@{#407854}

[Ben Goodger (Google), 2016-07-25 22:10:16]

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

[commit-bot: I haz the power, 2016-07-25 22:10:38]

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-07-25 22:17:41]

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

[commit-bot: I haz the power, 2016-07-25 22:17:42]

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...)
  ios-simulator-gn on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-gn/...)

[Ben Goodger (Google), 2016-07-25 22:30:51]

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

[commit-bot: I haz the power, 2016-07-25 22:31:13]

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-07-25 22:39:33]

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

[commit-bot: I haz the power, 2016-07-25 22:39:34]

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-07-25 22:42:03]

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

[commit-bot: I haz the power, 2016-07-25 22:42:52]

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-07-25 22:49:13]

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

[commit-bot: I haz the power, 2016-07-25 22:49:14]

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-07-25 23:49:25]

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

[commit-bot: I haz the power, 2016-07-25 23:50:06]

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-07-25 23:57:04]

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

[commit-bot: I haz the power, 2016-07-25 23:57:05]

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-07-26 00:14:46]

the few compile errors look trivial to resolve, so lgtm % those and the other
redness which at a glance appears come from InterfaceRegistry(nullptr) usage.

https://codereview.chromium.org/2182643003/diff/60001/services/shell/public/c...
File services/shell/public/cpp/lib/interface_registry.cc (right):

https://codereview.chromium.org/2182643003/diff/60001/services/shell/public/c...
services/shell/public/cpp/lib/interface_registry.cc:67:
iter->second->BindInterface(connection_->GetRemoteIdentity(),
None of the uses with a null connection need the remote identity, so can we just
pass a dummy Identity if connection_ is null?

[Ben Goodger (Google), 2016-07-26 02:50:07]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-07-26 02:50:08]

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

[commit-bot: I haz the power, 2016-07-26 02:50:24]

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-07-26 02:59:06]

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

[commit-bot: I haz the power, 2016-07-26 02:59:08]

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-07-26 13:56:41]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-07-26 13:56:41]

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

[commit-bot: I haz the power, 2016-07-26 13:56:50]

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-07-26 14:03:22]

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

[commit-bot: I haz the power, 2016-07-26 14:03:24]

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-07-26 14:17:17]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-07-26 14:17:17]

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

[commit-bot: I haz the power, 2016-07-26 14:17:33]

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-07-26 14:23:57]

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

[commit-bot: I haz the power, 2016-07-26 14:23:59]

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-07-26 14:41:26]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-07-26 14:41:26]

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

[commit-bot: I haz the power, 2016-07-26 14:41:36]

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-07-26 14:48:16]

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

[commit-bot: I haz the power, 2016-07-26 14:48:18]

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-07-26 16:57:05]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-07-26 16:57:06]

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

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

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-07-26 18:13:58]

Committed patchset #9 (id:160001)

[commit-bot: I haz the power, 2016-07-26 18:15:34]

Description was changed from

==========
Remove shell::Connection* parameter to InterfaceFactory<T>::Create()

Replace with shell::Identity, since that's all anyone used.

R=rockot@chromium.org
BUG=
==========

to

==========
Remove shell::Connection* parameter to InterfaceFactory<T>::Create()

Replace with shell::Identity, since that's all anyone used.

R=rockot@chromium.org
BUG=

Committed: https://crrev.com/0b326f1b91f8f70f94ddb11e51c15beb6cb2e286
Cr-Commit-Position: refs/heads/master@{#407854}
==========

[commit-bot: I haz the power, 2016-07-26 18:15:36]

Patchset 9 (id:??) landed as
https://crrev.com/0b326f1b91f8f70f94ddb11e51c15beb6cb2e286
Cr-Commit-Position: refs/heads/master@{#407854}