Separate target init from construction (#2320)

Author: kblok

lib/PuppeteerSharp/ChromeTargetManager.cs

@@ -199,6 +199,7 @@ private void OnTargetCreated(TargetCreatedResponse e)
                 }
 
                 var target = _targetFactoryFunc(e.TargetInfo, null);
+                target.Initialize();
                 _attachedTargetsByTargetId.AddItem(e.TargetInfo.TargetId, target);
             }
         }
@@ -288,6 +289,7 @@ await parent.SendAsync(
                 }
 
                 var workerTarget = _targetFactoryFunc(targetInfo, null);
+                workerTarget.Initialize();
                 _attachedTargetsByTargetId.AddItem(targetInfo.TargetId, workerTarget);
                 TargetAvailable?.Invoke(this, new TargetChangedArgs { Target = workerTarget });
                 return;
@@ -316,6 +318,8 @@ await parent.SendAsync(
             }
             else
             {
+                target.Initialize();
+
                 _attachedTargetsByTargetId.AddItem(targetInfo.TargetId, target);
                 _attachedTargetsBySessionId.TryAdd(session.Id, target);
             }

lib/PuppeteerSharp/FirefoxTargetManager.cs

@@ -117,6 +117,7 @@ private void OnTargetCreated(TargetCreatedResponse e)
             if (e.TargetInfo.Type == TargetType.Browser && e.TargetInfo.Attached)
             {
                 var browserTarget = _targetFactoryFunc(e.TargetInfo, null);
+                browserTarget.Initialize();
                 _availableTargetsByTargetId.AddItem(e.TargetInfo.TargetId, browserTarget);
                 FinishInitializationIfReady(e.TargetInfo.TargetId);
             }
@@ -129,6 +130,7 @@ private void OnTargetCreated(TargetCreatedResponse e)
                 return;
             }
 
+            target.Initialize();
             _availableTargetsByTargetId.AddItem(e.TargetInfo.TargetId, target);
             TargetAvailable?.Invoke(
                 this,

lib/PuppeteerSharp/PageTarget.cs

@@ -39,7 +39,7 @@ public override async Task<IPage> PageAsync()
             return await PageTask.ConfigureAwait(false);
         }
 
-        protected override void Initialize()
+        internal override void Initialize()
         {
             _ = InitializedTaskWrapper.Task.ContinueWith(
                 async initializedTask =>

lib/PuppeteerSharp/Target.cs

@@ -85,7 +85,7 @@ internal void TargetInfoChanged(TargetInfo targetInfo)
         /// <summary>
         /// Initializes the target.
         /// </summary>
-        protected virtual void Initialize()
+        internal virtual void Initialize()
         {
             IsInitialized = true;
             InitializedTaskWrapper.TrySetResult(InitializationStatus.Success);