Mojo: Make Core own a PlatformSupport, and plumb it through to Channel.

Mojo: Make Core own a PlatformSupport, and plumb it through to Channel.

And make SharedBufferDispatcher::Deserialize() use the PlatformSupport
from the Channel that's passed to it.

(For now, just initialize Core's platform_dispatcher_ with a
SimplePlatformSupport. Next, I'll have to add it to Core's ctor and do
all the required plumbing for that.)

R=darin@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290475

[darin (slow to review), 2014-08-18 23:42:58]

LGTM modulo comments below:

https://codereview.chromium.org/484893004/diff/1/mojo/system/channel.h
File mojo/system/channel.h (right):

https://codereview.chromium.org/484893004/diff/1/mojo/system/channel.h#newcode63
mojo/system/channel.h:63: // |platform_support| (typically owned by |Core|) must
remain alive |Channel|
nit: errant "|Channel|" in this comment?

https://codereview.chromium.org/484893004/diff/1/mojo/tools/data/unittests
File mojo/tools/data/unittests (right):

https://codereview.chromium.org/484893004/diff/1/mojo/tools/data/unittests#ne...
mojo/tools/data/unittests:18: #mojo_view_manager_lib_unittests
Is this change intentional?

[viettrungluu, 2014-08-19 00:02:32]

Thanks.

https://codereview.chromium.org/484893004/diff/1/mojo/system/channel.h
File mojo/system/channel.h (right):

https://codereview.chromium.org/484893004/diff/1/mojo/system/channel.h#newcode63
mojo/system/channel.h:63: // |platform_support| (typically owned by |Core|) must
remain alive |Channel|
On 2014/08/18 23:42:58, darin wrote:
> nit: errant "|Channel|" in this comment?

Done.

https://codereview.chromium.org/484893004/diff/1/mojo/tools/data/unittests
File mojo/tools/data/unittests (right):

https://codereview.chromium.org/484893004/diff/1/mojo/tools/data/unittests#ne...
mojo/tools/data/unittests:18: #mojo_view_manager_lib_unittests
On 2014/08/18 23:42:58, darin wrote:
> Is this change intentional?

Oops, no.

[viettrungluu, 2014-08-19 00:02:44]

The CQ bit was checked by viettrungluu@chromium.org

[commit-bot: I haz the power, 2014-08-19 00:05:48]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/viettrungluu@chromium.org/484893004/20001

[commit-bot: I haz the power, 2014-08-19 03:34:39]

Committed patchset #2 (20001) as 290475