118433 2013-07-05 15:07:17 -0700 AX: iOS: SVG paths are not offset by the parent bounding box 2013-07-08 14:04:47 -0700 1 1 1 Unclassified WebKit Accessibility 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 cfleizach cfleizach aboxhall apinheiro commit-queue ddkilzer dmazzoni jdiggs mario sam simon.fraser thorton oldest_to_newest 906491 0 cfleizach 2013-07-05 15:07:17 -0700 Asking for an SVG path will give it to you in the parent's view space. To correctly display on screen that needs to be offset by the parent's location within the web view 906496 1 206174 cfleizach 2013-07-05 15:56:40 -0700 Created attachment 206174 patch 906497 2 206175 cfleizach 2013-07-05 15:59:45 -0700 Created attachment 206175 patch 906541 3 sam 2013-07-05 21:26:01 -0700 Any way to test this? 906555 4 cfleizach 2013-07-05 22:55:58 -0700 (In reply to comment #3) > Any way to test this? I originally had the path data output point information, but it changed from on each build bot, so I had to remove the point location information. There's not a great way to test for this unfortunately that's reliable 906895 5 206175 thorton 2013-07-08 13:35:49 -0700 Comment on attachment 206175 patch Does this work for nested <svg>? 906896 6 thorton 2013-07-08 13:36:18 -0700 (In reply to comment #4) > (In reply to comment #3) > > Any way to test this? > > I originally had the path data output point information, but it changed from on each build bot, so I had to remove the point location information. There's not a great way to test for this unfortunately that's reliable This seems ... surprising. 906897 7 cfleizach 2013-07-08 13:42:16 -0700 (In reply to comment #5) > (From update of attachment 206175 [details]) > Does this work for nested <svg>? I'll have to check 906902 8 206175 commit-queue 2013-07-08 14:04:44 -0700 Comment on attachment 206175 patch Clearing flags on attachment: 206175 Committed r152464: <http://trac.webkit.org/changeset/152464> 906903 9 commit-queue 2013-07-08 14:04:47 -0700 All reviewed patches have been landed. Closing bug. 206174 2013-07-05 15:56:40 -0700 2013-07-05 15:59:45 -0700 patch patch text/plain 1964 cfleizach SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE1MjQyNikKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEzIEBACisyMDEzLTA3LTA1ICBDaHJpcyBG bGVpemFjaCAgPGNmbGVpemFjaEBhcHBsZS5jb20+CisKKyAgICAgICAgQVg6IGlPUzogU1ZHIHBh dGhzIGFyZSBub3Qgb2Zmc2V0IGJ5IHRoZSBwYXJlbnQgYm91bmRpbmcgYm94CisgICAgICAgIGh0 dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMTg0MzMKKworICAgICAgICBS ZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGFjY2Vzc2liaWxpdHkvQWNj ZXNzaWJpbGl0eVJlbmRlck9iamVjdC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpBY2Nlc3NpYmls aXR5UmVuZGVyT2JqZWN0OjplbGVtZW50UGF0aCk6CisKIDIwMTMtMDctMDUgIEJyZW50IEZ1bGdo YW0gIDxiZnVsZ2hhbUBhcHBsZS5jb20+CiAKICAgICAgICAgW1dpbmRvd3NdIElkZW50aWZ5IE9T LWxldmVsIGF0IGJ1aWxkLXRpbWUuCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9hY2Nlc3NpYmlsaXR5 L0FjY2Vzc2liaWxpdHlSZW5kZXJPYmplY3QuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJD b3JlL2FjY2Vzc2liaWxpdHkvQWNjZXNzaWJpbGl0eVJlbmRlck9iamVjdC5jcHAJKHJldmlzaW9u IDE1MjQyNikKKysrIFNvdXJjZS9XZWJDb3JlL2FjY2Vzc2liaWxpdHkvQWNjZXNzaWJpbGl0eVJl bmRlck9iamVjdC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTg1MSwxMiArODUxLDI0IEBACiAgICAg CiAgICAgcmV0dXJuIGZhbHNlOwogfQotICAgIAorCiBQYXRoIEFjY2Vzc2liaWxpdHlSZW5kZXJP YmplY3Q6OmVsZW1lbnRQYXRoKCkgY29uc3QKIHsKICNpZiBFTkFCTEUoU1ZHKQotICAgIGlmICht X3JlbmRlcmVyICYmIG1fcmVuZGVyZXItPmlzU1ZHU2hhcGUoKSAmJiB0b1JlbmRlclNWR1NoYXBl KG1fcmVuZGVyZXIpLT5oYXNQYXRoKCkpCi0gICAgICAgIHJldHVybiB0b1JlbmRlclNWR1NoYXBl KG1fcmVuZGVyZXIpLT5wYXRoKCk7CisgICAgaWYgKG1fcmVuZGVyZXIgJiYgbV9yZW5kZXJlci0+ aXNTVkdTaGFwZSgpICYmIHRvUmVuZGVyU1ZHU2hhcGUobV9yZW5kZXJlciktPmhhc1BhdGgoKSkg eworICAgICAgICBQYXRoIHBhdGggPSB0b1JlbmRlclNWR1NoYXBlKG1fcmVuZGVyZXIpLT5wYXRo KCk7CisgICAgICAgIAorICAgICAgICAvLyBUaGUgU1ZHIHBhdGggaXMgaW4gdGVybXMgb2YgdGhl IHBhcmVudCdzIGJvdW5kaW5nIGJveC4gVGhlIHBhdGggbmVlZHMgdG8gYmUgb2Zmc2V0IHRvIGZy YW1lIGNvb3JkaW5hdGVzLgorICAgICAgICBmb3IgKFJlbmRlck9iamVjdCogcGFyZW50ID0gbV9y ZW5kZXJlci0+cGFyZW50KCk7IHBhcmVudDsgcGFyZW50ID0gcGFyZW50LT5wYXJlbnQoKSkgewor ICAgICAgICAgICAgaWYgKHBhcmVudC0+aXNTVkdSb290KCkpIHsKKyAgICAgICAgICAgICAgICBM YXlvdXRQb2ludCBwYXJlbnRPZmZzZXQgPSBheE9iamVjdENhY2hlKCktPmdldE9yQ3JlYXRlKHBh cmVudCktPmVsZW1lbnRSZWN0KCkubG9jYXRpb24oKTsKKyAgICAgICAgICAgICAgICBwYXRoLnRy YW5zZm9ybShBZmZpbmVUcmFuc2Zvcm0oKS50cmFuc2xhdGUocGFyZW50T2Zmc2V0LngoKSwgcGFy ZW50T2Zmc2V0LnkoKSkpOworICAgICAgICAgICAgICAgIGJyZWFrOworICAgICAgICAgICAgfQor ICAgICAgICB9CisgICAgICAgIAorICAgICAgICByZXR1cm4gcGF0aDsKKyAgICB9CiAjZW5kaWYK ICAgICAKICAgICByZXR1cm4gUGF0aCgpOwo= 206175 2013-07-05 15:59:45 -0700 2013-07-08 14:04:43 -0700 patch patch text/plain 2126 cfleizach SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE1MjQyNikKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBACisyMDEzLTA3LTA1ICBDaHJpcyBG bGVpemFjaCAgPGNmbGVpemFjaEBhcHBsZS5jb20+CisKKyAgICAgICAgQVg6IGlPUzogU1ZHIHBh dGhzIGFyZSBub3Qgb2Zmc2V0IGJ5IHRoZSBwYXJlbnQgYm91bmRpbmcgYm94CisgICAgICAgIGh0 dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMTg0MzMKKworICAgICAgICBU aGUgcGF0aCBmb3IgU1ZHIHNoYXBlcyBpcyBpbiB0ZXJtcyBvZiBpdHMgcGFyZW50J3MgdmlldyBz cGFjZS4gV2UgbmVlZCB0byBvZmZzZXQKKyAgICAgICAgdGhlIHBhdGggYnkgdGhhdCBhbW91bnQg c28gdGhhdCBpdCdzIGluIHRoZSByaWdodCBsb2NhdGlvbi4KKworICAgICAgICBSZXZpZXdlZCBi eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGFjY2Vzc2liaWxpdHkvQWNjZXNzaWJpbGl0 eVJlbmRlck9iamVjdC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpBY2Nlc3NpYmlsaXR5UmVuZGVy T2JqZWN0OjplbGVtZW50UGF0aCk6CisKIDIwMTMtMDctMDUgIEJyZW50IEZ1bGdoYW0gIDxiZnVs Z2hhbUBhcHBsZS5jb20+CiAKICAgICAgICAgW1dpbmRvd3NdIElkZW50aWZ5IE9TLWxldmVsIGF0 IGJ1aWxkLXRpbWUuCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9hY2Nlc3NpYmlsaXR5L0FjY2Vzc2li aWxpdHlSZW5kZXJPYmplY3QuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL2FjY2Vz c2liaWxpdHkvQWNjZXNzaWJpbGl0eVJlbmRlck9iamVjdC5jcHAJKHJldmlzaW9uIDE1MjQyNikK KysrIFNvdXJjZS9XZWJDb3JlL2FjY2Vzc2liaWxpdHkvQWNjZXNzaWJpbGl0eVJlbmRlck9iamVj dC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTg1MSwxMiArODUxLDI0IEBACiAgICAgCiAgICAgcmV0 dXJuIGZhbHNlOwogfQotICAgIAorCiBQYXRoIEFjY2Vzc2liaWxpdHlSZW5kZXJPYmplY3Q6OmVs ZW1lbnRQYXRoKCkgY29uc3QKIHsKICNpZiBFTkFCTEUoU1ZHKQotICAgIGlmIChtX3JlbmRlcmVy ICYmIG1fcmVuZGVyZXItPmlzU1ZHU2hhcGUoKSAmJiB0b1JlbmRlclNWR1NoYXBlKG1fcmVuZGVy ZXIpLT5oYXNQYXRoKCkpCi0gICAgICAgIHJldHVybiB0b1JlbmRlclNWR1NoYXBlKG1fcmVuZGVy ZXIpLT5wYXRoKCk7CisgICAgaWYgKG1fcmVuZGVyZXIgJiYgbV9yZW5kZXJlci0+aXNTVkdTaGFw ZSgpICYmIHRvUmVuZGVyU1ZHU2hhcGUobV9yZW5kZXJlciktPmhhc1BhdGgoKSkgeworICAgICAg ICBQYXRoIHBhdGggPSB0b1JlbmRlclNWR1NoYXBlKG1fcmVuZGVyZXIpLT5wYXRoKCk7CisgICAg ICAgIAorICAgICAgICAvLyBUaGUgU1ZHIHBhdGggaXMgaW4gdGVybXMgb2YgdGhlIHBhcmVudCdz IGJvdW5kaW5nIGJveC4gVGhlIHBhdGggbmVlZHMgdG8gYmUgb2Zmc2V0IHRvIGZyYW1lIGNvb3Jk aW5hdGVzLgorICAgICAgICBmb3IgKFJlbmRlck9iamVjdCogcGFyZW50ID0gbV9yZW5kZXJlci0+ cGFyZW50KCk7IHBhcmVudDsgcGFyZW50ID0gcGFyZW50LT5wYXJlbnQoKSkgeworICAgICAgICAg ICAgaWYgKHBhcmVudC0+aXNTVkdSb290KCkpIHsKKyAgICAgICAgICAgICAgICBMYXlvdXRQb2lu dCBwYXJlbnRPZmZzZXQgPSBheE9iamVjdENhY2hlKCktPmdldE9yQ3JlYXRlKHBhcmVudCktPmVs ZW1lbnRSZWN0KCkubG9jYXRpb24oKTsKKyAgICAgICAgICAgICAgICBwYXRoLnRyYW5zZm9ybShB ZmZpbmVUcmFuc2Zvcm0oKS50cmFuc2xhdGUocGFyZW50T2Zmc2V0LngoKSwgcGFyZW50T2Zmc2V0 LnkoKSkpOworICAgICAgICAgICAgICAgIGJyZWFrOworICAgICAgICAgICAgfQorICAgICAgICB9 CisgICAgICAgIAorICAgICAgICByZXR1cm4gcGF0aDsKKyAgICB9CiAjZW5kaWYKICAgICAKICAg ICByZXR1cm4gUGF0aCgpOwo=