<?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>197801</bug_id>
          
          <creation_ts>2019-05-10 15:07:10 -0700</creation_ts>
          <short_desc>A service worker instance should be terminated when its SWServer is destroyed</short_desc>
          <delta_ts>2019-05-13 15:08:04 -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>Service Workers</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="youenn fablet">youennf</reporter>
          <assigned_to name="youenn fablet">youennf</assigned_to>
          <cc>beidson</cc>
    
    <cc>cdumez</cc>
    
    <cc>commit-queue</cc>
    
    <cc>ggaren</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1535204</commentid>
    <comment_count>0</comment_count>
    <who name="youenn fablet">youennf</who>
    <bug_when>2019-05-10 15:07:10 -0700</bug_when>
    <thetext>A service worker instance should be terminated when its SWServer is destroyed</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1535211</commentid>
    <comment_count>1</comment_count>
    <who name="youenn fablet">youennf</who>
    <bug_when>2019-05-10 15:30:21 -0700</bug_when>
    <thetext>&lt;rdar://problem/50587270&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1535212</commentid>
    <comment_count>2</comment_count>
      <attachid>369608</attachid>
    <who name="youenn fablet">youennf</who>
    <bug_when>2019-05-10 15:33:34 -0700</bug_when>
    <thetext>Created attachment 369608
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1535246</commentid>
    <comment_count>3</comment_count>
      <attachid>369608</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-05-10 16:33:44 -0700</bug_when>
    <thetext>Comment on attachment 369608
Patch

Clearing flags on attachment: 369608

