fix(executioncontext): follow up to properly adopt element handles (#…

…3857)

The `executionContextId` argument was missing, which made all
element handles to resolve in the main world. All our tests pass
atm, but this would've fired back when we exposed extension
execution contexts.

lib/ExecutionContext.js

@@ -186,6 +186,7 @@ class ExecutionContext {
     });
     const {object} = await this._client.send('DOM.resolveNode', {
       backendNodeId: nodeInfo.node.backendNodeId,
+      executionContextId: this._contextId,
     });
     return /** @type {Puppeteer.ElementHandle}*/(createJSHandle(this, object));
   }