Skip to content
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

If the application is switched to the background and then to the foreground, video streaming timeout occurs. #1471

Closed
zhouxin627 opened this issue Nov 14, 2019 · 7 comments
Assignees
Labels
bug A defect in the library manager-streaming-video Relating to the manager layer - video streaming transport-secondary Relating to the secondary transport
Projects

Comments

@zhouxin627
Copy link
Contributor

Bug Report

If the application is switched to the background and then to the foreground,
video streaming timeout occurs.

Preconditions

1.Primary transport: BT
Secondary transport: WIFI
※If it connected via USB, there is no problem.

Reproduction Steps
  1. Register App:
    ・App: navigation App for iOS
  2. Activate App
  3. Switch App to the background on iPhone
  4. Switch App to the foreground on iPhone
Expected Behavior
  1. SDL->App.OnHMIStatus(NONE->FULL,STREAMABLE)
  2. App starts video streaming
  3. Switch App to the background
  4. SDL<-App.OnHMIStatus(BACKGROUND)
  5. SDL<-App.OnServiceEndedCallback(11)
  6. App.resetStatus()
video protocol = nil;
videoStreamState = NotStreamable;
  1. Switch App to the foreground
  2. App.didEnterStateVideoStreamReady()
  3. App.sdl_startTCPSecondaryTransport()
  4. SDL<-App.OnHMIStatus(FULL)
  5. App restarts video streaming
Observed Behavior

There is no video data because the step8(App.didEnterStateVideoStreamReady)
is not called.

OS & Version Information
  • iOS Version: 10.2
  • SDL iOS Version: 6.3.0
  • Testing Against: SDL Core 5.1.3 + HMI running on our development environment.
Test Case, Sample Code, and / or Example App

Logs:Log.zip

Occurrence

100%

Recovery Method

None

@joeljfischer joeljfischer added the bug A defect in the library label Nov 14, 2019
@E-SAITO-TMC
Copy link

This is one of the issues that Toyota reported at the 11/18 Roadmap workshop.

@NicoleYarroch NicoleYarroch self-assigned this Dec 2, 2019
@NicoleYarroch
Copy link
Contributor

@E-SAITO-TMC is the Toyota team working on a fix for this issue?

@NicoleYarroch
Copy link
Contributor

@stefanek22 can you let us know if Ford is experiencing this issue as well?

@theresalech
Copy link
Contributor

Toyota team confirmed via email that they will be submitting a PR to resolve this issue by 2020-01-24.

@joeljfischer joeljfischer added manager-streaming-video Relating to the manager layer - video streaming transport-secondary Relating to the secondary transport labels Jan 28, 2020
@joeljfischer joeljfischer added this to To do in v6.6.0 Mar 18, 2020
@NicoleYarroch
Copy link
Contributor

@zhouxin627 I believe this issue was fixed in PR #1566. I know you submitted a PR (#1528) to fix the issue but I had to do an extensive refactor the secondary transport manager to fix some bugs. Can you test with PR #1566 to see if this bug has been fixed? Thank you.

@joeljfischer
Copy link
Contributor

joeljfischer commented Apr 8, 2020

Hello @zhouxin627, could you please take a look at Nicole's comment above and check this bug? We believe it has been fixed.

@zhouxin627
Copy link
Contributor Author

Hi @NicoleYarroch @joeljfischer, I'm sorry for the late reply. We have confirmed that #1471 has been fixed. But #1479 is still being confirmed. I'll let you know the result as soon as possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A defect in the library manager-streaming-video Relating to the manager layer - video streaming transport-secondary Relating to the secondary transport
Projects
No open projects
v6.6.0
  
Done
Development

No branches or pull requests

5 participants