Implement MojoGetBufferInformation(), part 1.

Implement MojoGetBufferInformation(), part 1.

This implements it at the Dispatcher level; it still needs to be plumbed
into Core and then the thunk table (and the NaCl SFI equivalent?).

R=vardhan@google.com

Committed: https://chromium.googlesource.com/external/mojo/+/63d9b93828f83a628dbeae7e3da907a007c5b3f3

[vardhan, 2016-03-08 21:36:57]

lgtm

https://codereview.chromium.org/1776033002/diff/1/mojo/edk/system/shared_buff...
File mojo/edk/system/shared_buffer_dispatcher.cc (right):

https://codereview.chromium.org/1776033002/diff/1/mojo/edk/system/shared_buff...
mojo/edk/system/shared_buffer_dispatcher.cc:241: // definition, more work will
be necessary. (See the definition of
nit: forgot to close the "(" (the one before "See"

[viettrungluu, 2016-03-08 21:43:46]

Thanks.

On 2016/03/08 21:36:57, vardhan wrote:
> lgtm
> 
>
https://codereview.chromium.org/1776033002/diff/1/mojo/edk/system/shared_buff...
> File mojo/edk/system/shared_buffer_dispatcher.cc (right):
> 
>
https://codereview.chromium.org/1776033002/diff/1/mojo/edk/system/shared_buff...
> mojo/edk/system/shared_buffer_dispatcher.cc:241: // definition, more work will
> be necessary. (See the definition of
> nit: forgot to close the "(" (the one before "See"

Nooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo.
Done.)

[viettrungluu, 2016-03-08 21:44:04]

Description was changed from

==========
Implement MojoGetBufferInformation(), part 1.

This implements it at the Dispatcher level; it still needs to be plumbed
into Core and then the thunk table (and the NaCl SFI equivalent?).

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

to

==========
Implement MojoGetBufferInformation(), part 1.

This implements it at the Dispatcher level; it still needs to be plumbed
into Core and then the thunk table (and the NaCl SFI equivalent?).

R=vardhan@google.com

Committed:
https://chromium.googlesource.com/external/mojo/+/63d9b93828f83a628dbeae7e3da...
==========

[viettrungluu, 2016-03-08 21:44:06]

Committed patchset #2 (id:20001) manually as
63d9b93828f83a628dbeae7e3da907a007c5b3f3 (presubmit successful).