191260 2018-11-05 06:47:08 -0800 Setting transceiver.direction = "inactive" is ignored 2018-12-06 03:39:57 -0800 1 1 1 Unclassified WebKit WebRTC Safari Technology Preview Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 ibc webkit-unassigned youennf oldest_to_newest 1475060 0 353859 ibc 2018-11-05 06:47:08 -0800 Created attachment 353859 Script that reproduces the issue. Just run it in Safari 12.1 console with "Unified-Plan" enabled. Scenario: * Have a pc with an audio transceiver for just sending audio (it does not receive audio from the remote), so it has `direction: "sendonly"`. * Call `transceiver.sender.replaceTrack(null)` and `transceiver.direction = "inactive"` and do a new SDP O/A. * No matter where `transceiver.direction = "inactive"` is set. The `transceiver.direction` remains "sendonly" and its m=audio section also remains with "a=sendonly". This works as expected in Chrome and Firefox. Script attached. 1478189 1 ibc 2018-11-13 16:05:58 -0800 Hi, any news about this issue? 1478222 2 youennf 2018-11-13 16:52:09 -0800 (In reply to Iñaki Baz from comment #1) > Hi, any news about this issue? Will look at it this week. 1478225 3 youennf 2018-11-13 16:57:55 -0800 Using WebKit Tot, your test case works as expected. Let's mark it as fixed for now. Please reopen if next STP version is not working as expected. 1480282 4 ibc 2018-11-19 13:02:15 -0800 Hi, I've tested in latest STP: Release 70 (Safari 12.1, WebKit 14607.1.13) The attached still fails (direction remains a=sendonly instead of a=inactive). Should I wait for next STP? 1485673 5 ibc 2018-12-06 03:39:57 -0800 ok, it works today in latest (Safari 12.1, WebKit 14607.1.15. Good work :) 353859 2018-11-05 06:47:08 -0800 2018-11-05 06:47:08 -0800 Script that reproduces the issue. Just run it in Safari 12.1 console with "Unified-Plan" enabled. transceiver-direction-inactive-is-ignored.js text/javascript 2690 ibc bGV0IG1pY1RyYWNrOwpsZXQgd2ViY2FtVHJhY2s7CmxldCBwYzE7CmxldCBwYzI7CmxldCBtaWNU cmFuc2NlaXZlcjsKCm5hdmlnYXRvci5tZWRpYURldmljZXMuZ2V0VXNlck1lZGlhKHsgYXVkaW86 IHRydWUgfSkKCS50aGVuKChzdHJlYW0pID0+Cgl7CgkJbWljVHJhY2sgPSBzdHJlYW0uZ2V0QXVk aW9UcmFja3MoKVswXTsKCX0pCgkudGhlbigoKSA9PgoJewoJCXBjMSA9IG5ldyBSVENQZWVyQ29u bmVjdGlvbigKCQkJewoJCQkJaWNlU2VydmVycyAgICA6IFtdLAoJCQkJYnVuZGxlUG9saWN5ICA6 ICdtYXgtYnVuZGxlJywKCQkJCXJ0Y3BNdXhQb2xpY3kgOiAncmVxdWlyZScsCgkJCQlzZHBTZW1h bnRpY3MgIDogJ3VuaWZpZWQtcGxhbicKCQkJfSk7CgoJCXBjMiA9IG5ldyBSVENQZWVyQ29ubmVj dGlvbigKCQkJewoJCQkJaWNlU2VydmVycyAgICA6IFtdLAoJCQkJYnVuZGxlUG9saWN5ICA6ICdt YXgtYnVuZGxlJywKCQkJCXJ0Y3BNdXhQb2xpY3kgOiAncmVxdWlyZScsCgkJCQlzZHBTZW1hbnRp Y3MgIDogJ3VuaWZpZWQtcGxhbicKCQkJfSk7CgoJCWNvbnNvbGUud2FybigncGMxOiBhZGRpbmcg bWljIHRyYWNrJyk7CgoJCW1pY1RyYW5zY2VpdmVyID0KCQkJcGMxLmFkZFRyYW5zY2VpdmVyKG1p Y1RyYWNrLCB7IGRpcmVjdGlvbjogJ3NlbmRvbmx5JyB9KTsKCX0pCgkudGhlbigoKSA9PiBuZWdv dGlhdGVTZHBzKCkpCgkudGhlbigoKSA9PgoJewoJCWNvbnNvbGUud2FybigncGMxOiByZW1vdmlu ZyBtaWMgdHJhY2sgdXNpbmcgcmVwbGFjZVRyYWNrKG51bGwpJyk7CgoJCS8vIE5PVEU6IFRoaXMg ZG9lcyBub3Qgd29yayBoZXJlLiBhPWRpcmVjdGlvbiByZW1haW5zICJzZW5kb25seSIgaW4KCQkv LyB0aGUgbmV3IG9mZmVyLgoJCW1pY1RyYW5zY2VpdmVyLmRpcmVjdGlvbiA9ICdpbmFjdGl2ZSc7 CgoJCXJldHVybiBtaWNUcmFuc2NlaXZlci5zZW5kZXIucmVwbGFjZVRyYWNrKG51bGwpOwoJfSkK CS50aGVuKCgpID0+Cgl7CgkJLy8gTk9URTogVGhpcyBkb2VzIG5vdCB3b3JrIG5laXRoZXIgaGVy ZS4gYT1kaXJlY3Rpb24gcmVtYWlucyAic2VuZG9ubHkiIGluCgkJLy8gdGhlIG5ldyBvZmZlci4K CQltaWNUcmFuc2NlaXZlci5kaXJlY3Rpb24gPSAnaW5hY3RpdmUnOwoJfSkKCS50aGVuKCgpID0+ IG5lZ290aWF0ZVNkcHMoKSkKCS5jYXRjaCgoZXJyb3IpID0+Cgl7CgkJY29uc29sZS5lcnJvcihl cnJvcik7Cgl9KTsKCmZ1bmN0aW9uIG5lZ290aWF0ZVNkcHMoKQp7CglsZXQgZXJyb3JlZCA9IGZh bHNlOwoKCXJldHVybiBQcm9taXNlLnJlc29sdmUoKQoJCS50aGVuKCgpID0+IHBjMS5jcmVhdGVP ZmZlcigpKQoJCS5jYXRjaCgoZXJyb3IpID0+CgkJewoJCQljb25zb2xlLmVycm9yKCdwYzEuY3Jl YXRlT2ZmZXIoKSBmYWlsZWQnKTsKCgkJCWVycm9yZWQgPSB0cnVlOwoJCQl0aHJvdyBlcnJvcjsK CQl9KQoJCS50aGVuKChvZmZlcikgPT4KCQl7CgkJCWNvbnNvbGUud2FybigncGMxLmNyZWF0ZU9m ZmVyKCkgc3VjY2VlZGVkOlxuJXMnLCBvZmZlci5zZHApOwoKCQkJcmV0dXJuIHBjMS5zZXRMb2Nh bERlc2NyaXB0aW9uKG9mZmVyKTsKCQl9KQoJCS5jYXRjaCgoZXJyb3IpID0+CgkJewoJCQlpZiAo ZXJyb3JlZCkKCQkJCXRocm93IGVycm9yOwoKCQkJY29uc29sZS5lcnJvcigncGMxLnNldExvY2Fs RGVzY3JpcHRpb24oKSBmYWlsZWQnKTsKCgkJCWVycm9yZWQgPSB0cnVlOwoJCQl0aHJvdyBlcnJv cjsKCQl9KQoJCS50aGVuKCgpID0+CgkJewoJCQljb25zdCBvZmZlciA9IHBjMS5sb2NhbERlc2Ny aXB0aW9uOwoKCQkJcmV0dXJuIHBjMi5zZXRSZW1vdGVEZXNjcmlwdGlvbihvZmZlcik7CgkJfSkK CQkuY2F0Y2goKGVycm9yKSA9PgoJCXsKCQkJaWYgKGVycm9yZWQpCgkJCQl0aHJvdyBlcnJvcjsK CgkJCWNvbnNvbGUuZXJyb3IoJ3BjMi5zZXRSZW1vdGVEZXNjcmlwdGlvbigpIGZhaWxlZCcpOwoK CQkJZXJyb3JlZCA9IHRydWU7CgkJCXRocm93IGVycm9yOwoJCX0pCgkJLnRoZW4oKCkgPT4gcGMy LmNyZWF0ZUFuc3dlcigpKQoJCS5jYXRjaCgoZXJyb3IpID0+CgkJewoJCQlpZiAoZXJyb3JlZCkK CQkJCXRocm93IGVycm9yOwoKCQkJY29uc29sZS5lcnJvcigncGMyLmNyZWF0ZUFuc3dlcigpIGZh aWxlZCcpOwoKCQkJZXJyb3JlZCA9IHRydWU7CgkJCXRocm93IGVycm9yOwoJCX0pCgkJLnRoZW4o KGFuc3dlcikgPT4gcGMyLnNldExvY2FsRGVzY3JpcHRpb24oYW5zd2VyKSkKCQkuY2F0Y2goKGVy cm9yKSA9PgoJCXsKCQkJaWYgKGVycm9yZWQpCgkJCQl0aHJvdyBlcnJvcjsKCgkJCWNvbnNvbGUu ZXJyb3IoJ3BjMi5zZXRMb2NhbERlc2NyaXB0aW9uKCkgZmFpbGVkJyk7CgoJCQllcnJvcmVkID0g dHJ1ZTsKCQkJdGhyb3cgZXJyb3I7CgkJfSkKCQkudGhlbigoKSA9PgoJCXsKCQkJY29uc3QgYW5z d2VyID0gcGMyLmxvY2FsRGVzY3JpcHRpb247CgoJCQlyZXR1cm4gcGMxLnNldFJlbW90ZURlc2Ny aXB0aW9uKGFuc3dlcik7CgkJfSkKCQkudGhlbigoKSA9PgoJCXsKCQkJY29uc29sZS53YXJuKCdw YzEuc2V0UmVtb3RlRGVzY3JpcHRpb24oKSBzdWNjZWVkZWQnKTsKCQl9KQoJCS5jYXRjaCgoZXJy b3IpID0+CgkJewoJCQlpZiAoZXJyb3JlZCkKCQkJCXRocm93IGVycm9yOwoKCQkJY29uc29sZS5l cnJvcigncGMyLnNldFJlbW90ZURlc2NyaXB0aW9uKCkgZmFpbGVkJyk7CgoJCQl0aHJvdyBlcnJv cjsKCQl9KTsKfQo=