Fenix: FNX2-13444 ⁃ [Bug] Inconsistencies with the ETP shield being highlighted

[vesta0]

Steps to reproduce

-On a fresh install, navigate to cnn.com

Expected behavior

-The ETP shield is highlighted and the CFR is shown

Actual behavior

• The ETP shield is not highlighted right away but if you tab on one of the cnn articles then the shield is highligherd and the CFR shows after the 2nd interaction.

Device information

• Android device: Pixel 4
• Fenix version: Latest Firefox Beta/Firefox Preview

[sv-ohorvath]

@vesta0 @Amejia481 I've found this issue on automation tests too but couldn't reproduce manually. I think it's the same issue, isn't it? https://github.com/mozilla-mobile/fenix/issues/11370

[Amejia481]

@vesta0 @Amejia481 I've found this issue on automation tests too but couldn't reproduce manually. I think it's the same issue, isn't it? #11370

@sv-ohorvath thanks for reporting I'm taking a look at it :)

[Amejia481]

Steps to reproduce

-On a fresh install, navigate to cnn.com

Expected behavior

-The ETP shield is highlighted and the CFR is shown

Actual behavior

* The ETP shield is not highlighted right away but if you tab on one of the cnn articles then the shield is highligherd and the CFR shows after the 2nd interaction.

Device information

* Android device: Pixel 4

* Fenix version: Latest Firefox Beta/Firefox Preview

@vesta0 is that always reproducible for you? I'm having a hard time replicating it.

[Amejia481]

Actual behavior

The ETP shield is not highlighted right away but if you tab on one of the cnn articles then the > shield is highligherd and the CFR shows after the 2nd interaction."

After the above happens and you navigate to another site and return to cnn.com, does the shield get highlighted?

[Amejia481]

I can replicate the issue on nightly. So far, what I could find it's that we are not getting a call on
onContentBlocked on the first load of a session, still I'm trying to figure out if it's an ac issue or gv one.

STR:

1. Open a new tab.
2. Go to ccn.com (The shield doesn't get highlighted).
3. If you fresh or try to go to ccn.com on the same tab the shield gets highlighted.

[Amejia481]

I opened a bugzilla ticket for the gv and privacy team to help us to further investigate https://bugzilla.mozilla.org/show_bug.cgi?id=1646392

[Amejia481]

It's been more than a week since the last time that I was able to reproduce. @vesta0 @ekager are you able to reproduce in nightly ?

[Amejia481]

I can replicate the issue on nightly. So far, what I could find it's that we are not getting a call on
onContentBlocked on the first load of a session, still I'm trying to figure out if it's an ac issue or gv one.

STR:

1. Open a new tab.

2. Go to ccn.com (The shield doesn't get highlighted).

3. If you fresh or try to go to ccn.com on the same tab the shield gets highlighted.

QA please help us to verify if this is still reproducible, thanks in advance.

[Amejia481]

It's been more than a week since the last time that I was able to reproduce. @vesta0 @ekager are you able to reproduce in nightly ?

@sv-ohorvath the test is still flaky?

[sv-ohorvath]

It's been more than a week since the last time that I was able to reproduce. @vesta0 @ekager are you able to reproduce in nightly ?

@sv-ohorvath the test is still flaky?

We haven't seen it fail recently, but it wasn't too frequent before either, so I can't really tell. I'll try to run it a few times tomorrow.

[softvision-miralobontiu]

I was able to reproduce this issue on the latest Beta build 79.0.0-beta.1 (#21820831) with HTC 10 (Android 8), and Samsung Galaxy Tab S6 (Android 9).
Maybe the logcat might help.
logcat.txt

[Amejia481]

I was able to reproduce this issue on the latest Beta build 79.0.0-beta.1 (#21820831) with HTC 10 (Android 8), and Samsung Galaxy Tab S6 (Android 9).
Maybe the logcat might help.
logcat.txt

@softvision-miralobontiu thanks!

This is the same behaviour that I see. I haven't been able to reproduce on nightly any more. I'm not sure what should be we do here @vesta0 @ekager?

[st3fan]

I added a release blocker label on this one because it sounds serious, but I think we could decide to ship without a fix.

@ekager @Amejia481 If we do not see this on Nightly, is it possible that a fix was not included in the 79 branch? Are we aware of any issues that were worked on related to this?

[softvision-miralobontiu]

I was able to reproduce this issue on the 79.0.0-beta.3 build with Samsung Galaxy Tab S6 (Android 9).
I'll remove the qa:needed label.

[ekager]

I can still repro on geckoNightlyDebug geckoBetaDebug variants locally and on my Nightly build.
Make sure you don't have any add-ons enabled while testing this because I noticed ublock can eat some of the trackers that we would report (not sure if that's expected?).

[Amejia481]

I can still repro on geckoNightlyDebug geckoBetaDebug variants locally and on my Nightly build.
Make sure you don't have any add-ons enabled while testing this because I noticed ublock can eat some of the trackers that we would report (not sure if that's expected?).

Thanks for the info, I was able to reproduce the issue in nightly, we are continue trying to find the source of the issue .

[Amejia481]

A patch has landed on GV , we are waiting for the new gv version get updated on ac and the subsequent version of ac that has the fixed land of Fenix

[Amejia481]

This should be fixed on nightly

[abodea]

Verified as fixed on the latest Nightly 7/13 with Google Pixel 4 XL (11).
Note that the ETP shield is being highlighted, I had more than 5 tries and it worked correctly.