Fix BidirectionalStreamTest#testFailures flake

Fix BidirectionalStreamTest#testFailures flake

Cancel during ResponseStep.ON_REQEST_HEADERS_SENT does not guarantee that
mResponseInfo will be null, because an onResponseHeadersReceived callback
might already been queued at the executor.

BUG=594432
Committed: https://crrev.com/3954c975719d59922ccc6e5935e74748bf63e8e7
Cr-Commit-Position: refs/heads/master@{#382444}

[xunjieli, 2016-03-18 15:22:27]

xunjieli@chromium.org changed reviewers:
+ mef@chromium.org

[mef, 2016-03-18 15:30:18]

https://codereview.chromium.org/1815163003/diff/1/components/cronet/android/t...
File
components/cronet/android/test/javatests/src/org/chromium/net/BidirectionalStreamTest.java
(right):

https://codereview.chromium.org/1815163003/diff/1/components/cronet/android/t...
components/cronet/android/test/javatests/src/org/chromium/net/BidirectionalStreamTest.java:815:
boolean expectResponseInfo, boolean expectError) {
remove |expectResponseInfo|.

[xunjieli, 2016-03-18 15:39:47]

Thanks!

https://codereview.chromium.org/1815163003/diff/1/components/cronet/android/t...
File
components/cronet/android/test/javatests/src/org/chromium/net/BidirectionalStreamTest.java
(right):

https://codereview.chromium.org/1815163003/diff/1/components/cronet/android/t...
components/cronet/android/test/javatests/src/org/chromium/net/BidirectionalStreamTest.java:815:
boolean expectResponseInfo, boolean expectError) {
On 2016/03/18 15:30:18, mef wrote:
> remove |expectResponseInfo|.

Done.

[mef, 2016-03-21 22:37:51]

lgtm

[xunjieli, 2016-03-21 23:03:02]

The CQ bit was checked by xunjieli@chromium.org

[commit-bot: I haz the power, 2016-03-21 23:03:36]

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

[commit-bot: I haz the power, 2016-03-22 00:00:50]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-22 00:01:55]

Description was changed from

==========
Fix BidirectionalStreamTest#testFailures flake

Cancel during ResponseStep.ON_REQEST_HEADERS_SENT does not guarantee that
mResponseInfo will be null, because an onResponseHeadersReceived callback
might already been queued at the executor.

BUG=594432
==========

to

==========
Fix BidirectionalStreamTest#testFailures flake

Cancel during ResponseStep.ON_REQEST_HEADERS_SENT does not guarantee that
mResponseInfo will be null, because an onResponseHeadersReceived callback
might already been queued at the executor.

BUG=594432

Committed: https://crrev.com/3954c975719d59922ccc6e5935e74748bf63e8e7
Cr-Commit-Position: refs/heads/master@{#382444}
==========

[commit-bot: I haz the power, 2016-03-22 00:01:56]

Patchset 2 (id:??) landed as
https://crrev.com/3954c975719d59922ccc6e5935e74748bf63e8e7
Cr-Commit-Position: refs/heads/master@{#382444}