<?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>229845</bug_id>
          
          <creation_ts>2021-09-02 18:52:41 -0700</creation_ts>
          <short_desc>Fix for debug mode with ENABLE_SERVICE_WORKER=OFF after r281706</short_desc>
          <delta_ts>2021-09-03 09:15:26 -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>WebKit Misc.</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="Stephan Szabo">stephan.szabo</reporter>
          <assigned_to name="Stephan Szabo">stephan.szabo</assigned_to>
          <cc>achristensen</cc>
    
    <cc>cdumez</cc>
    
    <cc>don.olmstead</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1790012</commentid>
    <comment_count>0</comment_count>
    <who name="Stephan Szabo">stephan.szabo</who>
    <bug_when>2021-09-02 18:52:41 -0700</bug_when>
    <thetext>There&apos;s an ASSERT that used m_serviceWorkerFetchTask outside of an #if ENABLE(SERVICE_WORKER) block.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1790017</commentid>
    <comment_count>1</comment_count>
      <attachid>437231</attachid>
    <who name="Stephan Szabo">stephan.szabo</who>
    <bug_when>2021-09-02 18:57:50 -0700</bug_when>
    <thetext>Created attachment 437231
Patch

I&apos;m not sure stylistically how we want to handle cases like this, so just put the only sometimes enabled bit on its own inside the if and made the assert go across lines.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1790195</commentid>
    <comment_count>2</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2021-09-03 09:14:37 -0700</bug_when>
    <thetext>Committed r281998 (241305@main): &lt;https://commits.webkit.org/241305@main&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 437231.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1790197</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2021-09-03 09:15:26 -0700</bug_when>
    <thetext>&lt;rdar://problem/82726075&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>437231</attachid>
            <date>2021-09-02 18:57:50 -0700</date>
            <delta_ts>2021-09-03 09:14:38 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>229845.1.diff</filename>
            <type>text/plain</type>
            <size>1652</size>
            <attacher name="Stephan Szabo">stephan.szabo</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9DaGFu
Z2VMb2cKaW5kZXggZDNiZjk0NTRhZTYxLi5hMzNjZGY0MzIyZTMgMTAwNjQ0Ci0tLSBhL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMg
KzEsMTYgQEAKKzIwMjEtMDktMDIgIFN0ZXBoYW4gU3phYm8gIDxzdGVwaGFuLnN6YWJvQHNvbnku
Y29tPgorCisgICAgICAgIEZpeCBmb3IgZGVidWcgbW9kZSB3aXRoIEVOQUJMRV9TRVJWSUNFX1dP
UktFUj1PRkYgYWZ0ZXIgcjI4MTcwNgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9MjI5ODQ1CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgUHV0IG1fc2VydmljZVdvcmtlckZldGNoVGFzayBjaGVjayBpbnRvICNp
Zi4KKworICAgICAgICBObyBuZXcgdGVzdHMsIGlzIGEgYnVpbGQgZml4IGZvciBoYXZpbmcgc2Vy
dmljZSB3b3JrZXJzIG9mZi4KKworICAgICAgICAqIE5ldHdvcmtQcm9jZXNzL05ldHdvcmtSZXNv
dXJjZUxvYWRlci5jcHA6CisKIDIwMjEtMDktMDIgIEFkaXR5YSBLZWVydGhpICA8YWtlZXJ0aGlA
YXBwbGUuY29tPgogCiAgICAgICAgIFtpT1NdIFNpbXBsaWZ5IGRhdGUgcGlja2VyIGxvZ2ljIGZv
ciBkYXRldGltZS1sb2NhbCBpbnB1dHMKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvTmV0d29y
a1Byb2Nlc3MvTmV0d29ya1Jlc291cmNlTG9hZGVyLmNwcCBiL1NvdXJjZS9XZWJLaXQvTmV0d29y
a1Byb2Nlc3MvTmV0d29ya1Jlc291cmNlTG9hZGVyLmNwcAppbmRleCAzZjliMWU1MzhkMzkuLjUz
M2JhYjM3Y2Y0ZCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9OZXR3b3JrUHJvY2Vzcy9OZXR3
b3JrUmVzb3VyY2VMb2FkZXIuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQvTmV0d29ya1Byb2Nlc3Mv
TmV0d29ya1Jlc291cmNlTG9hZGVyLmNwcApAQCAtNTI3LDcgKzUyNywxMSBAQCB2b2lkIE5ldHdv
cmtSZXNvdXJjZUxvYWRlcjo6dHJhbnNmZXJUb05ld1dlYlByb2Nlc3MoTmV0d29ya0Nvbm5lY3Rp
b25Ub1dlYlByb2NlcwogICAgIG1fY29ubmVjdGlvbiA9IG5ld0Nvbm5lY3Rpb247CiAgICAgbV9w
YXJhbWV0ZXJzLmlkZW50aWZpZXIgPSBuZXdDb3JlSWRlbnRpZmllcjsKIAotICAgIEFTU0VSVCht
X3Jlc3BvbnNlQ29tcGxldGlvbkhhbmRsZXIgfHwgbV9jYWNoZUVudHJ5V2FpdGluZ0ZvckNvbnRp
bnVlRGlkUmVjZWl2ZVJlc3BvbnNlIHx8IG1fc2VydmljZVdvcmtlckZldGNoVGFzayk7CisgICAg
QVNTRVJUKG1fcmVzcG9uc2VDb21wbGV0aW9uSGFuZGxlciB8fCBtX2NhY2hlRW50cnlXYWl0aW5n
Rm9yQ29udGludWVEaWRSZWNlaXZlUmVzcG9uc2UKKyNpZiBFTkFCTEUoU0VSVklDRV9XT1JLRVIp
CisgICAgICAgIHx8IG1fc2VydmljZVdvcmtlckZldGNoVGFzaworI2VuZGlmCisgICAgKTsKICAg
ICBib29sIHdpbGxXYWl0Rm9yQ29udGludWVEaWRSZWNlaXZlUmVzcG9uc2UgPSB0cnVlOwogICAg
IHNlbmQoTWVzc2FnZXM6OldlYlJlc291cmNlTG9hZGVyOjpEaWRSZWNlaXZlUmVzcG9uc2UgeyBt
X3Jlc3BvbnNlLCB3aWxsV2FpdEZvckNvbnRpbnVlRGlkUmVjZWl2ZVJlc3BvbnNlIH0pOwogfQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>