Skip to content

Commit

Permalink
For mozilla-mobile#11499 - Update tracking protection icon state
Browse files Browse the repository at this point in the history
Update the state of tracking protection indicator even when is not displayed
in order to avoid that scenario when the tracking protection became visible later
but its state is not the last one
  • Loading branch information
Ionut Cristian Bedregeanu committed Jan 7, 2022
1 parent 435306c commit 8cbb400
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -510,10 +510,6 @@ class DisplayToolbar internal constructor(
}

internal fun setTrackingProtectionState(state: Toolbar.SiteTrackingProtection) {
if (!indicators.contains(Indicators.TRACKING_PROTECTION)) {
return
}

views.trackingProtectionIndicator.siteTrackingProtection = state
updateSeparatorVisibility()
}
Expand Down
3 changes: 3 additions & 0 deletions docs/changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,9 @@ permalink: /changelog/
* [Gecko](https://github.com/mozilla-mobile/android-components/blob/main/buildSrc/src/main/java/Gecko.kt)
* [Configuration](https://github.com/mozilla-mobile/android-components/blob/main/.config.yml)

* **browser-toolbar**
* 🚒 Bug fixed [issue #11499](https://github.com/mozilla-mobile/android-components/issues/11499) - Update tracking protection icon state even when is not displayed

* **support-ktx**
* 🚒 Bug fixed [issue #11374](https://github.com/mozilla-mobile/android-components/issues/11374) - Restore immersive mode after interacting with other Windows.
* ⚠️ **This is a breaking change**: `OnWindowFocusChangeListener` parameter is removed from `Activity.enterToImmersiveMode()`. There was no way to guarantee that the argument knew to handle immersive mode. Now everything is handled internally.
Expand Down

0 comments on commit 8cbb400

Please sign in to comment.