[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