1258472 - [geckoview] Move Fennec-specific GeckoProfile initialization out of GeckoView

Status: RESOLVED FIXED

(Core Graveyard :: Embedding: APIs, defect)

Description

[Nick Alexander :nalexander [he/him]]

After Bug 1258450, we have:


nalexander@chocho ~/M/gecko> java -cp mobile/android/build/classycle/classycle-1.4.1.jar classycle.dependency.DependencyChecker -mergeInnerClasses -dependencies=@mobile/android/base/geckoview.ddf objdir-droid/gradle/build/mobile/android/app/intermediates/packaged/local/debug/classes.jar
show onlyShortestPaths allResults
Set [lib] has 133 classes.
Set [app] has 715 classes.
check [lib] independentOf [app]
  Unexpected dependencies found:
  org.mozilla.gecko.GeckoProfile
    -> org.mozilla.gecko.GeckoApp
    -> org.mozilla.gecko.db.BrowserDB
    -> org.mozilla.gecko.distribution.Distribution
    -> org.mozilla.gecko.GuestSession
    -> org.mozilla.gecko.db.StubBrowserDB
    -> org.mozilla.gecko.TelemetryContract
    -> org.mozilla.gecko.Telemetry
    -> org.mozilla.gecko.db.LocalBrowserDB
    -> org.mozilla.gecko.preferences.DistroSharedPrefsImport
<snip>

There are two things Fennec-specific here.  One is that the GeckoProfile yields access to the BrowserDB factory and instances.  I think this can just be turned around: Rather than GeckoProfile.get().getDB(), call BrowserDB.getDB(GeckoProfile.get()).

The latter is that a bunch of Fennec-specific initialization with Telemetry and Distributions is tied into GeckoProfile creation.  I suggest we just make this explicit by having GeckoProfile hand itself to the ambient GeckoInterface, which can then complete whatever initialization it wants to do.

If we play this correctly, we may be able to get rid of https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/db/StubBrowserDB.java entirely.  This is because the GeckoView package won't reference Fennec DBs at all anymore!

Comment 1

[Benjamin Smedberg]

Mass change of bugs in the "Embedding: GRE Core" component in preparation for archiving it. I believe that these are no longer relevant; but if they are, they should be reopened and moved into a component relevant to the code in question.

Comment 2

[Myk Melez [:myk] [@mykmelez]]

Reopening and moving to Core::Embedding: APIs, where we'll continue to triage and track GeckoView issues.

Comment 3

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: 1. Start first-run telemetry session outside of GeckoProfile (v1)

Move the first-run telemetry session from GeckoProfile to BrowserApp, so
there is no longer any dependency on Telemetry from inside GeckoProfile.

Comment 4

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: 2. Move guest mode state management to GeckoProfile (v1)

Because guest mode is intimately tied to the profile, it'd be hard to
keep guest mode out of geckoview code entirely, But we also don't want
any dependency on GuestSession from geckoview code, so this patch moves
the part of GuestSession that manages guest mode state to GeckoProfile.

Comment 5

[Nick Alexander :nalexander [he/him]]

Comment on attachment 8789839 [details] [diff] [review]
1. Start first-run telemetry session outside of GeckoProfile (v1)

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

Much better.

Comment 6

[Nick Alexander :nalexander [he/him]]

Comment on attachment 8789840 [details] [diff] [review]
2. Move guest mode state management to GeckoProfile (v1)

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

Looks good.

::: mobile/android/geckoview/src/main/java/org/mozilla/gecko/GeckoProfile.java
@@ +98,5 @@
>      private File mProfileDir;
>  
>      private Boolean mInGuestMode;
>  
> +    public static boolean shouldUseGuestMode(final Context context) {

I feel like these should be synchronized, but this whole approach could theoretically race since prefs are not atomic, so whatever :)

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/77797e21fa19
1. Start first-run telemetry session outside of GeckoProfile; r=nalexander
https://hg.mozilla.org/integration/mozilla-inbound/rev/5d2e0c859f9e
2. Move guest mode state management to GeckoProfile; r=nalexander

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/77797e21fa19
https://hg.mozilla.org/mozilla-central/rev/5d2e0c859f9e