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

Recreate the lock screen view controller #1483

Closed

Conversation

joeljfischer
Copy link
Contributor

@joeljfischer joeljfischer commented Dec 2, 2019

Fixes #1480

This PR is ready for review.

Risk

This PR makes no API changes.

Testing Plan

  • I have verified that I have not introduced new warnings in this PR (or explain why below)
  • I have run the unit tests with this PR
  • I have tested this PR against Core and verified behavior (if applicable, if not applicable, explain why below).

Unit Tests

No unit tests were added.

Core Tests

  • Displaying and hiding the lock screen verified 10x in a row.
  • Additional verification will be requested from the issue reporter.

Core version / branch / commit hash / module tested against: Sync Gen 3.4 19317_DEVTEST
HMI name / version / branch / commit hash / module tested against: n/a

Summary

This PR changes the SDLScreenshotViewController from being created once when the SDLLockScreenPresenter is initialized to creating it every time the lock screen is going to be created and destroying it when the lock screen dismisses.

Changelog

Bug Fixes
  • Fix the lock window not taking the app's window's characteristics if the app's window changes.

CLA

@joeljfischer
Copy link
Contributor Author

Closed in favor of #1509

@joeljfischer joeljfischer deleted the bugfix/issue-1480-recreate-lockVC-every-time branch January 2, 2020 17:00
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"_lockWindow.rootViewController = _screenshotViewController" is set too early
2 participants