Closed
Bug 1369413
Opened 8 years ago
Closed 8 years ago
Intermittent test_switch_remote_frame.py TestSwitchRemoteFrame.test_we_can_switch_to_a_remote_frame_by_index | IOError: Process killed because the connection to Marionette server is lost. Check gecko.log for errors
Categories
(Testing :: Marionette Client and Harness, defect)
Tracking
(firefox-esr52 unaffected, firefox53 unaffected, firefox54 unaffected, firefox55 fixed)
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox53 | --- | unaffected |
firefox54 | --- | unaffected |
firefox55 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: jessica)
References
Details
(Keywords: intermittent-failure, regression, Whiteboard: [stockwell fixed:other])
Attachments
(1 file)
1.63 KB,
patch
|
ato
:
review+
|
Details | Diff | Splinter Review |
Comment hidden (Intermittent Failures Robot) |
The problem here is:
1496328219064 Marionette TRACE 81 -> [0,49,"switchToFrame",{"focus":true,"id":0}]
JavaScript error: chrome://marionette/content/listener.js, line 1: SyntaxError: redeclaration of const INPUT_DATETIME_PREF
Seems to have been caused by https://hg.mozilla.org/integration/autoland/rev/f4448d4bcf98. So a regression from bug 1366188.
Jessica, can you please have a look?
PS: Your author name for the patch doesn't contain a valid email address.
status-firefox54:
--- → unaffected
status-firefox55:
--- → affected
Blocks: 1369570
Assignee | ||
Comment 3•8 years ago
|
||
Thanks for reporting this, I could not reproduce it locally.
But to avoid polluting the global scope, I will put the const inside my function or just use literal string in Preferences.get().
Flags: needinfo?(jjong)
When you check Orange Factor the issue only hits some platform and then specific builds. For example on Linux only ASAN builds seem to be affected.
Assignee | ||
Comment 5•8 years ago
|
||
Hi ato, I think the root cause might be loading the frame script multiple times in switchToFrame(), but that is just a guess and I don't really know the code, so I'll just avoid the issue by removing the const.
Attachment #8873778 -
Flags: review?(ato)
Updated•8 years ago
|
Attachment #8873778 -
Flags: review?(ato) → review+
Comment 6•8 years ago
|
||
(In reply to Jessica Jong [:jessica] from comment #5)
> Hi ato, I think the root cause might be loading the frame script multiple
> times in switchToFrame(), but that is just a guess and I don't really know
> the code, so I'll just avoid the issue by removing the const.
Yes, that is pretty worrying. We should look into that as part of
https://bugzilla.mozilla.org/show_bug.cgi?id=1157192.
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 9•8 years ago
|
||
(In reply to Andreas Tolfsen ‹:ato› from comment #6)
> (In reply to Jessica Jong [:jessica] from comment #5)
> > Hi ato, I think the root cause might be loading the frame script multiple
> > times in switchToFrame(), but that is just a guess and I don't really know
> > the code, so I'll just avoid the issue by removing the const.
>
> Yes, that is pretty worrying. We should look into that as part of
> https://bugzilla.mozilla.org/show_bug.cgi?id=1157192.
I see, thanks for the info.
Assignee | ||
Comment 10•8 years ago
|
||
Assignee: nobody → jjong
Keywords: checkin-needed
Comment 11•8 years ago
|
||
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/dbad3f1bb947
Remove the use of const INPUT_DATETIME_PREF. r=ato
Keywords: checkin-needed
Comment 12•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Updated•8 years ago
|
Whiteboard: [stockwell fixed:other]
Updated•8 years ago
|
status-firefox53:
--- → unaffected
status-firefox-esr52:
--- → unaffected
Comment hidden (Intermittent Failures Robot) |
Updated•2 years ago
|
Product: Testing → Remote Protocol
Comment 14•2 years ago
|
||
Moving bug to Testing::Marionette Client and Harness component per bug 1815831.
Component: Marionette → Marionette Client and Harness
Product: Remote Protocol → Testing
You need to log in
before you can comment on or make changes to this bug.
Description
•