<?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>179429</bug_id>
          
          <creation_ts>2017-11-08 10:17:06 -0800</creation_ts>
          <short_desc>Simplify ServiceWorkerContainer::jobResolvedWithRegistration()</short_desc>
          <delta_ts>2017-11-15 09:39:06 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebCore 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="Chris Dumez">cdumez</reporter>
          <assigned_to name="Chris Dumez">cdumez</assigned_to>
          <cc>beidson</cc>
    
    <cc>commit-queue</cc>
    
    <cc>ggaren</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>youennf</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1369473</commentid>
    <comment_count>0</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2017-11-08 10:17:06 -0800</bug_when>
    <thetext>Simplify ServiceWorkerContainer::jobResolvedWithRegistration().</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1369474</commentid>
    <comment_count>1</comment_count>
      <attachid>326338</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2017-11-08 10:18:32 -0800</bug_when>
    <thetext>Created attachment 326338
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1369514</commentid>
    <comment_count>2</comment_count>
      <attachid>326338</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2017-11-08 12:03:03 -0800</bug_when>
    <thetext>Comment on attachment 326338
Patch

Clearing flags on attachment: 326338

Committed r224591: &lt;https://trac.webkit.org/changeset/224591&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1369515</commentid>
    <comment_count>3</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2017-11-08 12:03:05 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1371937</commentid>
    <comment_count>4</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2017-11-15 09:39:06 -0800</bug_when>
    <thetext>&lt;rdar://problem/35562154&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>326338</attachid>
            <date>2017-11-08 10:18:32 -0800</date>
            <delta_ts>2017-11-08 12:03:03 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-179429-20171108101831.patch</filename>
            <type>text/plain</type>
            <size>3357</size>
            <attacher name="Chris Dumez">cdumez</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjI0NTg0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNTFjZTdiMDViMTYwZDky
