[prev in list] [next in list] [prev in thread] [next in thread]
List: selenium-commits
Subject: [Selenium-commits] [JIRA] Created: (SEL-172) setContext() doesn't
From: "Dan Fabulich (JIRA)" <jira () public ! thoughtworks ! org>
Date: 2005-12-16 1:24:00
Message-ID: 103485728.1134696240502.JavaMail.oqa-j2ee () openqa01 ! managed ! contegix ! com
[Download RAW message or body]
setContext() doesn't work on Netscape 7.2
-----------------------------------------
Key: SEL-172
URL: http://jira.openqa.org/browse/SEL-172
Project: Selenium
Type: Bug
Components: BrowserBot-Mozilla
Versions: 0.6
Environment: Netscape 7.2, Java Driver 0.6.0 (with Nelson's patch from SEL-168)
Reporter: Dan Fabulich
This breaks the RealDealIntegrationTest for the Java Driver 0.6.0.
I watched this run in a debugger; for some reason, on Netscape 7.2, the \
PageBot.prototype.setContext() function doesn't work. The context element is clearly \
there with its ID, but on Netscape 7.2 it claims that it can't find the element.
Don't ask me to explain why, but when I replace selenium-browserbot.js line 913 like \
thus, it works like a charm: 913c913
< context.innerHTML=strContext;
---
> document.getElementById("context").innerHTML=strContext;
These lines *should* be equivalent, as far as I know, but only the latter works in \
Netscape 7.2.
I'm not surprised that this action doesn't get adequate browser testing, because it's \
not really a useful verb outside of Selenium Driven Mode. Maybe the whole action \
should be removed/deprecated?
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.openqa.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
_______________________________________________
Selenium-commits mailing list
Selenium-commits@lists.public.thoughtworks.org
http://lists.public.thoughtworks.org/mailman/listinfo/selenium-commits
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic