sync compostor: Return resources async IPC

sync compostor: Return resources async IPC

Small step toward overall refactor outlined in bug. Make returning
resources an plain old async IPC that's sent whenever there are return
resources, and remove it from CommonBrowserParams that's sent in every
IPC.

On the renderer side, add OnMessageReceived to OutputSurface, and have
OuputSurface get the first chance of handling IPCs before Proxy.
Eventual aim is Proxy should not handle any IPC and should just be
removed.

BUG=609977
Committed: https://crrev.com/ec72a2edffebb31179ed7b1812ecb94509f8adfe
Cr-Commit-Position: refs/heads/master@{#392499}

[boliu, 2016-05-07 00:22:23]

boliu@chromium.org changed reviewers:
+ dcheng@chromium.org, hush@chromium.org

[boliu, 2016-05-07 00:22:24]

ptal

dcheng for ipc
hush for the rest

[dcheng, 2016-05-07 06:02:18]

rs lgtm for ipc changes

[hush (inactive), 2016-05-09 23:52:10]

lgtm

I supposed you are gonna do this IPC by IPC?

https://codereview.chromium.org/1958803003/diff/1/content/renderer/android/sy...
File content/renderer/android/synchronous_compositor_output_surface.h (left):

https://codereview.chromium.org/1958803003/diff/1/content/renderer/android/sy...
content/renderer/android/synchronous_compositor_output_surface.h:96: size_t
GetMemoryPolicy() const {
so... this function is not used by anybody?

[boliu, 2016-05-10 00:00:45]

On 2016/05/09 23:52:10, hush wrote:
> lgtm
> 
> I supposed you are gonna do this IPC by IPC?

Probably. In order of ease. And data that don't need to be sent and received
synchronously are the easiest.

https://codereview.chromium.org/1958803003/diff/1/content/renderer/android/sy...
File content/renderer/android/synchronous_compositor_output_surface.h (left):

https://codereview.chromium.org/1958803003/diff/1/content/renderer/android/sy...
content/renderer/android/synchronous_compositor_output_surface.h:96: size_t
GetMemoryPolicy() const {
On 2016/05/09 23:52:10, hush wrote:
> so... this function is not used by anybody?

nope

[boliu, 2016-05-10 00:00:55]

The CQ bit was checked by boliu@chromium.org

[commit-bot: I haz the power, 2016-05-10 00:01:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1958803003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1958803003/1

[commit-bot: I haz the power, 2016-05-10 00:52:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-10 00:53:11]

Description was changed from

==========
sync compostor: Return resources async IPC

Small step toward overall refactor outlined in bug. Make returning
resources an plain old async IPC that's sent whenever there are return
resources, and remove it from CommonBrowserParams that's sent in every
IPC.

On the renderer side, add OnMessageReceived to OutputSurface, and have
OuputSurface get the first chance of handling IPCs before Proxy.
Eventual aim is Proxy should not handle any IPC and should just be
removed.

BUG=609977
==========

to

==========
sync compostor: Return resources async IPC

Small step toward overall refactor outlined in bug. Make returning
resources an plain old async IPC that's sent whenever there are return
resources, and remove it from CommonBrowserParams that's sent in every
IPC.

On the renderer side, add OnMessageReceived to OutputSurface, and have
OuputSurface get the first chance of handling IPCs before Proxy.
Eventual aim is Proxy should not handle any IPC and should just be
removed.

BUG=609977

Committed: https://crrev.com/ec72a2edffebb31179ed7b1812ecb94509f8adfe
Cr-Commit-Position: refs/heads/master@{#392499}
==========

[commit-bot: I haz the power, 2016-05-10 00:53:11]

Patchset 1 (id:??) landed as
https://crrev.com/ec72a2edffebb31179ed7b1812ecb94509f8adfe
Cr-Commit-Position: refs/heads/master@{#392499}