Remove chrometask_chromescript from eval()-whitelist

RESOLVED FIXED in Firefox 68

Status

()

enhancement
P3
normal
RESOLVED FIXED
2 months ago
2 months ago

People

(Reporter: jallmann, Assigned: jallmann)

Tracking

(Blocks 1 bug)

unspecified
mozilla68
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox68 fixed)

Details

(Whiteboard: [domsecurity-backlog1])

Attachments

(1 attachment)

Assignee

Description

2 months ago

This occurence of eval() in a file used only for testing (and only for a single test, as it seems) triggers our eval()-Assertion:

https://searchfox.org/mozilla-central/source/testing/mochitest/tests/SimpleTest/ChromeTask.js#62

Under the given circumstances, it seems easiest to flip the eval-exception pref for this case.

Assignee

Comment 1

2 months ago

Flip pref to allow the use of eval() in this purely test-related case.

Assignee

Updated

2 months ago
Keywords: checkin-needed

Comment 2

2 months ago

Pushed by nbeleuzu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/45cb80d21409
Remove chrometask_chromescript from eval()-whitelist, r=ckerschb

Keywords: checkin-needed

Comment 3

2 months ago
bugherder
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.