User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b13pre) Gecko/20110303 Firefox/4.0b13pre Build Identifier: 20110304042311 Relevant crashes: https://crash-stats.mozilla.com/report/index/b19c98d5-1e50-46ff-b1a2-ba7572110304 https://crash-stats.mozilla.com/report/index/bp-7e01861a-462c-4814-ac7c-145942110304 They both are RegExp crashes, and it happens intermittently, but usually the first time loading a website when opening Fennec. Reproducible: Sometimes Steps to Reproduce: 1. Get a Samsung Continuum. 2. Download Fennec. 3. Install Fennec. 4. Open Fennec. 5. Load a website. Actual Results: Fennec crashes with a JS error. Expected Results: It should load the website.
tracking-fennec: --- → ?
Keywords: crash, mobile
Can you specify which website? does it reproduce on any page? Also, please add the buildID of fennec. (goto about:fennec, scroll to the bottom)
Summary: Fennec crashes due to JS errors on Samsung Continuum → Fennec crash [@ js::RegExp::executeInternal ] on Samsung Continuum
This device is in the Galaxy S class, and is running Android 2.1 with Linux 2.6.29, just like the other Galaxy S phones that experienced JIT crashes. We might need to extend the JIT blacklist to this device (SCH-I400).
It crashes on techcrunch.com, for example. Build: Mozilla/5.0 (android; Linux armv7l; rv:2.0b13pre) Gecko/20110304 Firefox/4.0b13pre Fennec/4.0b6pre
Instead of continuing to inflame this ugly pustule, I wonder if it's time to cut our losses and start refusing to install on the 2.1 galaxy s***. Skype is already doing this (dunno why though). If not now, when ... I had hoped to fix bug 604774 around November. Samsung and the carriers have just totally dropped the ball. (Yes, I have a galaxy s*** still running 2.1.)
Created attachment 517037 [details] [diff] [review] patch: add Continuum to JIT blacklist
Assignee: general → mbrubeck
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #517037 - Flags: review?(dvander)
There is a test build with this patch at: http://people.mozilla.com/~mbrubeck/fennec-4.0b6pre-mbrubeck-continuum.apk
Attachment #517037 - Flags: review?(dvander) → review+
Comment on attachment 517037 [details] [diff] [review] patch: add Continuum to JIT blacklist Requesting approval2.0: Trivial patch that fixes a crash by adding a new device to the JIT blocklist.
Attachment #517037 - Flags: approval2.0?
Whiteboard: [has patch][has review][needs approval]
Attachment #517037 - Flags: approval2.0? → approval2.0+
Whiteboard: [has patch][has review][needs approval] → [has patch][has review]
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Crash Signature: [@ js::RegExp::executeInternal ]
You need to log in before you can comment on or make changes to this bug.