browser(firefox): send screencastStarted after attachedToTarget (#3554)

microsoft · Aug 20, 2020 · 9f3a1b5 · 9f3a1b5
1 parent 5ba0254
commit 9f3a1b5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/browser_patches/firefox/BUILD_NUMBER b/browser_patches/firefox/BUILD_NUMBER
@@ -1,2 +1,2 @@
-1165
-Changed: [email protected] Thu Aug 20 13:05:37 PDT 2020
+1166
+Changed: [email protected] Thu Aug 20 13:55:50 PDT 2020
diff --git a/browser_patches/firefox/juggler/protocol/BrowserHandler.js b/browser_patches/firefox/juggler/protocol/BrowserHandler.js
@@ -33,13 +33,13 @@ class BrowserHandler {
       if (!this._shouldAttachToTarget(target))
         continue;
       const session = this._dispatcher.createSession();
-      target.initSession(session);
-      target.connectSession(session);
       this._attachedSessions.set(target, session);
       this._session.emitEvent('Browser.attachedToTarget', {
         sessionId: session.sessionId(),
         targetInfo: target.info()
       });
+      target.initSession(session);
+      target.connectSession(session);
     }
 
     this._eventListeners = [
@@ -92,12 +92,12 @@ class BrowserHandler {
     if (!this._shouldAttachToTarget(target))
       return;
     const session = this._dispatcher.createSession();
-    target.initSession(session);
     this._attachedSessions.set(target, session);
     this._session.emitEvent('Browser.attachedToTarget', {
       sessionId: session.sessionId(),
       targetInfo: target.info()
     });
+    target.initSession(session);
     sessions.push(session);
   }