988635 - Telemetry: Device orientation

Status: RESOLVED INCOMPLETE

(Firefox for Android Graveyard :: General, defect, P2)

Description

[Chenxia Liu [:liuche]]

Pretty self-explanatory: portrait or landscape.

This would especially be interesting to know especially for tablets (both large and small).

Comment 1

[Sola Ogunsakin [:sola]]

Attachment: bug-988635.patch

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 8400400 [details] [diff] [review]
bug-988635.patch

>diff --git a/mobile/android/base/GeckoScreenOrientation.java b/mobile/android/base/GeckoScreenOrientation.java

>+    public String telemetrySessionForCurrentOrientation() {

How about screenOrientationToTelemetryOrientation ? It kinda matches the other code already in the file.
You'd then pass mScreenOrientation to the method.

Comment 3

[Sola Ogunsakin [:sola]]

(In reply to Mark Finkle (:mfinkle) from comment #2)
> How about screenOrientationToTelemetryOrientation ? It kinda matches the
> other code already in the file.
> You'd then pass mScreenOrientation to the method.

sounds good!

Comment 4

[Sola Ogunsakin [:sola]]

Attachment: bug-988635.patch

-create screenOrientationToTelemetryOrientation(ScreenOrientation)

Comment 5

[Chenxia Liu [:liuche]]

Comment on attachment 8400898 [details] [diff] [review]
bug-988635.patch

Review of attachment 8400898 [details] [diff] [review]:
-----------------------------------------------------------------

It looks like this uses the GeckoApp inheritance to handle configuration changes (see GeckoApp.onConfigurationChanged). This means that anything that doesn't subclass GeckoApp won't get these tracked (e.g. Settings). It's possible that we don't care about the screen orientation in Settings though, but in that case, we need to explicitly close the session because this will be an "unmanaged" session.

::: mobile/android/base/TelemetryContract.java
@@ +33,5 @@
>          // Started when a user enters a given home panel.
>          // Session name is dynamic, encoded as "homepanel.1:<panel_id>"
>          public static final String HOME_PANEL = "homepanel.1:";
> +
> +        // Started when the device enters portrait mode.

Comment style, see comments on previous bugs.

Comment 6

[Chenxia Liu [:liuche]]

Adding a dependency on the session-closing bug, because that will allow us to track application-level sessions, instead of just GeckoApp level sessions (and omitting Settings).

Comment 7

[Chris Peterson [:cpeterson]]

Downgrading priority from P1 to P2

Comment 8

[BMO Automation]

We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.