Inline two methods of QuicChromiumClientSession

Inline two methods of QuicChromiumClientSession
which are only called from one place:

  RecordAndCloseSessionOnError
  CloseSessionOnErrorInner

Review-Url: https://codereview.chromium.org/2849823005
Cr-Commit-Position: refs/heads/master@{#468272}
Committed: https://chromium.googlesource.com/chromium/src/+/90f64ebe085b18db5d6ddac9aa231d1b76c2752b

[Ryan Hamilton, 2017-04-29 20:56:03]

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

[Zhongyi Shi, 2017-04-29 21:05:19]

lgtm

https://codereview.chromium.org/2849823005/diff/1/net/quic/chromium/quic_chro...
File net/quic/chromium/quic_chromium_client_session.h (right):

https://codereview.chromium.org/2849823005/diff/1/net/quic/chromium/quic_chro...
net/quic/chromium/quic_chromium_client_session.h:225: // Close the session
because of |error| and notifies the factory
nit: |net_error|

[Ryan Hamilton, 2017-05-01 02:01:52]

Description was changed from

==========
Inline two methods of QuicChromiumClientSession
which are only called from one place:

  RecordAndCloseSessionOnError
	CloseSessionOnErrorInner
==========

to

==========
Inline two methods of QuicChromiumClientSession
which are only called from one place:

  RecordAndCloseSessionOnError
  CloseSessionOnErrorInner
==========

[Ryan Hamilton, 2017-05-01 02:02:05]

Thanks!

https://codereview.chromium.org/2849823005/diff/1/net/quic/chromium/quic_chro...
File net/quic/chromium/quic_chromium_client_session.h (right):

https://codereview.chromium.org/2849823005/diff/1/net/quic/chromium/quic_chro...
net/quic/chromium/quic_chromium_client_session.h:225: // Close the session
because of |error| and notifies the factory
On 2017/04/29 21:05:19, Zhongyi Shi wrote:
> nit: |net_error|

Done.

[Ryan Hamilton, 2017-05-01 02:02:10]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2017-05-01 02:02:11]

The patchset sent to the CQ was uploaded after l-g-t-m from zhongyi@chromium.org
Link to the patchset: https://codereview.chromium.org/2849823005/#ps20001
(title: "fix")

[commit-bot: I haz the power, 2017-05-01 02:02:22]

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, 2017-05-01 03:07:16]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493604130344380,
"parent_rev": "09d695d182efcc32d08d4f7988b323d17d76d809", "commit_rev":
"90f64ebe085b18db5d6ddac9aa231d1b76c2752b"}

[commit-bot: I haz the power, 2017-05-01 03:08:03]

Description was changed from

==========
Inline two methods of QuicChromiumClientSession
which are only called from one place:

  RecordAndCloseSessionOnError
  CloseSessionOnErrorInner
==========

to

==========
Inline two methods of QuicChromiumClientSession
which are only called from one place:

  RecordAndCloseSessionOnError
  CloseSessionOnErrorInner

Review-Url: https://codereview.chromium.org/2849823005
Cr-Commit-Position: refs/heads/master@{#468272}
Committed:
https://chromium.googlesource.com/chromium/src/+/90f64ebe085b18db5d6ddac9aa23...
==========

[commit-bot: I haz the power, 2017-05-01 03:08:05]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/90f64ebe085b18db5d6ddac9aa23...