1719256 - TestHighlighterActor should not require to explicitly pass around the inspector actor ID

Status: RESOLVED FIXED

(DevTools :: Inspector, task, P3)

Description

[Julian Descottes [:jdescottes]]

For a few APIs, the test highlighter actor expects consumers to provide the inspector actor ID.

This makes no sense since the highlighter actor is a target scoped actor and can only be linked to a single inspector actor.

The issue is actually that on the server side, the test highlighter actor has no way to "find" the correct inspector actor unless it knows its ID. However we can make it so that the test highlighter front sets this up automatically when initializing and then it will avoid doing it in each and every test.

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1719256 - [devtools] initialize HighlighterTestActor with the correct inspector ID

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'DevTools::Inspector' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a39feedfa09a
[devtools] initialize HighlighterTestActor with the correct inspector ID r=nchevobbe

Comment 4

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/a39feedfa09a