Bug 224101 – Web Inspector: add Input domain for emitting keyboard and mouse events

• Status: NEW
• Component: Web Inspector

[Niek]

See bug #103530 -  9 years ago the Input domain was added, but at some point it seem to be removed again.

The Playwright team has working patches available that add this functionality: https://github.com/microsoft/playwright/blob/master/browser_patches/webkit/patches/bootstrap.diff

Would it be possible to have this merged upstream?

[Radar WebKit Bug Importer]

<rdar://problem/76370975>

[Sam Sneddon [:gsnedders]]

> at some point it seem to be removed again

see bug 131073

[Niek]

Thanks for providing the reference. I don't really understand the reasoning to remove it though: "It is for a Chrome feature (device screen sharing) we don't have.".

The Input domain is very useful for debugging and automation, right now there's no way to fire mouse or keyboard events using the debugging protocol, that's a severe limitation. I'd be more than happy to prepare a patch to add it again, in case you're open to re-add it of course.

[Blaze Burg]

(In reply to Niek from comment #3)
> Thanks for providing the reference. I don't really understand the reasoning
> to remove it though: "It is for a Chrome feature (device screen sharing) we
> don't have.".
> 
> The Input domain is very useful for debugging and automation, right now
> there's no way to fire mouse or keyboard events using the debugging
> protocol, that's a severe limitation. I'd be more than happy to prepare a
> patch to add it again, in case you're open to re-add it of course.

Generally speaking, we remove code that has no ship plan and no test coverage, such as post-fork Chrome features in Web Inspector Protocol.

How do you plan to write tests for the functionality of Input domain? I didn't see a single test in the diff that you linked–hopefully this is just an oversight.

[Niek]

The linked patch is not meant to be merged as-is, I can add tests for the functionality - no problem. But before I start work on this I want to be sure it's something you are willing to merge upstream.

In my opinion the Input domain is not just a "post-fork Chrome feature", it's an essential feature of any remote debugging implementation.