Log source_dependency in HTTP2_SESSION_SEND_HEADERS.

Log source_dependency in HTTP2_SESSION_SEND_HEADERS.

Pass in HttpStreamFactoryImpl::Job's NetLog source as source_dependency
to SpdyHttpStream and BidirectionalStreamSpdyImpl, where it is saved in
a private member.

Save HttpProxyClientSocketWrapper's NetLog source in
SpdyProxyClientSocket constructor (NetLog object was already passed in).

Add SpdyStream::Delegate::source_dependency() and log it in
SpdySession::CreateHeaders().

Add dummy implementation to test-only classes FuzzerDelegate,
SessionOpeningDelegate, ClosingDelegate, and StreamDelegateBase.

Example NetLog excerpt:

1567: HTTP_STREAM_JOB
https://clients1.google.com/
Start Time: 2017-03-29 09:09:35.617

t=9692 [st= 0] +HTTP_STREAM_JOB  [dt=74]
                --> alternative_service = "unknown :0"
                --> original_url = "https://clients1.google.com/"
                --> priority = "LOWEST"
                --> source_dependency = 1566 (HTTP_STREAM_JOB_CONTROLLER)
                --> url = "https://clients1.google.com/"
[...]
t=9764 [st=72]    HTTP2_SESSION_POOL_IMPORTED_SESSION_FROM_SOCKET
                  --> source_dependency = 1576 (HTTP2_SESSION)
t=9766 [st=74]    HTTP_STREAM_JOB_BOUND_TO_REQUEST
                  --> source_dependency = 1565 (URL_REQUEST)
t=9766 [st=74] -HTTP_STREAM_JOB


1576: HTTP2_SESSION
clients1.google.com:443 (DIRECT)
Start Time: 2017-03-29 09:09:35.688

t= 9763 [st=  0] +HTTP2_SESSION  [dt=318+]
                  --> host = "clients1.google.com:443"
                  --> proxy = "DIRECT"
t= 9763 [st=  0]    HTTP2_SESSION_INITIALIZED
                    --> protocol = "h2"
                    --> source_dependency = 1573 (SOCKET)
[...]
t= 9768 [st=  5]    HTTP2_SESSION_SEND_HEADERS
                    --> exclusive = true
                    --> fin = false
                    --> has_priority = true
                    --> :method: POST
                        :authority: clients1.google.com
                        :scheme: https
                        :path: /tbproxy/af/query?client=Chromium
                        content-length: 113
                        content-type: text/proto
                        user-agent: [...]
                        accept-encoding: gzip, deflate, br
                        accept-language: en-US,en;q=0.8
                    --> parent_stream_id = 0
                    --> source_dependency = 1567 (HTTP_STREAM_JOB)
                    --> stream_id = 1
                    --> weight = 147
t= 9769 [st=  6]    HTTP2_SESSION_SEND_DATA
                    --> fin = true
                    --> size = 113
                    --> stream_id = 1
[...]

1567 HTTP_STREAM_JOB's HTTP2_SESSION_POOL_IMPORTED_SESSION_FROM_SOCKET already
had a source_dependency pointing to 1576 HTTP2_SESSION before this CL.  This CL
adds the reverse direction: 1576 HTTP2_SESSION's HTTP2_SESSION_SEND_HEADERS
now has a source_dependency pointing to 1567 HTTP_STREAM_JOB.

BUG=705967
Review-Url: https://codereview.chromium.org/2783683002
Cr-Commit-Position: refs/heads/master@{#460365}
Committed: https://chromium.googlesource.com/chromium/src/+/3d95ca9fd487e0a0dd2617eee0f683eea7a7bed0

[Bence, 2017-03-28 13:46:04]

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

[commit-bot: I haz the power, 2017-03-28 13:46:24]

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, 2017-03-28 14:17:39]

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

[commit-bot: I haz the power, 2017-03-28 14:17:40]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Bence, 2017-03-28 14:24:39]

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

[commit-bot: I haz the power, 2017-03-28 14:24:53]

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, 2017-03-28 15:50:58]

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

