'Set Selenium Timeout' does not pass timeout variable to selenium

[emanlove]

It appears the 'Set Selenium Timeout' keyword does not actually set the timeout. The code as currently written

 def set_selenium_timeout(self, seconds):
 # . . .
 old_timeout = self.get_selenium_timeout()
 self._timeout_in_secs = robot.utils.timestr_to_secs(seconds)
 return old_timeout

 def set_selenium_implicit_wait(self, seconds):
# . . .

and the code as I suspect it should be

 def set_selenium_timeout(self, seconds):
 # . . .
 old_timeout = self.get_selenium_timeout()
 self._timeout_in_secs = robot.utils.timestr_to_secs(seconds)
 for browser in self._cache.browsers:
 browser.set_script_timeout(self._timeout_in_secs)
 return old_timeout

 def set_selenium_implicit_wait(self, seconds):
# . . .

Also it appears this feature is untested within the rf-s2l acceptance tests.

[j1z0]

Ed, I believe this is one we have discussed already. Thus I'm assuming this is an old ticket we forgot to close.