For #11390 - Removes redundant session observer init for ETP overlay
parent
27c7ef5ad4
commit
11697c1363
|
@ -147,8 +147,7 @@ class BrowserFragment : BaseBrowserFragment(), UserInteractionHandler {
|
||||||
) {
|
) {
|
||||||
browserToolbarView.view
|
browserToolbarView.view
|
||||||
}
|
}
|
||||||
session?.register(toolbarSessionObserver, this, autoPause = true)
|
session?.register(toolbarSessionObserver, viewLifecycleOwner, autoPause = true)
|
||||||
updateEngineBottomMargin()
|
|
||||||
|
|
||||||
if (settings.shouldShowFirstTimePwaFragment) {
|
if (settings.shouldShowFirstTimePwaFragment) {
|
||||||
session?.register(
|
session?.register(
|
||||||
|
@ -174,16 +173,6 @@ class BrowserFragment : BaseBrowserFragment(), UserInteractionHandler {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
private fun updateEngineBottomMargin() {
|
|
||||||
val toolbarSessionObserver = TrackingProtectionOverlay(
|
|
||||||
context = requireContext(),
|
|
||||||
settings = requireContext().settings()
|
|
||||||
) {
|
|
||||||
browserToolbarView.view
|
|
||||||
}
|
|
||||||
getSessionById()?.register(toolbarSessionObserver, this, autoPause = true)
|
|
||||||
}
|
|
||||||
|
|
||||||
override fun onResume() {
|
override fun onResume() {
|
||||||
super.onResume()
|
super.onResume()
|
||||||
getSessionById()?.let {
|
getSessionById()?.let {
|
||||||
|
|
Loading…
Reference in New Issue