Add ApplicationImplBase::Bind().

Add ApplicationImplBase::Bind().

(And make the constructor not take an Application request.)

This is to allow implementations to be instantiated before the
message/run loop.

R=vardhan@google.com

Committed: https://chromium.googlesource.com/external/mojo/+/21a4f27fb68ddfa590f5d34b0543cf56357d1a63

[vardhan, 2016-05-19 02:39:36]

lgtm

https://codereview.chromium.org/1993743004/diff/1/mojo/ui/content_viewer_app.cc
File mojo/ui/content_viewer_app.cc (right):

https://codereview.chromium.org/1993743004/diff/1/mojo/ui/content_viewer_app....
mojo/ui/content_viewer_app.cc:68: // TODO(vtl): This is leaky, since
|ApplicationImpl| doesn't own itself.
cc jeffbrown?

[viettrungluu, 2016-05-19 16:53:24]

Description was changed from

==========
Add ApplicationImplBase::Bind().

(And make the constructor not take an Application request.)

This is to allow implementations to be instantiated before the
message/run loop.

R=vardhan@google.com
==========

to

==========
Add ApplicationImplBase::Bind().

(And make the constructor not take an Application request.)

This is to allow implementations to be instantiated before the
message/run loop.

R=vardhan@google.com

Committed:
https://chromium.googlesource.com/external/mojo/+/21a4f27fb68ddfa590f5d34b054...
==========

[viettrungluu, 2016-05-19 16:53:26]

Committed patchset #1 (id:1) manually as
21a4f27fb68ddfa590f5d34b0543cf56357d1a63 (presubmit successful).