feat: explicitly support getNodeByNodeId

also remove readonly

Author: b10902118

src/Inspector.ts

@@ -507,8 +507,9 @@ export class Inspector extends EventEmitter {
     this.buildNodeTree(root);
   }
 
-  // for checking node tracked only
-  _getNodeByNodeId(nodeId: number): Readonly<InspectorNode> | undefined {
+  // mainly for checking node tracked
+  // also useful for mapping manually requested NodeId to InspectorNode
+  getNodeByNodeId(nodeId: number): InspectorNode | undefined {
     const node = this.idToNode.get(nodeId);
     return node;
   }

src/InspectorDOM.ts

@@ -36,7 +36,7 @@ export class InspectorNode {
 
   get tracked(): boolean {
     // check if the node is still tracked by inspector
-    const node = this.inspector._getNodeByNodeId(this._cdpNode.nodeId);
+    const node = this.inspector.getNodeByNodeId(this._cdpNode.nodeId);
     return node === this;
   }