[🐛 Bug]: Using CDP in multiple tabs

[grayglj]

What happened?

Hi,To inspect network traffic, i use AttachToTarget to switch session between multiple tabs in chome, but the newly created session
always be bound with the last opened tab.

How can we reproduce the issue?

//fist: open google manually
//second:open manual stackoverflow
public void Test(ChromeDriver currentDriver)
{
    DevToolsSession originalSession = currentDriver.GetDevToolsSession();

    //google's taregetId xxxxxx1(ex) 
    //stackoverflow targetId: xxxxxx2(ex) 
    var allTargets = originalSession.Domains.Target.GetTargets().Result.Where(t => t.Type == "page").ToList();

    foreach (var target in allTargets)
    {
        currentDriver.CloseDevToolsSession();

        //newSession's ActiveSessionId and attachedTargetid are null now.
        var newSession = currentDriver.GetDevToolsSession();

        var attachResult = newSession.Domains.Target.AttachToTarget(target.TargetId).Result;

        //here,newSession's ActiveSessionId and attachedTargetid have value.
        //but the attachedTargetid always be same as xxxxxx2 which last opened in Chrome.
        string fordebug = "";
    }
}

Relevant log output

no logs.

Operating System

Windows 10

Selenium version

4.16.2

What are the browser(s) and version(s) where you see this issue?

Chrome 120.0.6099.71 (Official Build) (64-bit)

What are the browser driver(s) and version(s) where you see this issue?

ChromeDriver 120.0.6099.71

Are you using Selenium Grid?

no

[github-actions]

@grayglj, thank you for creating this issue. We will troubleshoot it as soon as we can.

---

Info for maintainers

Triage this issue by using labels.

If information is missing, add a helpful comment and then I-issue-template label.

If the issue is a question, add the I-question label.

If the issue is valid but there is no time to troubleshoot it, consider adding the help wanted label.

If the issue requires changes or fixes from an external project (e.g., ChromeDriver, GeckoDriver, MSEdgeDriver, W3C), add the applicable G-* label, and it will provide the correct link and auto-close the issue.

After troubleshooting the issue, please add the R-awaiting answer label.

Thank you!

[diemol]

I know that you need to create a new DevTools session per tab, I just do not know how to do that in C#.

@nvborisenko do you know how to do that?

[nvborisenko]

Session implementation supposed to be initialized before creating new tabs in the browser. Behind of scene it attaches some unpredictable target (probably the latest one, hopefully the active one), and use auto-attach CDP feature. So then Session can "track" newly created targets.

More details about initialization: https://github.com/SeleniumHQ/selenium/blob/5b1b4490979a579716381a42f77ac5e4cf8f5e7b/dotnet/src/webdriver/DevTools/DevToolsSession.cs#L367C9-L367C9

                var targets = await this.domains.Target.GetTargets().ConfigureAwait(false);
                foreach (var target in targets)
                {
                    if (target.Type == "page")
                    {
                        this.attachedTargetId = target.TargetId;
                        LogTrace("Found Target ID {0}.", this.attachedTargetId);
                        break;
                    }
                }

@EdwinVanVliet tries to improve it, especially your case (network event monitoring for different tabs #10603).

[titusfortner]

@nvborisenko does #13330 fix this, then?

[nvborisenko]

Potentially, not 100% sure. That PR brings a possibility to "pause" newly opened tab in browser. Will see.

[titusfortner]

@nvborisenko is there something we know we want to do here beyond what we've done for 4.17?

[diemol]

The referenced PR was merged, and if something is still needed we can always reopen this issue.

[github-actions]

This issue has been automatically locked since there has not been any recent activity since it was closed. Please open a new issue for related bugs.