Fix net_unittests build errors when enable_bidirectional_stream is set.

Fix net_unittests build errors when enable_bidirectional_stream is set.

1. Use int instead of int64.
2. Define ENABLE_BIDIRECTIONAL_STREAM on net_unittests target.

BUG=516342
Committed: https://crrev.com/e666e9a71d427a7c3efdc39b1aed8ea5cba844d4
Cr-Commit-Position: refs/heads/master@{#367205}

[mef, 2015-12-30 21:48:36]

Description was changed from

==========
Fix net_unittests build errors when enable_bidirectional_stream is set.

BUG=516342
==========

to

==========
Fix net_unittests build errors when enable_bidirectional_stream is set.

1. Use int instead of int64.
2. Define ENABLE_BIDIRECTIONAL_STREAM on net_unittests target.

BUG=516342
==========

[mef, 2015-12-30 21:48:36]

mef@chromium.org changed reviewers:
+ luqui@chromium.org, thakis@chromium.org

[mef, 2015-12-30 22:31:30]

PTAL, this fixes build errors reported on official cronet builders.

[mef, 2015-12-30 22:34:09]

On 2015/12/30 22:31:30, mef wrote:
> PTAL, this fixes build errors reported on official cronet builders.

Here is the error:

In file included from ../../net/http/bidirectional_stream.h:13:0,
                 from ../../net/http/bidirectional_stream_unittest.cc:5:
../../net/http/bidirectional_stream_job.h:16:2: error: #error Only include this
if ENABLE_BIDIRECTIONAL_STREAM is defined
 #error Only include this if ENABLE_BIDIRECTIONAL_STREAM is defined
  ^
In file included from ../../net/http/bidirectional_stream_unittest.cc:5:0:
../../net/http/bidirectional_stream.h:18:2: error: #error Only include this if
ENABLE_BIDIRECTIONAL_STREAM is defined
 #error Only include this if ENABLE_BIDIRECTIONAL_STREAM is defined

[Nico, 2015-12-30 22:36:44]

lgtm to get your bots green, but https://codereview.chromium.org/1550843002/
seems like a structurally better fix

[mef, 2015-12-30 23:15:35]

On 2015/12/30 22:36:44, Nico wrote:
> lgtm to get your bots green, but https://codereview.chromium.org/1550843002/
> seems like a structurally better fix

I agree, and will be happy to land that next week.

[mef, 2015-12-30 23:16:11]

The CQ bit was checked by mef@chromium.org

[commit-bot: I haz the power, 2015-12-30 23:16:48]

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

[commit-bot: I haz the power, 2015-12-30 23:41:36]

Description was changed from

==========
Fix net_unittests build errors when enable_bidirectional_stream is set.

1. Use int instead of int64.
2. Define ENABLE_BIDIRECTIONAL_STREAM on net_unittests target.

BUG=516342
==========

to

==========
Fix net_unittests build errors when enable_bidirectional_stream is set.

1. Use int instead of int64.
2. Define ENABLE_BIDIRECTIONAL_STREAM on net_unittests target.

BUG=516342
==========

[commit-bot: I haz the power, 2015-12-30 23:41:37]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-30 23:43:00]

Description was changed from

==========
Fix net_unittests build errors when enable_bidirectional_stream is set.

1. Use int instead of int64.
2. Define ENABLE_BIDIRECTIONAL_STREAM on net_unittests target.

BUG=516342
==========

to

==========
Fix net_unittests build errors when enable_bidirectional_stream is set.

1. Use int instead of int64.
2. Define ENABLE_BIDIRECTIONAL_STREAM on net_unittests target.

BUG=516342

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

[commit-bot: I haz the power, 2015-12-30 23:43:01]

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