Make WaitSyncPoint go through command buffers.

Make WaitSyncPoint go through command buffers.

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

[piman, 2013-02-26 01:36:43]

gman: please review
jschuh: OWNERS for gpu_messages.h (removing an IPC, moving it to command
buffers).

[greggman, 2013-02-26 02:07:15]

Looks like InsertSyncPointCHROMIUM is missing in cmd_buffer_functions.txt etc.

no?

[piman, 2013-02-26 02:19:24]

On Mon, Feb 25, 2013 at 6:07 PM, <gman@chromium.org> wrote:

> Looks like InsertSyncPointCHROMIUM is missing in cmd_buffer_functions.txt
> etc.
>

That one won't move to command buffers, because it would need a Finish to
get the sync point name (being global it needs to be allocated by the GPU
process) - and that would defeat the point of this.
I'm planning to expose it in GLES2Interface though, but was planning to do
so in a follow-up.


> no?
>
>
https://codereview.chromium.**org/12330129/<https://codereview.chromium.org/1...
>

[greggman, 2013-02-26 02:24:34]

lgtm

okay then,

[jschuh, 2013-02-26 20:25:01]

To expedite, it looks like you're just moving that ID from straight IPC into the
command buffer?

[piman, 2013-02-26 20:59:47]

On Tue, Feb 26, 2013 at 12:25 PM, <jschuh@chromium.org> wrote:

> To expedite, it looks like you're just moving that ID from straight IPC
> into the
> command buffer?
>

Exactly.


>
>
https://codereview.chromium.**org/12330129/<https://codereview.chromium.org/1...
>

[jschuh, 2013-02-26 21:05:27]

okay. not really different from a security perspective, so rubberstamp lgtm.

[commit-bot: I haz the power, 2013-02-26 21:12:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/12330129/3001

[commit-bot: I haz the power, 2013-02-27 01:29:53]

Change committed as 184820