Bug 1258472 - 1. Start first-run telemetry session outside of GeckoProfile; r=nalexander

Move the first-run telemetry session from GeckoProfile to BrowserApp, so
there is no longer any dependency on Telemetry from inside GeckoProfile.
This commit is contained in:
Jim Chen 2016-09-14 13:33:09 -04:00
parent f6ee6d53f2
commit dd14f982bb
2 changed files with 6 additions and 5 deletions

View File

@ -933,8 +933,14 @@ public class BrowserApp extends GeckoApp
});
}
}
// Don't bother trying again to show the v1 minimal first run.
prefs.edit().putBoolean(FirstrunAnimationContainer.PREF_FIRSTRUN_ENABLED, false).apply();
// We have no intention of stopping this session. The FIRSTRUN session
// ends when the browsing session/activity has ended. All events
// during firstrun will be tagged as FIRSTRUN.
Telemetry.startUISession(TelemetryContract.Session.FIRSTRUN);
}
} finally {
StrictMode.setThreadPolicy(savedPolicy);

View File

@ -938,11 +938,6 @@ public final class GeckoProfile {
if (!isDefaultSet) {
// only set as default if this is the first profile we're creating
profileSection.setProperty("Default", 1);
// We have no intention of stopping this session. The FIRSTRUN session
// ends when the browsing session/activity has ended. All events
// during firstrun will be tagged as FIRSTRUN.
Telemetry.startUISession(TelemetryContract.Session.FIRSTRUN);
}
parser.addSection(profileSection);