[commit-bot: I haz the power, 2017-03-28 15:50:59]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Bence, 2017-03-28 17:44:53]

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

[commit-bot: I haz the power, 2017-03-28 17:45:14]

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, 2017-03-28 18:59:07]

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

[commit-bot: I haz the power, 2017-03-28 18:59:08]

Dry run: This issue passed the CQ dry run.

[Bence, 2017-03-28 19:01:27]

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

[Bence, 2017-03-28 19:01:28]

Ryan: PTAL.  Thank you.

Note that NetLogSource is an enum plus an uint32_t, so I believe it takes up 64
bits.  I think copying and storing it a few times is worth the benefit of
clearer logs.

[Ryan Hamilton, 2017-03-28 21:17:59]

I think this LGTM, but can you include an example txt output of what ends up
being added to #events as a result of this?

[Bence, 2017-03-29 13:14:54]

Description was changed from

==========
Log source_dependency in HTTP2_SESSION_SEND_HEADERS.

Pass in HttpStreamFactoryImpl::Job's NetLog source as source_dependency
to SpdyHttpStream and BidirectionalStreamSpdyImpl, where it is saved in
a private member.

Save HttpProxyClientSocketWrapper's NetLog source in
SpdyProxyClientSocket constructor (NetLog object was already passed in).

Add SpdyStream::Delegate::source_dependency() and log it in
SpdySession::CreateHeaders().

Add dummy implementation to test-only classes FuzzerDelegate,
SessionOpeningDelegate, ClosingDelegate, and StreamDelegateBase.

BUG=705967
==========

to

==========
Log source_dependency in HTTP2_SESSION_SEND_HEADERS.

Pass in HttpStreamFactoryImpl::Job's NetLog source as source_dependency
to SpdyHttpStream and BidirectionalStreamSpdyImpl, where it is saved in
a private member.

Save HttpProxyClientSocketWrapper's NetLog source in
SpdyProxyClientSocket constructor (NetLog object was already passed in).

Add SpdyStream::Delegate::source_dependency() and log it in
SpdySession::CreateHeaders().

Add dummy implementation to test-only classes FuzzerDelegate,
SessionOpeningDelegate, ClosingDelegate, and StreamDelegateBase.

Example NetLog excerpt:

1567: HTTP_STREAM_JOB
https://clients1.google.com/
Start Time: 2017-03-29 09:09:35.617

t=9692 [st= 0] +HTTP_STREAM_JOB  [dt=74]
                --> alternative_service = "unknown :0"
                --> original_url = "https://clients1.google.com/"
                --> priority = "LOWEST"
                --> source_dependency = 1566 (HTTP_STREAM_JOB_CONTROLLER)
                --> url = "https://clients1.google.com/"
[...]
t=9764 [st=72]    HTTP2_SESSION_POOL_IMPORTED_SESSION_FROM_SOCKET
                  --> source_dependency = 1576 (HTTP2_SESSION)
t=9766 [st=74]    HTTP_STREAM_JOB_BOUND_TO_REQUEST
                  --> source_dependency = 1565 (URL_REQUEST)
t=9766 [st=74] -HTTP_STREAM_JOB


1576: HTTP2_SESSION
clients1.google.com:443 (DIRECT)
Start Time: 2017-03-29 09:09:35.688

t= 9763 [st=  0] +HTTP2_SESSION  [dt=318+]
                  --> host = "clients1.google.com:443"
                  --> proxy = "DIRECT"
t= 9763 [st=  0]    HTTP2_SESSION_INITIALIZED
                    --> protocol = "h2"
                    --> source_dependency = 1573 (SOCKET)
[...]
t= 9768 [st=  5]    HTTP2_SESSION_SEND_HEADERS
                    --> exclusive = true
                    --> fin = false
                    --> has_priority = true
                    --> :method: POST
                        :authority: clients1.google.com
                        :scheme: https
                        :path: /tbproxy/af/query?client=Chromium
                        content-length: 113
                        content-type: text/proto
                        user-agent: [...]
                        accept-encoding: gzip, deflate, br
                        accept-language: en-US,en;q=0.8
                    --> parent_stream_id = 0
                    --> source_dependency = 1567 (HTTP_STREAM_JOB)
                    --> stream_id = 1
                    --> weight = 147
