kotovalexarian-likes-github/teamcapybara--capybara
1
0
Fork 0
mirror of https://github.com/teamcapybara/capybara.git synced 2022-11-09 12:08:07 -05:00
Code Releases Activity

Only FF needs the frame workaround

Browse source
This commit is contained in:
Thomas Walpole 2018-09-11 11:15:07 -07:00
parent 3692f5abf2
commit 5ca9b25685
2 changed files with 10 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
lib/capybara/selenium/driver.rb
View file

@ -143,11 +143,8 @@ class Capybara::Selenium::Driver < Capybara::Driver::Base
handles.clear
browser.switch_to.default_content
when :parent
# would love to use browser.switch_to.parent_frame here
# but it has an issue if the current frame is removed from within it
handles.pop
browser.switch_to.default_content
handles.each { |fh| browser.switch_to.frame(fh) }
browser.switch_to.parent_frame
else
handles << frame.native
browser.switch_to.frame(frame.native)

9
lib/capybara/selenium/driver_specializations/marionette_driver.rb
View file

@ -32,6 +32,15 @@ module Capybara::Selenium::Driver::MarionetteDriver
# No modal was opened - page has refreshed - ignore
end
def switch_to_frame(frame)
return super unless frame == :parent
# geckodriver/firefox has an issue if the current frame is removed from within it
# so we have to move to the default_content and iterate back through the frames
handles = @frame_handles[current_window_handle]
browser.switch_to.default_content
handles.tap(&:pop).each { |fh| browser.switch_to.frame(fh) }
end
private
def build_node(native_node)