163443 2016-10-14 04:36:02 -0700 [GTK] Default WebKitWebsiteDataManager is always leaked in WebKitWebContext 2016-10-16 02:34:01 -0700 1 1 1 Unclassified WebKit WebKitGTK WebKit Local Build Unspecified Unspecified RESOLVED FIXED Gtk P2 Normal --- 1 cgarcia webkit-unassigned berto bugs-noreply commit-queue gustavo mcatanzaro mrobinson oldest_to_newest 1240223 0 cgarcia 2016-10-14 04:36:02 -0700 We should adopt the ref. 1240224 1 291614 cgarcia 2016-10-14 04:38:11 -0700 Created attachment 291614 Patch 1240229 2 commit-queue 2016-10-14 04:41:01 -0700 Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API 1240351 3 291614 mcatanzaro 2016-10-14 11:06:17 -0700 Comment on attachment 291614 Patch Cool, though impact is limited to non-default web contexts because we still always leak the default web context ;) 1240707 4 cgarcia 2016-10-16 02:34:01 -0700 Committed r207391: <http://trac.webkit.org/changeset/207391> 291614 2016-10-14 04:38:11 -0700 2016-10-14 11:06:17 -0700 Patch wk2-website-data-leak.diff text/plain 1629 cgarcia ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No YW5nZUxvZwppbmRleCA3OTM4MTU2Li44NjBiZmI1IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0 Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMTcg QEAKIDIwMTYtMTAtMTQgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t PgogCisgICAgICAgIFtHVEtdIERlZmF1bHQgV2ViS2l0V2Vic2l0ZURhdGFNYW5hZ2VyIGlzIGFs d2F5cyBsZWFrZWQgaW4gV2ViS2l0V2ViQ29udGV4dAorICAgICAgICBodHRwczovL2J1Z3Mud2Vi a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTYzNDQzCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9C T0RZIChPT1BTISkuCisKKyAgICAgICAgQWRvcHQgdGhlIHJlZmVyZW5jZSByZXR1cm5lZCBieSB3 ZWJraXRXZWJzaXRlRGF0YU1hbmFnZXJDcmVhdGUoKS4KKworICAgICAgICAqIFVJUHJvY2Vzcy9B UEkvZ3RrL1dlYktpdFdlYkNvbnRleHQuY3BwOgorICAgICAgICAod2Via2l0V2ViQ29udGV4dENv bnN0cnVjdGVkKToKKworMjAxNi0xMC0xNCAgQ2FybG9zIEdhcmNpYSBDYW1wb3MgIDxjZ2FyY2lh QGlnYWxpYS5jb20+CisKICAgICAgICAgTWVtb3J5UHJlc3N1cmVIYW5kbGVyIHNob3VsZG4ndCBr bm93IGhvdyB0byByZWxlYXNlIFdlYkNvcmUgbWVtb3J5CiAgICAgICAgIGh0dHBzOi8vYnVncy53 ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNjA0OTcKIApkaWZmIC0tZ2l0IGEvU291cmNlL1dl YktpdDIvVUlQcm9jZXNzL0FQSS9ndGsvV2ViS2l0V2ViQ29udGV4dC5jcHAgYi9Tb3VyY2UvV2Vi S2l0Mi9VSVByb2Nlc3MvQVBJL2d0ay9XZWJLaXRXZWJDb250ZXh0LmNwcAppbmRleCA0NGE0OWI4 Li44MzY1MWFjIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL2d0ay9X ZWJLaXRXZWJDb250ZXh0LmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL2d0 ay9XZWJLaXRXZWJDb250ZXh0LmNwcApAQCAtMjc4LDcgKzI3OCw3IEBAIHN0YXRpYyB2b2lkIHdl YmtpdFdlYkNvbnRleHRDb25zdHJ1Y3RlZChHT2JqZWN0KiBvYmplY3QpCiAgICAgcHJpdi0+cHJv Y2Vzc1Bvb2wgPSBXZWJQcm9jZXNzUG9vbDo6Y3JlYXRlKGNvbmZpZ3VyYXRpb24pOwogCiAgICAg aWYgKCFwcml2LT53ZWJzaXRlRGF0YU1hbmFnZXIpCi0gICAgICAgIHByaXYtPndlYnNpdGVEYXRh TWFuYWdlciA9IHdlYmtpdFdlYnNpdGVEYXRhTWFuYWdlckNyZWF0ZSh3ZWJzaXRlRGF0YVN0b3Jl Q29uZmlndXJhdGlvbkZvcldlYlByb2Nlc3NQb29sQ29uZmlndXJhdGlvbihjb25maWd1cmF0aW9u KSk7CisgICAgICAgIHByaXYtPndlYnNpdGVEYXRhTWFuYWdlciA9IGFkb3B0R1JlZih3ZWJraXRX ZWJzaXRlRGF0YU1hbmFnZXJDcmVhdGUod2Vic2l0ZURhdGFTdG9yZUNvbmZpZ3VyYXRpb25Gb3JX ZWJQcm9jZXNzUG9vbENvbmZpZ3VyYXRpb24oY29uZmlndXJhdGlvbikpKTsKIAogICAgIHByaXYt PnJlcXVlc3RNYW5hZ2VyID0gcHJpdi0+cHJvY2Vzc1Bvb2wtPnN1cHBsZW1lbnQ8V2ViU291cEN1 c3RvbVByb3RvY29sUmVxdWVzdE1hbmFnZXI+KCk7CiAK