Remove the options on locator.isHidden

It did accepts options which were strict and timeout. Strict is enabled
and cannot be changed when working with the locator API. Timeout is no
longer useful when working with isHidden.

common/locator.go

@@ -246,28 +246,17 @@ func (l *Locator) IsVisible() (bool, error) {
 
 // IsHidden returns true if the element matches the locator's
 // selector and is hidden. Otherwise, returns false.
-func (l *Locator) IsHidden(opts goja.Value) (bool, error) {
-	l.log.Debugf("Locator:IsHidden", "fid:%s furl:%q sel:%q opts:%+v", l.frame.ID(), l.frame.URL(), l.selector, opts)
+func (l *Locator) IsHidden() (bool, error) {
+	l.log.Debugf("Locator:IsHidden", "fid:%s furl:%q sel:%q", l.frame.ID(), l.frame.URL(), l.selector)
 
-	copts := NewFrameIsHiddenOptions()
-	if err := copts.Parse(l.ctx, opts); err != nil {
-		return false, fmt.Errorf("parsing is hidden options: %w", err)
-	}
-	hidden, err := l.isHidden(copts)
+	hidden, err := l.frame.isHidden(l.selector, &FrameIsHiddenOptions{Strict: true})
 	if err != nil {
 		return false, fmt.Errorf("checking is %q hidden: %w", l.selector, err)
 	}
 
 	return hidden, nil
 }
 
-// isHidden is like IsHidden but takes parsed options and does not
-// throw an error.
-func (l *Locator) isHidden(opts *FrameIsHiddenOptions) (bool, error) {
-	opts.Strict = true
-	return l.frame.isHidden(l.selector, opts)
-}
-
 // Fill out the element using locator's selector with strict mode on.
 func (l *Locator) Fill(value string, opts goja.Value) {
 	l.log.Debugf(

tests/locator_test.go

@@ -344,7 +344,7 @@ func TestLocatorElementState(t *testing.T) {
 		{
 			"hidden",
 			`() => document.getElementById('inputText').style.visibility = 'hidden'`,
-			func(l *common.Locator) bool { resp, _ := l.IsHidden(nil); return !resp },
+			func(l *common.Locator) bool { resp, _ := l.IsHidden(); return !resp },
 		},
 		{
 			"readOnly",