watir-webdriver 0.6.11 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f701ec966d5fba22ef3505c0df91d8db1581b04e
-  data.tar.gz: c5002bc487ed49c06385469e0b531b23ecd6b11b
+  metadata.gz: 3c59e0947d61d3444072fba9199dbb1923ef78bb
+  data.tar.gz: cea68a6afbae78f341d70c43e276697cd297b42a
 SHA512:
-  metadata.gz: ad1194e189f237e24d3ff125332b038ca9361ed427d5dd51c15c61da29c9d7da24cd5a3df8303434d1eeea4f0d622e82a7d8932dd1510966a4763e2878a19047
-  data.tar.gz: 79f65f913cc04a54ed1a5f25a4717bbed04ba1a3529679c9647a40d752491f17cdb96b24254584def0a78b6af1b6ece720e602b5111317fe4fb71ede41540e06
+  metadata.gz: 118d8dac0739c61e992e684c3f7ea86744142bf27f68f10bd766f3f930724c5d7c7475b9ba02d2fb11dda3ff33c582ddf02227a5933e23b8c07d39409fcb1473
+  data.tar.gz: 76b6e52b21db799697ccf8777debbeaebd14bbb5e19e60825c6f7c6a5ccce387c964dd647e1506f5dc4c8a6315264b8f9816f77cd234987e219be7741399b723

data/.travis.yml

@@ -9,18 +9,20 @@ notifications:
   irc: "irc.freenode.net#watir"
 before_script:
   - support/travis.sh
+script: bundle exec rake $RAKE_TASK
 env:
-  - WATIR_WEBDRIVER_BROWSER=firefox
-  - WATIR_WEBDRIVER_BROWSER=firefox ALWAYS_LOCATE=false
-  - WATIR_WEBDRIVER_BROWSER=firefox PREFER_CSS=1 SELECTOR_STATS=1
-  - WATIR_WEBDRIVER_BROWSER=chrome
-  - WATIR_WEBDRIVER_BROWSER=chrome ALWAYS_LOCATE=false
-  - WATIR_WEBDRIVER_BROWSER=chrome PREFER_CSS=1 SELECTOR_STATS=1
-  - WATIR_WEBDRIVER_BROWSER=phantomjs
-  - WATIR_WEBDRIVER_BROWSER=phantomjs ALWAYS_LOCATE=false
-  - WATIR_WEBDRIVER_BROWSER=phantomjs PREFER_CSS=1 SELECTOR_STATS=1
+  - RAKE_TASK=spec WATIR_WEBDRIVER_BROWSER=firefox
+  - RAKE_TASK=spec WATIR_WEBDRIVER_BROWSER=firefox ALWAYS_LOCATE=false
+  - RAKE_TASK=spec WATIR_WEBDRIVER_BROWSER=firefox PREFER_CSS=1 SELECTOR_STATS=1
+  - RAKE_TASK=spec WATIR_WEBDRIVER_BROWSER=chrome
+  - RAKE_TASK=spec WATIR_WEBDRIVER_BROWSER=chrome ALWAYS_LOCATE=false
+  - RAKE_TASK=spec WATIR_WEBDRIVER_BROWSER=chrome PREFER_CSS=1 SELECTOR_STATS=1
+  # - RAKE_TASK=spec WATIR_WEBDRIVER_BROWSER=phantomjs
+  # - RAKE_TASK=spec WATIR_WEBDRIVER_BROWSER=phantomjs ALWAYS_LOCATE=false
+  # - RAKE_TASK=spec WATIR_WEBDRIVER_BROWSER=phantomjs PREFER_CSS=1 SELECTOR_STATS=1
+  - RAKE_TASK=yard:doctest
 matrix:
   allow_failures:
-    - env: WATIR_WEBDRIVER_BROWSER=phantomjs
-    - env: WATIR_WEBDRIVER_BROWSER=phantomjs ALWAYS_LOCATE=false
-    - env: WATIR_WEBDRIVER_BROWSER=phantomjs PREFER_CSS=1 SELECTOR_STATS=1
+    - env: RAKE_TASK=spec WATIR_WEBDRIVER_BROWSER=phantomjs
+    - env: RAKE_TASK=spec WATIR_WEBDRIVER_BROWSER=phantomjs ALWAYS_LOCATE=false
+    - env: RAKE_TASK=spec WATIR_WEBDRIVER_BROWSER=phantomjs PREFER_CSS=1 SELECTOR_STATS=1

