Add a integration test suite for BlobChannel.

Add a integration test suite for BlobChannel.

This test suite exercises the sender and receiver portions of
BlobChannel in integration, using a transport-agnostic proxy delegate
to connect the two together.

Also added:
* a string-based overload for CalculateBlobId();
* promoted CreateBlobDataPtr() to a common location.

R=haibinlu@chromium.org
CC=wez@chromium.org
BUG=600719
Committed: https://crrev.com/a62165f3e7ae61c2d8ddd3585aa5d120c8b670aa
Cr-Commit-Position: refs/heads/master@{#392648}

[Kevin M, 2016-05-09 23:49:20]

Description was changed from

==========
Add a integration test suite for BlobChannel.

This test suite exercises the sender and receiver portions of
BlobChannel in integration, using a transport-agnostic proxy delegate
to connect the two together.

Also added:
* a string-based overload for CalculateBlobId()
* promoted CreateBlobDataPtr() to a common location.

R=haibinlu@chromium.org
CC=wez@chromium.org
BUG=600719
==========

to

==========
Add a integration test suite for BlobChannel.

This test suite exercises the sender and receiver portions of
BlobChannel in integration, using a transport-agnostic proxy delegate
to connect the two together.

Also added:
* a string-based overload for CalculateBlobId();
* promoted CreateBlobDataPtr() to a common location.

R=haibinlu@chromium.org
CC=wez@chromium.org
BUG=600719
==========

[haibinlu, 2016-05-10 00:18:38]

lgtm

https://codereview.chromium.org/1962193002/diff/1/blimp/net/blob_channel/blob...
File blimp/net/blob_channel/blob_channel_integration_test.cc (right):

https://codereview.chromium.org/1962193002/diff/1/blimp/net/blob_channel/blob...
blimp/net/blob_channel/blob_channel_integration_test.cc:48:
receiver_->OnBlobReceived(id, data);
PostTask to simulate async nature of sender and receive communication?

[Kevin M, 2016-05-10 00:54:23]

https://codereview.chromium.org/1962193002/diff/1/blimp/net/blob_channel/blob...
File blimp/net/blob_channel/blob_channel_integration_test.cc (right):

https://codereview.chromium.org/1962193002/diff/1/blimp/net/blob_channel/blob...
blimp/net/blob_channel/blob_channel_integration_test.cc:48:
receiver_->OnBlobReceived(id, data);
On 2016/05/10 00:18:38, haibinlu wrote:
> PostTask to simulate async nature of sender and receive communication?

Done.

[Kevin M, 2016-05-10 00:55:48]

The CQ bit was checked by kmarshall@chromium.org

[Kevin M, 2016-05-10 00:55:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
haibinlu@chromium.org
Link to the patchset: https://codereview.chromium.org/1962193002/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2016-05-10 00:56:20]

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

[Kevin M, 2016-05-10 00:59:21]

The CQ bit was unchecked by kmarshall@chromium.org

[Kevin M, 2016-05-10 17:10:01]

The CQ bit was checked by kmarshall@chromium.org

[Kevin M, 2016-05-10 17:11:23]

The CQ bit was checked by kmarshall@chromium.org

[Kevin M, 2016-05-10 17:11:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
haibinlu@chromium.org
Link to the patchset: https://codereview.chromium.org/1962193002/#ps60001
(title: "remove redundant callback")

[commit-bot: I haz the power, 2016-05-10 17:11:53]

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

[commit-bot: I haz the power, 2016-05-10 18:17:01]

Description was changed from

==========
Add a integration test suite for BlobChannel.

This test suite exercises the sender and receiver portions of
BlobChannel in integration, using a transport-agnostic proxy delegate
to connect the two together.

Also added:
* a string-based overload for CalculateBlobId();
* promoted CreateBlobDataPtr() to a common location.

R=haibinlu@chromium.org
CC=wez@chromium.org
BUG=600719
==========

to

==========
Add a integration test suite for BlobChannel.

This test suite exercises the sender and receiver portions of
BlobChannel in integration, using a transport-agnostic proxy delegate
to connect the two together.

Also added:
* a string-based overload for CalculateBlobId();
* promoted CreateBlobDataPtr() to a common location.

R=haibinlu@chromium.org
CC=wez@chromium.org
BUG=600719
==========

[commit-bot: I haz the power, 2016-05-10 18:17:02]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-10 18:19:29]

Description was changed from

==========
Add a integration test suite for BlobChannel.

This test suite exercises the sender and receiver portions of
BlobChannel in integration, using a transport-agnostic proxy delegate
to connect the two together.

Also added:
* a string-based overload for CalculateBlobId();
* promoted CreateBlobDataPtr() to a common location.

R=haibinlu@chromium.org
CC=wez@chromium.org
BUG=600719
==========

to

==========
Add a integration test suite for BlobChannel.

This test suite exercises the sender and receiver portions of
BlobChannel in integration, using a transport-agnostic proxy delegate
to connect the two together.

Also added:
* a string-based overload for CalculateBlobId();
* promoted CreateBlobDataPtr() to a common location.

R=haibinlu@chromium.org
CC=wez@chromium.org
BUG=600719

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

[commit-bot: I haz the power, 2016-05-10 18:19:30]

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