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

Replace deprecated tab preview renderer #2814

Merged
merged 1 commit into from
May 3, 2024

Conversation

samsymons
Copy link
Contributor

@samsymons samsymons commented May 3, 2024

Task/Issue URL: https://app.asana.com/0/414235014887631/1207228085347616/f
Tech Design URL:
CC:

Description:

This PR replaces usage of UIGraphicsBeginImageContextWithOptions with UIGraphicsImageRenderer.

UIGraphicsBeginImageContextWithOptions was deprecated in iOS 17, and is reported to be unstable. We're seeing crashes with it ourselves, and the assertion suggests migrating to the new API, so here we are.

Steps to test this PR:

  1. Stress test tab previews - create a bunch of tabs, check that the tab list renders previews as expected in both modes, close a bunch of tabs, etc.

Orientation Testing:

  • Portrait
  • Landscape

Device Testing:

  • iPhone SE (1st Gen)
  • iPhone 8
  • iPhone X
  • iPhone 14 Pro
  • iPad

OS Testing:

  • iOS 14
  • iOS 15
  • iOS 16

Theme Testing:

  • Light theme
  • Dark theme

Internal references:

Software Engineering Expectations
Technical Design Template

…nderer.

The former is deprecated, and is known to cause issues on iOS 17.
@samsymons samsymons requested review from a team, mallexxx and Bunn and removed request for a team and mallexxx May 3, 2024 02:19
@samsymons samsymons merged commit e13bc13 into main May 3, 2024
40 of 42 checks passed
@samsymons samsymons deleted the sam/replace-deprecated-preview-renderer branch May 3, 2024 13:21
samsymons added a commit that referenced this pull request May 6, 2024
* main:
  Enable field validation for Sync payloads (#2807)
  Fix up address bar spoofing tests to match new behaviours. (#2743)
  Release 7.118.0-4 (#2821)
  Settings: onAppear changed to onDisappear (#2817) (#2820)
  Manage Expired Subscription (#2764)
  Replace deprecated tab preview renderer (#2814)
  Release 7.118.0-3 (#2819)
  Update release notes.
  Settings: onAppear changed to onDisappear (#2817)
  Release 7.118.0-2 (#2815)
  Improve VPN underlying error detail (#2813)
  Break the Subscription module’s dependency on BSK (#2804)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants