Use Binding<> instead of InterfaceImpl in ViewManagerClientImpl

Use Binding<> instead of InterfaceImpl in ViewManagerClientImpl

This way we can set up the service_ connection in the constructor
instead of in a separate function.

R=sky@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/38a20faf4f91db3016be479885cbdf3585118f62

[jamesr, 2014-11-13 00:30:13]

jamesr@chromium.org changed reviewers:
+ sky@chromium.org

[sky, 2014-11-13 00:34:39]

LGTM

https://codereview.chromium.org/713313002/diff/60001/mojo/services/public/cpp...
File mojo/services/public/cpp/view_manager/lib/view_manager_client_impl.cc
(right):

https://codereview.chromium.org/713313002/diff/60001/mojo/services/public/cpp...
mojo/services/public/cpp/view_manager/lib/view_manager_client_impl.cc:104:
service_ = binding_.client();
move to initializer list? You'll have to move service_ to after binding_ in the
declaration, but that should be fine.

https://codereview.chromium.org/713313002/diff/60001/mojo/services/public/cpp...
File mojo/services/public/cpp/view_manager/lib/view_manager_client_impl.h
(right):

https://codereview.chromium.org/713313002/diff/60001/mojo/services/public/cpp...
mojo/services/public/cpp/view_manager/lib/view_manager_client_impl.h:155: bool
delete_on_error_;
const

[jamesr, 2014-11-13 21:52:45]

Committed patchset #4 (id:60001) manually as
38a20faf4f91db3016be479885cbdf3585118f62 (presubmit successful).