Closed
Bug 947317
Opened 12 years ago
Closed 11 years ago
MarionetteException after a tap() closes a frame - MarionetteException: {u'message': u'"toString" is read-only', u'error': u"error occurred while processing 'findElement"}
Categories
(Remote Protocol :: Marionette, defect, P1)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1065933
People
(Reporter: zcampbell, Unassigned)
References
Details
(Whiteboard: [u=marionette-user c=marionette-server p=2][touch][affects=b2g])
When adding a contact from the dialer after we close the Contacts frame Marionette cannot use the find element command.
The contacts frame can be opened in different ways - ie just by normal Contacts app operation or via the dialer. When coming back from adding a contact via the dialer there is a wait for the button we tap to be not displayed. In normal operation this works fine but when Contacts is opened via the dialer it involves changing frames.
The "wait for not displayed" would technically be performed on the wrong frame, or perhaps on both frames as Gaia transitions between them.
this can be replicated by running on a device:
tests/functional/dialer/test_dialer_add_contact.py
Reporter | ||
Comment 2•12 years ago
|
||
Updating to major, causes device freeze and blocks test coverage.
Severity: normal → major
Updated•11 years ago
|
Keywords: regression
Comment 3•11 years ago
|
||
needinfoing myself and will try pick it up next week, currently on PTO
Flags: needinfo?(dburns)
Comment 4•11 years ago
|
||
This appears to not be a regression since it was caused by a change to a bug.
Change happened in https://github.com/mozilla-b2g/gaia/commit/9bd709d310f26dc4e3c4d6a1a51a3e4567558375#diff-92508c5b8215627dee173d2d7e2d1f53
Flags: needinfo?(dburns)
Keywords: regression
Reporter | ||
Comment 5•11 years ago
|
||
(In reply to David Burns :automatedtester from comment #4)
> This appears to not be a regression since it was caused by a change to a
> bug.
>
> Change happened in
> https://github.com/mozilla-b2g/gaia/commit/
> 9bd709d310f26dc4e3c4d6a1a51a3e4567558375#diff-
> 92508c5b8215627dee173d2d7e2d1f53
Just reviewing this commit; there's flawed logic in this code anyway so I am going to revert it and also work around this Marionette bug at the same time.
It means this test will no longer be xfailed, but the Marionette bug is still outstanding as Marionette should handle closing frames like this. I'm sure this bug is a dupe of a known Marionette issue where a tap closes a frame and Marionette hangs but I can't find it just yet.
Whiteboard: [xfail]
Reporter | ||
Comment 6•11 years ago
|
||
Sorry I cannot find the old bug for this but I know we've dealt with this problem before where a tap closes the frame that Marionette is working on and Marionette can't continue, as if it's lost the reference to the frame and doesn't fall back onto one to keep the session alive.
This is almost certainly a regression but given the time frames I'll leave this bug open as a new one.
Updated•11 years ago
|
Whiteboard: [u=marionette-user c=marionette-server p=2]
Updated•11 years ago
|
Whiteboard: [u=marionette-user c=marionette-server p=2] → [u=marionette-user c=marionette-server p=2][touch]
Updated•11 years ago
|
Keywords: ateam-marionette-goal,
ateam-marionette-userinput
Updated•11 years ago
|
Priority: -- → P2
Comment 7•11 years ago
|
||
Probably related to bug 909511.
Updated•11 years ago
|
Priority: P2 → P1
Updated•11 years ago
|
Whiteboard: [u=marionette-user c=marionette-server p=2][touch] → [u=marionette-user c=marionette-server p=2][touch][affects=webqa]
Updated•11 years ago
|
Keywords: ateam-marionette-goal,
ateam-marionette-userinput
Reporter | ||
Updated•11 years ago
|
Whiteboard: [u=marionette-user c=marionette-server p=2][touch][affects=webqa] → [u=marionette-user c=marionette-server p=2][touch][affects=b2g]
Comment 8•11 years ago
|
||
This bug doesn't appear to have steps to reproduce, and is probably now a duplicate of bug 1065933. If we see this issue again then we should catch the Marionette exception when an action is expected to close the current frame.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
Updated•2 years ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•