Minor cleanup in SpdySession.

Minor cleanup in SpdySession.

* Remove out-of-date comment in OnStreamEnd(); inline |buffer|.
* Remove non-HTTP/2 code path in OnStreamPadding().

Committed: https://crrev.com/13304a1d415dc258fb3058aa9236df4555d5800d
Cr-Commit-Position: refs/heads/master@{#434871}

[Bence, 2016-11-28 21:05:04]

The CQ bit was checked by bnc@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-28 21:06:04]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-28 21:14:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-28 21:14:12]

Dry run: Try jobs failed on following builders:
  android_cronet on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_cron...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Bence, 2016-11-28 21:26:16]

The CQ bit was checked by bnc@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-28 21:27:35]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-28 23:57:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-28 23:57:19]

Dry run: This issue passed the CQ dry run.

[Bence, 2016-11-29 00:28:43]

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

[Bence, 2016-11-29 00:28:44]

Ryan: PTAL.  Thank you.

https://codereview.chromium.org/2539563002/diff/1/net/spdy/spdy_session.cc
File net/spdy/spdy_session.cc (left):

https://codereview.chromium.org/2539563002/diff/1/net/spdy/spdy_session.cc#ol...
net/spdy/spdy_session.cc:2559:
stream->AddRawReceivedBytes(last_compressed_frame_len_);
Return around line 2479 makes this branch dead.

[Ryan Hamilton, 2016-11-29 03:32:01]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2016-11-29 03:32:02]

lgtm

[commit-bot: I haz the power, 2016-11-29 03:32:34]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-29 03:37:22]

CQ is committing da patch.

Bot data: {"patchset_id": 10002, "attempt_start_ts": 1480390321660660,
"parent_rev": "e8dfc8466859034d19653c4e75fdeef2f9535c6f", "commit_rev":
"f2af0567b2f3679f048d17ffa36bf35176e2bdf6"}

[commit-bot: I haz the power, 2016-11-29 03:37:47]

Committed patchset #2 (id:10002)

[commit-bot: I haz the power, 2016-11-29 03:43:23]

Description was changed from

==========
Minor cleanup in SpdySession.

* Remove out-of-date comment in OnStreamEnd(); inline |buffer|.
* Remove non-HTTP/2 code path in OnStreamPadding().
==========

to

==========
Minor cleanup in SpdySession.

* Remove out-of-date comment in OnStreamEnd(); inline |buffer|.
* Remove non-HTTP/2 code path in OnStreamPadding().

Committed: https://crrev.com/13304a1d415dc258fb3058aa9236df4555d5800d
Cr-Commit-Position: refs/heads/master@{#434871}
==========

[commit-bot: I haz the power, 2016-11-29 03:43:24]

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