Closed
Bug 1289292
Opened 9 years ago
Closed 9 years ago
[Presentation WebAPI] terminate PresentationConnection locally while fail to establish control channel
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
People
(Reporter: schien, Assigned: schien)
References
Details
Attachments
(2 files, 1 obsolete file)
19.57 KB,
patch
|
schien
:
review+
|
Details | Diff | Splinter Review |
48 bytes,
text/x-github-pull-request
|
jocheng
:
approval-mozilla-b2g48+
|
Details | Review |
In current m-c, PresentationSessionInfo::Close didn't handle failure during control channel establishment.
https://hg.mozilla.org/mozilla-central/file/tip/dom/presentation/PresentationSessionInfo.cpp#l303
Assignee | ||
Comment 1•9 years ago
|
||
Make sure |Shutdown| is invoked when fail to establish control channel or fail to send terminate command.
Attachment #8774981 -
Flags: review?(bugs)
Assignee | ||
Comment 2•9 years ago
|
||
Comment 3•9 years ago
|
||
Comment on attachment 8774981 [details] [diff] [review]
local-terminate-while-fail.patch
>+function testConnectionAvailable() {
>+ return new Promise(function(aResolve, aReject) {
>+ info('Receiver: --- testConnectionAvailable ---');
>+ ok(navigator.presentation, 'Receiver: navigator.presentation should be available.');
>+ ok(navigator.presentation.receiver, 'Receiver: navigator.presentation.receiver should be available.');
>+
>+ navigator.presentation.receiver.connectionList
>+ .then((aList) => {
>+ is(aList.connections.length, 1, 'Should get one conncetion.');
connection
Attachment #8774981 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 4•9 years ago
|
||
update according to review comment, carry r+.
Attachment #8774981 -
Attachment is obsolete: true
Attachment #8775403 -
Flags: review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 5•9 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 1276378
User impact if declined: cannot initiate page terminate on receiver side
Testing completed: yes
Risk to taking this patch (and alternatives if risky): low
String or UUID changes made by this patch: n/a
Attachment #8775407 -
Flags: approval-mozilla-b2g48?
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4730019ab44c
Shutdown session info while fail to establish control channel. r=smaug
Keywords: checkin-needed
Comment 7•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox50:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Comment 8•9 years ago
|
||
Comment on attachment 8775407 [details] [review]
pull request for tv 2.6
Approve for TV2.6
Attachment #8775407 -
Flags: approval-mozilla-b2g48? → approval-mozilla-b2g48+
Assignee | ||
Comment 9•9 years ago
|
||
Comment on attachment 8775407 [details] [review]
pull request for tv 2.6
ni @xeonchen for uplift.
Flags: needinfo?(xeonchen)
Comment 10•9 years ago
|
||
status-b2g-v2.6:
--- → fixed
Flags: needinfo?(xeonchen)
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•