1156489 - Forward nsISimpleContentPolicy hooks from child to parent process: dom/base/test/test_simplecontentpolicy.html

Status: NEW

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

Description

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

This is the second part of bug 1153852: we need to make nsISimpleContentPolicy work in e10s.

Comment 2

[Blake Kaplan (:mrbkap) (inactive)]

Bill, is this something we need to fix?

Comment 3

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

Yeah, I guess. The whole thing has kind of been superseded by WebExtensions. But we do expose this interface and it was always my intention to make it work in e10s. However, I don't see many add-ons using it.

Comment 4

[Chris Peterson [:cpeterson]]

The dom/base/test/test_sync_xhr_timer.xhtml test is disabled for e10s on all platforms because of this bug.

Comment 5

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

(In reply to Chris Peterson [:cpeterson] from comment #4)
> The dom/base/test/test_sync_xhr_timer.xhtml test is disabled for e10s on all
> platforms because of this bug.

Why? It doesn't appear to use this code.

Comment 6

[Andrew McCreight [:mccr8]]

(In reply to Chris Peterson [:cpeterson] from comment #4)
> The dom/base/test/test_sync_xhr_timer.xhtml test is disabled for e10s on all
> platforms because of this bug.

That's an annotation (in mochitest.ini) for test_simplecontentpolicy.html, not test_sync_xhr_timer.xhtml.

Comment 7

[Andrew McCreight [:mccr8]]

It looks like the XHR test is enabled everywhere except Android.

Comment 8

[Chris Peterson [:cpeterson]]

Thanks. Yes, it is the test_simplecontentpolicy.html test that is disabled. I misread the mochitest.ini.