<?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>113709</bug_id>
          
          <creation_ts>2013-04-01 10:45:48 -0700</creation_ts>
          <short_desc>[WK2] Sandbox extension tracking is incorrect when restoring a session</short_desc>
          <delta_ts>2013-04-01 11:04:34 -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>WebKit2</component>
          <version>528+ (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="Alexey Proskuryakov">ap</reporter>
          <assigned_to name="Alexey Proskuryakov">ap</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>867214</commentid>
    <comment_count>0</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2013-04-01 10:45:48 -0700</bug_when>
    <thetext>When a session is restored, a load is sometimes started and immediately replaced with another one. This erroneously clears pending provisional sandbox extension.

&lt;rdar://problem/13490123&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>867221</commentid>
    <comment_count>1</comment_count>
      <attachid>195993</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2013-04-01 10:53:33 -0700</bug_when>
    <thetext>Created attachment 195993
proposed fix</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>867232</commentid>
    <comment_count>2</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2013-04-01 11:04:34 -0700</bug_when>
    <thetext>Committed &lt;http://trac.webkit.org/changeset/147344&gt;.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>195993</attachid>
            <date>2013-04-01 10:53:33 -0700</date>
            <delta_ts>2013-04-01 11:00:22 -0700</delta_ts>
            <desc>proposed fix</desc>
            <filename>PendingSandboxExtension.txt</filename>
            <type>text/plain</type>
            <size>3601</size>
            <attacher name="Alexey Proskuryakov">ap</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDE0NzMzOCkKKysrIFNvdXJjZS9XZWJLaXQyL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIxIEBACisyMDEzLTA0LTAxICBBbGV4ZXkg
UHJvc2t1cnlha292ICA8YXBAYXBwbGUuY29tPgorCisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS8x
MzQ5MDEyMz4gU2FuZGJveCBleHRlbnNpb24gdHJhY2tpbmcgaXMgaW5jb3JyZWN0IHdoZW4gcmVz
dG9yaW5nIGEgc2Vzc2lvbgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9MTEzNzA5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK
KyAgICAgICAgKiBXZWJQcm9jZXNzL1dlYlBhZ2UvV2ViUGFnZS5jcHA6CisgICAgICAgIChXZWJL
aXQ6OldlYlBhZ2U6OlNhbmRib3hFeHRlbnNpb25UcmFja2VyOjpkaWRTdGFydFByb3Zpc2lvbmFs
TG9hZCk6IEFkZGVkIGFuIGFzc2VydGlvbgorICAgICAgICB0byBjYXRjaCBleHRlbnNpb24gdHJh
Y2tpbmcgbG9naWMgZXJyb3JzIHRoYXQgY291bGQgcmVzdWx0IGluIGdpdmluZyBhbiBleHRlbnNp
b24gdG8KKyAgICAgICAgbm9uLWxvY2FsIHBhZ2VzLgorICAgICAgICAoV2ViS2l0OjpXZWJQYWdl
OjpTYW5kYm94RXh0ZW5zaW9uVHJhY2tlcjo6ZGlkQ29tbWl0UHJvdmlzaW9uYWxMb2FkKTogSnVz
dCBsaWtlCisgICAgICAgIGJlbG93LCBjbGVhcmluZyBtX3BlbmRpbmdQcm92aXNpb25hbFNhbmRi
b3hFeHRlbnNpb24gYXBwZWFycyBpbmNvcnJlY3QsIGV2ZW4gdGhvdWdoCisgICAgICAgIEkgYW0g
bm90IGF3YXJlIG9mIGFueSBzY2VuYXJpb3Mgd2hlcmUgdGhpcyBoYXBwZW5zIGluIHByYWN0aWNl
LgorICAgICAgICAoV2ViS2l0OjpXZWJQYWdlOjpTYW5kYm94RXh0ZW5zaW9uVHJhY2tlcjo6ZGlk
RmFpbFByb3Zpc2lvbmFsTG9hZCk6IFRoZSBsb2FkIG1heQorICAgICAgICBiZSBmYWlsaW5nIGV4
YWN0bHkgYmVjYXVzZSBhIG5ldyBvbmUgY2FuY2VscyBpdCwgc28gd2Ugc2hvdWxkbid0IGJlIHBy
ZXZlbnRpbmcKKyAgICAgICAgdGhlIG5ldyBvbmUgZnJvbSB1c2luZyBpdHMgZXh0ZW5zaW9uLgor
CiAyMDEzLTAzLTMwICBDc2FiYSBPc3p0cm9nb27DoWMgIDxvc3N5QHdlYmtpdC5vcmc+CiAKICAg
ICAgICAgVW5yZXZpZXdlZCBidWlsZGZpeCBhZnRlciByMTQ3MjYwLgpJbmRleDogU291cmNlL1dl
YktpdDIvV2ViUHJvY2Vzcy9XZWJQYWdlL1dlYlBhZ2UuY3BwCj09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJj
ZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9XZWJQYWdlLmNwcAkocmV2aXNpb24gMTQ2OTIz
KQorKysgU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQYWdlL1dlYlBhZ2UuY3BwCSh3b3Jr
aW5nIGNvcHkpCkBAIC0zMTUzLDYgKzMxNTMsOCBAQCB2b2lkIFdlYlBhZ2U6OlNhbmRib3hFeHRl
bnNpb25UcmFja2VyOjpkCiAgICAgaWYgKCFtX3Byb3Zpc2lvbmFsU2FuZGJveEV4dGVuc2lvbikK
ICAgICAgICAgcmV0dXJuOwogCisgICAgQVNTRVJUKCFtX3Byb3Zpc2lvbmFsU2FuZGJveEV4dGVu
c2lvbiB8fCBmcmFtZS0+Y29yZUZyYW1lKCktPmxvYWRlcigpLT5wcm92aXNpb25hbERvY3VtZW50
TG9hZGVyKCktPnVybCgpLmlzTG9jYWxGaWxlKCkpOworCiAgICAgbV9wcm92aXNpb25hbFNhbmRi
b3hFeHRlbnNpb24tPmNvbnN1bWUoKTsKIH0KIApAQCAtMzE2MSwxNCArMzE2MywxMyBAQCB2b2lk
IFdlYlBhZ2U6OlNhbmRib3hFeHRlbnNpb25UcmFja2VyOjpkCiAgICAgaWYgKCFmcmFtZS0+aXNN
YWluRnJhbWUoKSkKICAgICAgICAgcmV0dXJuOwogCi0gICAgLy8gR2VuZXJhbGx5LCB0aGVyZSBz
aG91bGQgYmUgbm8gcGVuZGluZyBleHRlbnNpb24gYXQgdGhpcyBzdGFnZSwgYnV0IHdlIGNhbiBo
YXZlIG9uZSBpZiBVSSBwcm9jZXNzCi0gICAgLy8gaGFzIGFuIG91dCBvZiBkYXRlIGlkZWEgb2Yg
V2ViUHJvY2VzcyBzdGF0ZSwgYW5kIGluaXRpYXRlcyBhIGxvYWQgb3IgcmVsb2FkIHdpdGhvdXQg
c3RvcHBpbmcgYW4gZXhpc3Rpbmcgb25lLgotICAgIG1fcGVuZGluZ1Byb3Zpc2lvbmFsU2FuZGJv
eEV4dGVuc2lvbiA9IG51bGxwdHI7Ci0KICAgICBpZiAobV9jb21taXR0ZWRTYW5kYm94RXh0ZW5z
aW9uKQogICAgICAgICBtX2NvbW1pdHRlZFNhbmRib3hFeHRlbnNpb24tPnJldm9rZSgpOwogCiAg
ICAgbV9jb21taXR0ZWRTYW5kYm94RXh0ZW5zaW9uID0gbV9wcm92aXNpb25hbFNhbmRib3hFeHRl
bnNpb24ucmVsZWFzZSgpOworCisgICAgLy8gV2UgY2FuIGFsc28gaGF2ZSBhIG5vbi1udWxsIG1f
cGVuZGluZ1Byb3Zpc2lvbmFsU2FuZGJveEV4dGVuc2lvbiBpZiBhIG5ldyBsb2FkIGlzIGJlaW5n
IHN0YXJ0ZWQuCisgICAgLy8gVGhpcyBleHRlbnNpb24gaXMgbm90IGNsZWFyZWQsIGJlY2F1c2Ug
aXQgZG9lcyBub3QgcGVydGFpbiB0byB0aGUgZmFpbGVkIGxvYWQsIGFuZCB3aWxsIGJlIG5lZWRl
ZC4KIH0KIAogdm9pZCBXZWJQYWdlOjpTYW5kYm94RXh0ZW5zaW9uVHJhY2tlcjo6ZGlkRmFpbFBy
b3Zpc2lvbmFsTG9hZChXZWJGcmFtZSogZnJhbWUpCkBAIC0zMTc2LDE1ICszMTc3LDE1IEBAIHZv
aWQgV2ViUGFnZTo6U2FuZGJveEV4dGVuc2lvblRyYWNrZXI6OmQKICAgICBpZiAoIWZyYW1lLT5p
c01haW5GcmFtZSgpKQogICAgICAgICByZXR1cm47CiAKLSAgICAvLyBHZW5lcmFsbHksIHRoZXJl
IHNob3VsZCBiZSBubyBwZW5kaW5nIGV4dGVuc2lvbiBhdCB0aGlzIHN0YWdlLCBidXQgd2UgY2Fu
IGhhdmUgb25lIGlmIFVJIHByb2Nlc3MKLSAgICAvLyBoYXMgYW4gb3V0IG9mIGRhdGUgaWRlYSBv
ZiBXZWJQcm9jZXNzIHN0YXRlLCBhbmQgaW5pdGlhdGVzIGEgbG9hZCBvciByZWxvYWQgd2l0aG91
dCBzdG9wcGluZyBhbiBleGlzdGluZyBvbmUuCi0gICAgbV9wZW5kaW5nUHJvdmlzaW9uYWxTYW5k
Ym94RXh0ZW5zaW9uID0gbnVsbHB0cjsKLQogICAgIGlmICghbV9wcm92aXNpb25hbFNhbmRib3hF
eHRlbnNpb24pCiAgICAgICAgIHJldHVybjsKIAogICAgIG1fcHJvdmlzaW9uYWxTYW5kYm94RXh0
ZW5zaW9uLT5yZXZva2UoKTsKICAgICBtX3Byb3Zpc2lvbmFsU2FuZGJveEV4dGVuc2lvbiA9IG51
bGxwdHI7CisKKyAgICAvLyBXZSBjYW4gYWxzbyBoYXZlIGEgbm9uLW51bGwgbV9wZW5kaW5nUHJv
dmlzaW9uYWxTYW5kYm94RXh0ZW5zaW9uIGlmIGEgbmV3IGxvYWQgaXMgYmVpbmcgc3RhcnRlZAor
ICAgIC8vIChub3RhYmx5LCBpZiB0aGUgY3VycmVudCBvbmUgZmFpbHMgYmVjYXVzZSB0aGUgbmV3
IG9uZSBjYW5jZWxzIGl0KS4gVGhpcyBleHRlbnNpb24gaXMgbm90IGNsZWFyZWQsCisgICAgLy8g
YmVjYXVzZSBpdCBkb2VzIG5vdCBwZXJ0YWluIHRvIHRoZSBmYWlsZWQgbG9hZCwgYW5kIHdpbGwg
YmUgbmVlZGVkLgogfQogCiBib29sIFdlYlBhZ2U6Omhhc0xvY2FsRGF0YUZvclVSTChjb25zdCBL
VVJMJiB1cmwpCg==
</data>
<flag name="review"
          id="217976"
          type_id="1"
          status="+"
          setter="andersca"
    />
          </attachment>
      

    </bug>

</bugzilla>