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

Memory leak fixed #118

Merged
merged 1 commit into from
May 24, 2021
Merged

Memory leak fixed #118

merged 1 commit into from
May 24, 2021

Conversation

tomasstrba
Copy link
Contributor

Task/Issue URL: https://app.asana.com/0/1177771139624306/1200300006780048/f
CC:

Description:
TabViewModel released immediately from TabCollectionViewModel after its tab is closed. Prevents video/audio from playing after the closing.

Steps to test this PR:

  1. Open empty tab + a new tab with youtube.com and start a video
  2. Close the youtube tab and make sure no audio is playing

Internal references:

Software Engineering Expectations
Technical Design Template
When ready for review, remember to post the PR in MM

Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

(bitrise is having problems - I think we need to sort out the test that regularly breaks AppStateChangePublisherTests.swift:103 )

@brindy brindy self-assigned this May 20, 2021
@tomasstrba tomasstrba merged commit 30843e9 into develop May 24, 2021
@tomasstrba tomasstrba deleted the tom/memory-leak branch May 24, 2021 15:01
samsymons added a commit that referenced this pull request May 26, 2021
# By Tomas Strba (4) and others
# Via GitHub
* develop:
  Update the Chrome user agent (#122)
  TabViewModel released immediately from TabCollectionViewModel after its tab is closed (#118)
  User agent updated (#117)
  History Storage - Please do not merge (#108)
  use the temporary unprotected sites list and update the frequency to 30 minutes (#114)
  update email autofill (#113)
  Version 0.6.16
  Force the content blocking rules publisher onto the main thread. (#112)
  Update to BrowserServicesKit 1.1.1 (#111)
  Zoom menu items; Menu Items validation (#105)
  check file headers programatically (#110)
  Fixed up filenames in file headers (#109)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
#	DuckDuckGo/Bookmarks/Services/Bookmark.xcdatamodeld/Bookmark.xcdatamodel/contents
#	Unit Tests/Bookmarks/Services/LocalBookmarkStoreTests.swift
#	Unit Tests/FileSystem/CoreDataEncryptionTests.swift
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