Fix sporadic crash at the end of gles2 ppapi plugin

Fix sporadic crash at the end of gles2 ppapi plugin

Fix plugin so that frames can still be painted after the decoder has
been destroyed.

BUG=NONE
TEST=gles2 consistently runs without crashing

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=96196

[vrk (LEFT CHROMIUM), 2011-08-09 22:14:58]

PictureReady() was still being called after the DecoderFlushDone() destroyed the
video_decoder_ field.

However, it wasn't the plugin calling PictureReady() but the PaintFinished()
callback was calling PictureReady on the saved |pictures_pending_paint_| list.
This used video_decoder_->pp_resource() as a param to PictureReady() and
therefore segfaulted in this scenario.

[Ami GONE FROM CHROMIUM, 2011-08-09 22:25:16]

http://codereview.chromium.org/7607014/diff/1/ppapi/examples/gles2/gles2.cc
File ppapi/examples/gles2/gles2.cc (right):

http://codereview.chromium.org/7607014/diff/1/ppapi/examples/gles2/gles2.cc#n...
ppapi/examples/gles2/gles2.cc:396: void GLES2DemoInstance::Render(const
PP_PictureBuffer_Dev& buffer) {
From offline convo:
- inline Render into PaintPicture, renaming it to PaintStart
- replace the while() loop at the bottom of PaintFinished with an if test, since
it can only run one iteration, anyway.

[vrk (LEFT CHROMIUM), 2011-08-09 22:44:39]

http://codereview.chromium.org/7607014/diff/1/ppapi/examples/gles2/gles2.cc
File ppapi/examples/gles2/gles2.cc (right):

http://codereview.chromium.org/7607014/diff/1/ppapi/examples/gles2/gles2.cc#n...
ppapi/examples/gles2/gles2.cc:396: void GLES2DemoInstance::Render(const
PP_PictureBuffer_Dev& buffer) {
On 2011/08/09 22:25:16, Ami Fischman wrote:
> From offline convo:
> - inline Render into PaintPicture, renaming it to PaintStart
> - replace the while() loop at the bottom of PaintFinished with an if test,
since
> it can only run one iteration, anyway.

Done.

[Ami GONE FROM CHROMIUM, 2011-08-09 23:02:12]

LGTM

[vrk (LEFT CHROMIUM), 2011-08-09 23:09:33]

+brettw OWNERS review?

Also added OWNERS file in the gles2 directory.

[brettw, 2011-08-10 16:00:10]

LGTM