80736 2012-03-09 15:09:45 -0800 CSSStyleDeclaration.getPropertyValue() for 'border-radius' returns null when value is set 2012-04-14 14:05:23 -0700 1 1 1 Unclassified WebKit CSS 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 epiceric24 joethomas antaryami.pandia joethomas koivisto macpherson menard shanestephens webkit.review.bot oldest_to_newest 575282 0 epiceric24 2012-03-09 15:09:45 -0800 There is an inconsistency with how the CSSStyleDeclaration reports the values of shorthand css properties. Also seen in Chrome. Notably, the querying the value for 'border-radius' like so returns null: // (Assume 'rule' is a CSSStyleRule object which has a the border-radius set) rule.style.getPropertyValue('border-radius') Other shorthand properties such as 'border' or 'background' will return their value. Thanks! 599448 1 136496 epiceric24 2012-04-10 11:39:43 -0700 Created attachment 136496 Demo page of bug 602523 2 137180 joethomas 2012-04-13 16:51:33 -0700 Created attachment 137180 ProposedPatch 602524 3 137181 joethomas 2012-04-13 16:58:04 -0700 Created attachment 137181 Reduced test case 602696 4 137180 webkit.review.bot 2012-04-14 14:05:18 -0700 Comment on attachment 137180 ProposedPatch Clearing flags on attachment: 137180 Committed r114207: <http://trac.webkit.org/changeset/114207> 602697 5 webkit.review.bot 2012-04-14 14:05:23 -0700 All reviewed patches have been landed. Closing bug. 136496 2012-04-10 11:39:43 -0700 2012-04-10 11:39:43 -0700 Demo page of bug borderRadiusShorthandBug.html text/html 1784 epiceric24 PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KICAgIDxtZXRhIGh0dHAtZXF1aXY9IkNvbnRl bnQtdHlwZSIgY29udGVudD0idGV4dC9odG1sOyBjaGFyc2V0PXV0Zi04Ij4KICAgIDx0aXRsZT5U YWxlIG9mIHRoZSBNaXNzaW5nIEJvcmRlciBSYWRpdXMgU2hvcnRoYW5kPC90aXRsZT4KPC9oZWFk Pgo8c3R5bGUgdHlwZT0idGV4dC9jc3MiIG1lZGlhPSJzY3JlZW4iPgogICAgYm9keSB7CiAgICAg ICAgd2lkdGg6IDgwJTsKICAgICAgICBtYXgtd2lkdGg6IDYwMHB4OwogICAgICAgIG1hcmdpbjog MCBhdXRvOwogICAgICAgIGZvbnQtZmFtaWx5OiBoZWx2ZXRpY2EsIHNhbnMtc2VyaWY7CiAgICB9 CiAgICBwI2JvcmRlclJhZGl1cyB7CiAgICAgICAgYm9yZGVyLXJhZGl1czogMTVweDsKICAgICAg ICBiYWNrZ3JvdW5kLWNvbG9yOiAjQ0NDOwogICAgICAgIHBhZGRpbmc6IDE1cHg7CiAgICB9CiAg ICAubW9ub3NwYWNlIHsKICAgICAgICBmb250LWZhbWlseTogbW9ub3NwYWNlOwogICAgfQogICAg YnV0dG9uLCBpbnB1dCB7CiAgICAgICAgZm9udC1zaXplOiAyNXB4OwogICAgfQo8L3N0eWxlPgo8 Ym9keT4KICAgIDxoMT5XZWJraXQgYnVnIC0gUmV0dXJuaW5nIG51bGwgZm9yIGJvcmRlci1yYWRp dXM8L2gxPgogICAgPHAgaWQ9ImJvcmRlclJhZGl1cyI+CiAgICAgICAgVGhpcyBwYXJhZ3JhcGgg aGFzIGEgcnVsZSB3aGljaCBkZWZpbmVzIGEgYm9yZGVyIHJhZGl1cyBvZiAxNXB4LgogICAgPC9w PgogICAgPHA+CiAgICAgICAgVXNpbmcgd2luZG93LmdldE1hdGNoZWRDU1NSdWxlcygpLCB3ZSBj YW4gYWNjZXNzIHRoZSBydWxlcyBzdHlsaW5nIHRoZSBwYXJhZ3JhcGgKICAgICAgICBhYm92ZS4g V2hlbiB3ZSBnZXQgdGhlIHJ1bGVzLCB3ZSBjYW4gYW5hbHl6ZSB0aGUgY29udGFpbmVkIENTU1N0 eWxlRGVjbGFyYXRpb24gb2JqZWN0cwogICAgICAgIHRvIGNoZWNrIGZvciB0aGUgJ2JvcmRlci1y YWRpdXMnIHZhbHVlIHVzaW5nIAogICAgICAgIDxzcGFuIGNsYXNzPSJtb25vc3BhY2UiPnJ1bGUu c3R5bGUuZ2V0UHJvcGVydHlWYWx1ZSgnYm9yZXItcmFkaXVzJyk8L3NwYW4+LgogICAgPC9wPgog ICAgPGJ1dHRvbiBpZD0iZ2V0VmFsdWVCdXR0b24iPkdldCBCb3JkZXIgUmFkaXVzIFZhbHVlPC9i dXR0b24+CiAgICA8aW5wdXQgaWQ9InJlc3VsdHNGaWVsZCIgdHlwZT0idGV4dCI+PC9pbnB1dD4K ICAgIDxzY3JpcHQgdHlwZT0idGV4dC9qYXZhc2NyaXB0Ij4KICAgICAgICB2YXIgYnRuID0gZG9j dW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ2dldFZhbHVlQnV0dG9uJyk7CiAgICAgICAgdmFyIHJlc3Vs dHMgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgncmVzdWx0c0ZpZWxkJyk7CiAgICAgICAgdmFy IHBhcmEgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgnYm9yZGVyUmFkaXVzJyk7CiAgICAgICAg CiAgICAgICAgYnRuLmFkZEV2ZW50TGlzdGVuZXIoJ2NsaWNrJywgZnVuY3Rpb24oZSkgewogICAg ICAgICAgICByZXN1bHQgPSB3aW5kb3cuZ2V0TWF0Y2hlZENTU1J1bGVzKHBhcmEpWzBdLnN0eWxl LmdldFByb3BlcnR5VmFsdWUoJ2JvcmRlci1yYWRpdXMnKTsKICAgICAgICAgICAgCiAgICAgICAg ICAgIGlmKHJlc3VsdCA9PT0gbnVsbCkgewogICAgICAgICAgICAgICAgcmVzdWx0ID0gJ251bGwn OwogICAgICAgICAgICB9CiAgICAgICAgICAgIAogICAgICAgICAgICByZXN1bHRzLnZhbHVlID0g cmVzdWx0OwogICAgICAgIH0sIGZhbHNlKTsKICAgICAgICAKICAgICAgICAKICAgIDwvc2NyaXB0 Pgo8L2JvZHk+CjwvaHRtbD4= 137180 2012-04-13 16:51:33 -0700 2012-04-14 14:05:18 -0700 ProposedPatch patch text/plain 4312 joethomas SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDExNDE4MykKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBACisyMDEyLTA0LTEzICBKb2UgVGhv bWFzICA8am9ldGhvbWFzQG1vdG9yb2xhLmNvbT4KKworICAgICAgICBDU1NTdHlsZURlY2xhcmF0 aW9uLmdldFByb3BlcnR5VmFsdWUoKSBmb3IgJ2JvcmRlci1yYWRpdXMnIHJldHVybnMgbnVsbCB3 aGVuIHZhbHVlIGlzIHNldAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1 Zy5jZ2k/aWQ9ODA3MzYKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor ICAgICAgICBBZGRlZCBzdXBwb3J0IGZvciBmZXRjaGluZyB0aGUgdmFsdWUgb2YgJ2JvcmRlci1y YWRpdXMnIENTUyBwcm9wZXJ0eSB3aGljaCB3YXMgbWlzc2luZy4KKworICAgICAgICBUZXN0OiBm YXN0L2Nzcy9ib3JkZXItcmFkaXVzLXByb3BlcnR5LXZhbHVlLmh0bWwKKworICAgICAgICAqIGNz cy9TdHlsZVByb3BlcnR5U2V0LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OlN0eWxlUHJvcGVydHlT ZXQ6OmdldFByb3BlcnR5VmFsdWUpOgorCiAyMDEyLTA0LTEzICBKZXIgTm9ibGUgIDxqZXIubm9i bGVAYXBwbGUuY29tPgogCiAgICAgICAgIFZpZGVvIGF0IGFwcGxlLmNvbSBnZXRzIHN0YW5kYXJk IGNvbnRyb2xzIGluIGFkZGl0aW9uIHRvIGN1c3RvbSBjb250cm9scyBhZnRlciByZXR1cm5pbmcg ZnJvbSBmdWxsIHNjcmVlbgpJbmRleDogU291cmNlL1dlYkNvcmUvY3NzL1N0eWxlUHJvcGVydHlT ZXQuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL2Nzcy9TdHlsZVByb3BlcnR5U2V0 LmNwcAkocmV2aXNpb24gMTE0MTc0KQorKysgU291cmNlL1dlYkNvcmUvY3NzL1N0eWxlUHJvcGVy dHlTZXQuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xNzEsNiArMTcxLDggQEAgU3RyaW5nIFN0eWxl UHJvcGVydHlTZXQ6OmdldFByb3BlcnR5VmFsdQogICAgICAgICAgICAgcmV0dXJuIHZhbHVlLT5j c3NUZXh0KCk7CiAgICAgfQogI2VuZGlmCisgICAgY2FzZSBDU1NQcm9wZXJ0eUJvcmRlclJhZGl1 czoKKyAgICAgICAgcmV0dXJuIGdldDRWYWx1ZXMoYm9yZGVyUmFkaXVzU2hvcnRoYW5kKCkpOwog ICAgIGRlZmF1bHQ6CiAgICAgICAgICAgcmV0dXJuIFN0cmluZygpOwogICAgIH0KSW5kZXg6IExh eW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJ KHJldmlzaW9uIDExNDE4MykKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5 KQpAQCAtMSwzICsxLDE1IEBACisyMDEyLTA0LTEzICBKb2UgVGhvbWFzICA8am9ldGhvbWFzQG1v dG9yb2xhLmNvbT4KKworICAgICAgICBDU1NTdHlsZURlY2xhcmF0aW9uLmdldFByb3BlcnR5VmFs dWUoKSBmb3IgJ2JvcmRlci1yYWRpdXMnIHJldHVybnMgbnVsbCB3aGVuIHZhbHVlIGlzIHNldAor ICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9ODA3MzYKKwor ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBBZGRlZCBzdXBw b3J0IGZvciBmZXRjaGluZyB0aGUgdmFsdWUgb2YgJ2JvcmRlci1yYWRpdXMnIENTUyBwcm9wZXJ0 eSB3aGljaCB3YXMgbWlzc2luZy4KKworICAgICAgICAqIGZhc3QvY3NzL2JvcmRlci1yYWRpdXMt cHJvcGVydHktdmFsdWUtZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAgICAgKiBmYXN0L2Nzcy9i b3JkZXItcmFkaXVzLXByb3BlcnR5LXZhbHVlLmh0bWw6IEFkZGVkLgorCiAyMDEyLTA0LTEzICBT aW1vbiBGcmFzZXIgIDxzaW1vbi5mcmFzZXJAYXBwbGUuY29tPgogCiAgICAgICAgIFJlYmFzZWxp bmUgc29tZSBjb21wb3NpdGluZyB0ZXN0cy4gU29tZSB3ZXJlIGFmZmVjdGVkIGJ5IHRoZSBjb2xv cnNwYWNlIG9mCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L2Nzcy9ib3JkZXItcmFkaXVzLXByb3Bl cnR5LXZhbHVlLWV4cGVjdGVkLnR4dAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L2Nz cy9ib3JkZXItcmFkaXVzLXByb3BlcnR5LXZhbHVlLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkK KysrIExheW91dFRlc3RzL2Zhc3QvY3NzL2JvcmRlci1yYWRpdXMtcHJvcGVydHktdmFsdWUtZXhw ZWN0ZWQudHh0CShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDE0IEBACitUZXN0IGZvciBCdWc6IDgw NzM2IC0gQ1NTU3R5bGVEZWNsYXJhdGlvbi5nZXRQcm9wZXJ0eVZhbHVlKCkgZm9yICdib3JkZXIt cmFkaXVzJyByZXR1cm5zIG51bGwgd2hlbiB2YWx1ZSBpcyBzZXQKKworT24gc3VjY2VzcywgeW91 IHdpbGwgc2VlIGEgc2VyaWVzIG9mICJQQVNTIiBtZXNzYWdlcywgZm9sbG93ZWQgYnkgIlRFU1Qg Q09NUExFVEUiLgorCisKK1BBU1Mgc3R5bGUuZ2V0UHJvcGVydHlWYWx1ZSgnYm9yZGVyLXJhZGl1 cycpIGlzICc1cHgnCitQQVNTIHN0eWxlLmdldFByb3BlcnR5VmFsdWUoJ2JvcmRlci10b3AtbGVm dC1yYWRpdXMnKSBpcyAnNXB4JworUEFTUyBzdHlsZS5nZXRQcm9wZXJ0eVZhbHVlKCdib3JkZXIt dG9wLXJpZ2h0LXJhZGl1cycpIGlzICc1cHgnCitQQVNTIHN0eWxlLmdldFByb3BlcnR5VmFsdWUo J2JvcmRlci1ib3R0b20tbGVmdC1yYWRpdXMnKSBpcyAnNXB4JworUEFTUyBzdHlsZS5nZXRQcm9w ZXJ0eVZhbHVlKCdib3JkZXItYm90dG9tLXJpZ2h0LXJhZGl1cycpIGlzICc1cHgnCitQQVNTIHN1 Y2Nlc3NmdWxseVBhcnNlZCBpcyB0cnVlCisKK1RFU1QgQ09NUExFVEUKKwpJbmRleDogTGF5b3V0 VGVzdHMvZmFzdC9jc3MvYm9yZGVyLXJhZGl1cy1wcm9wZXJ0eS12YWx1ZS5odG1sCj09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvY3NzL2JvcmRlci1yYWRpdXMtcHJvcGVydHktdmFsdWUu aHRtbAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvY3NzL2JvcmRlci1yYWRpdXMt cHJvcGVydHktdmFsdWUuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwyNyBAQAorPCFET0NU WVBFIEhUTUw+Cis8aHRtbD4KKzxzdHlsZT4KKyAjZWxlbWVudC1jb250YWluZXIgeworICAgYm9y ZGVyLXJhZGl1czo1cHg7Cit9Cis8L3N0eWxlPgorPGRpdiBpZD0iZWxlbWVudC1jb250YWluZXIi PjwvZGl2PgorPHNjcmlwdCBzcmM9Ii4uL2pzL3Jlc291cmNlcy9qcy10ZXN0LXByZS5qcyI+PC9z Y3JpcHQ+CisKKzxzY3JpcHQ+CitpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKQorICAg IGxheW91dFRlc3RDb250cm9sbGVyLmR1bXBBc1RleHQoKTsKKworZGVzY3JpcHRpb24oIlRlc3Qg Zm9yIEJ1ZzogODA3MzYgLSBDU1NTdHlsZURlY2xhcmF0aW9uLmdldFByb3BlcnR5VmFsdWUoKSBm b3IgJ2JvcmRlci1yYWRpdXMnIHJldHVybnMgbnVsbCB3aGVuIHZhbHVlIGlzIHNldCIpOworCit2 YXIgZWxlbWVudCA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJlbGVtZW50LWNvbnRhaW5lciIp OwordmFyIHN0eWxlID0gd2luZG93LmdldE1hdGNoZWRDU1NSdWxlcyhlbGVtZW50KVswXS5zdHls ZTsKK3Nob3VsZEJlKCJzdHlsZS5nZXRQcm9wZXJ0eVZhbHVlKCdib3JkZXItcmFkaXVzJykiLCIn NXB4JyIpOworc2hvdWxkQmUoInN0eWxlLmdldFByb3BlcnR5VmFsdWUoJ2JvcmRlci10b3AtbGVm dC1yYWRpdXMnKSIsIic1cHgnIik7CitzaG91bGRCZSgic3R5bGUuZ2V0UHJvcGVydHlWYWx1ZSgn Ym9yZGVyLXRvcC1yaWdodC1yYWRpdXMnKSIsIic1cHgnIik7CitzaG91bGRCZSgic3R5bGUuZ2V0 UHJvcGVydHlWYWx1ZSgnYm9yZGVyLWJvdHRvbS1sZWZ0LXJhZGl1cycpIiwiJzVweCciKTsKK3No b3VsZEJlKCJzdHlsZS5nZXRQcm9wZXJ0eVZhbHVlKCdib3JkZXItYm90dG9tLXJpZ2h0LXJhZGl1 cycpIiwiJzVweCciKTsKKzwvc2NyaXB0PgorCis8c2NyaXB0IHNyYz0iLi4vanMvcmVzb3VyY2Vz L2pzLXRlc3QtcG9zdC5qcyI+PC9zY3JpcHQ+Cis8L2h0bWw+Cg== 137181 2012-04-13 16:58:04 -0700 2012-04-13 16:59:27 -0700 Reduced test case testcase.html text/html 310 joethomas PCFET0NUWVBFPgo8aHRtbD4KPHN0eWxlPgogI2VsZW1lbnQtY29udGFpbmVyIHsKICAgYm9yZGVy LXJhZGl1czo1cHg7Cn0KPC9zdHlsZT4KPGRpdiBpZD0iZWxlbWVudC1jb250YWluZXIiPjwvZGl2 PgoKPHNjcmlwdD4KdmFyIGVsZW1lbnQgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgiZWxlbWVu dC1jb250YWluZXIiKTsKdmFyIHN0eWxlID0gd2luZG93LmdldE1hdGNoZWRDU1NSdWxlcyhlbGVt ZW50KVswXS5zdHlsZTsKYWxlcnQoc3R5bGUuZ2V0UHJvcGVydHlWYWx1ZSgiYm9yZGVyLXJhZGl1 cyIpKTsKPC9zY3JpcHQ+Cgo8L2h0bWw+Cg==