1302809 - Total URI Count Probe Also Counts Non-User initiated URIs

Status: RESOLVED WONTFIX

(Toolkit :: Telemetry, defect, P2)

Description

[John Dorlus [:Silne30]]

Steps to Reproduce:
1.) Open Firefox Nightly with a new profile
2.) Go to second tab and load about:telemetry and look at scalars

Expected result:
Scalars should read:
Name: browser.engagement.max_concurrent_window_count, Value:1
Name: browser.engagement.max_concurrent_tab_count, Value: 2

Actual Result:
Scalars Read:
Name: browser.engagement.total_uri_count, Value: 1
Name: browser.engagement.max_concurrent_window_count, Value:1
Name: browser.engagement.unique_domains_count, Value: 1
Name: browser.engagement.max_concurrent_tab_count, Value: 2

Total URI Count should not be counting this URI for the nightly page.

Comment 1

[Alessio Placitelli [:Dexter]]

Yes, it looks like the welcome page gets counted. We could probably blacklist the URL from the following prefs:

- "startup.homepage_welcome_url"
- "browser.usedOnWindows10.introURL"

I'm not sure if it's worth doing that though, is this would only skew our numbers of 1 or 2 (depending on the platform), since it's happening only on new profiles on the first run.

Brendan, any opinion on this?

Comment 2

[brendan c]

I think we can ignore it if it's just counting the first run page seen by users of a new profile. It's not exactly a user-initiated page visit, but I agree that it seems like a minor case.

Thanks for the thorough QA John-- even though we can live with this one, this is exactly the kind of edge case that normally bites us!

Comment 3

[John Dorlus [:Silne30]]

(In reply to brendan c from comment #2)
> I think we can ignore it if it's just counting the first run page seen by
> users of a new profile. It's not exactly a user-initiated page visit, but I
> agree that it seems like a minor case.
> 
> Thanks for the thorough QA John-- even though we can live with this one,
> this is exactly the kind of edge case that normally bites us!

What a relief. Saves me from having to rewrite 12 test cases.

Comment 4

[John Dorlus [:Silne30]]

Resolving as won't fix.