Fix flow control enforcement condition.

Fix flow control enforcement condition.

The receiving window as known and obeyed by the peer is
|unacked_recv_window_bytes_| less than |recv_window_size_|, this should be
reflected in the check in DecreaseRecvWindowSize().  Otherwise
|DCHECK_GE(recv_window_size_, unacked_recv_window_bytes_)| might fail in
IncreaseRecvWindowSize().

This CL fixes that in both SpdySession and SpdyStream.

BUG=478836
Committed: https://crrev.com/2fc6a45506266335ff1fec2bf9a58f896ea7cf43
Cr-Commit-Position: refs/heads/master@{#327285}

[Bence, 2015-04-17 17:35:11]

bnc@chromium.org changed reviewers:
+ rch@chromium.org

[Bence, 2015-04-17 17:35:11]

Ryan, PTAL.  Mille grazie.

[Ryan Hamilton, 2015-04-17 17:42:11]

I suspect we should have a regression test for this.

[Bence, 2015-04-24 13:49:04]

Ryan, PTAL.  Thank you.

[Ryan Hamilton, 2015-04-27 15:27:18]

LGTM, mod a nit in the test.

https://codereview.chromium.org/1051213006/diff/60001/net/spdy/spdy_session_u...
File net/spdy/spdy_session_unittest.cc (right):

https://codereview.chromium.org/1051213006/diff/60001/net/spdy/spdy_session_u...
net/spdy/spdy_session_unittest.cc:3716: const int32 second_packet_size = 400;
nit: should these be "packet" sizes, or "frame" sizes?

[Bence, 2015-04-28 13:33:18]

Thank you.

https://codereview.chromium.org/1051213006/diff/60001/net/spdy/spdy_session_u...
File net/spdy/spdy_session_unittest.cc (right):

https://codereview.chromium.org/1051213006/diff/60001/net/spdy/spdy_session_u...
net/spdy/spdy_session_unittest.cc:3716: const int32 second_packet_size = 400;
On 2015/04/27 15:27:18, Ryan Hamilton wrote:
> nit: should these be "packet" sizes, or "frame" sizes?

These are frame sizes.  Good catch.  Done.

[Bence, 2015-04-28 13:33:25]

The CQ bit was checked by bnc@chromium.org

[Bence, 2015-04-28 13:33:26]

The patchset sent to the CQ was uploaded after l-g-t-m from rch@chromium.org
Link to the patchset: https://codereview.chromium.org/1051213006/#ps80001
(title: "s/packet/data frame/g")

[commit-bot: I haz the power, 2015-04-28 13:33:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1051213006/80001

[commit-bot: I haz the power, 2015-04-28 14:42:39]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-04-28 14:44:30]

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