data/CHANGES.md

@@ -1,3 +1,20 @@
+### 0.7.0
+
+* Allow finding all elements with prefer_css
+* Add support for yard-doctest (#287)
+* Update from HTML spec (#296)
+* Support tag_name call on Frames and IFrames (#293 & #294)
+* Increased performance by caching elements by default where possible (#307)
+* Improved handling of elements that go stale during lookup (#291, thanks @titusfortner)
+* Fix element location issues when switching between IFrames (#286, thanks @titusfortner)
+* Fix creation of an IFrameCollection based on selector (#299, thanks @titusfortner)
+* Fix window handling with closed windows (#290 & 282, thanks @titusfortner)
+* Prevent running checkers on a closed window (#283, thanks @titusfortner)
+* Allow taking actions without triggering run checkers (#283, thanks @titusfortner)
+* Fix bug when ElementCollection#[] returns existing elements for non-existing selector (#309)
+* Fix bug when Wait would never execute block with 0 timeout (#312)
+* Fix race condition with IFrameCollection#to_a (#317)
+
 ### 0.6.11 (2014-09-23)
 
 * Fix namespacing issue (#265, thanks @titusfortner)

data/LICENSE

@@ -1,6 +1,6 @@
 (the MIT License)
 
-Copyright (c) 2009-2014 Jari Bakken
+Copyright (c) 2009-2015 Jari Bakken
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/README.md

@@ -38,6 +38,13 @@ watir-webdriver uses [watirspec](http://github.com/watir/watirspec) for testing.
 
 Specs specific to watir-webdriver are found in spec/*_spec.rb, with watirspec in spec/watirspec.
 
+Doctests
+--------
+
+watir-webdriver uses [yard-doctest](https://github.com/p0deje/yard-doctest) for testing documentation examples.
+
+    rake yard:doctest
+
 API docs
 --------
 
@@ -71,4 +78,4 @@ Note on Patches/Pull Requests
 Copyright
 ---------
 
-Copyright (c) 2009-2014 Jari Bakken. See LICENSE for details.
+Copyright (c) 2009-2015 Jari Bakken. See LICENSE for details.

data/Rakefile

@@ -20,10 +20,10 @@ namespace :spec do
   end
 end
 
-task :default => :spec
+task :default => [:spec, 'yard:doctest']
 
 namespace :html5 do
-  SPEC_URI  = "http://www.whatwg.org/specs/web-apps/current-work/"
+  SPEC_URI  = "https://www.whatwg.org/specs/web-apps/current-work/"
   SPEC_PATH = "support/html5.html"
 
   task :html_lib => :lib do
@@ -91,6 +91,11 @@ YARD::Rake::YardocTask.new do |task|
   task.options = %w[--debug] # this is pretty slow, so nice with some output
 end
 
+require 'yard-doctest'
+YARD::Doctest::RakeTask.new do |task|
+  task.doctest_opts = ['-v']
+end
+
 namespace :changes do
   task :differ do
     require './support/version_differ'

data/lib/watir-webdriver/alert.rb

@@ -14,7 +14,7 @@ module Watir
     #
     # @example
     #   browser.alert.text
-    #   #=> "This is alert text"
+    #   #=> "ok"
     #
     # @return [String]
     #
@@ -70,6 +70,10 @@ module Watir
     #
     # Returns true if alert, confirm or prompt is present and false otherwise.
     #
+    # @example
+    #   browser.alert.exists?
+    #   #=> true
+    #
 
     def exists?
       assert_exists

data/lib/watir-webdriver/attribute_helper.rb

@@ -68,30 +68,26 @@ module Watir
 
     def define_string_attribute(mname, aname)
       define_method mname do
-        assert_exists
-        @element.attribute(aname).to_s
+        attribute_value(aname).to_s
       end
     end
 
     def define_boolean_attribute(mname, aname)
       define_method mname do
-        assert_exists
-        @element.attribute(aname) == "true"
+        attribute_value(aname) == "true"
       end
     end
 
     def define_int_attribute(mname, aname)
       define_method mname do
-        assert_exists
-        value = @element.attribute(aname)
+        value = attribute_value(aname)
         value && Integer(value)
       end
     end
 
     def define_float_attribute(mname, aname)
       define_method mname do
-        assert_exists
-        value = @element.attribute(aname)
+        value = attribute_value(aname)
         value && Float(value)
       end
     end

data/lib/watir-webdriver/browser.rb

@@ -18,7 +18,7 @@ module Watir
       # Creates a Watir::Browser instance and goes to URL.
       #
       # @example
-      #   browser = Watir::Browser.start "www.google.com", :chrome
+      #   browser = Watir::Browser.start "www.google.com", :firefox
       #   #=> #<Watir::Browser:0x..fa45a499cb41e1752 url="http://www.google.com" title="Google">
       #
       # @param [String] url
@@ -65,7 +65,7 @@ module Watir
     # Goes to the given URL.
     #
     # @example
-    #   browser.goto "www.google.com"
+    #   browser.goto "www.watir.com"
     #
     # @param [String] uri The url.
     # @return [String] The url you end up at.
@@ -100,9 +100,9 @@ module Watir
     # Returns URL of current page.
     #
     # @example
-    #   browser.goto "http://www.google.com"
+    #   browser.goto "www.watir.com"
     #   browser.url
-    #   #=> "http://www.google.com"
+    #   #=> "http://watir.com/"
     #
     # @return [String]
     #
@@ -116,9 +116,9 @@ module Watir
     # Returns title of current page.
     #
     # @example
-    #   browser.goto "http://www.google.com"
+    #   browser.goto "www.watir.com"
     #   browser.title
-    #   #=> "Google"
+    #   #=> "Watir.com | Web Application Testing in Ruby"
     #
     # @return [String]
     #
@@ -152,9 +152,9 @@ module Watir
     # Returns browser name.
     #
     # @example
-    #   browser = Watir::Browser.new :chrome
+    #   browser = Watir::Browser.new :firefox
     #   browser.name
-    #   #=> :chrome
+    #   #=> :firefox
     #
     # @return [Symbol]
     #
@@ -243,14 +243,9 @@ module Watir
     # `return` explicitly.
     #
     # @example Check that Ajax requests are completed with jQuery
-    #   browser.execute_script("return jQuery.active") == '0'
+    #   browser.execute_script("return jQuery.active") == 0
     #   #=> true
     #
-    # @example Get inner HTML of element
-    #   span = browser.span(class: "someclass")
-    #   browser.execute_script "return arguments[0].innerHTML", span
-    #   #=> "Span innerHTML"
-    #
     # @param [String] script JavaScript snippet to execute
     # @param *args Arguments will be available in the given script in the 'arguments' pseudo-array
     #
@@ -266,7 +261,7 @@ module Watir
     # Sends sequence of keystrokes to currently active element.
     #
     # @example
-    #   browser.goto "http://www.google.com"
+    #   browser.goto "www.google.com"
     #   browser.send_keys "Watir", :return
     #
     # @param [String, Symbol] *args
@@ -299,8 +294,8 @@ module Watir
     #   browser.add_checker do |page|
     #     page.text.include?("Server Error") and puts "Application exception or 500 error!"
     #   end
-    #   browser.goto "www.mywebsite.com/page-with-error"
-    #   "Server error! (RuntimeError)"
+    #   browser.goto "www.watir.com/404"
+    #   "Application exception or 500 error!"
     #
     # @param [#call] checker Object responding to call
     # @yield Checker block
@@ -325,8 +320,8 @@ module Watir
     #     page.text.include?("Server Error") and puts "Application exception or 500 error!"
     #   end
     #   browser.add_checker checker
-    #   browser.goto "www.mywebsite.com/page-with-error"
-    #   "Server error! (RuntimeError)"
+    #   browser.goto "www.watir.com/404"
+    #   "Application exception or 500 error!"
     #   browser.disable_checker checker
     #   browser.refresh
     #
@@ -340,7 +335,26 @@ module Watir
     #
 
     def run_checkers
-      @error_checkers.each { |e| e.call(self) }
+      @error_checkers.each { |e| e.call(self) } if !@error_checkers.empty? && window.present?
+    end
+
+    #
+    # Executes a block without running error checkers.
+    #
+    # @example
+    #   browser.without_checkers do
+    #     browser.element(:name => "new_user_button").click
+    #   end
+    #
+    # @yieldparam [Watir::Browser]
+    #
+
+    def without_checkers
+      current_checkers = @error_checkers
+      @error_checkers = []
+      yield(self)
+    ensure
+      @error_checkers = current_checkers
     end
 
     #
@@ -350,7 +364,10 @@ module Watir
     #
 
     def exist?
-      not @closed
+      assert_exists
+      true
+    rescue Exception::NoMatchingWindowFoundException, Exception::Error
+      false
     end
     alias_method :exists?, :exist?
 
@@ -363,11 +380,14 @@ module Watir
     def assert_exists
       if @closed
         raise Exception::Error, "browser was closed"
+      elsif !window.present?
+        raise Exception::NoMatchingWindowFoundException, "browser window was closed"
       else
         driver.switch_to.default_content
         true
       end
     end
+    alias_method :assert_not_stale, :assert_exists
 
     def reset!
       # no-op

data/lib/watir-webdriver/element_collection.rb

@@ -42,7 +42,6 @@ module Watir
 
     #
     # Get the element at the given index.
-    # Note that this is 0-indexed and not compatible with older Watir implementations.
     #
     # Also note that because of Watir's lazy loading, this will return an Element
     # instance even if the index is out of bounds.
@@ -52,7 +51,7 @@ module Watir
     #
 
     def [](idx)
-      to_a[idx] || element_class.new(@parent, :index => idx)
+      to_a[idx] || element_class.new(@parent, @selector.merge(:index => idx))
     end
 
     #

data/lib/watir-webdriver/elements/button.rb

@@ -23,15 +23,13 @@ module Watir
     #
 
     def text
-      assert_exists
-
-      tn = @element.tag_name.downcase
+      tn = tag_name
 
       case tn
       when 'input'
-        @element.attribute(:value)
+        value
       when 'button'
-        @element.text
+        super
       else
         raise Exception::Error, "unknown tag name for button: #{tn}"
       end

data/lib/watir-webdriver/elements/checkbox.rb

@@ -17,14 +17,7 @@ module Watir
     #
 
     def set(bool = true)
-      assert_exists
-      assert_enabled
-
-      if @element.selected?
-        @element.click unless bool
-      else
-        @element.click if bool
-      end
+      set? == bool ? assert_enabled : click
     end
 
     #
@@ -34,14 +27,12 @@ module Watir
 
     def set?
       assert_exists
-      @element.selected?
+      element_call { @element.selected? }
     end
 
     #
     # Unsets checkbox.
     #
-    # Same as +set(false)+
-    #
 
     def clear
       set false

data/lib/watir-webdriver/elements/element.rb

@@ -60,15 +60,12 @@ module Watir
     # Returns true if two elements are equal.
     #
     # @example
-    #   browser.a(:id => "foo") == browser.a(:id => "foo")
+    #   browser.text_field(:name => "new_user_first_name") == browser.text_field(:name => "new_user_first_name")
     #   #=> true
     #
 
     def ==(other)
-      return false unless other.kind_of? self.class
-
-      assert_exists
-      @element == other.wd
+      other.kind_of?(self.class) && wd == other.wd
     end
     alias_method :eql?, :==
 
@@ -84,7 +81,7 @@ module Watir
 
     def text
       assert_exists
-      @element.text
+      element_call { @element.text }
     end
 
     #
@@ -95,7 +92,7 @@ module Watir
 
     def tag_name
       assert_exists
-      @element.tag_name.downcase
+      element_call { @element.tag_name.downcase }
     end
 
     #
@@ -104,13 +101,13 @@ module Watir
     # and may not work at all.
     #
     # @example Click an element
-    #   element.click
+    #   browser.element(:name => "new_user_button").click
     #
     # @example Click an element with shift key pressed
-    #   element.click(:shift)
+    #   browser.element(:name => "new_user_button").click(:shift)
     #
     # @example Click an element with several modifier keys pressed
-    #   element.click(:shift, :control)
+    #   browser.element(:name => "new_user_button").click(:shift, :control)
     #
     # @param [:shift, :alt, :control, :command, :meta] Modifier key(s) to press while clicking.
     #
@@ -119,17 +116,19 @@ module Watir
       assert_exists
       assert_enabled
 
-      if modifiers.any?
-        assert_has_input_devices_for "click(#{modifiers.join ', '})"
+      element_call do
+        if modifiers.any?
+          assert_has_input_devices_for "click(#{modifiers.join ', '})"
 
-        action = driver.action
-        modifiers.each { |mod| action.key_down mod }
-        action.click @element
-        modifiers.each { |mod| action.key_up mod }
+          action = driver.action
+          modifiers.each { |mod| action.key_down mod }
+          action.click @element
+          modifiers.each { |mod| action.key_up mod }
 
-        action.perform
-      else
-        @element.click
+          action.perform
+        else
+          @element.click
+        end
       end
 
       run_checkers
@@ -140,14 +139,14 @@ module Watir
     # Note that browser support may vary.
     #
     # @example
-    #   browser.a(:id => "foo").double_click
+    #   browser.element(:name => "new_user_button").double_click
     #
 
     def double_click
       assert_exists
       assert_has_input_devices_for :double_click
 
-      driver.action.double_click(@element).perform
+      element_call { driver.action.double_click(@element).perform }
       run_checkers
     end
 
@@ -156,14 +155,14 @@ module Watir
     # Note that browser support may vary.
     #
     # @example
-    #   browser.a(:id => "foo").right_click
+    #   browser.element(:name => "new_user_button").right_click
     #
 
     def right_click
       assert_exists
       assert_has_input_devices_for :right_click
 
-      driver.action.context_click(@element).perform
+      element_call { driver.action.context_click(@element).perform }
       run_checkers
     end
 
@@ -172,14 +171,14 @@ module Watir
     # Note that browser support may vary.
     #
     # @example
-    #   browser.a(:id => "foo").hover
+    #   browser.element(:name => "new_user_button").hover
     #
 
     def hover
       assert_exists
       assert_has_input_devices_for :hover
 
-      driver.action.move_to(@element).perform
+      element_call { driver.action.move_to(@element).perform }
     end
 
     #
@@ -187,9 +186,9 @@ module Watir
     # Note that browser support may vary.
     #
     # @example
-    #    a = browser.div(:id => "draggable")
-    #    b = browser.div(:id => "droppable")
-    #    a.drag_and_drop_on b
+    #   a = browser.div(:id => "draggable")
+    #   b = browser.div(:id => "droppable")
+    #   a.drag_and_drop_on b
     #
 
     def drag_and_drop_on(other)
@@ -197,9 +196,11 @@ module Watir
       assert_exists
       assert_has_input_devices_for :drag_and_drop_on
 
-      driver.action.
-             drag_and_drop(@element, other.wd).
-             perform
+      element_call do
+        driver.action.
+               drag_and_drop(@element, other.wd).
+               perform
+      end
     end
 
     #
@@ -207,7 +208,7 @@ module Watir
     # Note that browser support may vary.
     #
     # @example
-    #    browser.div(:id => "draggable").drag_and_drop_by 100, -200
+    #   browser.div(:id => "draggable").drag_and_drop_by 100, -200
     #
     # @param [Fixnum] right_by
     # @param [Fixnum] down_by
@@ -217,16 +218,18 @@ module Watir
       assert_exists
       assert_has_input_devices_for :drag_and_drop_by
 
-      driver.action.
-             drag_and_drop_by(@element, right_by, down_by).
-             perform
+      element_call do
+        driver.action.
+               drag_and_drop_by(@element, right_by, down_by).
+               perform
+      end
     end
 
     #
     # Flashes (change background color far a moment) element.
     #
     # @example
-    #    browser.div(:id => "draggable").flash
+    #   browser.text_field(:name => "new_user_first_name").flash
     #
 
     def flash
@@ -250,44 +253,40 @@ module Watir
     #
 
     def value
-      assert_exists
-
-      begin
-        @element.attribute('value') || ''
-      rescue Selenium::WebDriver::Error::InvalidElementStateError
-        ""
-      end
+      attribute_value('value') || ''
+    rescue Selenium::WebDriver::Error::InvalidElementStateError
+      ''
     end
 
     #
     # Returns given attribute value of element.
     #
     # @example
-    #   browser.a(:id => "foo").attribute_value "href"
-    #   #=> "http://watir.com"
+    #   browser.a(:id => "link_2").attribute_value "title"
+    #   #=> "link_title_2"
     #
     # @param [String] attribute_name
-    # @return [String]
+    # @return [String, nil]
     #
 
     def attribute_value(attribute_name)
       assert_exists
-      @element.attribute attribute_name
+      element_call { @element.attribute attribute_name }
     end
 
     #
     # Returns outer (inner + element itself) HTML code of element.
     #
     # @example
-    #   browser.div(:id => "foo").html
-    #   #=> "<div id=\"foo\"><a>Click</a></div>"
+    #   browser.div(:id => 'foo').outer_html
+    #   #=> "<div id=\"foo\"><a href=\"#\">hello</a></div>"
     #
     # @return [String]
     #
 
     def outer_html
       assert_exists
-      execute_atom(:getOuterHtml, @element).strip
+      element_call { execute_atom(:getOuterHtml, @element) }.strip
     end
 
     alias_method :html, :outer_html
@@ -296,29 +295,30 @@ module Watir
     # Returns inner HTML code of element.
     #
     # @example
-    #   browser.div(:id => "foo").html
-    #   #=> "<a>Click</a>"
+    #   browser.div(:id => 'foo').inner_html
+    #   #=> "<a href=\"#\">hello</a>"
     #
     # @return [String]
     #
 
     def inner_html
       assert_exists
-      execute_atom(:getInnerHtml, @element).strip
+      element_call { execute_atom(:getInnerHtml, @element) }.strip
     end
 
     #
     # Sends sequence of keystrokes to element.
     #
     # @example
-    #   browser.div(:id => "foo").send_keys "Watir", :return
+    #   browser.text_field(:name => "new_user_first_name").send_keys "Watir", :return
     #
     # @param [String, Symbol] *args
     #
 
     def send_keys(*args)
       assert_exists
-      @element.send_keys(*args)
+      assert_writable
+      element_call { @element.send_keys(*args) }
     end
 
     #
@@ -330,7 +330,7 @@ module Watir
 
     def focus
       assert_exists
-      driver.execute_script "return arguments[0].focus()", @element
+      element_call { driver.execute_script "return arguments[0].focus()", @element }
     end
 
     #
@@ -341,7 +341,7 @@ module Watir
 
     def focused?
       assert_exists
-      @element == driver.switch_to.active_element
+      element_call { @element == driver.switch_to.active_element }
     end
 
     #
@@ -349,9 +349,9 @@ module Watir
     # Note that you may omit "on" from event name.
     #
     # @example
-    #   browser.a(:id => "foo").fire_event :click
-    #   browser.a(:id => "foo").fire_event "mousemove"
-    #   browser.a(:id => "foo").fire_event "onmouseover"
+    #   browser.button(:name => "new_user_button").fire_event :click
+    #   browser.button(:name => "new_user_button").fire_event "mousemove"
+    #   browser.button(:name => "new_user_button").fire_event "onmouseover"
     #
     # @param [String, Symbol] event_name
     #
@@ -360,7 +360,7 @@ module Watir
       assert_exists
       event_name = event_name.to_s.sub(/^on/, '').downcase
 
-      execute_atom :fireEvent, @element, event_name
+      element_call { execute_atom :fireEvent, @element, event_name }
     end
 
     #
@@ -370,7 +370,7 @@ module Watir
     def parent
       assert_exists
 
-      e = execute_atom :getParentElement, @element
+      e = element_call { execute_atom :getParentElement, @element }
 
       if e.kind_of?(Selenium::WebDriver::Element)
         Watir.element_class_for(e.tag_name.downcase).new(@parent, :element => e)
@@ -402,7 +402,7 @@ module Watir
 
     def visible?
       assert_exists
-      @element.displayed?
+      element_call { @element.displayed? }
     end
 
     #
@@ -414,7 +414,7 @@ module Watir
 
     def present?
       exists? && visible?
-    rescue Selenium::WebDriver::Error::ObsoleteElementError, UnknownObjectException
+    rescue Selenium::WebDriver::Error::StaleElementReferenceError, UnknownObjectException
       # if the element disappears between the exists? and visible? calls,
       # consider it not present.
       false
@@ -424,10 +424,8 @@ module Watir
     # Returns given style property of this element.
     #
     # @example
-    #   browser.a(:id => "foo").style
-    #   #=> "display: block"
-    #   browser.a(:id => "foo").style "display"
-    #   #=> "block"
+    #   browser.button(:value => "Delete").style           #=> "border: 4px solid red;"
+    #   browser.button(:value => "Delete").style("border") #=> "4px solid red"
     #
     # @param [String] property
     # @return [String]
@@ -436,7 +434,7 @@ module Watir
     def style(property = nil)
       if property
         assert_exists
-        @element.style property
+        element_call { @element.style property }
       else
         attribute_value("style").to_s.strip
       end
@@ -497,39 +495,35 @@ module Watir
   protected
 
     def assert_exists
-      if @element and not Watir.always_locate?
-        assert_not_stale
-        return
+      begin
+        assert_not_stale if @element ||= @selector[:element]
+      rescue UnknownObjectException => ex
+        raise ex if @selector[:element] || !Watir.always_locate?
       end
 
-      @element = @selector[:element]
-
-      if @element
-        assert_not_stale
-      else
-        @element = locate
+      @element ||= locate
 
-        unless @element
-          raise UnknownObjectException, "unable to locate element, using #{selector_string}"
-        end
+      unless @element
+        raise UnknownObjectException, "unable to locate element, using #{selector_string}"
       end
     end
 
     def assert_not_stale
+      @parent.assert_not_stale
+      @parent.switch_to! if @parent.is_a? IFrame
       @element.enabled? # do a staleness check - any wire call will do.
     rescue Selenium::WebDriver::Error::ObsoleteElementError => ex
       # don't cache a stale element - it will never come back
-      @element = nil
+      reset!
       raise UnknownObjectException, "#{ex.message} - #{selector_string}"
     end
 
     def reset!
-      @parent.reset!
       @element = nil
     end
 
     def locate
-      @parent.assert_exists
+      @parent.is_a?(IFrame) ? @parent.switch_to! : @parent.assert_exists
       locator_class.new(@parent.wd, @selector, self.class.attribute_list).locate
     end
 
@@ -545,11 +539,15 @@ module Watir
 
     def attribute?(attribute)
       assert_exists
-      !!execute_atom(:getAttribute, @element, attribute.to_s.downcase)
+      element_call do
+        !!execute_atom(:getAttribute, @element, attribute.to_s.downcase)
+      end
     end
 
     def assert_enabled
-      raise ObjectDisabledException, "object is disabled #{selector_string}" unless @element.enabled?
+      unless element_call { @element.enabled? }
+        raise ObjectDisabledException, "object is disabled #{selector_string}"
+      end
     end
 
     def assert_writable
@@ -572,6 +570,15 @@ module Watir
       end
     end
 
+    def element_call
+      yield
+    rescue Selenium::WebDriver::Error::StaleElementReferenceError
+      raise unless Watir.always_locate?
+      reset!
+      assert_exists
+      retry
+    end
+
     def method_missing(meth, *args, &blk)
       method = meth.to_s
       if method =~ ElementLocator::WILDCARD_ATTRIBUTE