Closed Bug 1170385 Opened 5 years ago Closed 5 years ago

csp-on-violate-policy message not sent in browser tests with e10s

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
e10s + ---
firefox41 --- fixed

People

(Reporter: marcosc, Assigned: marcosc)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

For some reason, the csp-on-violate-policy message is not dispatched to browser tests. However, it works fine in non-e10s.
Blocks: e10s-tests
tracking-e10s: --- → +
Flags: needinfo?(wmccloskey)
If this is just for tests, one easy way to fix this would be to add "csp-on-violate-policy" to the list here:
https://dxr.mozilla.org/mozilla-central/source/toolkit/components/addoncompat/RemoteAddonsParent.jsm#356
Flags: needinfo?(wmccloskey)
(In reply to Bill McCloskey (:billm) from comment #1)
> If this is just for tests, one easy way to fix this would be to add
> "csp-on-violate-policy" to the list here:
> https://dxr.mozilla.org/mozilla-central/source/toolkit/components/
> addoncompat/RemoteAddonsParent.jsm#356

Ohhh!!! let me try! :D
Wholly crap, that worked! Awesome! :D preparing patch.
Comment on attachment 8614295 [details] [diff] [review]
Adds csp-on-violate-policy to TOPIC_WHITELIST of RemoteAddonsParent.jsm + enables depedent tests

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

::: toolkit/components/addoncompat/RemoteAddonsParent.jsm
@@ +360,4 @@
>    "dom-window-destroyed",
>    "inner-window-destroyed",
>    "outer-window-destroyed",
> +  "csp-on-violate-policy"

Please put a comma at the end.
Attachment #8614295 - Flags: review?(wmccloskey) → review+
> Please put a comma at the end.

Done.
https://hg.mozilla.org/mozilla-central/rev/97f13e9f8a64
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.