<?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>187309</bug_id>
          
          <creation_ts>2018-07-03 16:25:49 -0700</creation_ts>
          <short_desc>[mac-wk1] Layout Test editing/selection/navigation-clears-editor-state.html is a flaky failure</short_desc>
          <delta_ts>2018-10-25 09:45:08 -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>HTML Editing</component>
          <version>Other</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=187389</see_also>
    
    <see_also>https://bugs.webkit.org/show_bug.cgi?id=190890</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ryan Haddad">ryanhaddad</reporter>
          <assigned_to name="Saam Barati">saam</assigned_to>
          <cc>ap</cc>
    
    <cc>commit-queue</cc>
    
    <cc>mark.lam</cc>
    
    <cc>realdawei</cc>
    
    <cc>rniwa</cc>
    
    <cc>saam</cc>
    
    <cc>tsavell</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>wenson_hsieh</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1439051</commentid>
    <comment_count>0</comment_count>
    <who name="Ryan Haddad">ryanhaddad</who>
    <bug_when>2018-07-03 16:25:49 -0700</bug_when>
    <thetext>The following layout test is flaky on macOS WK1

editing/selection/navigation-clears-editor-state.html

Probable cause:

Unknown. This test was added in https://trac.webkit.org/changeset/232434/webkit.
It looks like it was flaky since the start, but it has become very flaky recently. The test was modified as part of https://trac.webkit.org/changeset/233236/webkit, which may be the culprit.

Flakiness Dashboard:

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&amp;tests=editing%2Fselection%2Fnavigation-clears-editor-state.html

--- /Volumes/Data/slave/highsierra-release-tests-wk1/build/layout-test-results/editing/selection/navigation-clears-editor-state-expected.txt
+++ /Volumes/Data/slave/highsierra-release-tests-wk1/build/layout-test-results/editing/selection/navigation-clears-editor-state-actual.txt
@@ -3,7 +3,7 @@
 On success, you will see a series of &quot;PASS&quot; messages, followed by &quot;TEST COMPLETE&quot;.
 
 
