fix: don't force DevTools tab target to Type=Other; sync TargetManager test

With the previous commit's filter, tab targets auto-attach. The DevTools
tab target's URL matches isDevToolsPageTarget so we wrap it as a
CdpDevToolsTarget — and that class hard-overrode Type to TargetType.Other,
making Targets()/PagesAsync() treat the tab as a page, then try to send
Page.enable on a tab session. Removing the override matches upstream
(DevToolsTarget extends PageTarget with no type override).

ShouldHandleTargets was written for the pre-filter world where tabs
weren't attached. Synced it to upstream's version, which accounts for the
tab/page pair on each new page.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: kblok

lib/PuppeteerSharp.Tests/TargetManagerTests/TargetManagerTests.cs

@@ -30,38 +30,41 @@ public TargetManagerTests() : base()
         public async Task ShouldHandleTargets()
         {
             var targetManager = (Browser as CdpBrowser)!.TargetManager;
-            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(2));
+            var initialTargetCount = targetManager.GetAvailableTargets().Values.Count;
+
+            // There could be an conditional extra prerender target.
+            Assert.That(initialTargetCount, Is.EqualTo(3).Or.EqualTo(4));
 
             Assert.That(await Context.PagesAsync(), Is.Empty);
-            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(2));
+            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(initialTargetCount));
 
             var page = await Context.NewPageAsync();
             Assert.That((await Context.PagesAsync()), Has.Length.EqualTo(1));
-            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(3));
+            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(initialTargetCount + 2));
 
             await page.GoToAsync(TestConstants.EmptyPage);
             Assert.That((await Context.PagesAsync()), Has.Length.EqualTo(1));
-            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(3));
+            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(initialTargetCount + 2));
 
             var frameTask = page.WaitForFrameAsync(target => target.Url == TestConstants.EmptyPage);
             await FrameUtils.AttachFrameAsync(page, "frame1", TestConstants.EmptyPage);
             await frameTask.WithTimeout();
             Assert.That((await Context.PagesAsync()), Has.Length.EqualTo(1));
-            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(3));
+            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(initialTargetCount + 2));
             Assert.That(page.Frames, Has.Length.EqualTo(2));
 
             frameTask = page.WaitForFrameAsync(target => target.Url == TestConstants.CrossProcessUrl + "/empty.html");
             await FrameUtils.AttachFrameAsync(page, "frame2", TestConstants.CrossProcessUrl + "/empty.html");
             await frameTask.WithTimeout();
             Assert.That((await Context.PagesAsync()), Has.Length.EqualTo(1));
-            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(4));
+            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(initialTargetCount + 3));
             Assert.That(page.Frames, Has.Length.EqualTo(3));
 
             frameTask = page.WaitForFrameAsync(target => target.Url == TestConstants.CrossProcessUrl + "/empty.html");
             await FrameUtils.AttachFrameAsync(page, "frame3", TestConstants.CrossProcessUrl + "/empty.html");
             await frameTask.WithTimeout();
             Assert.That((await Context.PagesAsync()), Has.Length.EqualTo(1));
-            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(5));
+            Assert.That(targetManager.GetAvailableTargets().Values, Has.Count.EqualTo(initialTargetCount + 4));
             Assert.That(page.Frames, Has.Length.EqualTo(4));
         }
     }

lib/PuppeteerSharp/Cdp/CdpDevToolsTarget.cs

@@ -20,8 +20,5 @@ internal CdpDevToolsTarget(
             : base(targetInfo, session, context, targetManager, sessionFactory, defaultViewport, screenshotTaskQueue)
         {
         }
-
-        /// <inheritdoc/>
-        public override TargetType Type => TargetType.Other;
     }
 }