<?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>237181</bug_id>
          
          <creation_ts>2022-02-24 20:28:58 -0800</creation_ts>
          <short_desc>[GTK] The setting WebKitMinimumFontSize is not reset by the testing framework, leading to flaky tests</short_desc>
          <delta_ts>2022-04-27 12:05:13 -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>Tools / Tests</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</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="Arcady Goldmints-Orlov">crzwdjk</reporter>
          <assigned_to name="Arcady Goldmints-Orlov">crzwdjk</assigned_to>
          <cc>cgarcia</cc>
    
    <cc>changseok</cc>
    
    <cc>lmoura</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1845896</commentid>
    <comment_count>0</comment_count>
    <who name="Arcady Goldmints-Orlov">crzwdjk</who>
    <bug_when>2022-02-24 20:28:58 -0800</bug_when>
    <thetext>The test fast/forms/validation-message-minimum-font-size.html uses the setting WebKitMinimumFontSize, which on GTK does not appear to be getting reset between tests, leading to failures in tests that run in the same test runner process, which results in test flakiness. The code to set this setting on GTK was introduced by r288666.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1846836</commentid>
    <comment_count>1</comment_count>
    <who name="ChangSeok Oh">changseok</who>
    <bug_when>2022-02-28 16:29:08 -0800</bug_when>
    <thetext>(In reply to Arcady Goldmints-Orlov from comment #0)
&gt; The test fast/forms/validation-message-minimum-font-size.html uses the
&gt; setting WebKitMinimumFontSize, which on GTK does not appear to be getting
&gt; reset between tests, leading to failures in tests that run in the same test
&gt; runner process, which results in test flakiness. The code to set this
&gt; setting on GTK was introduced by r288666.

May I ask what tests are flaky in particular?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1846838</commentid>
    <comment_count>2</comment_count>
    <who name="Arcady Goldmints-Orlov">crzwdjk</who>
    <bug_when>2022-02-28 16:43:10 -0800</bug_when>
    <thetext>If you are looking to reproduce it, the command I have been using is:

Tools/Scripts/run-webkit-tests --no-new --child-processes=1 --no-show fast/forms/validation-message-minimum-font-size.html fast/text/international</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1857706</commentid>
    <comment_count>3</comment_count>
    <who name="ChangSeok Oh">changseok</who>
    <bug_when>2022-04-03 15:20:50 -0700</bug_when>
    <thetext>(In reply to Arcady Goldmints-Orlov from comment #2)
&gt; If you are looking to reproduce it, the command I have been using is:
&gt; 
&gt; Tools/Scripts/run-webkit-tests --no-new --child-processes=1 --no-show
&gt; fast/forms/validation-message-minimum-font-size.html fast/text/international

Unfortunately, I could not reproduce the issue in two different Linux distributions: Ubuntu 21.10 and Fedora 35.

In my investigation, a WebPreferences instance is ephemeral so each test creates its own WebPreferences rather than sharing one among tests.

Would you test if the attached hotfix resolves your issue?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1857707</commentid>
    <comment_count>4</comment_count>
      <attachid>456521</attachid>
    <who name="ChangSeok Oh">changseok</who>
    <bug_when>2022-04-03 15:21:42 -0700</bug_when>
    <thetext>Created attachment 456521
A hotfix</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1864853</commentid>
    <comment_count>5</comment_count>
    <who name="Arcady Goldmints-Orlov">crzwdjk</who>
    <bug_when>2022-04-27 11:14:04 -0700</bug_when>
    <thetext>This was fixed by r292681, I will remove the line from the TestExpectation.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1864859</commentid>
    <comment_count>6</comment_count>
      <attachid>458455</attachid>
    <who name="Arcady Goldmints-Orlov">crzwdjk</who>
    <bug_when>2022-04-27 11:37:10 -0700</bug_when>
    <thetext>Created attachment 458455
[fast-cq] Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1864873</commentid>
    <comment_count>7</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-04-27 12:04:23 -0700</bug_when>
    <thetext>Committed r293523 (250054@main): &lt;https://commits.webkit.org/250054@main&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 458455.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1864874</commentid>
    <comment_count>8</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2022-04-27 12:05:13 -0700</bug_when>
    <thetext>&lt;rdar://problem/92412612&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>456521</attachid>
            <date>2022-04-03 15:21:42 -0700</date>
            <delta_ts>2022-04-27 11:37:06 -0700</delta_ts>
            <desc>A hotfix</desc>
            <filename>hotfix.patch</filename>
            <type>text/plain</type>
            <size>1052</size>
            <attacher name="ChangSeok Oh">changseok</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL1dlYktpdFRlc3RSdW5uZXIvVGVzdENvbnRyb2xsZXIuY3BwIGIv
VG9vbHMvV2ViS2l0VGVzdFJ1bm5lci9UZXN0Q29udHJvbGxlci5jcHAKaW5kZXggYTAxY2VhNDI3
NzFhLi4yZTRkMjQyZjc4NTEgMTAwNjQ0Ci0tLSBhL1Rvb2xzL1dlYktpdFRlc3RSdW5uZXIvVGVz
dENvbnRyb2xsZXIuY3BwCisrKyBiL1Rvb2xzL1dlYktpdFRlc3RSdW5uZXIvVGVzdENvbnRyb2xs
ZXIuY3BwCkBAIC05Niw2ICs5Niw4IEBACiAjaW5jbHVkZSA8d3RmL3RleHQvQ1N0cmluZy5oPgog
I2luY2x1ZGUgPHd0Zi90ZXh0L1N0cmluZ0NvbmNhdGVuYXRlTnVtYmVycy5oPgogCisjaW5jbHVk
ZSAiV2ViUHJlZmVyZW5jZXNEZWZpbml0aW9ucy5oIgorCiAjaWYgUExBVEZPUk0oQ09DT0EpCiAj
aW5jbHVkZSA8V2ViS2l0L1dLQ29udGV4dFByaXZhdGVNYWMuaD4KICNpbmNsdWRlIDxXZWJLaXQv
V0tQYWdlUHJpdmF0ZU1hYy5oPgpAQCAtOTU1LDYgKzk1Nyw5IEBAIHRlbXBsYXRlPHR5cGVuYW1l
IEY+IHN0YXRpYyB2b2lkIGJhdGNoVXBkYXRlUHJlZmVyZW5jZXMoV0tQcmVmZXJlbmNlc1JlZiBw
cmVmZXJlCiAKIHZvaWQgVGVzdENvbnRyb2xsZXI6OnJlc2V0UHJlZmVyZW5jZXNUb0NvbnNpc3Rl
bnRWYWx1ZXMoY29uc3QgVGVzdE9wdGlvbnMmIG9wdGlvbnMpCiB7CisgICAgaWYgKFdLUHJlZmVy
ZW5jZXNHZXRNaW5pbXVtRm9udFNpemUocGxhdGZvcm1QcmVmZXJlbmNlcygpKSkKKyAgICAgICAg
V0tQcmVmZXJlbmNlc1NldE1pbmltdW1Gb250U2l6ZShwbGF0Zm9ybVByZWZlcmVuY2VzKCksIGRv
dWJsZShERUZBVUxUX1ZBTFVFX0ZPUl9NaW5pbXVtRm9udFNpemUpKTsKKwogICAgIGJhdGNoVXBk
YXRlUHJlZmVyZW5jZXMocGxhdGZvcm1QcmVmZXJlbmNlcygpLCBbb3B0aW9ucywgZW5hYmxlQWxs
RXhwZXJpbWVudGFsRmVhdHVyZXMgPSBtX2VuYWJsZUFsbEV4cGVyaW1lbnRhbEZlYXR1cmVzXSAo
YXV0byBwcmVmZXJlbmNlcykgewogICAgICAgICBXS1ByZWZlcmVuY2VzUmVzZXRUZXN0UnVubmVy
T3ZlcnJpZGVzKHByZWZlcmVuY2VzKTsKIAo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>458455</attachid>
            <date>2022-04-27 11:37:10 -0700</date>
            <delta_ts>2022-04-27 12:04:24 -0700</delta_ts>
            <desc>[fast-cq] Patch</desc>
            <filename>bug-237181-20220427143709.patch</filename>
            <type>text/plain</type>
            <size>1534</size>
            <attacher name="Arcady Goldmints-Orlov">crzwdjk</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjkzNDk1CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu
Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggY2MzZjhhNDYwNmY1YmVlODYxNmE5
ODY4MDIzNTU1NDk4NjQxOTUxYS4uNWEwNmQyMDJkYzJiMTIxYzU4OTk5OThiNjdhZjVhN2M2MmRi
ZDE5YSAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz
L0NoYW5nZUxvZwpAQCAtMSwzICsxLDEyIEBACisyMDIyLTA0LTI3ICBBcmNhZHkgR29sZG1pbnRz
LU9ybG92ICA8YWdvbGRtaW50c0BpZ2FsaWEuY29tPgorCisgICAgICAgIFtHVEtdIFRoZSBzZXR0
aW5nIFdlYktpdE1pbmltdW1Gb250U2l6ZSBpcyBub3QgcmVzZXQgYnkgdGhlIHRlc3RpbmcgZnJh
bWV3b3JrLCBsZWFkaW5nIHRvIGZsYWt5IHRlc3RzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr
aXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMzcxODEKKworICAgICAgICBVbnJldmlld2VkIHRlc3Qg
Z2FyZGVuaW5nLgorCisgICAgICAgICogcGxhdGZvcm0vZ3RrL1Rlc3RFeHBlY3RhdGlvbnM6IFJl
bW92ZSBza2lwIG5vdyB0aGF0IHRoZSB0ZXN0IGRvZXNuJ3QgY2F1c2UgcHJvYmxlbXMuCisKIDIw
MjItMDQtMjYgIFRydWl0dCBTYXZlbGwgIDx0c2F2ZWxsQGFwcGxlLmNvbT4KIAogICAgICAgICBD
bGVhbnVwIGV4cGVjdGF0aW9ucyBmb3IgNyBpbXBvcnRlZC93M2Mvd2ViLXBsYXRmb3JtLXRlc3Rz
L2h0bWwvc2VtYW50aWNzL2ludGVyYWN0aXZlLWVsZW1lbnRzLyB0ZXN0cwpkaWZmIC0tZ2l0IGEv
TGF5b3V0VGVzdHMvcGxhdGZvcm0vZ3RrL1Rlc3RFeHBlY3RhdGlvbnMgYi9MYXlvdXRUZXN0cy9w
bGF0Zm9ybS9ndGsvVGVzdEV4cGVjdGF0aW9ucwppbmRleCAwMzI5ZjcyY2NhMGUxYjAwOGZhNjE0
YTI2NGU5NzAwNjYwZDg3ZjNkLi44NzFkZWJjNTkwMThkMTI5YTgwOTI4YzM3NTQ1ZmIzZmY5MWU3
ZTRmIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9ndGsvVGVzdEV4cGVjdGF0aW9u
cworKysgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9ndGsvVGVzdEV4cGVjdGF0aW9ucwpAQCAtMTk5
LDkgKzE5OSw2IEBAIHdlYmtpdC5vcmcvYi8yMDg5ODggd2VieHIgWyBTa2lwIF0KICMgQ3Jhc2gg
b24gRVdTIGJvdC4KIHdlYnJ0Yy92cDgtdGhlbi1oMjY0LWdwdS1wcm9jZXNzLWNyYXNoLmh0bWwg
WyBTa2lwIF0KIAotIyBUaGlzIHRlc3QgaW50cm9kdWNlcyBmbGFraW5lc3MgZHVlIHRvIGlzc3Vl
cyB3aXRoIHJlc2V0dGluZyBXZWJQcmVmZXJlbmNlcy4KLXdlYmtpdC5vcmcvYi8yMzcxODEgZmFz
dC9mb3Jtcy92YWxpZGF0aW9uLW1lc3NhZ2UtbWluaW11bS1mb250LXNpemUuaHRtbCBbIFNraXAg
XQotCiAjLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8v
Ly8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vCiAjIEVuZCBvZiBUcmlhZ2Vk
IEV4cGVjdGF0aW9ucwogIyBMZWdhY3kgRXhwZWN0YXRpb25zIHNlY3Rpb25zIGJlbG93Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>