-PASS freed more than 60 documents
+FAIL freed fewer than 60 documents
 PASS successfullyParsed is true
 
 TEST COMPLETE</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1439111</commentid>
    <comment_count>1</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2018-07-03 19:32:12 -0700</bug_when>
    <thetext>Ugh... conservative GC is hard :(</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1439467</commentid>
    <comment_count>2</comment_count>
    <who name="Saam Barati">saam</who>
    <bug_when>2018-07-05 11:42:15 -0700</bug_when>
    <thetext>We can just make the threshold smaller.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1439556</commentid>
    <comment_count>3</comment_count>
      <attachid>344366</attachid>
    <who name="Saam Barati">saam</who>
    <bug_when>2018-07-05 14:23:22 -0700</bug_when>
    <thetext>Created attachment 344366
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1439557</commentid>
    <comment_count>4</comment_count>
      <attachid>344366</attachid>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2018-07-05 14:25:36 -0700</bug_when>
    <thetext>Comment on attachment 344366
patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1439558</commentid>
    <comment_count>5</comment_count>
      <attachid>344367</attachid>
    <who name="Saam Barati">saam</who>
    <bug_when>2018-07-05 14:29:35 -0700</bug_when>
    <thetext>Created attachment 344367
patch for landing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1439661</commentid>
    <comment_count>6</comment_count>
      <attachid>344367</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2018-07-05 18:35:38 -0700</bug_when>
    <thetext>Comment on attachment 344367
patch for landing

Clearing flags on attachment: 344367

Committed r233555: &lt;https://trac.webkit.org/changeset/233555&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1439662</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2018-07-05 18:35:39 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1439663</commentid>
    <comment_count>8</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2018-07-05 18:36:21 -0700</bug_when>
    <thetext>&lt;rdar://problem/41873906&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>344366</attachid>
            <date>2018-07-05 14:23:22 -0700</date>
            <delta_ts>2018-07-05 14:29:35 -0700</delta_ts>
            <desc>patch</desc>
            <filename>a-backup.diff</filename>
            <type>text/plain</type>
            <size>2324</size>
            <attacher name="Saam Barati">saam</attacher>
            
              <data encoding="base64">SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDIzMzUzOSkKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29y
a2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBACisyMDE4LTA3LTA1ICBTYWFtIEJhcmF0aSAgPHNi
YXJhdGlAYXBwbGUuY29tPgorCisgICAgICAgIFttYWMtd2sxXSBMYXlvdXQgVGVzdCBlZGl0aW5n
L3NlbGVjdGlvbi9uYXZpZ2F0aW9uLWNsZWFycy1lZGl0b3Itc3RhdGUuaHRtbCBpcyBhIGZsYWt5
IGZhaWx1cmUKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lk
PTE4NzMwOQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg
IFRyeSB0byBtYWtlIHRoaXMgdGVzdCBsZXNzIGZsYWt5IGJ5IG1ha2luZyBwZXJjZW50cyB0aHJl
c2hvbGQgb2YgZnJlZWQKKyAgICAgICAgb2JqZWN0cyBzbWFsbGVyLgorCisgICAgICAgICogZWRp
dGluZy9zZWxlY3Rpb24vbmF2aWdhdGlvbi1jbGVhcnMtZWRpdG9yLXN0YXRlLWV4cGVjdGVkLnR4
dDoKKyAgICAgICAgKiBlZGl0aW5nL3NlbGVjdGlvbi9uYXZpZ2F0aW9uLWNsZWFycy1lZGl0b3It
c3RhdGUuaHRtbDoKKwogMjAxOC0wNy0wNCAgUnlvc3VrZSBOaXdhICA8cm5pd2FAd2Via2l0Lm9y
Zz4KIAogICAgICAgICBZb3V0dWJlIHZpZGVvIHBhZ2VzIGNyYXNoIGFmdGVyIGEgY291cGxlIG9m
IG1pbnV0ZXMKSW5kZXg6IExheW91dFRlc3RzL2VkaXRpbmcvc2VsZWN0aW9uL25hdmlnYXRpb24t
Y2xlYXJzLWVkaXRvci1zdGF0ZS1leHBlY3RlZC50eHQKPT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVz
dHMvZWRpdGluZy9zZWxlY3Rpb24vbmF2aWdhdGlvbi1jbGVhcnMtZWRpdG9yLXN0YXRlLWV4cGVj
dGVkLnR4dAkocmV2aXNpb24gMjMzNTM5KQorKysgTGF5b3V0VGVzdHMvZWRpdGluZy9zZWxlY3Rp
b24vbmF2aWdhdGlvbi1jbGVhcnMtZWRpdG9yLXN0YXRlLWV4cGVjdGVkLnR4dAkod29ya2luZyBj
b3B5KQpAQCAtMyw3ICszLDcgQEAgVGhpcyB0ZXN0cyBuYXZpZ2F0aW5nIGF3YXkgZnJvbSBhIGRv
Y3VtZQogT24gc3VjY2VzcywgeW91IHdpbGwgc2VlIGEgc2VyaWVzIG9mICJQQVNTIiBtZXNzYWdl
cywgZm9sbG93ZWQgYnkgIlRFU1QgQ09NUExFVEUiLgogCiAKLVBBU1MgZnJlZWQgbW9yZSB0aGFu
IDYwIGRvY3VtZW50cworUEFTUyBmcmVlZCBtb3JlIHRoYW4gMTUlIG9mIGRvY3VtZW50cwogUEFT
UyBzdWNjZXNzZnVsbHlQYXJzZWQgaXMgdHJ1ZQogCiBURVNUIENPTVBMRVRFCkluZGV4OiBMYXlv
dXRUZXN0cy9lZGl0aW5nL3NlbGVjdGlvbi9uYXZpZ2F0aW9uLWNsZWFycy1lZGl0b3Itc3RhdGUu
aHRtbAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9lZGl0aW5nL3NlbGVjdGlvbi9uYXZpZ2F0
aW9uLWNsZWFycy1lZGl0b3Itc3RhdGUuaHRtbAkocmV2aXNpb24gMjMzNTM5KQorKysgTGF5b3V0
VGVzdHMvZWRpdGluZy9zZWxlY3Rpb24vbmF2aWdhdGlvbi1jbGVhcnMtZWRpdG9yLXN0YXRlLmh0
bWwJKHdvcmtpbmcgY29weSkKQEAgLTU5LDEwICs1OSwxMCBAQCBhc3luYyBmdW5jdGlvbiBydW5U
ZXN0KCkKICAgICAgICAgICAgIEdDQ29udHJvbGxlci5jb2xsZWN0KCk7CiAgICAgICAgICAgICBs
ZXQgYWZ0ZXIgPSBpbnRlcm5hbHMubnVtYmVyT2ZMaXZlRG9jdW1lbnRzKCk7CiAgICAgICAgICAg
ICBsZXQgZGVsdGEgPSBiZWZvcmUgLSBhZnRlcjsKLSAgICAgICAgICAgIGlmIChkZWx0YSA+IDAu
MyAqIGNvdW50KQotICAgICAgICAgICAgICAgIGRlYnVnKCJQQVNTIGZyZWVkIG1vcmUgdGhhbiA2
MCBkb2N1bWVudHMiKTsKKyAgICAgICAgICAgIGlmIChkZWx0YSA+IDAuMTUgKiBjb3VudCkKKyAg
ICAgICAgICAgICAgICBkZWJ1ZygiUEFTUyBmcmVlZCBtb3JlIHRoYW4gMTUlIG9mIGRvY3VtZW50
cyIpOwogICAgICAgICAgICAgZWxzZQotICAgICAgICAgICAgICAgIGRlYnVnKCJGQUlMIGZyZWVk
IGZld2VyIHRoYW4gNjAgZG9jdW1lbnRzIik7CisgICAgICAgICAgICAgICAgZGVidWcoIkZBSUwg
ZnJlZWQgZmV3ZXIgdGhhbiAxNSUgb2YgZG9jdW1lbnRzIik7CiAgICAgICAgICAgICBmaW5pc2hK
U1Rlc3QoKTsgICAgICAgICAgICAgICAgCiAgICAgICAgIH07CiAgICAgfQo=
</data>
<flag name="review"
          id="362419"
          type_id="1"
          status="+"
          setter="mark.lam"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>344367</attachid>
            <date>2018-07-05 14:29:35 -0700</date>
            <delta_ts>2018-07-05 18:35:38 -0700</delta_ts>
            <desc>patch for landing</desc>
            <filename>a-backup.diff</filename>
            <type>text/plain</type>
            <size>2317</size>
            <attacher name="Saam Barati">saam</attacher>
            
              <data encoding="base64">SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDIzMzUzOSkKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29y
a2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBACisyMDE4LTA3LTA1ICBTYWFtIEJhcmF0aSAgPHNi
YXJhdGlAYXBwbGUuY29tPgorCisgICAgICAgIFttYWMtd2sxXSBMYXlvdXQgVGVzdCBlZGl0aW5n
L3NlbGVjdGlvbi9uYXZpZ2F0aW9uLWNsZWFycy1lZGl0b3Itc3RhdGUuaHRtbCBpcyBhIGZsYWt5
IGZhaWx1cmUKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lk
PTE4NzMwOQorCisgICAgICAgIFJldmlld2VkIGJ5IE1hcmsgTGFtLgorCisgICAgICAgIFRyeSB0
byBtYWtlIHRoaXMgdGVzdCBsZXNzIGZsYWt5IGJ5IG1ha2luZyBwZXJjZW50cyB0aHJlc2hvbGQg
b2YgZnJlZWQKKyAgICAgICAgb2JqZWN0cyBzbWFsbGVyLgorCisgICAgICAgICogZWRpdGluZy9z
ZWxlY3Rpb24vbmF2aWdhdGlvbi1jbGVhcnMtZWRpdG9yLXN0YXRlLWV4cGVjdGVkLnR4dDoKKyAg
ICAgICAgKiBlZGl0aW5nL3NlbGVjdGlvbi9uYXZpZ2F0aW9uLWNsZWFycy1lZGl0b3Itc3RhdGUu
aHRtbDoKKwogMjAxOC0wNy0wNCAgUnlvc3VrZSBOaXdhICA8cm5pd2FAd2Via2l0Lm9yZz4KIAog
ICAgICAgICBZb3V0dWJlIHZpZGVvIHBhZ2VzIGNyYXNoIGFmdGVyIGEgY291cGxlIG9mIG1pbnV0
ZXMKSW5kZXg6IExheW91dFRlc3RzL2VkaXRpbmcvc2VsZWN0aW9uL25hdmlnYXRpb24tY2xlYXJz
LWVkaXRvci1zdGF0ZS1leHBlY3RlZC50eHQKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZWRp
dGluZy9zZWxlY3Rpb24vbmF2aWdhdGlvbi1jbGVhcnMtZWRpdG9yLXN0YXRlLWV4cGVjdGVkLnR4
dAkocmV2aXNpb24gMjMzNTM5KQorKysgTGF5b3V0VGVzdHMvZWRpdGluZy9zZWxlY3Rpb24vbmF2
aWdhdGlvbi1jbGVhcnMtZWRpdG9yLXN0YXRlLWV4cGVjdGVkLnR4dAkod29ya2luZyBjb3B5KQpA
QCAtMyw3ICszLDcgQEAgVGhpcyB0ZXN0cyBuYXZpZ2F0aW5nIGF3YXkgZnJvbSBhIGRvY3VtZQog
T24gc3VjY2VzcywgeW91IHdpbGwgc2VlIGEgc2VyaWVzIG9mICJQQVNTIiBtZXNzYWdlcywgZm9s
bG93ZWQgYnkgIlRFU1QgQ09NUExFVEUiLgogCiAKLVBBU1MgZnJlZWQgbW9yZSB0aGFuIDYwIGRv
Y3VtZW50cworUEFTUyBmcmVlZCBtb3JlIHRoYW4gMTUlIG9mIGRvY3VtZW50cwogUEFTUyBzdWNj
ZXNzZnVsbHlQYXJzZWQgaXMgdHJ1ZQogCiBURVNUIENPTVBMRVRFCkluZGV4OiBMYXlvdXRUZXN0
cy9lZGl0aW5nL3NlbGVjdGlvbi9uYXZpZ2F0aW9uLWNsZWFycy1lZGl0b3Itc3RhdGUuaHRtbAo9
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9lZGl0aW5nL3NlbGVjdGlvbi9uYXZpZ2F0aW9uLWNs
ZWFycy1lZGl0b3Itc3RhdGUuaHRtbAkocmV2aXNpb24gMjMzNTM5KQorKysgTGF5b3V0VGVzdHMv
ZWRpdGluZy9zZWxlY3Rpb24vbmF2aWdhdGlvbi1jbGVhcnMtZWRpdG9yLXN0YXRlLmh0bWwJKHdv
cmtpbmcgY29weSkKQEAgLTU5LDEwICs1OSwxMCBAQCBhc3luYyBmdW5jdGlvbiBydW5UZXN0KCkK
ICAgICAgICAgICAgIEdDQ29udHJvbGxlci5jb2xsZWN0KCk7CiAgICAgICAgICAgICBsZXQgYWZ0
ZXIgPSBpbnRlcm5hbHMubnVtYmVyT2ZMaXZlRG9jdW1lbnRzKCk7CiAgICAgICAgICAgICBsZXQg
ZGVsdGEgPSBiZWZvcmUgLSBhZnRlcjsKLSAgICAgICAgICAgIGlmIChkZWx0YSA+IDAuMyAqIGNv
dW50KQotICAgICAgICAgICAgICAgIGRlYnVnKCJQQVNTIGZyZWVkIG1vcmUgdGhhbiA2MCBkb2N1
bWVudHMiKTsKKyAgICAgICAgICAgIGlmIChkZWx0YSA+IDAuMTUgKiBjb3VudCkKKyAgICAgICAg
ICAgICAgICBkZWJ1ZygiUEFTUyBmcmVlZCBtb3JlIHRoYW4gMTUlIG9mIGRvY3VtZW50cyIpOwog
ICAgICAgICAgICAgZWxzZQotICAgICAgICAgICAgICAgIGRlYnVnKCJGQUlMIGZyZWVkIGZld2Vy
IHRoYW4gNjAgZG9jdW1lbnRzIik7CisgICAgICAgICAgICAgICAgZGVidWcoIkZBSUwgZnJlZWQg
bGVzcyB0aGFuIDE1JSBvZiBkb2N1bWVudHMiKTsKICAgICAgICAgICAgIGZpbmlzaEpTVGVzdCgp
OyAgICAgICAgICAgICAgICAKICAgICAgICAgfTsKICAgICB9Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>