839208 - Test indexed-db on Fennec

Status: RESOLVED WORKSFORME

(Add-on SDK Graveyard :: General, defect)

Description

[Will Bamberg [:wbamberg]]

We should test indexed-db on Fennec.

Comment 1

[Matteo Ferretti [:zer0] [:matteo]]

Just a note: if `index-db` is not working on Fennec for any reason, until the proper support we need to add the module compatibility as follow:

module.metadata = {
  "stability": "experimental",
  "engines": {
    "Firefox": "*"
  }
};

Also, if this module is not potentially working on Thunderbird or we don't want to check it, we should limit the compatibility to Firefox and Fennec:

module.metadata = {
  "stability": "experimental",
  "engines": {
    "Firefox": "*",
    "Fennec": "*"
  }
};

Basically `"engines"` can be omitted only when there are no specific platform issues.

See: https://github.com/mozilla/addon-sdk/wiki/JEP-Metadata for further details.

Comment 2

[Gregg Lind (Fx Strategy and Insights - Shield - Heartbeat )]

I am having tons of trouble getting my phone to work with `adb`, so if someone else can take this.... 

   cfx test -f indexed-db -a fennec-on-device --mobile-app fennec --force-mobile -b /Users/glind/Downloads/android-sdk-macosx/platform-tools/adb

that would be helpful.

Comment 3

[Gregg Lind (Fx Strategy and Insights - Shield - Heartbeat )]

Does cfx test still work on mobile?  Mobile nightly?  I see it start up, but get no useful output.

Some output:

Launching mobile application with intent name org.mozilla.fennec
Pushing the addon to your device
Starting: Intent { act=android.activity.MAIN cmp=org.mozilla.fennec/.App (has extras) }
info: addon-sdk: [JavaScript Error: "[Exception... "'No matching logins' when calling method: [nsILoginManagerStorage::removeLogin]"  nsresult: "0x8057001e (NS_ERROR_XPC_JS_THREW_STRING)"  location: "JS frame :: jar:jar:file:///mnt/asec/org.mozilla.fennec-1/pkg.apk!/omni.ja!/components/nsLoginManager.js :: LoginManager.prototype.removeLogin :: line 381"  data: no]" {file: "jar:jar:file:///mnt/asec/org.mozilla.fennec-1/pkg.apk!/omni.ja!/components/nsLoginManager.js" line: 381}]
info: addon-sdk: [JavaScript Error: "[Exception... "'No matching logins' when calling method: [nsILoginManagerStorage::removeLogin]"  nsresult: "0x8057001e (NS_ERROR_XPC_JS_THREW_STRING)"  location: "JS frame :: jar:jar:file:///mnt/asec/org.mozilla.fennec-1/pkg.apk!/omni.ja!/components/nsLoginManager.js :: LoginManager.prototype.removeLogin :: line 381"  data: no]" {file: "jar:jar:file:///mnt/asec/org.mozilla.fennec-1/pkg.apk!/omni.ja!/components/nsLoginManager.js" line: 381}]
info: addon-sdk: [JavaScript Warning: "TypeError: setting a property that has only a getter" {file: "resource://gre/modules/JNI.jsm" line: 73}]
info: addon-sdk: [JavaScript Warning: "TypeError: setting a property that has only a getter" {file: "resource://gre/modules/JNI.jsm" line: 22}]

Comment 4

[Matteo Ferretti [:zer0] [:matteo]]

Not sure about it, I suggest to add the flag `-v` at your command line, it should be displayed the proper output. Let me know if still not working!

Comment 5

[Gregg Lind (Fx Strategy and Insights - Shield - Heartbeat )]

Attachment: results of cfx mobile test run

Comment 6

[Gregg Lind (Fx Strategy and Insights - Shield - Heartbeat )]

Works on mobile / fennec 21.

IMPORTANT:  use the `-v` flag!

Comment 7

[Matteo Ferretti [:zer0] [:matteo]]

I think we can close the bug, then…! Thanks for the test!

Comment 8

[Matteo Ferretti [:zer0] [:matteo]]

BTW let me know about the comment 1, if `engines` property need to be specified.