<?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>210305</bug_id>
          
          <creation_ts>2020-04-09 14:23:44 -0700</creation_ts>
          <short_desc>[iOS]: New test landed for r259762 landed broken: editing/editable-region/hit-test-overlap.html</short_desc>
          <delta_ts>2020-04-09 14:50:57 -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>New Bugs</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=209888</see_also>
          <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>
          <dependson>209888</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ryan Haddad">ryanhaddad</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>dbates</cc>
    
    <cc>webkit-bot-watchers-bugzilla</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1639870</commentid>
    <comment_count>0</comment_count>
    <who name="Ryan Haddad">ryanhaddad</who>
    <bug_when>2020-04-09 14:23:44 -0700</bug_when>
    <thetext>[iOS]: New test landed for https://trac.webkit.org/changeset/259762/webkit landed broken: editing/editable-region/hit-test-overlap.html

--- /Volumes/Data/slave/ios-simulator-13-release-tests-wk2/build/layout-test-results/editing/editable-region/hit-test-overlap-expected.txt
+++ /Volumes/Data/slave/ios-simulator-13-release-tests-wk2/build/layout-test-results/editing/editable-region/hit-test-overlap-actual.txt
@@ -9,10 +9,11 @@
 PASS (x = 281, y = 69, width = 249, height = 201) contains editable elements.
 
 Composited overlap:
-PASS (x = 29, y = 343, width = 251, height = 201) does not contain editable elements.
-PASS (x = 270, y = 400, width = 10, height = 10) does not contain editable elements.
+FAIL (x = 29, y = 343, width = 251, height = 201) should not contain editable elements, but did.
+FAIL (x = 270, y = 400, width = 10, height = 10) should not contain editable elements, but did.
 PASS (x = 281, y = 343, width = 249, height = 201) contains editable elements.
 PASS successfullyParsed is true
+Some tests failed.
 
 TEST COMPLETE</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1639873</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2020-04-09 14:24:34 -0700</bug_when>
    <thetext>&lt;rdar://problem/61539886&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1639882</commentid>
    <comment_count>2</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2020-04-09 14:37:27 -0700</bug_when>
    <thetext>Test is expected to fail as written. There was some confusion (or I became confused) with regards to earlier revisions of the bug #209888 that had -_mayContainEditableElementsInRect only returning true if the frontmost layer didn&apos;t contain an editable element. I had coded the test with this behavior at the time, but it&apos;s not the final behavior I went with: -_mayContainEditableElementsInRect returns true if any hit layer contains an editable element.

Thinking about it now, it may be possible to actually get what the test was trying to achieve....</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1639883</commentid>
    <comment_count>3</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2020-04-09 14:38:38 -0700</bug_when>
    <thetext>(In reply to Daniel Bates from comment #2)
&gt; Thinking about it now, it may be possible to actually get what the test was
&gt; trying to achieve....

I&apos;ll achieve it in stages. First, for this bug, I&apos;ll just update the test to match the landed behavior. Then I will file a bug for new behavior.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1639889</commentid>
    <comment_count>4</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2020-04-09 14:50:57 -0700</bug_when>
    <thetext>Committed r259831: &lt;https://trac.webkit.org/changeset/259831&gt;</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>