cc: Remove some DCHECKs in CopyOutputRequest code.

cc: Remove some DCHECKs in CopyOutputRequest code.

These DCHECKs fire if we ever commit to the impl side twice without
performing a draw in between. There are various ways that this
could happen, and it doesn't really matter to the CopyOutputRequest
code anyways, so remove the DCHECKs and deal with the case gracefully
instead.

Test: LayerTreeHostTestAsyncTwoReadbacksWithoutDraw

BUG=263486
R=enne,piman

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

[enne (OOO), 2013-07-23 18:33:36]

Is it easy to modify one of your existing unit tests to exercise this?

[piman, 2013-07-23 18:34:05]

lgtm

[danakj, 2013-07-23 18:35:42]

On 2013/07/23 18:33:36, enne wrote:
> Is it easy to modify one of your existing unit tests to exercise this?

I'll try!

[danakj, 2013-07-23 18:59:45]

Test added LayerTreeHostTestAsyncTwoReadbacksWithoutDraw

[enne (OOO), 2013-07-23 19:00:50]

lgtm, thanks!

[commit-bot: I haz the power, 2013-07-23 19:06:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/danakj@chromium.org/19668008/13001

[commit-bot: I haz the power, 2013-07-23 19:06:32]

Failed to trigger a try job on chromium_presubmit
HTTP Error 400: Bad Request

[commit-bot: I haz the power, 2013-07-23 19:09:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/danakj@chromium.org/19668008/18001

[commit-bot: I haz the power, 2013-07-23 23:30:13]

Change committed as 213270