Closed
Bug 1263878
Opened 8 years ago
Closed 8 years ago
Intermittent test_presentation_tcp_receiver_oop.html | The control channel is closed normally. - got 2152923159, expected +0
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
People
(Reporter: cbook, Assigned: CuveeHsu)
References
()
Details
(Keywords: intermittent-failure, Whiteboard: [ft:conndevices])
Attachments
(2 files, 2 obsolete files)
2.94 KB,
patch
|
CuveeHsu
:
review+
|
Details | Diff | Splinter Review |
48 bytes,
text/x-github-pull-request
|
jocheng
:
approval-mozilla-b2g48+
|
Details | Review |
https://treeherder.mozilla.org/logviewer.html#?job_id=25601048&repo=mozilla-inbound 10:04:26 INFO - 271 INFO TEST-UNEXPECTED-FAIL | dom/presentation/tests/mochitest/test_presentation_tcp_receiver_oop.html | The control channel is closed normally. - got 2152923159, expected +0
Reporter | ||
Comment 1•8 years ago
|
||
junior: you might want to take a look at this i guess :)
Flags: needinfo?(juhsu)
Assignee | ||
Comment 2•8 years ago
|
||
I'm not sure if it's a dup of Bug 1245560 or not. (I rename the file in bug 1148307) I keep the needinfo and will investigate it. Stay tuned.
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Updated•8 years ago
|
Flags: needinfo?(juhsu)
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 13•8 years ago
|
||
This is pretty frequent. Please investigate or disable the test ASAP.
Flags: needinfo?(juhsu)
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Updated•8 years ago
|
Flags: needinfo?(juhsu)
Assignee | ||
Comment 15•8 years ago
|
||
Sorry for wrongly cleaning the needinfo twice. I found a weird fact that |NotifyReceiverReady| is before|HandleSessionRequest|. I'm still working on it.
Comment 16•8 years ago
|
||
should be the same as bug 1261595, let's try my patch first.
Attachment #8744763 -
Flags: review?(juhsu)
Assignee | ||
Comment 17•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=b1daf7f34752
Assignee | ||
Comment 18•8 years ago
|
||
Here's the patch for randomized session ID. Do you think if we still need to extend the time-out?
Assignee: nobody → juhsu
Attachment #8744769 -
Flags: review?(schien)
Updated•8 years ago
|
Attachment #8744763 -
Attachment is obsolete: true
Attachment #8744763 -
Flags: review?(juhsu)
Comment 19•8 years ago
|
||
Comment on attachment 8744769 [details] [diff] [review] randomized session id Review of attachment 8744769 [details] [diff] [review]: ----------------------------------------------------------------- let's try this patch first.
Attachment #8744769 -
Flags: review?(schien) → review+
Assignee | ||
Comment 20•8 years ago
|
||
a minor change. carry r+ Let's wait for the treeherder green
Attachment #8744769 -
Attachment is obsolete: true
Attachment #8744787 -
Flags: review+
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Comment hidden (Intermittent Failures Robot) |
Comment 22•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/97a52fc1216a
Keywords: checkin-needed
Reporter | ||
Comment 23•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/97a52fc1216a
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Reporter | ||
Updated•8 years ago
|
Target Milestone: mozilla48 → mozilla49
Comment 24•8 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/e4f374edfb91f304b2b44b0f09694b32a1b5d6cd
status-firefox48:
--- → fixed
Comment hidden (Intermittent Failures Robot) |
Comment 28•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/0b564c344657
status-firefox47:
--- → fixed
Assignee | ||
Comment 29•8 years ago
|
||
NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings. [Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 1263878 User impact if declined: Every patches for Presentation API need two versions of them, one for m-c and another one for b2g. Testing completed: manual testing passed Risk to taking this patch (and alternatives if risky): no risk since this patch only has impacts on tests String or UUID changes made by this patch:
Attachment #8758528 -
Flags: approval-mozilla-b2g48?
Assignee | ||
Updated•8 years ago
|
Whiteboard: [ft:conndevices]
Updated•8 years ago
|
blocking-b2g: --- → 2.6+
Comment 30•8 years ago
|
||
Comment on attachment 8758528 [details] [review] randomized session id (v2.6) Approve for TV 2.6
Attachment #8758528 -
Flags: approval-mozilla-b2g48? → approval-mozilla-b2g48+
Updated•8 years ago
|
status-b2g-v2.6:
--- → affected
Comment 31•8 years ago
|
||
https://github.com/mozilla-b2g/gecko-b2g/commit/d278d24cbc21363c90992b970c4dbb02ff4b6e88
You need to log in
before you can comment on or make changes to this bug.
Description
•