172128 2017-05-15 10:49:16 -0700 [WK2][macOS][iOS] Allow Network process to access nsurlstoraged's top level domain data cache 2017-05-15 14:22:03 -0700 1 1 1 Unclassified WebKit WebKit2 WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 bfulgham bfulgham achristensen ap bfulgham commit-queue ggaren oldest_to_newest 1308717 0 bfulgham 2017-05-15 10:49:16 -0700 We extended the sandbox last cycle to permit access to nsurlstoraged's top level domain data cache. At the time, we believed this was only needed in the WebContent process. However, further logging and feedback from the networking teams reveal that the Network process should also have been granted this access. This bug allows the Network Process to access the nsurlstoraged's data cache. 1308720 1 bfulgham 2017-05-15 10:51:56 -0700 <rdar://problem/30871490> 1308722 2 310149 bfulgham 2017-05-15 10:58:23 -0700 Created attachment 310149 Patch 1308750 3 achristensen 2017-05-15 12:19:13 -0700 Is there an explanation about why this is only needed on certain OS versions? Will this be permanent? Should we have done it before? 1308752 4 bfulgham 2017-05-15 12:29:17 -0700 (In reply to Alex Christensen from comment #3) > Is there an explanation about why this is only needed on certain OS > versions? Will this be permanent? Should we have done it before? It's not needed on older OS's since the cache was not really being used. We want it to be used in an upcoming release, and beyond. 1308781 5 310149 achristensen 2017-05-15 13:53:54 -0700 Comment on attachment 310149 Patch In bugzilla I have no comment on the existence of such an upcoming release, much less our plans if such a thing exists. But r=me 1308794 6 310149 commit-queue 2017-05-15 14:22:01 -0700 Comment on attachment 310149 Patch Clearing flags on attachment: 310149 Committed r216877: <http://trac.webkit.org/changeset/216877> 1308795 7 commit-queue 2017-05-15 14:22:03 -0700 All reviewed patches have been landed. Closing bug. 310149 2017-05-15 10:58:23 -0700 2017-05-15 14:22:01 -0700 Patch bug-172128-20170515105826.patch text/plain 2208 bfulgham SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDIxNjg2MykKKysrIFNvdXJjZS9XZWJLaXQyL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE0IEBACisyMDE3LTA1LTE1ICBCcmVudCBG dWxnaGFtICA8YmZ1bGdoYW1AYXBwbGUuY29tPgorCisgICAgICAgIFtXSzJdW21hY09TXVtpT1Nd IEFsbG93IE5ldHdvcmsgcHJvY2VzcyB0byBhY2Nlc3MgbnN1cmxzdG9yYWdlZCdzIHRvcCBsZXZl bCBkb21haW4gZGF0YSBjYWNoZSAKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv d19idWcuY2dpP2lkPTE3MjEyOAorICAgICAgICA8cmRhcjovL3Byb2JsZW0vMzA4NzE0OTA+CisK KyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBOZXR3b3Jr UHJvY2Vzcy9tYWMvY29tLmFwcGxlLldlYktpdC5OZXR3b3JrUHJvY2Vzcy5zYi5pbjoKKyAgICAg ICAgKiBSZXNvdXJjZXMvU2FuZGJveFByb2ZpbGVzL2lvcy9jb20uYXBwbGUuV2ViS2l0Lk5ldHdv cmtpbmcuc2I6CisKIDIwMTctMDUtMTUgIEd3YW5nIFlvb24gSHdhbmcgIDx5b29uQGlnYWxpYS5j b20+CiAKICAgICAgICAgW1dQRV0gVW5yZXZpZXdlZCBkZWJ1ZyBidWlsZCBmaXgKSW5kZXg6IFNv dXJjZS9XZWJLaXQyL05ldHdvcmtQcm9jZXNzL21hYy9jb20uYXBwbGUuV2ViS2l0Lk5ldHdvcmtQ cm9jZXNzLnNiLmluCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJLaXQyL05ldHdvcmtQcm9jZXNz L21hYy9jb20uYXBwbGUuV2ViS2l0Lk5ldHdvcmtQcm9jZXNzLnNiLmluCShyZXZpc2lvbiAyMTY4 NjIpCisrKyBTb3VyY2UvV2ViS2l0Mi9OZXR3b3JrUHJvY2Vzcy9tYWMvY29tLmFwcGxlLldlYktp dC5OZXR3b3JrUHJvY2Vzcy5zYi5pbgkod29ya2luZyBjb3B5KQpAQCAtNTUsNiArNTUsMTEgQEAK IChkZWZpbmUgKGhvbWUtbGl0ZXJhbCBob21lLXJlbGF0aXZlLWxpdGVyYWwpCiAgICAgKGxpdGVy YWwgKHN0cmluZy1hcHBlbmQgKHBhcmFtICJIT01FX0RJUiIpIGhvbWUtcmVsYXRpdmUtbGl0ZXJh bCkpKQogCisjaWYgX19NQUNfT1NfWF9WRVJTSU9OX01JTl9SRVFVSVJFRCA+PSAxMDEzMDAKKzs7 IENGTmV0d29yaworKGFsbG93IGZpbGUtcmVhZC1kYXRhIChwYXRoICIvcHJpdmF0ZS92YXIvZGIv bnN1cmxzdG9yYWdlZC9kYWZzYURhdGEuYmluIikpCisjZW5kaWYKKwogOzsgUmVtb3ZlIHdoZW4g PHJkYXI6Ly9wcm9ibGVtLzI5NjQ2MDk0PiBpcyBmaXhlZC4KIChkZWZpbmUgKEhFWC1wYXR0ZXJu LW1hdGNoLWdlbmVyYXRvciBwYXR0ZXJuLWRlc2NyaXB0b3IpCiAgICAgKGxldHJlYyAoKHBhdHRl cm4tc3RyaW5nICIiKSkKSW5kZXg6IFNvdXJjZS9XZWJLaXQyL1Jlc291cmNlcy9TYW5kYm94UHJv ZmlsZXMvaW9zL2NvbS5hcHBsZS5XZWJLaXQuTmV0d29ya2luZy5zYgo9PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBT b3VyY2UvV2ViS2l0Mi9SZXNvdXJjZXMvU2FuZGJveFByb2ZpbGVzL2lvcy9jb20uYXBwbGUuV2Vi S2l0Lk5ldHdvcmtpbmcuc2IJKHJldmlzaW9uIDIxNjg2MikKKysrIFNvdXJjZS9XZWJLaXQyL1Jl c291cmNlcy9TYW5kYm94UHJvZmlsZXMvaW9zL2NvbS5hcHBsZS5XZWJLaXQuTmV0d29ya2luZy5z Ygkod29ya2luZyBjb3B5KQpAQCAtNjEsNiArNjEsMTAgQEAKIAogKG5ldHdvcmstY2xpZW50IChy ZW1vdGUgdGNwKSAocmVtb3RlIHVkcCkpCiAKKzs7IGFsbG93IDNyZCBwYXJ0eSBhcHBsaWNhdGlv bnMgdG8gYWNjZXNzIG5zdXJsc3RvcmFnZWQncyB0b3AgbGV2ZWwgZG9tYWluIGRhdGEgY2FjaGUK KyhhbGxvdy13ZWxsLWtub3duLXN5c3RlbS1ncm91cC1jb250YWluZXItc3VicGF0aC1yZWFkCisg ICAgIi9zeXN0ZW1ncm91cC5jb20uYXBwbGUubnN1cmxzdG9yYWdlZHJlc291cmNlcy9MaWJyYXJ5 L2RhZnNhRGF0YS5iaW4iKQorCiA7OyBTZWN1cml0eSBmcmFtZXdvcmsKIChhbGxvdyBtYWNoLWxv b2t1cAogICAgIChnbG9iYWwtbmFtZSAiY29tLmFwcGxlLm9jc3BkIikK