[Pepper GL] Use shared state to fix a memory leak in MapSub GL extension.

When use the Pepper MapTexSubImage2DCHROMIUM() and the Compositor API together, it may cause memory leak, and then run out of memory. The problem is because the Compositor API uses sync point to sync texture producer and the Chrome compositor, it does not need any IPC, so the PpapiCommandBufferProxy::last_state_[1] will not get updated, so the client does not known when the GPU process finishes accessing the memory, and then the memory can not be reused forever. This change add a shared state in PpapiCommandBufferProxy. It allows PpapiCommandBufferProxy updating state from a shared memory without an IPC. It fix this issue.

BUG=411004
Committed: https://crrev.com/a206fb49cc1af7e022fd5d3028271323b00f3c6d
Cr-Commit-Position: refs/heads/master@{#294012}

[Peng, 2014-09-08 20:57:27]

penghuang@chromium.org changed reviewers:
+ piman@chromium.org

[piman, 2014-09-08 21:02:03]

piman@chromium.org changed reviewers:
+ tsepez@chromium.org

[piman, 2014-09-08 21:02:04]

Awesome, LGTM. Adding security person for ppapi_messages.h

https://codereview.chromium.org/547733002/diff/80001/ppapi/proxy/ppb_graphics...
File ppapi/proxy/ppb_graphics_3d_proxy.cc (right):

https://codereview.chromium.org/547733002/diff/80001/ppapi/proxy/ppb_graphics...
ppapi/proxy/ppb_graphics_3d_proxy.cc:173: //   
ppapi::proxy::SerializedHandle::SHARED_MEMORY);
nit: bogus comment?

[Peng, 2014-09-08 21:04:53]

penghuang@chromium.org changed reviewers:
+ dmichael@chromium.org

[Peng, 2014-09-08 21:05:12]

penghuang@chromium.org changed reviewers:
+ brettw@chromium.org

[Peng, 2014-09-08 21:10:35]

On 2014/09/08 21:02:04, piman (Very slow to review) wrote:
> Awesome, LGTM. Adding security person for ppapi_messages.h
> 
>
https://codereview.chromium.org/547733002/diff/80001/ppapi/proxy/ppb_graphics...
> File ppapi/proxy/ppb_graphics_3d_proxy.cc (right):
> 
>
https://codereview.chromium.org/547733002/diff/80001/ppapi/proxy/ppb_graphics...
> ppapi/proxy/ppb_graphics_3d_proxy.cc:173: //   
> ppapi::proxy::SerializedHandle::SHARED_MEMORY);
> nit: bogus comment

Hi David, PTAL. Thanks.

[Peng, 2014-09-08 21:12:23]

https://codereview.chromium.org/547733002/diff/80001/ppapi/proxy/ppb_graphics...
File ppapi/proxy/ppb_graphics_3d_proxy.cc (right):

https://codereview.chromium.org/547733002/diff/80001/ppapi/proxy/ppb_graphics...
ppapi/proxy/ppb_graphics_3d_proxy.cc:173: //   
ppapi::proxy::SerializedHandle::SHARED_MEMORY);
On 2014/09/08 21:02:04, piman (Very slow to review) wrote:
> nit: bogus comment?

Done.

[Tom Sepez, 2014-09-08 21:16:39]

Messages LGTM

[dmichael (off chromium), 2014-09-08 22:56:04]

pepper OWNER lgtm

[Peng, 2014-09-08 23:00:35]

The CQ bit was checked by penghuang@chromium.org

[commit-bot: I haz the power, 2014-09-08 23:02:02]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/penghuang@chromium.org/547733002/120001

[commit-bot: I haz the power, 2014-09-09 02:36:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-09 02:36:43]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Peng, 2014-09-09 03:36:17]

On 2014/09/09 02:36:43, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium.linux
>
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

Hi brettw, PTAL for ppapi/think/DEPS. Thanks.

[brettw, 2014-09-09 17:31:12]

I don't understand why you made the change in the deps file. Were you seeing an
error? The old version looks correct to me.

[Peng, 2014-09-09 18:17:50]

On 2014/09/09 17:31:12, brettw wrote:
> I don't understand why you made the change in the deps file. Were you seeing
an
> error? The old version looks correct to me.

I though ppapi/thunk should not use headers from base/, so I added it. Seems I
was wrong. Thanks for your comment.

[Peng, 2014-09-09 18:18:12]

The CQ bit was checked by penghuang@chromium.org

[commit-bot: I haz the power, 2014-09-09 18:21:18]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/penghuang@chromium.org/547733002/140001

[commit-bot: I haz the power, 2014-09-09 21:28:52]

Committed patchset #8 (id:140001) as b48187dc25fed1a79b5cb6b0bb55dafe4d5a0237

[commit-bot: I haz the power, 2014-09-10 03:55:45]

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