Status

defect
--
critical
VERIFIED FIXED
9 years ago
9 years ago

People

(Reporter: ioana.chiorean, Assigned: mfinkle)

Tracking

Firefox Tracking Flags

(fennec2.0b2+)

Details

Attachments

(1 attachment)

Build Identifier: Mozilla/5.0 (Maemo; Linux armv7l; en-US; rv:2.0b8pre)
Gecko/20101029 Firefox/4.0b8pre Fennec/4.0b2pre 

Device: Nokia N900

Steps to Reproduce:
1. Open Fennec
2. Go to URL BAr-> Bookmarks -> choose AMO

Actual Results:  
- crash @ mozilla::net::HttpChannelParentListener::GetInterface 

Expected Results:  
- page loads ok

Note: Crash Report
http://crash-stats.mozilla.com/report/index/1dde78d1-c50d-469d-af06-bee742101029
tracking-fennec: --- → 2.0b2+
Crashes on Android too: Nexus One

This is a regression from bug 566478
https://bugzilla.mozilla.org loads fine (this is a SSL site)

https://addons.mozilla.org is an EV site
Just tested https://paypal.com and it works fine on Android
Ioana says https://paypal.com works fine on Maemo too
Yep, I can reproduce this on the Nexus One.
OS: Maemo → All
Severity: normal → critical
Wuh-oh:

kaie@56656 147 if (!mActiveChannel && !mActiveChannel->mTabParent)
kaie@56656 148   return NS_NOINTERFACE; 

That should be ||
(In reply to comment #7)
> Wuh-oh:
> 
> kaie@56656 147 if (!mActiveChannel && !mActiveChannel->mTabParent)
> kaie@56656 148   return NS_NOINTERFACE; 
> 
> That should be ||

Testing. Patch for you soon Josh
Posted patch patchSplinter Review
Yep, the simple fix stops the crash and AMO is shown with it's EV colors and info
Assignee: nobody → mark.finkle
Attachment #486919 - Flags: review?(josh)
Comment on attachment 486919 [details] [diff] [review]
patch

Ship it ship it ship it.
Attachment #486919 - Flags: review?(josh) → review+
pushed:
http://hg.mozilla.org/mozilla-central/rev/3b73e3b88653
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
verified FIXED on build:
Mozilla/5.0 (Android; Linux armv71; rv:2.0b8pre) Gecko/20101029 Namoroka/4.0b8pre Fennec/4.0b2pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.