Closed
Bug 1097221
Opened 11 years ago
Closed 11 years ago
Windows opened due to an action outside of marionette are not reflected in marionette's set of window handles
Categories
(Remote Protocol :: Marionette, defect)
Remote Protocol
Marionette
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 941749
People
(Reporter: chmanchester, Assigned: chmanchester)
References
Details
Attachments
(1 file)
1.90 KB,
patch
|
Details | Diff | Splinter Review |
This is probably usually something of an edge case, but when we do something that results in a window opening, we sometimes don't end up with a reference to that window, so we can't switch to it and activate marionette for further testing. Mozmill tests do this sort of thing from time to time.
Assignee | ||
Comment 1•11 years ago
|
||
Test case demonstrating the issue. My reading of the webdriver spec is that we should know about the newly opened tab. Is that right? If not, we might need some other notion of "activating" marionette in certain windows to convert mozmill tests.
Attachment #8520846 -
Flags: feedback?(dburns)
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → cmanchester
Status: NEW → ASSIGNED
Comment 2•11 years ago
|
||
If a window has been opened in a browser that has a marionette server session active then we need to know about it. If say we have a new window created via something that doesnt speak to marionette server then if we don't know about it then it's ok to not know about it.
Assignee | ||
Comment 3•11 years ago
|
||
(In reply to David Burns :automatedtester from comment #2)
> If a window has been opened in a browser that has a marionette server
> session active then we need to know about it. If say we have a new window
> created via something that doesnt speak to marionette server then if we
> don't know about it then it's ok to not know about it.
In this case I think we need to know about the newly opened window, but I would appreciate feedback on the test case because it could be a grey area.
Comment 4•11 years ago
|
||
I wonder if this is a dupe of bug 941749
Assignee | ||
Comment 5•11 years ago
|
||
(In reply to David Burns :automatedtester from comment #4)
> I wonder if this is a dupe of bug 941749
I'd say it is!
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
Comment 6•11 years ago
|
||
I'm confused! So what is the relationship between window_handles and available tabs inside a browser window? The testcase as attached here opens a new tab but not a new window.
Flags: needinfo?(dburns)
Comment 7•11 years ago
|
||
A tab is a window too. It's a child window and not the HNWD. Please read http://w3c.github.io/webdriver/webdriver-spec.html#widl-WebDriver-getWindowHandles-sequence-DOMString.
Flags: needinfo?(dburns)
Comment 8•11 years ago
|
||
So how are you able to get the real number of top-level windows then? I was not able to find something in that referenced document. This is somewhat important for Mozmill related tests. Tabs are not windows in chrome scope.
Flags: needinfo?(dburns)
Comment 9•11 years ago
|
||
you'll always be able to be able to execute javascript to find out that we can wrap on the client side. FOr the most part people will want to switch top browsing contexts which means tabs and we should cater for that
Flags: needinfo?(dburns)
Comment 10•11 years ago
|
||
Comment on attachment 8520846 [details] [diff] [review]
Test case demonstrating the issue
Review of attachment 8520846 [details] [diff] [review]:
-----------------------------------------------------------------
clearing flag since we have a dupe and I commented
Attachment #8520846 -
Flags: feedback?(dburns)
Updated•3 years ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•