Closed
Bug 779490
Opened 12 years ago
Closed 12 years ago
Unable to install an application with unicode characters (UTF-8) within the name
Categories
(Firefox for Android Graveyard :: Web Apps (PWAs), defect, P1)
Tracking
(firefox16 unaffected, firefox17 verified)
VERIFIED
FIXED
Firefox 17
Tracking | Status | |
---|---|---|
firefox16 | --- | unaffected |
firefox17 | --- | verified |
People
(Reporter: aaronmt, Assigned: wesj)
Details
(Keywords: regression, Whiteboard: [blocking-webrtandroid1+])
Attachments
(1 file)
1.43 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
E/GeckoConsole( 4296): [JavaScript Error: "DOMApplicationRegistry: Could not parse JSON: [Exception... "Component returned failure code: 0x8050000e (NS_ERROR_ILLEGAL_INPUT) [nsIScriptableUnicodeConverter.ConvertToUnicode]" nsresult: "0x8050000e (NS_ERROR_ILLEGAL_INPUT)" location: "JS frame :: chrome://browser/content/browser.js :: <TOP_LEVEL> :: line 5895" data: no]" {file: "resource://gre/modules/Webapps.jsm" line: 126}] 1. Go to http://ed.agadak.net/app/ 2. Install Ed's app that should contain Japanese characters in the name This is only only currently affecting mozilla-central (might be a regression from bug 766604). { "name": "テスト Ed Agadak テスト", "launch_path": "/app", "default_locale": "en-US", "icons": { "32": "/favicon.ico", "64": "/favicon.ico" }, "description": "Ed Agadak", "locales": {}, "developer": { "url": "http://ed.agadak.net", "name": "Edward Lee" } } -- Nightly (08/01) Samsung Galaxy Nexus (Android 4.1.1)
Reporter | ||
Updated•12 years ago
|
Whiteboard: [blocking-webrtandroid1?]
Comment 1•12 years ago
|
||
Hmm...bug 738298 also landed recently too, I think it's possible it might be that one too. Fabrice - Do you know? I think it's failing in a function called "_loadJSONASync" in Webapps.jsm.
Assignee | ||
Comment 4•12 years ago
|
||
fabrice, your code is throwing on that particular manifest. Are you going to be able to look into it?
Comment 5•12 years ago
|
||
(In reply to Wesley Johnston (:wesj) from comment #4) > fabrice, your code is throwing on that particular manifest. Are you going to > be able to look into it? yes
Assignee | ||
Comment 6•12 years ago
|
||
Assigning this so I don't feel like I'm ignoring it then. If you are too busy fabrice, I can grab this (but I'm not sure what you're doing wrong...)
Assignee: nobody → fabrice
Comment 7•12 years ago
|
||
What's strange about this bug is that it only appears to happen on Android. Why is that the case? Is it cause Android added additional UTF-8 handling in bug 766604?
Updated•12 years ago
|
Priority: -- → P1
Updated•12 years ago
|
Whiteboard: [blocking-webrtandroid1?] → [blocking-webrtandroid1+]
Assignee | ||
Comment 8•12 years ago
|
||
Actually, I made us not even use this anymore and removing it fixes this.
Assignee: fabrice → wjohnston
Attachment #650397 -
Flags: review?(mark.finkle)
Comment 9•12 years ago
|
||
Comment on attachment 650397 [details] [diff] [review] Patch Less is better. I am tempted to have you remove the two "Got callback" logs too.
Attachment #650397 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 11•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/62f8e90be342 Hmm.. 16 is weird because the platform patch isn't there. So... no.
Comment 12•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/62f8e90be342
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 17
Reporter | ||
Updated•12 years ago
|
Status: RESOLVED → VERIFIED
Updated•11 years ago
|
tracking-fennec: ? → ---
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•