Zjg2NjY3MDMzZTdhODMyMTlkNmJhZTE3MC4uYzliM2QzMWYwMzVlZWYwMWU0ZWM1M2FmNjdlMzQ0
N2M5MTM3MjAyMiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSw1ICsxLDE1IEBACiAyMDE3LTExLTA4ICBDaHJp
cyBEdW1leiAgPGNkdW1lekBhcHBsZS5jb20+CiAKKyAgICAgICAgU2ltcGxpZnkgU2VydmljZVdv
cmtlckNvbnRhaW5lcjo6am9iUmVzb2x2ZWRXaXRoUmVnaXN0cmF0aW9uKCkKKyAgICAgICAgaHR0
cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE3OTQyOQorCisgICAgICAgIFJl
dmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogd29ya2Vycy9zZXJ2aWNlL1Nl
cnZpY2VXb3JrZXJDb250YWluZXIuY3BwOgorICAgICAgICAoV2ViQ29yZTo6U2VydmljZVdvcmtl
ckNvbnRhaW5lcjo6am9iUmVzb2x2ZWRXaXRoUmVnaXN0cmF0aW9uKToKKworMjAxNy0xMS0wOCAg
Q2hyaXMgRHVtZXogIDxjZHVtZXpAYXBwbGUuY29tPgorCiAgICAgICAgIFtTZXJ2aWNlIFdvcmtl
cnNdIE1ha2Ugc3VyZSBTV1NlcnZlclJlZ2lzdHJhdGlvbiB1cGRhdGUgZnVuY3Rpb25zIGFsd2F5
cyBtZXNzYWdlIGJhY2sgdGhlIGNvbm5lY3Rpb24gdGhhdCBzY2hlZHVsZWQgdGhlIGpvYgogICAg
ICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTc5NDI4CiAKZGlm
ZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3dvcmtlcnMvc2VydmljZS9TZXJ2aWNlV29ya2VyQ29u
dGFpbmVyLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3dvcmtlcnMvc2VydmljZS9TZXJ2aWNlV29ya2Vy
Q29udGFpbmVyLmNwcAppbmRleCAwYTlkM2UwMmE3OGU2ZTZhODk0ZTNhMmU0MWQyZTE4YTUxYjU5
N2MwLi5hOTlhM2Y1Y2Y1ZjlhMWM3NDExMzZlMGUyMzQwZWVhN2UxNzAyMjM5IDEwMDY0NAotLS0g
YS9Tb3VyY2UvV2ViQ29yZS93b3JrZXJzL3NlcnZpY2UvU2VydmljZVdvcmtlckNvbnRhaW5lci5j
cHAKKysrIGIvU291cmNlL1dlYkNvcmUvd29ya2Vycy9zZXJ2aWNlL1NlcnZpY2VXb3JrZXJDb250
YWluZXIuY3BwCkBAIC0zNDUsMjkgKzM0NSwxOSBAQCB2b2lkIFNlcnZpY2VXb3JrZXJDb250YWlu
ZXI6OmpvYlJlc29sdmVkV2l0aFJlZ2lzdHJhdGlvbihTZXJ2aWNlV29ya2VySm9iJiBqb2IsCiAg
ICAgICAgIHJldHVybjsKICAgICB9CiAKLSAgICAvLyBGSVhNRTogSW1wbGVtZW50IHByb3BlciBz
ZWxlY3Rpb24gb2Ygc2VydmljZSB3b3JrZXJzLgotICAgIGF1dG8qIGluc3RhbGxpbmdTZXJ2aWNl
V29ya2VyID0gY29udGV4dC0+YWN0aXZlU2VydmljZVdvcmtlcigpOwogICAgIEFTU0VSVChkYXRh
Lmluc3RhbGxpbmdTZXJ2aWNlV29ya2VySWRlbnRpZmllcik7Ci0gICAgaWYgKCFpbnN0YWxsaW5n
U2VydmljZVdvcmtlciB8fCBpbnN0YWxsaW5nU2VydmljZVdvcmtlci0+aWRlbnRpZmllcigpICE9
ICpkYXRhLmluc3RhbGxpbmdTZXJ2aWNlV29ya2VySWRlbnRpZmllcikgewotICAgICAgICBjb250
ZXh0LT5zZXRBY3RpdmVTZXJ2aWNlV29ya2VyKFNlcnZpY2VXb3JrZXI6OmNyZWF0ZSgqY29udGV4
dCwgKmRhdGEuaW5zdGFsbGluZ1NlcnZpY2VXb3JrZXJJZGVudGlmaWVyLCBkYXRhLnNjcmlwdFVS
TCkpOwotICAgICAgICBpbnN0YWxsaW5nU2VydmljZVdvcmtlciA9IGNvbnRleHQtPmFjdGl2ZVNl
cnZpY2VXb3JrZXIoKTsKLSAgICB9CisgICAgYXV0byBpbnN0YWxsaW5nU2VydmljZVdvcmtlcklk
ZW50aWZpZXIgPSAqZGF0YS5pbnN0YWxsaW5nU2VydmljZVdvcmtlcklkZW50aWZpZXI7CiAKICAg
ICBSZWZQdHI8U2VydmljZVdvcmtlclJlZ2lzdHJhdGlvbj4gcmVnaXN0cmF0aW9uID0gbV9yZWdp
c3RyYXRpb25zLmdldChkYXRhLmtleSk7Ci0gICAgaWYgKCFyZWdpc3RyYXRpb24pIHsKLSAgICAg
ICAgLy8gQ3VycmVudGx5IHRoZSBvbmx5IHJlZ2lzdHJhdGlvbnMgdGhhdCBjYW4gYmUgY3JlYXRl
ZCBmb3IgdGhlIGZpcnN0IHRpbWUgaGVyZSBzaG91bGQgYmUgSW5zdGFsbGluZy4KLSAgICAgICAg
QVNTRVJUKGRhdGEuaW5zdGFsbGluZ1NlcnZpY2VXb3JrZXJJZGVudGlmaWVyKTsKLSAgICAgICAg
YXV0byBpbnN0YWxsaW5nSWRlbnRpZmllciA9ICpkYXRhLmluc3RhbGxpbmdTZXJ2aWNlV29ya2Vy
SWRlbnRpZmllcjsKLSAgICAgICAgCisgICAgaWYgKCFyZWdpc3RyYXRpb24pCiAgICAgICAgIHJl
Z2lzdHJhdGlvbiA9IFNlcnZpY2VXb3JrZXJSZWdpc3RyYXRpb246OmNyZWF0ZSgqY29udGV4dCwg
KnRoaXMsIFdURk1vdmUoZGF0YSkpOwotICAgICAgICByZWdpc3RyYXRpb24tPnVwZGF0ZVN0YXRl
RnJvbVNlcnZlcihTZXJ2aWNlV29ya2VyUmVnaXN0cmF0aW9uU3RhdGU6Okluc3RhbGxpbmcsIGlu
c3RhbGxpbmdJZGVudGlmaWVyKTsKLSAgICAgICAgQVNTRVJUKHJlZ2lzdHJhdGlvbi0+aW5zdGFs
bGluZygpKTsKIAotICAgICAgICBpbnN0YWxsaW5nU2VydmljZVdvcmtlciA9IHJlZ2lzdHJhdGlv
bi0+aW5zdGFsbGluZygpOwotICAgIH0KKyAgICByZWdpc3RyYXRpb24tPnVwZGF0ZVN0YXRlRnJv
bVNlcnZlcihTZXJ2aWNlV29ya2VyUmVnaXN0cmF0aW9uU3RhdGU6Okluc3RhbGxpbmcsIGluc3Rh
bGxpbmdTZXJ2aWNlV29ya2VySWRlbnRpZmllcik7CisgICAgQVNTRVJUKHJlZ2lzdHJhdGlvbi0+
aW5zdGFsbGluZygpKTsKKyAgICByZWdpc3RyYXRpb24tPmluc3RhbGxpbmcoKS0+dXBkYXRlV29y
a2VyU3RhdGUoU2VydmljZVdvcmtlclN0YXRlOjpJbnN0YWxsaW5nLCBTZXJ2aWNlV29ya2VyOjpE
b05vdEZpcmVTdGF0ZUNoYW5nZUV2ZW50KTsKIAotICAgIGluc3RhbGxpbmdTZXJ2aWNlV29ya2Vy
LT51cGRhdGVXb3JrZXJTdGF0ZShTZXJ2aWNlV29ya2VyU3RhdGU6Okluc3RhbGxpbmcsIFNlcnZp
Y2VXb3JrZXI6OkRvTm90RmlyZVN0YXRlQ2hhbmdlRXZlbnQpOwotICAgIHJlZ2lzdHJhdGlvbi0+
c2V0SW5zdGFsbGluZ1dvcmtlcihpbnN0YWxsaW5nU2VydmljZVdvcmtlcik7CisgICAgLy8gRklY
TUU6IEltcGxlbWVudCBwcm9wZXIgc2VsZWN0aW9uIG9mIHNlcnZpY2Ugd29ya2Vycy4KKyAgICBj
b250ZXh0LT5zZXRBY3RpdmVTZXJ2aWNlV29ya2VyKHJlZ2lzdHJhdGlvbi0+aW5zdGFsbGluZygp
KTsKIAogICAgIExPRyhTZXJ2aWNlV29ya2VyLCAiQ29udGFpbmVyICVwIHJlc29sdmVkIGpvYiB3
aXRoIHJlZ2lzdHJhdGlvbiAlcCIsIHRoaXMsIHJlZ2lzdHJhdGlvbi5nZXQoKSk7CiAK
</data>

          </attachment>
      

    </bug>

</bugzilla>