<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>224101</bug_id>
          
          <creation_ts>2021-04-02 02:40:32 -0700</creation_ts>
          <short_desc>Web Inspector: add Input domain for emitting keyboard and mouse events</short_desc>
          <delta_ts>2021-04-12 02:49:12 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Web Inspector</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Niek">mail</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bburg</cc>
    
    <cc>gsnedders</cc>
    
    <cc>hi</cc>
    
    <cc>inspector-bugzilla-changes</cc>
    
    <cc>smoley</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1746438</commentid>
    <comment_count>0</comment_count>
    <who name="Niek">mail</who>
    <bug_when>2021-04-02 02:40:32 -0700</bug_when>
    <thetext>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?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1748131</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2021-04-07 16:21:20 -0700</bug_when>
    <thetext>&lt;rdar://problem/76370975&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1748550</commentid>
    <comment_count>2</comment_count>
    <who name="Sam Sneddon [:gsnedders]">gsnedders</who>
    <bug_when>2021-04-08 16:53:20 -0700</bug_when>
    <thetext>&gt; at some point it seem to be removed again

see bug 131073</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1748606</commentid>
    <comment_count>3</comment_count>
    <who name="Niek">mail</who>
    <bug_when>2021-04-08 21:54:13 -0700</bug_when>
    <thetext>Thanks for providing the reference. I don&apos;t really understand the reasoning to remove it though: &quot;It is for a Chrome feature (device screen sharing) we don&apos;t have.&quot;.

The Input domain is very useful for debugging and automation, right now there&apos;s no way to fire mouse or keyboard events using the debugging protocol, that&apos;s a severe limitation. I&apos;d be more than happy to prepare a patch to add it again, in case you&apos;re open to re-add it of course.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1748851</commentid>
    <comment_count>4</comment_count>
    <who name="Blaze Burg">bburg</who>
    <bug_when>2021-04-09 13:51:33 -0700</bug_when>
    <thetext>(In reply to Niek from comment #3)
&gt; Thanks for providing the reference. I don&apos;t really understand the reasoning
&gt; to remove it though: &quot;It is for a Chrome feature (device screen sharing) we
&gt; don&apos;t have.&quot;.
&gt; 
&gt; The Input domain is very useful for debugging and automation, right now
&gt; there&apos;s no way to fire mouse or keyboard events using the debugging
&gt; protocol, that&apos;s a severe limitation. I&apos;d be more than happy to prepare a
&gt; patch to add it again, in case you&apos;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&apos;t see a single test in the diff that you linked–hopefully this is just an oversight.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1749165</commentid>
    <comment_count>5</comment_count>
    <who name="Niek">mail</who>
    <bug_when>2021-04-12 02:49:12 -0700</bug_when>
    <thetext>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&apos;s something you are willing to merge upstream.

In my opinion the Input domain is not just a &quot;post-fork Chrome feature&quot;, it&apos;s an essential feature of any remote debugging implementation.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>