Test: disable Flash protected mode

RESOLVED WONTFIX

Status

()

defect
RESOLVED WONTFIX
5 years ago
4 years ago

People

(Reporter: benjamin, Assigned: benjamin)

Tracking

unspecified
mozilla38
x86_64
Windows 7
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox36 fixed, firefox37 fixed, firefox38 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

5 years ago
The results of bug 1111791 indicate that Flash protected mode doubles or more the rate of Flash crashes and hangs for Firefox users. In order to validate that experiment against a larger audience and also get better crash-stats, we intend to disable protected mode by default up to beta.

Flash problems are one of the major reasons people are stopping using Firefox. This change is definitely trading off security for better functionality, which is a tough call, but it's something we're going to try on beta.

No decision has been made yet about rolling this out to release users.
Assignee

Comment 1

5 years ago
Attachment #8546843 - Flags: review?(johnath)
Assignee

Updated

5 years ago
Depends on: 1112709
Assignee

Comment 2

5 years ago
Comment on attachment 8546843 [details] [diff] [review]
bug1119941.diff

Approval Request Comment for Firefox 36
[Risks and why]: Protected mode reduces the impact of other security exploits, but appears to cause a significant increase in crashes and hangs as well as general browser jank. This is a significant change that will require careful evaluation after it has been deployed to the beta audience.
Attachment #8546843 - Flags: approval-mozilla-aurora?
Comment on attachment 8546843 [details] [diff] [review]
bug1119941.diff

I'm marking this r+ based exclusively on the context I have, which is that Benjamin wants it clear that I agree to this test on beta, since there is an explicit trade-off between security and stability at play. I do agree to that on beta for the purposes of understanding how much this stability issue impacts our broader user base, however:

* I have not reviewed the code that uses this pref
* I have not tested this change
* I am explicitly *not* supporting this in release, at least not yet. This patch doesn't have a kill switch in it for exiting beta, but I trust Benjamin to be watching closely and to avoid accidental migration to release.
Attachment #8546843 - Flags: review?(johnath) → review+
Comment on attachment 8546843 [details] [diff] [review]
bug1119941.diff

[Triage Comment]
after the merge day
Attachment #8546843 - Flags: approval-mozilla-beta+
Attachment #8546843 - Flags: approval-mozilla-aurora?
Attachment #8546843 - Flags: approval-mozilla-aurora+
Assignee

Comment 5

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/7e0bcec5e4f5
https://hg.mozilla.org/releases/mozilla-aurora/rev/8b8a825927f1
https://hg.mozilla.org/releases/mozilla-beta/rev/e91fcba59c18

Sylvestre, should we file a separate bug to track that we still need to make a decision about this for FF36?
Flags: needinfo?(sledru)
Target Milestone: --- → mozilla38
https://hg.mozilla.org/mozilla-central/rev/7e0bcec5e4f5
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
(In reply to Benjamin Smedberg  [:bsmedberg] from comment #5)
> Sylvestre, should we file a separate bug to track that we still need to make
> a decision about this for FF36?
I would prefer that yes. Thanks
Flags: needinfo?(sledru)
Assignee

Updated

5 years ago
Depends on: 1120747
Assignee

Updated

5 years ago
Depends on: 1120993
Assignee

Comment 8

5 years ago
We're going to back this out at least for 36b1. Sylvestre says this should be backed out of the beta branch and GECKO360b1_2015011222_RELBRANCH.
See Also: → 1123245
Assignee

Comment 10

4 years ago
Comment on attachment 8546843 [details] [diff] [review]
bug1119941.diff

We've decided to deploy this along with bug 1125891 in beta 5, as a test, which will be backed out in beta 9 and will not ride to 36 release.
Attachment #8546843 - Flags: approval-mozilla-beta+ → approval-mozilla-beta?
Attachment #8546843 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Exploratory testing was performed today to check Flash player compatibility, using Firefox 36 Beta 5 (BuildID=20150129200438), on: Windows Vista x86, Windows 7 x64, Ubuntu 14.04 x64, and Mac OS X 10.9.5. Testing revealed only one new issue, filed in bug 1127884 (not sure if in any way related to this).

Detailed test results are in: https://etherpad.mozilla.org/Fx36b5.
Assignee

Updated

4 years ago
Depends on: 1131345
Assignee

Comment 13

4 years ago
To close the bugzilla loops, we disabled this on all channels in bug 1120993.
Resolution: FIXED → WONTFIX
You need to log in before you can comment on or make changes to this bug.