Closed
Bug 1369413
Opened 7 years ago
Closed 7 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 |
Filed by: rvandermeulen [at] mozilla.com https://treeherder.mozilla.org/logviewer.html#?job_id=103714962&repo=mozilla-inbound https://queue.taskcluster.net/v1/task/RCefXO11QoSXe04aaclftQ/runs/0/artifacts/public/logs/live_backing.log
Comment hidden (Intermittent Failures Robot) |
Comment 2•7 years ago
|
||
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.
Updated•7 years ago
|
status-firefox54:
--- → unaffected
status-firefox55:
--- → affected
Assignee | ||
Comment 3•7 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)
Comment 4•7 years ago
|
||
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•7 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•7 years ago
|
Attachment #8873778 -
Flags: review?(ato) → review+
Comment 6•7 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•7 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•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=f7d5c27c5e13701e74a1b4c174fc8cf8c481dcbc&group_state=expanded
Assignee: nobody → jjong
Keywords: checkin-needed
Comment 11•7 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•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/dbad3f1bb947
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Updated•7 years ago
|
Whiteboard: [stockwell fixed:other]
Updated•7 years ago
|
status-firefox53:
--- → unaffected
status-firefox-esr52:
--- → unaffected
Comment hidden (Intermittent Failures Robot) |
Updated•1 year ago
|
Product: Testing → Remote Protocol
Comment 14•1 year 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
•