Pass (at least) Session to Session.Observer methods

[pocmo]

Our Session.Observer interfaces currently looks like this:

    interface Observer {
        fun onUrlChanged() = Unit
        fun onProgress() = Unit
        fun onLoadingStateChanged() = Unit
        fun onNavigationStateChanged() = Unit
        fun onSearch() = Unit
        fun onSecurityChanged() = Unit
        fun onCustomTabConfigChanged() = Unit
    }

It would be helpful if we at least pass the Session object to those methods. Otherwise it's sometimes cumbersome to get the Session instance from inside the observer.

┆Issue is synchronized with this Jira Task

[pocmo]

I wonder if we should pass the updated value to the methods too?

E.g.:

fun onUrlChanged(session: Session)

// vs.

fun onUrlChanged(session: Session, url: String)

Of course I could just do session.url. However in a multi-threaded environment this value could already be a different one.