Let canvas decide how to handle the lost resource: WebGL part

If the client code reports the resource is lost, the mailbox should be
removed and also release the reference to DrawingBuffer, so that make
sure the DrawingBuffer has a chance to be destroyed.
    
BUG=390960
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179089

[Hongbo Min, 2014-07-24 14:56:55]

kbr@, this CL is for WebGL canvas to handle the case when the resource is lost,
reported from client code.

Could you please take a review? Thanks.

[dshwang, 2014-07-28 06:56:57]

non-owner lgtm

[Ken Russell (switch to Gerrit), 2014-07-28 21:25:13]

LGTM

https://codereview.chromium.org/415123002/diff/1/Source/platform/graphics/gpu...
File Source/platform/graphics/gpu/DrawingBufferTest.cpp (right):

https://codereview.chromium.org/415123002/diff/1/Source/platform/graphics/gpu...
Source/platform/graphics/gpu/DrawingBufferTest.cpp:318:
TEST_F(DrawingBufferTest, verifyClearDrawingBufferIfResourceLost)
This test looks pretty good, but the name could be clearer. Could you consider
naming it something like verifyDrawingBufferStaysAliveIfResourcesAreLost?

[Hongbo Min, 2014-07-29 00:46:47]

https://codereview.chromium.org/415123002/diff/1/Source/platform/graphics/gpu...
File Source/platform/graphics/gpu/DrawingBufferTest.cpp (right):

https://codereview.chromium.org/415123002/diff/1/Source/platform/graphics/gpu...
Source/platform/graphics/gpu/DrawingBufferTest.cpp:318:
TEST_F(DrawingBufferTest, verifyClearDrawingBufferIfResourceLost)
On 2014/07/28 21:25:13, Ken Russell wrote:
> This test looks pretty good, but the name could be clearer. Could you consider
> naming it something like verifyDrawingBufferStaysAliveIfResourcesAreLost?

Done.

[Hongbo Min, 2014-07-29 00:46:49]

https://codereview.chromium.org/415123002/diff/1/Source/platform/graphics/gpu...
File Source/platform/graphics/gpu/DrawingBufferTest.cpp (right):

https://codereview.chromium.org/415123002/diff/1/Source/platform/graphics/gpu...
Source/platform/graphics/gpu/DrawingBufferTest.cpp:318:
TEST_F(DrawingBufferTest, verifyClearDrawingBufferIfResourceLost)
On 2014/07/28 21:25:13, Ken Russell wrote:
> This test looks pretty good, but the name could be clearer. Could you consider
> naming it something like verifyDrawingBufferStaysAliveIfResourcesAreLost?

Done.

[Hongbo Min, 2014-07-29 00:47:07]

The CQ bit was checked by hongbo.min@intel.com

[commit-bot: I haz the power, 2014-07-29 00:48:14]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/hongbo.min@intel.com/415123002/20001

[commit-bot: I haz the power, 2014-07-29 02:38:14]

Change committed as 179089