[SPDY] Count closed-stream DATA frames for session flow control

[SPDY] Count closed-stream DATA frames for session flow control

This ensures that both sides' flow control states stay in sync even if one side resets a stream.

BUG=269873
R=rch@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=217107

[akalin, 2013-08-08 23:04:28]

+rch for review

[akalin, 2013-08-12 21:27:19]

On 2013/08/08 23:04:28, akalin wrote:
> +rch for review

ping!

[Ryan Hamilton, 2013-08-12 21:32:06]

sorry for the delay, this is probably lgtm, but I'm confused about the
disconnect between the CL description and the comment in the test.

https://codereview.chromium.org/22610006/diff/1/net/spdy/spdy_session_unittes...
File net/spdy/spdy_session_unittest.cc (right):

https://codereview.chromium.org/22610006/diff/1/net/spdy/spdy_session_unittes...
net/spdy/spdy_session_unittest.cc:3181: // receive window size to decrease, but
it should cause the unacked
The first part of this comment seems to contradict the description in the CL. 
Am I reading it wrong?

[akalin, 2013-08-12 21:41:19]

PTAL!

https://codereview.chromium.org/22610006/diff/1/net/spdy/spdy_session_unittes...
File net/spdy/spdy_session_unittest.cc (right):

https://codereview.chromium.org/22610006/diff/1/net/spdy/spdy_session_unittes...
net/spdy/spdy_session_unittest.cc:3181: // receive window size to decrease, but
it should cause the unacked
On 2013/08/12 21:32:06, Ryan Hamilton wrote:
> The first part of this comment seems to contradict the description in the CL. 
> Am I reading it wrong?

The CL description was imprecise. I updated it to hopefully be clearer.

[Ryan Hamilton, 2013-08-12 21:52:11]

LGTM!

[akalin, 2013-08-12 22:55:02]

Committed patchset #2 manually as r217107 (presubmit successful).