From ae9d06bee557f19af508339be5616c5c8e7d8256 Mon Sep 17 00:00:00 2001 From: Sawyer Blatz Date: Tue, 23 Jun 2020 14:05:55 -0700 Subject: [PATCH] [fenix] For https://github.com/mozilla-mobile/fenix/issues/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 4991968da5..791971685c 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() }