<?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>200649</bug_id>
          
          <creation_ts>2019-08-12 17:02:06 -0700</creation_ts>
          <short_desc>Add a way to opt out of kern TCSM for layout tests</short_desc>
          <delta_ts>2019-08-13 20:38:15 -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>JavaScriptCore</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</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="Saam Barati">saam</reporter>
          <assigned_to name="Saam Barati">saam</assigned_to>
          <cc>ap</cc>
    
    <cc>benjamin</cc>
    
    <cc>commit-queue</cc>
    
    <cc>fpizlo</cc>
    
    <cc>ggaren</cc>
    
    <cc>gskachkov</cc>
    
    <cc>guijemont</cc>
    
    <cc>jbedard</cc>
    
    <cc>keith_miller</cc>
    
    <cc>mark.lam</cc>
    
    <cc>msaboff</cc>
    
    <cc>rmorisset</cc>
    
    <cc>ticaiolima</cc>
    
    <cc>tzagallo</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>ysuzuki</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1560611</commentid>
    <comment_count>0</comment_count>
    <who name="Saam Barati">saam</who>
    <bug_when>2019-08-12 17:02:06 -0700</bug_when>
    <thetext>...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1560612</commentid>
    <comment_count>1</comment_count>
    <who name="Saam Barati">saam</who>
    <bug_when>2019-08-12 17:02:38 -0700</bug_when>
    <thetext>&lt;rdar://problem/51304923&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1560655</commentid>
    <comment_count>2</comment_count>
      <attachid>376127</attachid>
    <who name="Saam Barati">saam</who>
    <bug_when>2019-08-12 19:06:13 -0700</bug_when>
    <thetext>Created attachment 376127
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1560670</commentid>
    <comment_count>3</comment_count>
      <attachid>376127</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2019-08-12 20:26:43 -0700</bug_when>
    <thetext>Comment on attachment 376127
patch

How does one pass an option of this kind to WebKitTestRunner’s WebContent process?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1560972</commentid>
    <comment_count>4</comment_count>
    <who name="Saam Barati">saam</who>
    <bug_when>2019-08-13 15:06:02 -0700</bug_when>
    <thetext>(In reply to Alexey Proskuryakov from comment #3)
&gt; Comment on attachment 376127 [details]
&gt; patch
&gt; 
&gt; How does one pass an option of this kind to WebKitTestRunner’s WebContent
&gt; process?

Set the environment variable &quot;JSC_useKernTCSM=false&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561006</commentid>
    <comment_count>5</comment_count>
      <attachid>376223</attachid>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2019-08-13 16:33:37 -0700</bug_when>
    <thetext>Created attachment 376223
set-environment-webkitpy

Should we be setting this webkitpy? Or in WebKitTestRunner/DumpRenderTree?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561009</commentid>
    <comment_count>6</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2019-08-13 17:10:05 -0700</bug_when>
    <thetext>&gt; Should we be setting this webkitpy? Or in WebKitTestRunner/DumpRenderTree?

I don&apos;t think that there is any practical difference. Theoretically, setting it in WebKitTestRunner/DumpRenderTree is slightly better because running them manually (without run-webkit-tests) will result in the same behavior as with run-webkit-tests.

In addition to layout tests, JSC tests probably need this too.

Also, we should probably use a separate bug for the adoption.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561058</commentid>
    <comment_count>7</comment_count>
      <attachid>376127</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-08-13 20:38:13 -0700</bug_when>
    <thetext>Comment on attachment 376127
patch

Clearing flags on attachment: 376127

Committed r248661: &lt;https://trac.webkit.org/changeset/248661&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561059</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-08-13 20:38:15 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>376127</attachid>
            <date>2019-08-12 19:06:13 -0700</date>
            <delta_ts>2019-08-13 20:38:13 -0700</delta_ts>
            <desc>patch</desc>
            <filename>a-backup.diff</filename>
            <type>text/plain</type>
            <size>2087</size>
            <attacher name="Saam Barati">saam</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMjQ4NTgzKQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE1IEBA
CisyMDE5LTA4LTEyICBTYWFtIEJhcmF0aSAgPHNiYXJhdGlAYXBwbGUuY29tPgorCisgICAgICAg
IEFkZCBhIHdheSB0byBvcHQgb3V0IG9mIGtlcm4gVENTTSBmb3IgbGF5b3V0IHRlc3RzCisgICAg
ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMDA2NDkKKyAgICAg
ICAgPHJkYXI6Ly9wcm9ibGVtLzUxMzA0OTIzPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E
WSAoT09QUyEpLgorCisgICAgICAgICogYXNzZW1ibGVyL0NQVS5jcHA6CisgICAgICAgIChKU0M6
OmlzS2VyblRDU01BdmFpbGFibGUpOgorICAgICAgICAqIHJ1bnRpbWUvT3B0aW9ucy5oOgorCiAy
MDE5LTA4LTEyICBTYW0gV2VpbmlnICA8d2VpbmlnQGFwcGxlLmNvbT4KIAogICAgICAgICBSZXBs
YWNlIG11bHRpcGFyYW1ldGVyIG92ZXJsb2FkcyBvZiBhcHBlbmQoKSBpbiBTdHJpbmdCdWlsZGVy
IGFzIGEgZmlyc3Qgc3RlcCB0b3dhcmQgc3RhbmRhcmRpemluZ2luZyBvbiB0aGUgZmxleGlibGVB
cHBlbmQoKSBpbXBsZW1lbnRhdGlvbgpJbmRleDogU291cmNlL0phdmFTY3JpcHRDb3JlL2Fzc2Vt
Ymxlci9DUFUuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9hc3NlbWJs
ZXIvQ1BVLmNwcAkocmV2aXNpb24gMjQ4NTgzKQorKysgU291cmNlL0phdmFTY3JpcHRDb3JlL2Fz
c2VtYmxlci9DUFUuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0zNiw2ICszNiw5IEBAIG5hbWVzcGFj
ZSBKU0MgewogI2lmIChDUFUoWDg2KSB8fCBDUFUoWDg2XzY0KSkgJiYgT1MoREFSV0lOKQogYm9v
bCBpc0tlcm5UQ1NNQXZhaWxhYmxlKCkKIHsKKyAgICBpZiAoIU9wdGlvbnM6OnVzZUtlcm5UQ1NN
KCkpCisgICAgICAgIHJldHVybiBmYWxzZTsKKwogICAgIHVpbnQzMl90IHZhbCA9IDA7CiAgICAg
c2l6ZV90IHZhbFNpemUgPSBzaXplb2YodmFsKTsKICAgICBpbnQgcmMgPSBzeXNjdGxieW5hbWUo
Imtlcm4udGNzbV9hdmFpbGFibGUiLCAmdmFsLCAmdmFsU2l6ZSwgTlVMTCwgMCk7CkluZGV4OiBT
b3VyY2UvSmF2YVNjcmlwdENvcmUvcnVudGltZS9PcHRpb25zLmgKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL3J1bnRpbWUvT3B0aW9ucy5oCShyZXZpc2lvbiAyNDg1ODMpCisr
KyBTb3VyY2UvSmF2YVNjcmlwdENvcmUvcnVudGltZS9PcHRpb25zLmgJKHdvcmtpbmcgY29weSkK
QEAgLTExMyw2ICsxMTMsNyBAQCBjb25zdGV4cHIgYm9vbCBlbmFibGVXZWJBc3NlbWJseVN0cmVh
bWluCiAjZW5kaWYKIAogI2RlZmluZSBKU0NfT1BUSU9OUyh2KSBcCisgICAgdihib29sLCB1c2VL
ZXJuVENTTSwgdHJ1ZSwgTm9ybWFsLCAiTm90ZTogdGhpcyBuZWVkcyB0byBnbyBiZWZvcmUgb3Ro
ZXIgb3B0aW9ucyBzaW5jZSB0aGV5IGRlcGVuZCBvbiB0aGlzIHZhbHVlLiIpIFwKICAgICB2KGJv
b2wsIHZhbGlkYXRlT3B0aW9ucywgZmFsc2UsIE5vcm1hbCwgImNyYXNoZXMgaWYgbWlzLXR5cGVk
IEpTQyBvcHRpb25zIHdlcmUgcGFzc2VkIHRvIHRoZSBWTSIpIFwKICAgICB2KHVuc2lnbmVkLCBk
dW1wT3B0aW9ucywgMCwgTm9ybWFsLCAiZHVtcHMgSlNDIG9wdGlvbnMgKDAgPSBOb25lLCAxID0g
T3ZlcnJpZGRlbiBvbmx5LCAyID0gQWxsLCAzID0gVmVyYm9zZSkiKSBcCiAgICAgdihvcHRpb25T
dHJpbmcsIGNvbmZpZ0ZpbGUsIG51bGxwdHIsIE5vcm1hbCwgImZpbGUgdG8gY29uZmlndXJlIEpT
QyBvcHRpb25zIGFuZCBsb2dnaW5nIGxvY2F0aW9uIikgXAo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>376223</attachid>
            <date>2019-08-13 16:33:37 -0700</date>
            <delta_ts>2019-08-13 16:33:37 -0700</delta_ts>
            <desc>set-environment-webkitpy</desc>
            <filename>patch.patch</filename>
            <type>text/plain</type>
            <size>1556</size>
            <attacher name="Jonathan Bedard">jbedard</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDI0ODY0MSkKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDE0IEBACisyMDE5LTA4LTEzICBKb25hdGhhbiBCZWRhcmQgIDxqYmVkYXJkQGFwcGxlLmNv
bT4KKworICAgICAgICBBZGQgYSB3YXkgdG8gb3B0IG91dCBvZiBrZXJuIFRDU00gZm9yIGxheW91
dCB0ZXN0cworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9
MjAwNjQ5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg
KiBTY3JpcHRzL3dlYmtpdHB5L3BvcnQvZHJpdmVyLnB5OgorICAgICAgICAoRHJpdmVyLl9zZXR1
cF9lbnZpcm9uX2Zvcl9kcml2ZXIpOiBEaXNhYmxlIFRDU00gaW4gSmF2YVNjcmlwdENvcmUgd2hl
biBydW5uaW5nCisgICAgICAgIGxheW91dCB0ZXN0cy4KKwogMjAxOS0wOC0xMyAgSm9obiBXaWxh
bmRlciAgPHdpbGFuZGVyQGFwcGxlLmNvbT4KIAogICAgICAgICBSZXNvdXJjZSBMb2FkIFN0YXRp
c3RpY3M6IFN3aXRjaCBOU1VSTFNlc3Npb24gb24gdG9wIG5hdmlnYXRpb24gdG8gcHJldmFsZW50
IHJlc291cmNlIHdpdGggdXNlciBpbnRlcmFjdGlvbgpJbmRleDogVG9vbHMvU2NyaXB0cy93ZWJr
aXRweS9wb3J0L2RyaXZlci5weQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9TY3JpcHRzL3dlYmtpdHB5
L3BvcnQvZHJpdmVyLnB5CShyZXZpc2lvbiAyNDg2NDEpCisrKyBUb29scy9TY3JpcHRzL3dlYmtp
dHB5L3BvcnQvZHJpdmVyLnB5CSh3b3JraW5nIGNvcHkpCkBAIC00MzQsNiArNDM0LDkgQEAgY2xh
c3MgRHJpdmVyKG9iamVjdCk6CiAgICAgICAgIGVudmlyb25tZW50WydTUUxJVEVfRVhFTVBUX1BB
VEhfRlJPTV9WTk9ERV9HVUFSRFMnXSA9IG9zLnBhdGgucmVhbHBhdGgoZW52aXJvbm1lbnRbJ0RV
TVBSRU5ERVJUUkVFX1RFTVAnXSkKICAgICAgICAgZW52aXJvbm1lbnRbJ19fWFBDX1NRTElURV9F
WEVNUFRfUEFUSF9GUk9NX1ZOT0RFX0dVQVJEUyddID0gZW52aXJvbm1lbnRbJ1NRTElURV9FWEVN
UFRfUEFUSF9GUk9NX1ZOT0RFX0dVQVJEUyddCiAKKyAgICAgICAgZW52aXJvbm1lbnRbJ0pTQ191
c2VLZXJuVENTTSddID0gJzAnCisgICAgICAgIGVudmlyb25tZW50WydfX1hQQ19KU0NfdXNlS2Vy
blRDU00nXSA9ICcwJworCiAgICAgICAgIGlmIHN5cy5wbGF0Zm9ybS5zdGFydHN3aXRoKCdsaW51
eCcpOgogICAgICAgICAgICAgIyBDdXJyZW50bHkgb24gV2ViS2l0MiwgdGhlcmUgaXMgbm8gQVBJ
IGZvciBzZXR0aW5nIHRoZSBhcHBsaWNhdGlvbiBjYWNoZSBkaXJlY3RvcnkuCiAgICAgICAgICAg
ICAjIEVhY2ggd29ya2VyIHNob3VsZCBoYXZlIGl0J3Mgb3duIGFuZCBpdCBzaG91bGQgYmUgY2xl
YW5lZCBhZnRlcndhcmRzLgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>