Status

()

defect
P1
normal
VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: dougt, Assigned: mbrubeck)

Tracking

unspecified
All
Android
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

Attachments

(1 attachment)

Reporter

Description

8 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

8 years ago
Assignee: nobody → mbrubeck
Assignee

Comment 1

8 years ago
Posted patch patchSplinter Review
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

8 years ago
Status: NEW → ASSIGNED
Reporter

Comment 2

8 years ago
try loading about:config
Assignee

Comment 3

8 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
Closed: 8 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+
You need to log in before you can comment on or make changes to this bug.