t= 9769 [st=  6]    HTTP2_SESSION_SEND_DATA
                    --> fin = true
                    --> size = 113
                    --> stream_id = 1
[...]

1567 HTTP_STREAM_JOB's HTTP2_SESSION_POOL_IMPORTED_SESSION_FROM_SOCKET already
had a source_dependency pointing to 1576 HTTP2_SESSION before this CL.  This CL
adds the reverse direction: 1576 HTTP2_SESSION's HTTP2_SESSION_SEND_HEADERS
now has a source_dependency pointing to 1567 HTTP_STREAM_JOB.

BUG=705967
==========

[Bence, 2017-03-29 13:15:33]

The CQ bit was checked by bnc@chromium.org

[commit-bot: I haz the power, 2017-03-29 13:15:58]

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-03-29 13:20:39]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490793333454290,
"parent_rev": "debb4fc168676c5acc3353b9c841c24bc483aa7b", "commit_rev":
"3d95ca9fd487e0a0dd2617eee0f683eea7a7bed0"}

[commit-bot: I haz the power, 2017-03-29 13:21:26]

Description was changed from

==========
Log source_dependency in HTTP2_SESSION_SEND_HEADERS.

Pass in HttpStreamFactoryImpl::Job's NetLog source as source_dependency
to SpdyHttpStream and BidirectionalStreamSpdyImpl, where it is saved in
a private member.

Save HttpProxyClientSocketWrapper's NetLog source in
SpdyProxyClientSocket constructor (NetLog object was already passed in).

Add SpdyStream::Delegate::source_dependency() and log it in
SpdySession::CreateHeaders().

Add dummy implementation to test-only classes FuzzerDelegate,
SessionOpeningDelegate, ClosingDelegate, and StreamDelegateBase.

Example NetLog excerpt:

1567: HTTP_STREAM_JOB
https://clients1.google.com/
Start Time: 2017-03-29 09:09:35.617

t=9692 [st= 0] +HTTP_STREAM_JOB  [dt=74]
                --> alternative_service = "unknown :0"
                --> original_url = "https://clients1.google.com/"
                --> priority = "LOWEST"
                --> source_dependency = 1566 (HTTP_STREAM_JOB_CONTROLLER)
                --> url = "https://clients1.google.com/"
[...]
t=9764 [st=72]    HTTP2_SESSION_POOL_IMPORTED_SESSION_FROM_SOCKET
                  --> source_dependency = 1576 (HTTP2_SESSION)
t=9766 [st=74]    HTTP_STREAM_JOB_BOUND_TO_REQUEST
                  --> source_dependency = 1565 (URL_REQUEST)
t=9766 [st=74] -HTTP_STREAM_JOB


1576: HTTP2_SESSION
clients1.google.com:443 (DIRECT)
Start Time: 2017-03-29 09:09:35.688

t= 9763 [st=  0] +HTTP2_SESSION  [dt=318+]
                  --> host = "clients1.google.com:443"
                  --> proxy = "DIRECT"
t= 9763 [st=  0]    HTTP2_SESSION_INITIALIZED
                    --> protocol = "h2"
                    --> source_dependency = 1573 (SOCKET)
[...]
t= 9768 [st=  5]    HTTP2_SESSION_SEND_HEADERS
                    --> exclusive = true
                    --> fin = false
                    --> has_priority = true
                    --> :method: POST
                        :authority: clients1.google.com
                        :scheme: https
                        :path: /tbproxy/af/query?client=Chromium
                        content-length: 113
                        content-type: text/proto
                        user-agent: [...]
                        accept-encoding: gzip, deflate, br
                        accept-language: en-US,en;q=0.8
                    --> parent_stream_id = 0
                    --> source_dependency = 1567 (HTTP_STREAM_JOB)
                    --> stream_id = 1
                    --> weight = 147
t= 9769 [st=  6]    HTTP2_SESSION_SEND_DATA
                    --> fin = true
                    --> size = 113
                    --> stream_id = 1
[...]

