That's definitely true, this method has generally only been used with page_checker. We don't have tests for it, and no one on the current core team is using it. It's definitely a good piece for cleanup, if you're willing to take it on and submit a github pull request, please do. Otherwise, you can submit a ticket on jira - http://jira.openqa.org/browse/WTR - let us know if you need help with either of those.
-c On Thu, Apr 22, 2010 at 11:00 AM, joedio <joe...@comcast.net> wrote: > In Watir 1.6.5 it appears that the method check_for_http_error does > NOT support IE8, as there is NO check for "8.0", only for "7.0", or a > default to IE6. I noticed this because some pages I've hit in my tests > run in IE8 are NOT raising NavigationException's even when one has > occurred. > > Here's the current method: > > # File lib/watir/page-container.rb, line 10 > > def check_for_http_error > # check for IE7 > n = self.document.invoke('parentWindow').navigator.appVersion > m=/MSIE\s(.*?);/.match( n ) > if m and m[1] =='7.0' > if m = /HTTP (\d\d\d.*)/.match( self.title ) > raise NavigationException, m[1] > end > else > # assume its IE6 > url = self.document.location.href > if /shdoclc.dll/.match(url) > m = /id=IEText.*?>(.*?)</i.match(self.html) > raise NavigationException, m[1] if m > end > end > > false > end > > -- > Before posting, please read http://watir.com/support. In short: search > before you ask, be nice. > > You received this message because you are subscribed to > http://groups.google.com/group/watir-general > To post: watir-general@googlegroups.com > -- Before posting, please read http://watir.com/support. In short: search before you ask, be nice. You received this message because you are subscribed to http://groups.google.com/group/watir-general To post: watir-general@googlegroups.com