Check brave_network_delegate_base.cc:GetWebContentsFromProcessAndFrameId

[iefremov]

https://github.com/brave/brave-core/blob/master/browser/net/brave_network_delegate_base.cc#L36

Using render_frame_id instead of render_frame_tree_node_id in the return statement seems like a typo, but fixing it causes the browser tests breakage.

The same issue is in the upsream code, see chrome_content_browser_client.cc:GetWebContentsFromProcessAndFrameId

Need to investigate this and fix or add an explaining comment.

Brave version (brave://version info)

Brave	0.59.1 Chromium: 71.0.3578.53 (Developer Build) (64-bit)
Revision	599fb72e740eab9d10609f1ed28a2ad669f82c1a-refs/branch-heads/3578@{#669}

Reproducible on current release:

• Does it reproduce on brave-browser dev/beta builds?
  yes

[iefremov]

@bbondy FYI

[iefremov]

Fixed after migration to NetworkService