1567 HTTP_STREAM_JOB's HTTP2_SESSION_POOL_IMPORTED_SESSION_FROM_SOCKET already
had a source_dependency pointing to 1576 HTTP2_SESSION before this CL.  This CL
adds the reverse direction: 1576 HTTP2_SESSION's HTTP2_SESSION_SEND_HEADERS
now has a source_dependency pointing to 1567 HTTP_STREAM_JOB.

BUG=705967
==========

to

==========
Log source_dependency in HTTP2_SESSION_SEND_HEADERS.

Pass in HttpStreamFactoryImpl::Job's NetLog source as source_dependency
to SpdyHttpStream and BidirectionalStreamSpdyImpl, where it is saved in
a private member.

Save HttpProxyClientSocketWrapper's NetLog source in
SpdyProxyClientSocket constructor (NetLog object was already passed in).

Add SpdyStream::Delegate::source_dependency() and log it in
SpdySession::CreateHeaders().

Add dummy implementation to test-only classes FuzzerDelegate,
SessionOpeningDelegate, ClosingDelegate, and StreamDelegateBase.

Example NetLog excerpt:

1567: HTTP_STREAM_JOB
https://clients1.google.com/
Start Time: 2017-03-29 09:09:35.617

t=9692 [st= 0] +HTTP_STREAM_JOB  [dt=74]
                --> alternative_service = "unknown :0"
                --> original_url = "https://clients1.google.com/"
                --> priority = "LOWEST"
                --> source_dependency = 1566 (HTTP_STREAM_JOB_CONTROLLER)
                --> url = "https://clients1.google.com/"
[...]
t=9764 [st=72]    HTTP2_SESSION_POOL_IMPORTED_SESSION_FROM_SOCKET
                  --> source_dependency = 1576 (HTTP2_SESSION)
t=9766 [st=74]    HTTP_STREAM_JOB_BOUND_TO_REQUEST
                  --> source_dependency = 1565 (URL_REQUEST)
t=9766 [st=74] -HTTP_STREAM_JOB


1576: HTTP2_SESSION
clients1.google.com:443 (DIRECT)
Start Time: 2017-03-29 09:09:35.688

t= 9763 [st=  0] +HTTP2_SESSION  [dt=318+]
                  --> host = "clients1.google.com:443"
                  --> proxy = "DIRECT"
t= 9763 [st=  0]    HTTP2_SESSION_INITIALIZED
                    --> protocol = "h2"
                    --> source_dependency = 1573 (SOCKET)
[...]
t= 9768 [st=  5]    HTTP2_SESSION_SEND_HEADERS
                    --> exclusive = true
                    --> fin = false
                    --> has_priority = true
                    --> :method: POST
                        :authority: clients1.google.com
                        :scheme: https
                        :path: /tbproxy/af/query?client=Chromium
                        content-length: 113
                        content-type: text/proto
                        user-agent: [...]
                        accept-encoding: gzip, deflate, br
                        accept-language: en-US,en;q=0.8
                    --> parent_stream_id = 0
                    --> source_dependency = 1567 (HTTP_STREAM_JOB)
                    --> stream_id = 1
                    --> weight = 147
t= 9769 [st=  6]    HTTP2_SESSION_SEND_DATA
                    --> fin = true
                    --> size = 113
                    --> stream_id = 1
[...]

1567 HTTP_STREAM_JOB's HTTP2_SESSION_POOL_IMPORTED_SESSION_FROM_SOCKET already
had a source_dependency pointing to 1576 HTTP2_SESSION before this CL.  This CL
adds the reverse direction: 1576 HTTP2_SESSION's HTTP2_SESSION_SEND_HEADERS
now has a source_dependency pointing to 1567 HTTP_STREAM_JOB.

BUG=705967

Review-Url: https://codereview.chromium.org/2783683002
Cr-Commit-Position: refs/heads/master@{#460365}
Committed:
https://chromium.googlesource.com/chromium/src/+/3d95ca9fd487e0a0dd2617eee0f6...
==========

[commit-bot: I haz the power, 2017-03-29 13:21:27]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/3d95ca9fd487e0a0dd2617eee0f6...