Fix max_concurrent_streams == 0 behavior.

Fix max_concurrent_streams == 0 behavior.

|max_concurrent_streams == 0| means no more stream can be created.  It does not
mean there is no limit on the number of streams.

Also add regression test.  I locally verified that the test fails without the
fix.

Also remove SpdySession::max_concurrent_streams() accessor that was only used in
one place, in a test that is friends with SpdySession anyway so it can directly
access the member in question.

BUG=481009
Committed: https://crrev.com/16f6d4649cd3cc28a416c155dcde5c26eaddafb3
Cr-Commit-Position: refs/heads/master@{#370723}

[Bence, 2016-01-21 14:20:39]

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

[Bence, 2016-01-21 14:20:39]

Ryan: PTAL.  Thank you.

[Ryan Hamilton, 2016-01-21 14:52:46]

lgtm

[Bence, 2016-01-21 16:52:02]

The CQ bit was checked by bnc@chromium.org

[commit-bot: I haz the power, 2016-01-21 16:52:34]

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

[commit-bot: I haz the power, 2016-01-21 17:36:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-21 17:36:53]

Description was changed from

==========
Fix max_concurrent_streams == 0 behavior.

|max_concurrent_streams == 0| means no more stream can be created.  It does not
mean there is no limit on the number of streams.

Also add regression test.  I locally verified that the test fails without the
fix.

Also remove SpdySession::max_concurrent_streams() accessor that was only used in
one place, in a test that is friends with SpdySession anyway so it can directly
access the member in question.

BUG=481009
==========

to

==========
Fix max_concurrent_streams == 0 behavior.

|max_concurrent_streams == 0| means no more stream can be created.  It does not
mean there is no limit on the number of streams.

Also add regression test.  I locally verified that the test fails without the
fix.

Also remove SpdySession::max_concurrent_streams() accessor that was only used in
one place, in a test that is friends with SpdySession anyway so it can directly
access the member in question.

BUG=481009

Committed: https://crrev.com/16f6d4649cd3cc28a416c155dcde5c26eaddafb3
Cr-Commit-Position: refs/heads/master@{#370723}
==========

[commit-bot: I haz the power, 2016-01-21 17:36:54]

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