Bug 1651838 Comment 3 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

I think this is just a bug in how we compute the compatibility key for android.

In the current version the key is something like this:

```
[Compatibility]
LastVersion=80.0a1_20200707110539/20200707110539
LastOSABI=Android_x86_64-gcc3
LastPlatformDir=/data/user/0/org.mozilla.geckoview_example
LastAppDir=/data/user/0/org.mozilla.geckoview_example
```

Note the `/data/user/0/org.mozilla.geckoview_example` points to where our Android "dummy" executable is. This only captures when the data directory changes, but not when the executable data changes, we should do something like this

```
[Compatibility]
LastVersion=80.0a1_20200707110539/20200707110539
LastOSABI=Android_x86_64-gcc3
LastPlatformDir=/data/app/org.mozilla.geckoview_example-QyhEDWT7jIwY3rqYDJmCSA==/base.apk
LastAppDir=/data/user/0/org.mozilla.geckoview_example
```

where `LastPlatformDir` captures changes in the executable folder (and would catch this case) and `LastAppDir` catches changes in the data folder, which happens when the app is moved to removable storage (need to double check this last bit)
I think this is just a bug in how we compute the StartupCache's compatibility key for android.

In the current version the key is something like this:

```
[Compatibility]
LastVersion=80.0a1_20200707110539/20200707110539
LastOSABI=Android_x86_64-gcc3
LastPlatformDir=/data/user/0/org.mozilla.geckoview_example
LastAppDir=/data/user/0/org.mozilla.geckoview_example
```

Note the `/data/user/0/org.mozilla.geckoview_example` points to where our Android "dummy" executable is. This only captures when the data directory changes, but not when the executable data changes, we should do something like this

```
[Compatibility]
LastVersion=80.0a1_20200707110539/20200707110539
LastOSABI=Android_x86_64-gcc3
LastPlatformDir=/data/app/org.mozilla.geckoview_example-QyhEDWT7jIwY3rqYDJmCSA==/base.apk
LastAppDir=/data/user/0/org.mozilla.geckoview_example
```

where `LastPlatformDir` captures changes in the executable folder (and would catch this case) and `LastAppDir` catches changes in the data folder, which happens when the app is moved to removable storage (need to double check this last bit)

Back to Bug 1651838 Comment 3