search

MobileChromeApps / mobile-chrome-apps

Chrome apps on Android and iOS
BSD 3-Clause "New" or "Revised" License
2.5k stars 347 forks source link

IDB wiped after v0.6.0 introduced Crosswalk update #554

Open felixhammerl opened 9 years ago

felixhammerl commented 9 years ago

Hi guys,

apparently, 0.6.0 brought an update to a newer version of the Crosswalk webview, which wiped the existing IndexedDB our app uses. Is there any way to avoid this going forward? It kind of sucks to wipe our user's app databases because we receive a new version of the webview.

Cheers Felix

agrieve commented 9 years ago

Ouch! That does suck! :(

@huningxin - do you know if this is a known issue with Crosswalk?

tanx commented 9 years ago

This is especially bad because we store the user's private PGP key in IDB. This bugs basically leads to complete data loss if users don't have the key on another device :(

agrieve commented 9 years ago

@fujunwei - can you help with this?

fujunwei commented 9 years ago

I'd glad to do this. I am trying to reproduce this issue.

fujunwei commented 9 years ago

I use cordova-plugin-indexedDB plugin to create databases and add new record. it works finely. It would be helpful if you could describe test steps or simplify your usage scenario to reproduce the issue. Thanks.

tanx commented 9 years ago

Well we shiped the previous version of our using cca v0.5.1 which had an older version of the crosswalk webview if I understand correctly. Then we shiped the current version of our app using cca v0.6.0 which updated crosswalk. The IDB was empty for users that upgraded.

Is that enough info? Thanks

fujunwei commented 9 years ago

Do you mean users installed the apk which generated by cca v0.5.1, data can be stored int the IDB, but after users update the apk which generated by cca v0.6.0. the IDB was wiped?

felixhammerl commented 9 years ago

yes, exactly. we had a binary built with v0.5.1 in the appstore which allowed people to use idb. i assume the crosswalk update coming with v0.6.0 overwrote the pre-existing idb.

agrieve commented 9 years ago

For reference - v0.5.1 used Crosswalk v8 (Chrome 37), and v0.6.0 uses Crosswalk v11 (Chrome 40). Neither used cordova-plugin-indexedDB, but instead just used the built-in IndexedDB.

fujunwei commented 9 years ago

It can be reproduced after upgrading app from Crosswalk 8 to Crosswalk 11, but it works finely from Crosswalk 9 to Crosswalk 11 and the same version upgrade.

fujunwei commented 9 years ago

We had approach to fix it, and created a JIRA ticket to keep track of this. https://crosswalk-project.org/jira/browse/XWALK-4047

tanx commented 9 years ago

Great thx for looking into this

fujunwei commented 9 years ago

The IndexedDB wasn't wiped but the upgraded Crosswalk didn't used it. You can find it in /data/data/org.example.testing/app_xwalkcore/IndexedDB. You can copy it manually under profile's subfolder /data/data/org.example.testing/app_xwalkcore/Default/IndexedDB, the name of the profile is "Default" by default.

fujunwei commented 9 years ago

@agrieve Because Crosswalk 11 is unmaintained, Could you please upgrade crosswalk in cca?