Closed
Bug 1240245
Opened 9 years ago
Closed 9 years ago
Disable data collection for new-tab page
Categories
(Firefox :: New Tab Page, defect)
Firefox
New Tab Page
Tracking
()
RESOLVED
FIXED
Firefox 46
Tracking | Status | |
---|---|---|
firefox46 | --- | fixed |
People
(Reporter: Dolske, Assigned: oyiptong)
References
Details
Attachments
(1 file)
In order to improve the Firefox onboarding experience, we'd like to remove the new-tab "tour" and associated privacy notice, since they interrupt the user's initial experience with Firefox. Looks like this data is currently not critical, and so the simplest route will be to just turn it off.
Assignee | ||
Comment 1•9 years ago
|
||
This would simply involve turning off the 'pings' to our servers in reportSitesAction, effectively making that function a noop.
https://dxr.mozilla.org/mozilla-central/source/browser/modules/DirectoryLinksProvider.jsm#531
The fetch should be kept so that we keep showing directory/default tiles.
Comment 2•9 years ago
|
||
The data from the newtab page is used to measure quality and engagement of the Directory Tiles product (the set of Mozilla related tiles shown to new/fresh profiles).
Without this data, the engagement team will not be able to properly optimize this experience. I worry about the impact to the end user and about losing our ability to properly measure engagement.
Flags: needinfo?(jcollings)
Comment 3•9 years ago
|
||
(In reply to Tim Spurway [:tspurway] from comment #2)
> The data from the newtab page is used to measure quality and engagement of
> the Directory Tiles product (the set of Mozilla related tiles shown to
> new/fresh profiles).
>
> Without this data, the engagement team will not be able to properly optimize
> this experience. I worry about the impact to the end user and about losing
> our ability to properly measure engagement.
Is it possible to count how often the pages are visited since presumably they would be hosted by Mozilla anyways? The links sent down from the directory/default tiles could be non-advertised links that users would only naturally get if they clicked on them from the new-tab page. These would not be unique per user but unique to the new-tab page origin so we could still track engagement.
Comment 4•9 years ago
|
||
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #3)
> Is it possible to count how often the pages are visited since presumably
> they would be hosted by Mozilla anyways?
You can definitely count clicks that way, but you can't count impressions. You need both in order to measure and optimize engagement and performance correctly. Most performance measures are ratios involving an impression count as denominator.
Comment 5•9 years ago
|
||
Hi all,
I manage the Directory Tiles on the +tab page and impressions and clicks data are needed in order for me to better understand what the users engage with. Without this data, I am going in blind and would not be able to make quick and effective marketing decisions.
I spoke with Nick C. about this and he mentioned a possible short term solution of being able to gather impressions the way we do with snippets on about:home (I can get clicks through GA). However, this would be a manual process to gather data and I prefer the original way of retrieving data. Worth exploring for the interim?
Updated•9 years ago
|
Flags: needinfo?(jcollings)
Comment 6•9 years ago
|
||
The other issue with 'turning off data collection' is that FX does an HTTP GET on the Tiles service (Onyx) once per day in order to fetch Directory Tiles. This is also considered data collection as the user's IP address, UA string, Locale, etc. is transmitted to Mozilla servers.
I don't think we can turn off data collection without disabling Directory Tiles entirely.
Comment 7•9 years ago
|
||
(In reply to Tim Spurway [:tspurway] from comment #6)
> The other issue with 'turning off data collection' is that FX does an HTTP
> GET on the Tiles service (Onyx) once per day in order to fetch Directory
> Tiles. This is also considered data collection as the user's IP address, UA
> string, Locale, etc. is transmitted to Mozilla servers.
Those are all transmitted via update requests, so I doubt that has any additional privacy implications, whereas the rest of the data collection might do.
Comment 8•9 years ago
|
||
As part of fixing this bug, the following two files/pages will also need to be updated:
https://dxr.mozilla.org/mozilla-central/source/browser/docs/DirectoryLinksProvider.rst
http://gecko.readthedocs.org/en/latest/browser/browser/DirectoryLinksProvider.html
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → oyiptong
Assignee | ||
Comment 9•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/31631/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/31631/
Attachment #8709989 -
Flags: review?(jaws)
Updated•9 years ago
|
Attachment #8709989 -
Flags: review?(jaws)
Comment 10•9 years ago
|
||
Comment on attachment 8709989 [details]
MozReview Request: Bug 1240245 - Disable data collection for new-tab page r?jaws
https://reviewboard.mozilla.org/r/31631/#review28345
Why is this only disabling it for release builds? I don't see any mention of release vs pre-release in the bug.
Assignee | ||
Comment 11•9 years ago
|
||
This patch is following my question on bug 1234272, comment 28.
It is just as easy to remove the data collection completely.
Assignee | ||
Comment 12•9 years ago
|
||
I take that back. Removing the ping from the code introduces a more invasive change, though delete-only
Assignee | ||
Comment 13•9 years ago
|
||
Comment on attachment 8709989 [details]
MozReview Request: Bug 1240245 - Disable data collection for new-tab page r?jaws
Review request updated; see interdiff: https://reviewboard.mozilla.org/r/31631/diff/1-2/
Attachment #8709989 -
Attachment description: MozReview Request: Bug 1240245 - Disable data collection for new-tab page in release builds r?jaws → MozReview Request: Bug 1240245 - Disable data collection for new-tab page r?jaws
Attachment #8709989 -
Flags: review?(jaws)
Assignee | ||
Comment 14•9 years ago
|
||
Comment on attachment 8709989 [details]
MozReview Request: Bug 1240245 - Disable data collection for new-tab page r?jaws
Review request updated; see interdiff: https://reviewboard.mozilla.org/r/31631/diff/2-3/
Comment 15•9 years ago
|
||
Comment on attachment 8709989 [details]
MozReview Request: Bug 1240245 - Disable data collection for new-tab page r?jaws
https://reviewboard.mozilla.org/r/31631/#review28413
::: browser/modules/DirectoryLinksProvider.jsm:521
(Diff revision 3)
> /**
> * Report some action on a newtab page (view, click)
> * @param sites Array of sites shown on newtab page
> * @param action String of the behavior to report
> * @param triggeringSiteIndex optional Int index of the site triggering action
> * @return download promise
> */
> reportSitesAction: function DirectoryLinksProvider_reportSitesAction(sites, action, triggeringSiteIndex) {
::: browser/modules/DirectoryLinksProvider.jsm:566
(Diff revision 3)
> - // Only send pings when enhancing tiles with an endpoint and valid action
> + // Bug 1240245 - We no longer send pings, but frequency capping and fetching
> + // tests depend on the following actions, so references to PING remain.
> let invalidAction = PING_ACTIONS.indexOf(action) == -1;
Please file a follow up bug to remove frequency capping.
Attachment #8709989 -
Flags: review?(jaws) → review+
Comment 16•9 years ago
|
||
Reporter | ||
Comment 17•9 years ago
|
||
Yay, thanks for the quick fix here!
Comment 18•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
Comment 19•9 years ago
|
||
[bugday-20160323]
Status: RESOLVED,FIXED -> UNVERIFIED
Comments:
STR: Not clear.
Developer specific testing
Component:
Name Firefox
Version 46.0b9
Build ID 20160322075646
Update Channel beta
User Agent Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0
OS Windows 7 SP1 x86_64
Expected Results:
Developer specific testing
Actual Results:
As expected
You need to log in
before you can comment on or make changes to this bug.
Description
•