Open Bug 1533412 Opened 6 years ago Updated 3 years ago

Include expanded principal event handlers for browser_markup_events_chrome_*.js

Categories

(DevTools :: Inspector, enhancement, P3)

enhancement

Tracking

(Not tracked)

People

(Reporter: miker, Unassigned)

References

Details

Bug 1532584 removed the video control from our event listener mochitests to prevent changes in platform code.

e.g. If the play control handler of the video element moved to a new line due to a code change then platform would need to fix this test, which is simply testing the event bubbles in privileged security contexts.

Sadly, this means we no longer test in an expanded security context.

This means that the following tests should include an event handler that has an extended security principle:

  • browser_markup_events_chrome_not_blocked.js
  • browser_markup_events_chrome_blocked.js

An expanded principal is specified as an array of origins:

["http://mozilla.org", "http://moz.org"]

The expanded principal subsumes every content principal it contains. The content principals do not subsume the expanded principal, even if the expanded principal only contains a single content principal.

So we need to somehow use a test to add an event handler to the target content page but the handler needs to have an expanded principle.

Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.