Status

defect
VERIFIED FIXED
4 years ago
10 months ago

People

(Reporter: snorp, Assigned: snorp)

Tracking

34 Branch
Firefox 48
ARM
Android

Firefox Tracking Flags

(firefox45 affected, firefox46 verified, firefox47 verified, firefox48 verified, fennec47+)

Details

Attachments

(1 attachment)

Presumably this is because the Nexus 9 has a 64-bit user land, and the 32-bit Flash plugin links to some of these. We should probably just block Flash on 64-bit.
(In reply to James Willcox (:snorp) (jwillcox@mozilla.com) from comment #0)
> Presumably this is because the Nexus 9 has a 64-bit user land, and the
> 32-bit Flash plugin links to some of these. We should probably just block
> Flash on 64-bit.

That sounds right to me
Actually we are able to load/link the plugin, it just crashes during init somewhere. JimDB is busted on 64-bit, so I don't know exactly where yet. Flash works in Dolphin on this device, so presumably we can make it work.
Duplicate of this bug: 1001624
tracking-fennec: --- → 47+
We should just block Flash on the Nexus 9
Assignee: nobody → snorp
Comment on attachment 8726248 [details] [diff] [review]
Try harder to disable Flash on Tegra devices

Review of attachment 8726248 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me provided that the previous checks are not made redundant by the new one.
Attachment #8726248 - Flags: review?(droeh) → review+

Comment 8

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/4beeb84a6696
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48

Comment 9

3 years ago
I am this gonna be uplifted? I am easily reproducing it on Nexus 9 (Android 6.0.1) by playing a video on cnn.com

Verified as fixed for Nightly.
Comment on attachment 8726248 [details] [diff] [review]
Try harder to disable Flash on Tegra devices

Approval Request Comment
[Feature/regressing bug #]: None
[User impact if declined]: Crashes when using Flash on Nexus 9
[Describe test coverage new/current, TreeHerder]: Nightly, Aurora
[Risks and why]: Low
[String/UUID change made/needed]: None
Attachment #8726248 - Flags: approval-mozilla-beta?
Attachment #8726248 - Flags: approval-mozilla-aurora?
Comment on attachment 8726248 [details] [diff] [review]
Try harder to disable Flash on Tegra devices

Crash fix for android, let's land this in beta.
Attachment #8726248 - Flags: approval-mozilla-beta?
Attachment #8726248 - Flags: approval-mozilla-beta+
Attachment #8726248 - Flags: approval-mozilla-aurora?
Attachment #8726248 - Flags: approval-mozilla-aurora+

Comment 14

3 years ago
Verified this with the steps mentioned in comment 9:  go to cnn.com and start a video. 
Working on both Aurora 47 (03/29) and 46 Beta 6 on Nexus 9 (Android 6.0.1)
Status: RESOLVED → VERIFIED

Updated

3 years ago
See Also: → 1272810

Updated

10 months 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.