Status

()

Firefox for Android
General
P1
normal
VERIFIED FIXED
6 years ago
5 years ago

People

(Reporter: dougt, Assigned: mbrubeck)

Tracking

unspecified
All
Android
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
in browser.js, onLocationChange().  chrome pages do not have contentDocuments.  so we end up passing undefined to java.  This, in turn, causes an exception to be raised when handling the Content:LocationChange message.


E/GeckoApp(28356): org.json.JSONException: No value for documentURI                                                                                                    
E/GeckoApp(28356):      at org.json.JSONObject.get(JSONObject.java:354)                                                                                                
E/GeckoApp(28356):      at org.json.JSONObject.getString(JSONObject.java:510)                                                                                          
E/GeckoApp(28356):      at org.mozilla.gecko.GeckoApp.handleMessage(GeckoApp.java:860)                                                                                 
E/GeckoApp(28356):      at org.mozilla.gecko.GeckoAppShell.handleGeckoMessage(GeckoAppShell.java:1501)                                                                 
E/GeckoApp(28356):      at org.mozilla.gecko.GeckoAppShell.nativeRun(Native Method)                                                                                    
E/GeckoApp(28356):      at org.mozilla.gecko.GeckoAppShell.nativeRun(Native Method)                                                                                    
E/GeckoApp(28356):      at org.mozilla.gecko.GeckoAppShell.runGecko(GeckoAppShell.java:447)                                                                            
E/GeckoApp(28356):      at org.mozilla.gecko.GeckoThread.run(GeckoThread.java:107)                                                                                     
I/GeckoApp(28356): Got message: Content:SecurityChange
(Assignee)

Updated

6 years ago
Assignee: nobody → mbrubeck
(Assignee)

Comment 1

6 years ago
Created attachment 581744 [details] [diff] [review]
patch

This should fix the problem, but I wasn't able to verify the fix because I couldn't reproduce the error.

Doug, what steps did you use to produce this?  I tried loading chrome://browser/content/about.xhtml but I did not see any errors or bugs.
Attachment #581744 - Flags: review?(mark.finkle)
(Assignee)

Updated

6 years ago
Status: NEW → ASSIGNED
(Reporter)

Comment 2

6 years ago
try loading about:config
(Assignee)

Comment 3

6 years ago
(In reply to Doug Turner (:dougt) from comment #2)
> try loading about:config

about:config loads fine for me in today's native android nightly (build ID 20111214, built from 221eccfa6a3f).  I didn't see any exceptions or errors logcat.
Attachment #581744 - Flags: review?(mark.finkle) → review+
pushed https://hg.mozilla.org/mozilla-central/rev/5ef118dcf1a1
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
I cannot reproduce this on the latest native fennec:
Mozilla/5.0(Android;Linux armv7l;rv:11.0a1)Gecko/20111220;Firefox/11.0a1 Fennec/11.0a1.
Device: HTC Desire (Android 2.2)
Status: RESOLVED → VERIFIED
tracking-fennec: --- → 11+
status-firefox11: --- → fixed
You need to log in before you can comment on or make changes to this bug.