Committed r245200: &lt;https://trac.webkit.org/changeset/245200&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1535248</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-05-10 16:33:45 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1535786</commentid>
    <comment_count>5</comment_count>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2019-05-13 14:32:09 -0700</bug_when>
    <thetext>What mechanism prevents two concurrent but independent sessions from sharing a service worker?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1535809</commentid>
    <comment_count>6</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2019-05-13 15:08:04 -0700</bug_when>
    <thetext>(In reply to Geoffrey Garen from comment #5)
&gt; What mechanism prevents two concurrent but independent sessions from sharing
&gt; a service worker?

We use a different SWServer instance per session ID. Service worker instances are owned by each SWServer instance so I do not believe different session can share service workers.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>369608</attachid>
            <date>2019-05-10 15:33:34 -0700</date>
            <delta_ts>2019-05-10 16:33:44 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-197801-20190510153333.patch</filename>
            <type>text/plain</type>
            <size>3303</size>
            <attacher name="youenn fablet">youennf</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjQ1MTc4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMzgwOWYyYTg1YTRmNGU5
YmVmZWM4NzcxNDdlNzg0MzU5OTUyODM2Ny4uNDU2Y2FlZmZmMzk5OTI3YjU0MGJmNDJhYmEzY2E1
ZmU1NzVlNDg0YiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDE5LTA1LTEwICBZb3Vl
bm4gRmFibGV0ICA8eW91ZW5uQGFwcGxlLmNvbT4KKworICAgICAgICBBIHNlcnZpY2Ugd29ya2Vy
IGluc3RhbmNlIHNob3VsZCBiZSB0ZXJtaW5hdGVkIHdoZW4gaXRzIFNXU2VydmVyIGlzIGRlc3Ry
b3llZAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTk3
ODAxCisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS81MDU4NzI3MD4KKworICAgICAgICBSZXZpZXdl
ZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBPbiBzZXNzaW9uIGRlc3RydWN0aW9uLCB0
aGUgU1dTZXJ2ZXIgaXMgZGVzdHJveWVkLgorICAgICAgICBBdCB0aGF0IHRpbWUsIGl0IHNob3Vs
ZCB0ZXJtaW5hdGUgYWxsIGl0cyBydW5uaW5nIHNlcnZpY2Ugd29ya2Vycy4KKyAgICAgICAgQ292
ZXJlZCBieSB1cGRhdGVkIEFQSSB0ZXN0LgorCisgICAgICAgICogd29ya2Vycy9zZXJ2aWNlL3Nl
cnZlci9TV1NlcnZlci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpTV1NlcnZlcjo6flNXU2VydmVy
KToKKwogMjAxOS0wNS0xMCAgWW91ZW5uIEZhYmxldCAgPHlvdWVubkBhcHBsZS5jb20+CiAKICAg
ICAgICAgQSBzZXJ2aWNlIHdvcmtlciBwcm9jZXNzIHNob3VsZCBhcHAgbmFwIHdoZW4gYWxsIGl0
cyBjbGllbnRzIGFwcCBuYXAKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3dvcmtlcnMvc2Vy
dmljZS9zZXJ2ZXIvU1dTZXJ2ZXIuY3BwIGIvU291cmNlL1dlYkNvcmUvd29ya2Vycy9zZXJ2aWNl
L3NlcnZlci9TV1NlcnZlci5jcHAKaW5kZXggOWQyZDMwNzQzNTJmNmNlNDMzNjUyYTVkMWYzYzJm
MjM5YWY4ZmM0Yy4uOTdiMzBjMTViYzkwYjVmYjhiNDBjZGQxY2YyM2I3YTBkZmU4NTEwZiAxMDA2
NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvd29ya2Vycy9zZXJ2aWNlL3NlcnZlci9TV1NlcnZlci5j
cHAKKysrIGIvU291cmNlL1dlYkNvcmUvd29ya2Vycy9zZXJ2aWNlL3NlcnZlci9TV1NlcnZlci5j
cHAKQEAgLTY5LDYgKzY5LDE0IEBAIFNXU2VydmVyOjp+U1dTZXJ2ZXIoKQogICAgIGF1dG8gY29u
bmVjdGlvbnMgPSBXVEZNb3ZlKG1fY29ubmVjdGlvbnMpOwogICAgIGNvbm5lY3Rpb25zLmNsZWFy
KCk7CiAKKyAgICBWZWN0b3I8U1dTZXJ2ZXJXb3JrZXIqPiBydW5uaW5nV29ya2VyczsKKyAgICBm
b3IgKGF1dG8mIHdvcmtlciA6IG1fcnVubmluZ09yVGVybWluYXRpbmdXb3JrZXJzLnZhbHVlcygp
KSB7CisgICAgICAgIGlmICh3b3JrZXItPmlzUnVubmluZygpKQorICAgICAgICAgICAgcnVubmlu
Z1dvcmtlcnMuYXBwZW5kKHdvcmtlci5wdHIoKSk7CisgICAgfQorICAgIGZvciAoYXV0byYgcnVu
bmluZ1dvcmtlciA6IHJ1bm5pbmdXb3JrZXJzKQorICAgICAgICB0ZXJtaW5hdGVXb3JrZXIoKnJ1
bm5pbmdXb3JrZXIpOworCiAgICAgYWxsU2VydmVycygpLnJlbW92ZSh0aGlzKTsKIH0KIApkaWZm
IC0tZ2l0IGEvVG9vbHMvQ2hhbmdlTG9nIGIvVG9vbHMvQ2hhbmdlTG9nCmluZGV4IDJkNTJlMWVl
NDRlMTA1OGIyOWJlZjI2ZTFlN2I1MzUxNzYxZmNlOTcuLmJmNDgyMzY5YzdlNmMzZGE2MTBhNTEy
YjE0ODY1Y2FlYjg5MjY3MzggMTAwNjQ0Ci0tLSBhL1Rvb2xzL0NoYW5nZUxvZworKysgYi9Ub29s
cy9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMyBAQAorMjAxOS0wNS0xMCAgWW91ZW5uIEZhYmxldCAg
PHlvdWVubkBhcHBsZS5jb20+CisKKyAgICAgICAgQSBzZXJ2aWNlIHdvcmtlciBpbnN0YW5jZSBz
aG91bGQgYmUgdGVybWluYXRlZCB3aGVuIGl0cyBTV1NlcnZlciBpcyBkZXN0cm95ZWQKKyAgICAg
ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE5NzgwMQorICAgICAg
ICA8cmRhcjovL3Byb2JsZW0vNTA1ODcyNzA+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ
IChPT1BTISkuCisKKyAgICAgICAgKiBUZXN0V2ViS2l0QVBJL1Rlc3RzL1dlYktpdENvY29hL1Nl
cnZpY2VXb3JrZXJCYXNpYy5tbToKKwogMjAxOS0wNS0xMCAgWW91ZW5uIEZhYmxldCAgPHlvdWVu
bkBhcHBsZS5jb20+CiAKICAgICAgICAgQSBzZXJ2aWNlIHdvcmtlciBwcm9jZXNzIHNob3VsZCBh
cHAgbmFwIHdoZW4gYWxsIGl0cyBjbGllbnRzIGFwcCBuYXAKZGlmZiAtLWdpdCBhL1Rvb2xzL1Rl
c3RXZWJLaXRBUEkvVGVzdHMvV2ViS2l0Q29jb2EvU2VydmljZVdvcmtlckJhc2ljLm1tIGIvVG9v
bHMvVGVzdFdlYktpdEFQSS9UZXN0cy9XZWJLaXRDb2NvYS9TZXJ2aWNlV29ya2VyQmFzaWMubW0K
aW5kZXggNGNkMWIzMzI4YWQwNmY4ZGZmZTk1MzRmMWQzNmYyODZhMDQxN2RmOS4uM2RkNGQ3YmNj
MjY3MjRhNzIxMTU4YjRiNWZkM2I3ZmFhZjc5NjI2ZCAxMDA2NDQKLS0tIGEvVG9vbHMvVGVzdFdl
YktpdEFQSS9UZXN0cy9XZWJLaXRDb2NvYS9TZXJ2aWNlV29ya2VyQmFzaWMubW0KKysrIGIvVG9v
bHMvVGVzdFdlYktpdEFQSS9UZXN0cy9XZWJLaXRDb2NvYS9TZXJ2aWNlV29ya2VyQmFzaWMubW0K
QEAgLTMyLDYgKzMyLDcgQEAKICNpbXBvcnQgPFdlYktpdC9XS1VSTFNjaGVtZUhhbmRsZXIuaD4K
ICNpbXBvcnQgPFdlYktpdC9XS1VSTFNjaGVtZVRhc2tQcml2YXRlLmg+CiAjaW1wb3J0IDxXZWJL
aXQvV0tXZWJWaWV3Q29uZmlndXJhdGlvblByaXZhdGUuaD4KKyNpbXBvcnQgPFdlYktpdC9XS1dl
YlZpZXdQcml2YXRlLmg+CiAjaW1wb3J0IDxXZWJLaXQvV0tXZWJzaXRlRGF0YVN0b3JlUHJpdmF0
ZS5oPgogI2ltcG9ydCA8V2ViS2l0L1dLV2Vic2l0ZURhdGFTdG9yZVJlZi5oPgogI2ltcG9ydCA8
V2ViS2l0L1dlYktpdC5oPgpAQCAtMTUwMiw2ICsxNTAzLDcgQEAgVEVTVChTZXJ2aWNlV29ya2Vy
cywgTm9uRGVmYXVsdFNlc3Npb25JRCkKICAgICBUZXN0V2ViS2l0QVBJOjpVdGlsOjpydW4oJmRv
bmUpOwogICAgIGRvbmUgPSBmYWxzZTsKIAorICAgIFt3ZWJWaWV3IF9jbG9zZV07CiAgICAgd2Vi
VmlldyA9IG51bGxwdHI7CiAKICAgICBFWFBFQ1RfVFJVRShbW05TRmlsZU1hbmFnZXIgZGVmYXVs
dE1hbmFnZXJdIGZpbGVFeGlzdHNBdFBhdGg6c2VydmljZVdvcmtlcnNQYXRoLnBhdGhdKTsK
</data>

          </attachment>
      

    </bug>

</bugzilla>