blallo
/
fenix
1
0
Fork 0
Code Releases Activity

No issue: Register observers with owner view

master
Emily Kager 2019-05-24 10:07:08 -07:00 committed by Sawyer Blatz
parent a676e964b5
commit 4e13de3ffd
3 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/src/main/java/org/mozilla/fenix/HomeActivity.kt
View File

@ -248,7 +248,7 @@ open class HomeActivity : AppCompatActivity() {
override fun onSessionAdded(session: Session) {
super.onSessionAdded(session)
session.register(singleSessionObserver)
session.register(singleSessionObserver, this@HomeActivity)
}
override fun onSessionRemoved(session: Session) {
@ -259,10 +259,10 @@ open class HomeActivity : AppCompatActivity() {
override fun onSessionsRestored() {
super.onSessionsRestored()
components.core.sessionManager.sessions.forEach {
it.register(singleSessionObserver)
it.register(singleSessionObserver, this@HomeActivity)
}
}
}.also { components.core.sessionManager.register(it) }
}.also { components.core.sessionManager.register(it, this) }
}
companion object {

4
app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt
View File

@ -750,7 +750,7 @@ class BrowserFragment : Fragment(), BackHandler, CoroutineScope {
updateBookmarkState(session)
}
}
getSessionById()?.register(observer)
getSessionById()?.register(observer, this)
return observer
}
@ -760,7 +760,7 @@ class BrowserFragment : Fragment(), BackHandler, CoroutineScope {
super.onSessionSelected(session)
(activity as HomeActivity).updateThemeForSession(session)
}
}.also { requireComponents.core.sessionManager.register(it) }
}.also { requireComponents.core.sessionManager.register(it, this) }
}
private fun findBookmarkedURL(session: Session?): Boolean {

2
app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt
View File

@ -540,7 +540,7 @@ class HomeFragment : Fragment(), CoroutineScope, AccountObserver {
emitSessionChanges()
}
}
requireComponents.core.sessionManager.register(observer)
requireComponents.core.sessionManager.register(observer, this)
return observer
}