-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Send ContentLauncher commands from tv-casting-app to tv-app #12913
Send ContentLauncher commands from tv-casting-app to tv-app #12913
Conversation
0d6a8be
to
9da1a3a
Compare
PR #12913: Size comparison from 5d10320 to 9da1a3a Increases (25 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (6 builds for mbed, p6)
Full report (32 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
6603846
to
6cd6e79
Compare
PR #12913: Size comparison from a525e63 to 6cd6e79 Increases (10 builds for k32w, linux, p6, qpg, telink)
Decreases (3 builds for p6)
Full report (12 builds for k32w, linux, p6, qpg, telink)
|
6cd6e79
to
d461885
Compare
PR #12913: Size comparison from a525e63 to d461885 Increases (10 builds for k32w, linux, p6, qpg, telink)
Decreases (3 builds for p6)
Full report (12 builds for k32w, linux, p6, qpg, telink)
|
PR #12913: Size comparison from 89aff25 to 243eec9 Increases (24 builds for efr32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (6 builds for mbed, p6)
Full report (30 builds for efr32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
243eec9
to
bdea82a
Compare
PR #12913: Size comparison from 2ddefed to bdea82a Increases (25 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (12 builds for esp32, mbed, nrfconnect, qpg, telink)
Full report (31 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved, if the documentation for SetConnectedSession is fixed to:
- Give some idea of when this makes sense to do (only if you just allocated the object)
- Say that this function is deprecated and should not be used.
/rebase |
Merged from master and resolved merge conflicts Using InvokeCommand instead of the callback api to LaunchURL Calling GetPeerIdForNode instead of GetPeerID
0d705d7
to
8c9607c
Compare
PR #12913: Size comparison from 569e7cc to 8c9607c Increases (25 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (9 builds for esp32, nrfconnect, telink)
Full report (31 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Problem
Change overview
What's in this PR
Testing
Tested that the tv-casting-app is able to send ContentLauncher.LaunchURL with test data to the tv-app. See tv-casting-app logs below: