<?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>228987</bug_id>
          
          <creation_ts>2021-08-11 02:20:43 -0700</creation_ts>
          <short_desc>ANGLE Cocoa compiles contents of ContextEAGL on mac</short_desc>
          <delta_ts>2021-08-12 06:02:19 -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>ANGLE</component>
          <version>WebKit Local 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="Kimmo Kinnunen">kkinnunen</reporter>
          <assigned_to name="Kimmo Kinnunen">kkinnunen</assigned_to>
          <cc>dino</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>kbr</cc>
    
    <cc>kkinnunen</cc>
    
    <cc>kondapallykalyan</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1783427</commentid>
    <comment_count>0</comment_count>
    <who name="Kimmo Kinnunen">kkinnunen</who>
    <bug_when>2021-08-11 02:20:43 -0700</bug_when>
    <thetext>ANGLE Cocoa compiles contents ContextEAGL on Mac, guards missing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1783428</commentid>
    <comment_count>1</comment_count>
      <attachid>435330</attachid>
    <who name="Kimmo Kinnunen">kkinnunen</who>
    <bug_when>2021-08-11 02:22:35 -0700</bug_when>
    <thetext>Created attachment 435330
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1783429</commentid>
    <comment_count>2</comment_count>
    <who name="EWS Watchlist">ews-watchlist</who>
    <bug_when>2021-08-11 02:23:25 -0700</bug_when>
    <thetext>Note that there are important steps to take when updating ANGLE. See https://trac.webkit.org/wiki/UpdatingANGLE</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1783596</commentid>
    <comment_count>3</comment_count>
      <attachid>435330</attachid>
    <who name="Kenneth Russell">kbr</who>
    <bug_when>2021-08-11 12:30:13 -0700</bug_when>
    <thetext>Comment on attachment 435330
Patch

Looks good to me. We need to be sure to either upstream this or avoid losing it while rolling ANGLE back down into WebKit. Do you want to point to this on https://bugs.chromium.org/p/angleproject/issues/detail?id=5505 ?

r+</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1783611</commentid>
    <comment_count>4</comment_count>
      <attachid>435330</attachid>
    <who name="Dean Jackson">dino</who>
    <bug_when>2021-08-11 13:15:26 -0700</bug_when>
    <thetext>Comment on attachment 435330
Patch

Before we can land this - please check that it doesn&apos;t break Catalyst.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1783746</commentid>
    <comment_count>5</comment_count>
    <who name="Kimmo Kinnunen">kkinnunen</who>
    <bug_when>2021-08-11 23:29:28 -0700</bug_when>
    <thetext>Upstreaming bug: https://bugs.chromium.org/p/angleproject/issues/detail?id=6271</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1783803</commentid>
    <comment_count>6</comment_count>
    <who name="Kimmo Kinnunen">kkinnunen</who>
    <bug_when>2021-08-12 05:57:48 -0700</bug_when>
    <thetext>(In reply to Dean Jackson from comment #4)
&gt; Comment on attachment 435330 [details]
&gt; Patch
&gt; 
&gt; Before we can land this - please check that it doesn&apos;t break Catalyst.

I checked that it compiles on Catalyst. The Catalyst doesn&apos;t yet have tests so I didn&apos;t spend time building an app to verify that WebGL works. 

The condition itself is same as in other files. 

Before, the condition was in 4 EAGL files and missing in 1 EAGL file.
After, the condition was in 5 EAGL files.

Based on this, I&apos;ll land.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1783804</commentid>
    <comment_count>7</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2021-08-12 06:01:34 -0700</bug_when>
    <thetext>Committed r280962 (240468@main): &lt;https://commits.webkit.org/240468@main&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 435330.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1783805</commentid>
    <comment_count>8</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2021-08-12 06:02:19 -0700</bug_when>
    <thetext>&lt;rdar://problem/81844871&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>435330</attachid>
            <date>2021-08-11 02:22:35 -0700</date>
            <delta_ts>2021-08-12 06:01:35 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-228987-20210811122233.patch</filename>
            <type>text/plain</type>
            <size>1559</size>
            <attacher name="Kimmo Kinnunen">kkinnunen</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjgwODgzCmRpZmYgLS1naXQgYS9Tb3VyY2UvVGhpcmRQYXJ0
eS9BTkdMRS9DaGFuZ2VMb2cgYi9Tb3VyY2UvVGhpcmRQYXJ0eS9BTkdMRS9DaGFuZ2VMb2cKaW5k
ZXggN2Y1NzgwODNhZmRmOTU5OTJkMDdiOWVhNjM4ODcwMzk1ODRiNWEyOS4uYTY1NjFjYmI2NThl
NTkyOTUzMDY1NzkyNjZiY2UzZmIxMzJhYTgzZSAxMDA2NDQKLS0tIGEvU291cmNlL1RoaXJkUGFy
dHkvQU5HTEUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9UaGlyZFBhcnR5L0FOR0xFL0NoYW5nZUxv
ZwpAQCAtMSwzICsxLDE0IEBACisyMDIxLTA4LTExICBLaW1tbyBLaW5udW5lbiAgPGtraW5udW5l
bkBhcHBsZS5jb20+CisKKyAgICAgICAgQU5HTEUgQ29jb2EgY29tcGlsZXMgY29udGVudHMgb2Yg
Q29udGV4dEVBR0wgb24gbWFjCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3df
YnVnLmNnaT9pZD0yMjg5ODcKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4K
KworICAgICAgICBBZGQgaW5jbHVkZSBndWFyZHMgdGhhdCBhcmUgY29uc2lzdGVudCB3aXRoIG90
aGVyIEVBR0wgZmlsZXMuCisKKyAgICAgICAgKiBzcmMvbGliQU5HTEUvcmVuZGVyZXIvZ2wvZWFn
bC9Db250ZXh0RUFHTC5jcHA6CisKIDIwMjEtMDgtMTEgIEtpbW1vIEtpbm51bmVuICA8a2tpbm51
bmVuQGFwcGxlLmNvbT4KIAogICAgICAgICBBTkdMRSBDb2NvYSBjb21waWxlcyBwYXJ0cyBvZiBI
TFNMIHRyYW5zbGF0b3IsIHZ1bGthbiB0cmFuc2xhdG9yLCBsaWJnbCBhcGksIGNhcHR1cmUgcmVk
dW5kYW50bHkKZGlmZiAtLWdpdCBhL1NvdXJjZS9UaGlyZFBhcnR5L0FOR0xFL3NyYy9saWJBTkdM
RS9yZW5kZXJlci9nbC9lYWdsL0NvbnRleHRFQUdMLmNwcCBiL1NvdXJjZS9UaGlyZFBhcnR5L0FO
R0xFL3NyYy9saWJBTkdMRS9yZW5kZXJlci9nbC9lYWdsL0NvbnRleHRFQUdMLmNwcAppbmRleCBk
MjE0ODUwOTFlMGZmZTA0NjVhMGI3MDczZWE5NzE3MDM5NWE1MjU5Li5lMTAwNTBlZjRlMzk3YjZi
YjU5ODk1MWZjYTY4MWQ1MWUyYjQyYzhlIDEwMDY0NAotLS0gYS9Tb3VyY2UvVGhpcmRQYXJ0eS9B
TkdMRS9zcmMvbGliQU5HTEUvcmVuZGVyZXIvZ2wvZWFnbC9Db250ZXh0RUFHTC5jcHAKKysrIGIv
U291cmNlL1RoaXJkUGFydHkvQU5HTEUvc3JjL2xpYkFOR0xFL3JlbmRlcmVyL2dsL2VhZ2wvQ29u
dGV4dEVBR0wuY3BwCkBAIC04LDYgKzgsMTAgQEAKIC8vICAgaU9TLXNwZWNpZmljIHN1YmNsYXNz
IG9mIENvbnRleHRHTC4KIC8vCiAKKyNpbXBvcnQgImNvbW1vbi9wbGF0Zm9ybS5oIgorCisjaWYg
ZGVmaW5lZChBTkdMRV9FTkFCTEVfRUFHTCkKKwogI2luY2x1ZGUgImxpYkFOR0xFL3JlbmRlcmVy
L2dsL2VhZ2wvQ29udGV4dEVBR0wuaCIKIAogI2luY2x1ZGUgImxpYkFOR0xFL0NvbnRleHQuaCIK
QEAgLTI0LDMgKzI4LDUgQEAgQ29udGV4dEVBR0w6OkNvbnRleHRFQUdMKGNvbnN0IGdsOjpTdGF0
ZSAmc3RhdGUsCiB7fQogCiB9ICAvLyBuYW1lc3BhY2UgcngKKworI2VuZGlmICAvLyBkZWZpbmVk
KEFOR0xFX0VOQUJMRV9FQUdMKQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>