We should also include telemetry on dismissing the

...dismissing the onboarding start pane.

I was talking to Anthony about onboarding telemetry and suddenly realized that we don't have telemetry for dismissing/canceling the onboarding screen. I'll want to request beta uplift for this, because it'll be really good to have these stats on release, and since the merge will be delayed until after Thanksgiving this should make it...?

Comment on attachment 8523333 [details] [diff] [review] Patch: Add telemetry for canceling/dismissing the start pane >diff --git a/mobile/android/base/StartPane.java b/mobile/android/base/StartPane.java My gut instinct is saying that we don't need to treat these as two different things. In both cases, the onboarding is dismissed by the user, even though it's technically two different ways. If you think we have a reason to track them separately, I could be swayed. If we could get by with a single Extra, I would suggest "firstrun-popup" or "firstrun-pane". Looks like we already use "firstrun-homepage" as the Extra for the Banner telemetry. >+ public void onCancel(DialogInterface dialog) { >+ // StartPane is closed by touching outside the dialog. >+ Telemetry.sendUIEvent(TelemetryContract.Event.ACTION, TelemetryContract.Method.DIALOG, "firstrun-cancel"); I do think this should be TelemetryContract.Event.CANCEL > private void addDismissHandler(View view) { > final GestureDetector gestureDetector = new GestureDetector(getActivity(), new GestureDetector.SimpleOnGestureListener() { > @Override > public boolean onSingleTapUp(MotionEvent e) { >+ Telemetry.sendUIEvent(TelemetryContract.Event.ACTION, TelemetryContract.Method.DIALOG, "firstrun-dismiss"); Same -> TelemetryContract.Event.CANCEL Even though I could be swayed on using two different Extras, I think we do want to use TelemetryContract.Event.CANCEL for the Event.

> > Even though I could be swayed on using two different Extras, I think we do > want to use TelemetryContract.Event.CANCEL for the Event. Yep, sounds good to me. https://hg.mozilla.org/integration/fx-team/rev/7fe78086ac8a

Comment on attachment 8523333 [details] [diff] [review] Patch: Add telemetry for canceling/dismissing the start pane Approval Request Comment [Feature/regressing bug #]: Onboarding telemetry [User impact if declined]: Won't be able to tell how many users exit onboarding without clicking either of our two button options [Describe test coverage new/current, TBPL]: beta try push: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=5f28a683df8a [Risks and why]: very low, just adding telemetry probes [String/UUID change made/needed]: none

Comment on attachment 8523333 [details] [diff] [review] Patch: Add telemetry for canceling/dismissing the start pane Approving for both Beta/Aurora due to low risk and high value of gathering the info.

https://hg.mozilla.org/releases/mozilla-aurora/rev/1a3f9aae6df7 https://hg.mozilla.org/releases/mozilla-beta/rev/3ef1a2619a92

Verified as fixed in builds: - 36.0a1 2014-11-25; - 35.0a2 2014-11-25; - 34 RC1; Device: Asus Transformer Tab (Android 4.2.1).