For mozilla-mobile#11499 - Update tracking protection icon state

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
ionutbedregeanu · Jan 7, 2022 · e4ff57c · e4ff57c
1 parent fbcb951
commit e4ff57c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/...rowser/toolbar/src/main/java/mozilla/components/browser/toolbar/display/DisplayToolbar.kt b/...rowser/toolbar/src/main/java/mozilla/components/browser/toolbar/display/DisplayToolbar.kt
@@ -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()
     }

diff --git a/docs/changelog.md b/docs/changelog.md
@@ -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
+
 * **feature-prompts**:
   * Removes deprecated constructor in `PromptFeature`.