From 13c8abe3dcec5d9e8558e14c7cb8314aa26625bd Mon Sep 17 00:00:00 2001 From: Sawyer Blatz Date: Tue, 23 Jun 2020 14:05:55 -0700 Subject: [PATCH] For #11613: Expand toolbar onUrlChanged --- .../org/mozilla/fenix/browser/BaseBrowserFragment.kt | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/browser/BaseBrowserFragment.kt b/app/src/main/java/org/mozilla/fenix/browser/BaseBrowserFragment.kt index 4991968da..791971685 100644 --- a/app/src/main/java/org/mozilla/fenix/browser/BaseBrowserFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/browser/BaseBrowserFragment.kt @@ -527,14 +527,7 @@ abstract class BaseBrowserFragment : Fragment(), UserInteractionHandler, Session ) session.register(observer = object : Session.Observer { - override fun onNavigationStateChanged( - session: Session, - canGoBack: Boolean, - canGoForward: Boolean - ) { - // Once https://bugzilla.mozilla.org/show_bug.cgi?id=1626338 is fixed, we can - // rely solely on `onLoadRequest` entirely, but as it stands that is not called - // for history navigation (back or forward). + override fun onUrlChanged(session: Session, url: String) { browserToolbarView.expand() }