1925088 - [WebCompat Reporter] Call the GeckoView API and implement the Glean Metrics for submitting the report

Status: RESOLVED FIXED

(Fenix :: General, task, P1)

Description

[Noah Bond [:007]]

Desktop code

https://searchfox.org/mozilla-central/rev/3b59c739df66574d94022a684596845cd05e7c65/browser/components/reportbrokensite/ReportBrokenSite.sys.mjs#662

https://phabricator.services.mozilla.com/D190649#change-05PBUDHHF45x

Comment 1

[Noah Bond [:007]]

Adding Glean dictionary link

Comment 2

[Noah Bond [:007]]

This ticket should wait until Bug 1921885 exposes the Gecko data we need, so we only report the data we need/have access to.

Comment 3

[Nicholas Poon [:Nick]]

Updating the tasks for this ticket:

• This ticket will now include tasks from Bug 1925099. We will call getWebCompatInfo from GeckoView, put the retrieved information into the corresponding ping metric, and submit the ping.
• We are awaiting input from UX about error states - what happens if there is no internet connection for example?
• Since there is a URL field retrieved from the GV call and also a URL field that the user can change, we want to ignore the GV URL field and use the URL field submitted from the form. The form's URL field gets prepopulated but the user could have opened the reporter with the wrong tab (or had nagivated away from the page since). It's also possible that they want to strip out sensitive bits of the URL.
• Some additional behaviour is that if the user changes the URL from the original URL in any way, we don't want to submit the tab specific data associated with the GV call. (Note that this logic could be changed if we come up with a better approach since it's possible that the user is just stripping out sensitive bits of the URL and not reporting a "different" URL)

Comment 4

[Nicholas Poon [:Nick]]

Attachment: Bug 1925088 - Part 3: Call the GeckoView API and implement the behaviour for submitting the WebCompat report

Comment 5

[Nicholas Poon [:Nick]]

Attachment: Bug 1925088 - Part 1: Refactor the WebCompat store and middlewares into the same package

Comment 6

[Nicholas Poon [:Nick]]

Attachment: Bug 1925088 - Part 2: Implement the highly sensitive Glean Metrics for submitting the WebCompat report

Comment 7

[Nicholas Poon [:Nick]]

Attachment: Webcompat Reporter Desktop - Data Collection Approval.pdf

Comment 8

[Nicholas Poon [:Nick]]

Attachment: WebCompat Reporter - Request for data collection review form.txt

Comment 9

[Nicholas Poon [:Nick]]

Note that all of the metrics that I am adding to fenix have already been approved for desktop so I have attached the desktop approval form as well

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6687b49ed72f
Part 1: Refactor the WebCompat store and middlewares into the same package r=android-reviewers,007

Comment 11

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/6687b49ed72f

Comment 12

[Roger Yang [:royang]]

Comment on attachment 9443858 [details]
WebCompat Reporter - Request for data collection review form.txt

Remove the need to do data review here. The sensitive data has been reviewed in Phabricator
https://phabricator.services.mozilla.com/D232122#8037249

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8024b5b1106b
Part 2: Implement the highly sensitive Glean Metrics for submitting the WebCompat report r=android-reviewers,data-stewards,TravisLong,tthibaud
https://hg.mozilla.org/integration/autoland/rev/12a1480b1a41
Part 3: Call the GeckoView API and implement the behaviour for submitting the WebCompat report r=android-reviewers,twisniewski,rsainani

Comment 14

[agoloman]

https://hg.mozilla.org/mozilla-central/rev/8024b5b1106b
https://hg.mozilla.org/mozilla-central/rev/12a1480b1a41