Remove invalid CHECK() from DesktopSessionAgent::OnCaptureCompleted.

Remove invalid CHECK() from DesktopSessionAgent::OnCaptureCompleted.

crrev.com/370480 added a CHECK() in DesktopSessionAgent. It was
added there in last minute during review and it wasn't properly
validated.

It's supposed to verify that the screen capturer actually uses
the shared memory given to it by verifying that frame->data() and
shared_memory->data() return the same address. On windows the capturer
maps uses shared_memory->handle() to allocate bitmap, so the same
file mapping ends up mapped in two different addresses. So
frame->data() and shared_memory->data() are different addresses, but
the data they contain is always the same.

BUG=575427
Committed: https://crrev.com/f545a8581d72cdf8a39d4d5fec23e728e2ca089f
Cr-Commit-Position: refs/heads/master@{#371321}

[Sergey Ulanov, 2016-01-25 20:49:23]

sergeyu@chromium.org changed reviewers:
+ joedow@chromium.org

[joedow, 2016-01-25 20:53:18]

lgtm

[Sergey Ulanov, 2016-01-25 20:55:01]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2016-01-25 20:56:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1630953003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1630953003/1

[commit-bot: I haz the power, 2016-01-25 21:41:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-25 21:44:03]

Description was changed from

==========
Remove invalid CHECK() from DesktopSessionAgent::OnCaptureCompleted.

crrev.com/370480 added a CHECK() in DesktopSessionAgent. It was
added there in last minute during review and it wasn't properly
validated.

It's supposed to verify that the screen capturer actually uses
the shared memory given to it by verifying that frame->data() and
shared_memory->data() return the same address. On windows the capturer
maps uses shared_memory->handle() to allocate bitmap, so the same
file mapping ends up mapped in two different addresses. So
frame->data() and shared_memory->data() are different addresses, but
the data they contain is always the same.

BUG=575427
==========

to

==========
Remove invalid CHECK() from DesktopSessionAgent::OnCaptureCompleted.

crrev.com/370480 added a CHECK() in DesktopSessionAgent. It was
added there in last minute during review and it wasn't properly
validated.

It's supposed to verify that the screen capturer actually uses
the shared memory given to it by verifying that frame->data() and
shared_memory->data() return the same address. On windows the capturer
maps uses shared_memory->handle() to allocate bitmap, so the same
file mapping ends up mapped in two different addresses. So
frame->data() and shared_memory->data() are different addresses, but
the data they contain is always the same.

BUG=575427

Committed: https://crrev.com/f545a8581d72cdf8a39d4d5fec23e728e2ca089f
Cr-Commit-Position: refs/heads/master@{#371321}
==========

[commit-bot: I haz the power, 2016-01-25 21:44:04]

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