New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 782297 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

src/net/spdy/chromium/spdy_session.cc missing xref info and links

Project Member Reported by pauljensen@chromium.org, Nov 7 2017

Issue description

Open this link:
https://cs.chromium.org/chromium/src/net/spdy/chromium/spdy_session.cc?type=cs&sq=package:chromium&l=3060
Notice that there are no links in the code.

spdy_session.h seems to be able to reference definitions in spdy_session.cc.

"Call Hierarchy" seems to be missing referrences from spdy_session.cc also:
1. Open
https://cs.chromium.org/chromium/src/net/socket/client_socket_pool.h?type=cs&q=AddHigherLayeredPool&sq=package:chromium&l=57
2. Click AddHigherLayeredPool
3. In the "Call Hierarchy" notice how it misses this spdy_session.cc call:
https://cs.chromium.org/chromium/src/net/spdy/chromium/spdy_session.cc?type=cs&q=AddHigherLayeredPool&sq=package:chromium&l=857
 

Comment 1 by emso@chromium.org, Nov 9 2017

Cc: aga...@chromium.org
Owner: flyboy@chromium.org
Status: Assigned (was: Untriaged)
Owner: aga...@chromium.org
Status: WontFix (was: Assigned)
It doesn't miss it. There are two methods named AddHigherLayeredPool. One of them (the one you click on in step 1) is a method on LowerLayeredPool. The other (the one being called by spdy_session.cc in step 3) is a method on ClientSocketHandle. Codesearch correctly determines that these are different methods on different classes, and doesn't confuse the XRefs.

Sign in to add a comment