From e90f1c9892476cdb42f93e0e5250c28a035203c4 Mon Sep 17 00:00:00 2001 From: Blake GH Date: Wed, 30 Oct 2019 11:13:48 -0500 Subject: [PATCH] Driver.pm screenshot method referenced incorrect Variable name $self->{browser} doesn't exist. Needed to be: $self->{browser_name} for the croak --- lib/Selenium/Remote/Driver.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/Selenium/Remote/Driver.pm b/lib/Selenium/Remote/Driver.pm index 3580ddc1..c5bf9f99 100644 --- a/lib/Selenium/Remote/Driver.pm +++ b/lib/Selenium/Remote/Driver.pm @@ -2181,7 +2181,7 @@ sub screenshot { my ($self, $params) = @_; $params //= { full => 0 }; - croak "Full page screenshot only supported on geckodriver" if $params->{full} && ( $self->{browser} ne 'firefox' ); + croak "Full page screenshot only supported on geckodriver" if $params->{full} && ( $self->{browser_name} ne 'firefox' ); my $res = { 'command' => $params->{'full'} == 1 ? 'mozScreenshotFull' : 'screenshot' }; return $self->_execute_command($res);