Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

[Bug] Clearing an exception for enhanced protection does not reload that page #7929

Closed
s-ankur opened this issue Jan 25, 2020 · 4 comments
Closed
Labels
b:etp Issues caused by enhanced tracking protection 🐞 bug Crashes, Something isn't working, .. Feature:TrackingProtection

Comments

@s-ankur
Copy link
Contributor

s-ankur commented Jan 25, 2020

Steps to reproduce

Add an exception for enhanced protection by disabing the sheild toggle
go to protection setting-> exceptions
clear that exception

Expected behavior

page is reloaded,
both the icon and the protection overlay say that protections are on for the site

Actual behavior

There is no reload but the overlay says that the protections are on
Screenshot_20200125-140336
At the same time, the sheild icon is broken
Screenshot_20200125-140340

Device information

  • Android device: Asus Zenfone m1
  • Fenix version:
    Nightly 200124 06:01 (Build #20240607)
    29.0.0, ae1ba0f02
    GV: 74.0a1-20200122094859

┆Issue is synchronized with this Jira Task

@s-ankur s-ankur added the 🐞 bug Crashes, Something isn't working, .. label Jan 25, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Jan 25, 2020
@kbrosnan kbrosnan added b:etp Issues caused by enhanced tracking protection Feature:TrackingProtection labels Feb 15, 2020
@lobontiumira
Copy link

Hi @s-ankur
I was not able to reproduce this issue on the latest Nightly build from 11/2 with Samsung Galaxy Tab S6 (Android 10).
Could you, please, retest on your device?
Thank you!

@lobontiumira lobontiumira removed the needs:triage Issue needs triage label Nov 2, 2020
@s-ankur
Copy link
Contributor Author

s-ankur commented Nov 2, 2020

@softvision-miralobontiu can you try it on this page https://senglehardt.com/test/trackingprotection/test_pages/fingerprinting.html

@lobontiumira
Copy link

Yes, after deleting the page from exceptions, going back to the tab doesn't reload the page.

@s-ankur
Copy link
Contributor Author

s-ankur commented Nov 27, 2020

Closing as duplicate of #6824

@s-ankur s-ankur closed this as completed Nov 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
b:etp Issues caused by enhanced tracking protection 🐞 bug Crashes, Something isn't working, .. Feature:TrackingProtection
Projects
None yet
Development

No branches or pull requests

3 participants