Add error handling to ownerFrame

Updates: #804
Related: #688

api/element_handle.go

@@ -25,7 +25,7 @@ type ElementHandle interface {
 	IsEnabled() bool
 	IsHidden() bool
 	IsVisible() bool
-	OwnerFrame() Frame
+	OwnerFrame() (Frame, error)
 	Press(key string, opts goja.Value)
 	Query(selector string) (ElementHandle, error)
 	QueryAll(selector string) ([]ElementHandle, error)

browser/mapping.go

@@ -230,10 +230,12 @@ func mapElementHandle(vu moduleVU, eh api.ElementHandle) mapping {
 		"isEnabled":     eh.IsEnabled,
 		"isHidden":      eh.IsHidden,
 		"isVisible":     eh.IsVisible,
-		"ownerFrame": func() *goja.Object {
-			f := eh.OwnerFrame()
-			mf := mapFrame(vu, f)
-			return rt.ToValue(mf).ToObject(rt)
+		"ownerFrame": func() (mapping, error) {
+			f, err := eh.OwnerFrame()
+			if err != nil {
+				return nil, err //nolint:wrapcheck
+			}
+			return mapFrame(vu, f), nil
 		},
 		"press":                  eh.Press,
 		"screenshot":             eh.Screenshot,

common/element_handle.go

@@ -970,7 +970,7 @@ func (h *ElementHandle) IsVisible() bool {
 }
 
 // OwnerFrame returns the frame containing this element.
-func (h *ElementHandle) OwnerFrame() api.Frame {
+func (h *ElementHandle) OwnerFrame() (api.Frame, error) {
 	fn := `
 		(node, injected) => {
 			return injected.getDocumentElement(node);
@@ -982,28 +982,31 @@ func (h *ElementHandle) OwnerFrame() api.Frame {
 	}
 	res, err := h.evalWithScript(h.ctx, opts, fn)
 	if err != nil {
-		k6ext.Panic(h.ctx, "getting document element: %w", err)
+		err = fmt.Errorf("getting document element: %w", err)
 	}
-	if res == nil {
-		return nil
+	if res == nil || err != nil {
+		return nil, err
 	}
 
-	documentHandle := res.(*ElementHandle)
+	documentHandle, ok := res.(*ElementHandle)
+	if !ok {
+		return nil, fmt.Errorf("unexpected result type while getting document element: %T", res)
+	}
 	defer documentHandle.Dispose()
 	if documentHandle.remoteObject.ObjectID == "" {
-		return nil
+		return nil, err
 	}
 
 	var node *cdp.Node
 	action := dom.DescribeNode().WithObjectID(documentHandle.remoteObject.ObjectID)
 	if node, err = action.Do(cdp.WithExecutor(h.ctx, h.session)); err != nil {
-		k6ext.Panic(h.ctx, "getting node in frame: %w", err)
+		return nil, fmt.Errorf("getting node in frame: %w", err)
 	}
 	if node == nil || node.FrameID == "" {
-		return nil
+		return nil, fmt.Errorf("no frame found for node: %w", err)
 	}
 
-	return h.frame.manager.getFrameByID(node.FrameID)
+	return h.frame.manager.getFrameByID(node.FrameID), nil
 }
 
 func (h *ElementHandle) Press(key string, opts goja.Value) {