169939 2017-03-21 18:12:20 -0700 REGRESSION(STP): rgb() with calc() containing variables doesn't work 2017-05-04 08:41:25 -0700 1 1 1 Unclassified WebKit CSS Safari Technology Preview Mac OS X 10.11 RESOLVED FIXED InRadar P2 Normal --- 1 gwhit hyatt ap hyatt simon.fraser webkit-bug-importer zalan oldest_to_newest 1289923 0 gwhit 2017-03-21 18:12:20 -0700 This is currently working in stable Safari but seems to have regressed: https://jsbin.com/metobigecu/1/edit?css,output Expected: Background is Green Actual: Red We discovered this while working on a demo for Microsoft Edge custom properties. FF fails due to no calc support in color functions and Chrome doesn't support float values in Calc(). Webkit seems to have the same issue for calc() but doesn't if you're using a variable. At any rate, would be great to see this fixed. 1290473 1 webkit-bug-importer 2017-03-23 09:37:25 -0700 <rdar://problem/31221151> 1293876 2 hyatt 2017-04-03 15:35:40 -0700 We brought consistency to this, so we fail whether you use a variable or not. 1302187 3 308413 hyatt 2017-04-27 11:22:56 -0700 Created attachment 308413 Patch for EWS 1304618 4 309046 hyatt 2017-05-04 07:53:05 -0700 Created attachment 309046 Patch 1304635 5 hyatt 2017-05-04 08:41:25 -0700 Landed in r216188. 308413 2017-04-27 11:22:56 -0700 2017-05-04 07:53:05 -0700 Patch for EWS patch.txt text/plain 851 hyatt SW5kZXg6IFNvdXJjZS9XZWJDb3JlL2Nzcy9DU1NDYWxjdWxhdGlvblZhbHVlLmNwcAo9PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9jc3MvQ1NTQ2FsY3VsYXRpb25WYWx1ZS5jcHAJKHJldmlz aW9uIDIxNTQxNykKKysrIFNvdXJjZS9XZWJDb3JlL2Nzcy9DU1NDYWxjdWxhdGlvblZhbHVlLmNw cAkod29ya2luZyBjb3B5KQpAQCAtNTk4LDcgKzU5OCw4IEBAIHByaXZhdGU6CiAgICAgICAgIGlm ICh1bml0Q2F0ZWdvcnkodHlwZSkgPT0gQ2FsY090aGVyKQogICAgICAgICAgICAgcmV0dXJuIGZh bHNlOwogICAgICAgICAKLSAgICAgICAgcmVzdWx0LT52YWx1ZSA9IENTU0NhbGNQcmltaXRpdmVW YWx1ZTo6Y3JlYXRlKENTU1ByaW1pdGl2ZVZhbHVlOjpjcmVhdGUodG9rZW4ubnVtZXJpY1ZhbHVl KCksIHR5cGUpLCB0b2tlbi5udW1lcmljVmFsdWVUeXBlKCkgPT0gSW50ZWdlclZhbHVlVHlwZSk7 CisgICAgICAgIGJvb2wgaXNJbnRlZ2VyID0gdG9rZW4ubnVtZXJpY1ZhbHVlVHlwZSgpID09IElu dGVnZXJWYWx1ZVR5cGUgfHwgKHRva2VuLm51bWVyaWNWYWx1ZVR5cGUoKSA9PSBOdW1iZXJWYWx1 ZVR5cGUgJiYgdG9rZW4ubnVtZXJpY1ZhbHVlKCkgPT0gdHJ1bmModG9rZW4ubnVtZXJpY1ZhbHVl KCkpKTsKKyAgICAgICAgcmVzdWx0LT52YWx1ZSA9IENTU0NhbGNQcmltaXRpdmVWYWx1ZTo6Y3Jl YXRlKENTU1ByaW1pdGl2ZVZhbHVlOjpjcmVhdGUodG9rZW4ubnVtZXJpY1ZhbHVlKCksIHR5cGUp LCBpc0ludGVnZXIpOwogICAgICAgICAKICAgICAgICAgcmV0dXJuIHRydWU7CiAgICAgfQo= 309046 2017-05-04 07:53:05 -0700 2017-05-04 07:59:45 -0700 Patch patch.txt text/plain 3295 hyatt SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDIxNjE4NSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBACisyMDE3LTA1LTA0ICBEYXZlIEh5 YXR0ICA8aHlhdHRAYXBwbGUuY29tPgorCisgICAgICAgIFJFR1JFU1NJT04oU1RQKTogcmdiKCkg d2l0aCBjYWxjKCkgY29udGFpbmluZyB2YXJpYWJsZXMgZG9lc24ndCB3b3JrCisgICAgICAgIGh0 dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNjk5MzkKKworICAgICAgICBS ZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBBZGRlZCBuZXcgdGVzdCBpbiBm YXN0L2Nzcy92YXJpYWJsZXMuCisKKyAgICAgICAgKiBjc3MvQ1NTQ2FsY3VsYXRpb25WYWx1ZS5j cHA6CisgICAgICAgIChXZWJDb3JlOjpDU1NDYWxjRXhwcmVzc2lvbk5vZGVQYXJzZXI6OnBhcnNl VmFsdWUpOgorICAgICAgICBUcmVhdCBmbG9hdHMgaW4gY2FsY3MgYXMgaW50ZWdlcnMgd2hlbiB3 ZSBjYW4uCisKIDIwMTctMDUtMDQgIENvbW1pdCBRdWV1ZSAgPGNvbW1pdC1xdWV1ZUB3ZWJraXQu b3JnPgogCiAgICAgICAgIFVucmV2aWV3ZWQsIHJvbGxpbmcgb3V0IHIyMTYxNzIuCkluZGV4OiBT b3VyY2UvV2ViQ29yZS9jc3MvQ1NTQ2FsY3VsYXRpb25WYWx1ZS5jcHAKPT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0g U291cmNlL1dlYkNvcmUvY3NzL0NTU0NhbGN1bGF0aW9uVmFsdWUuY3BwCShyZXZpc2lvbiAyMTU0 MTcpCisrKyBTb3VyY2UvV2ViQ29yZS9jc3MvQ1NTQ2FsY3VsYXRpb25WYWx1ZS5jcHAJKHdvcmtp bmcgY29weSkKQEAgLTU5OCw3ICs1OTgsOCBAQCBwcml2YXRlOgogICAgICAgICBpZiAodW5pdENh dGVnb3J5KHR5cGUpID09IENhbGNPdGhlcikKICAgICAgICAgICAgIHJldHVybiBmYWxzZTsKICAg ICAgICAgCi0gICAgICAgIHJlc3VsdC0+dmFsdWUgPSBDU1NDYWxjUHJpbWl0aXZlVmFsdWU6OmNy ZWF0ZShDU1NQcmltaXRpdmVWYWx1ZTo6Y3JlYXRlKHRva2VuLm51bWVyaWNWYWx1ZSgpLCB0eXBl KSwgdG9rZW4ubnVtZXJpY1ZhbHVlVHlwZSgpID09IEludGVnZXJWYWx1ZVR5cGUpOworICAgICAg ICBib29sIGlzSW50ZWdlciA9IHRva2VuLm51bWVyaWNWYWx1ZVR5cGUoKSA9PSBJbnRlZ2VyVmFs dWVUeXBlIHx8ICh0b2tlbi5udW1lcmljVmFsdWVUeXBlKCkgPT0gTnVtYmVyVmFsdWVUeXBlICYm IHRva2VuLm51bWVyaWNWYWx1ZSgpID09IHRydW5jKHRva2VuLm51bWVyaWNWYWx1ZSgpKSk7Cisg ICAgICAgIHJlc3VsdC0+dmFsdWUgPSBDU1NDYWxjUHJpbWl0aXZlVmFsdWU6OmNyZWF0ZShDU1NQ cmltaXRpdmVWYWx1ZTo6Y3JlYXRlKHRva2VuLm51bWVyaWNWYWx1ZSgpLCB0eXBlKSwgaXNJbnRl Z2VyKTsKICAgICAgICAgCiAgICAgICAgIHJldHVybiB0cnVlOwogICAgIH0KSW5kZXg6IExheW91 dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHJl dmlzaW9uIDIxNjE4NSkKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpA QCAtMSwzICsxLDEzIEBACisyMDE3LTA1LTA0ICBEYXZlIEh5YXR0ICA8aHlhdHRAYXBwbGUuY29t PgorCisgICAgICAgIFJFR1JFU1NJT04oU1RQKTogcmdiKCkgd2l0aCBjYWxjKCkgY29udGFpbmlu ZyB2YXJpYWJsZXMgZG9lc24ndCB3b3JrCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn L3Nob3dfYnVnLmNnaT9pZD0xNjk5MzkKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9P UFMhKS4KKworICAgICAgICAqIGZhc3QvY3NzL3ZhcmlhYmxlcy9jYWxjLWZsb2F0LXRvLWludC1l eHBlY3RlZC5odG1sOiBBZGRlZC4KKyAgICAgICAgKiBmYXN0L2Nzcy92YXJpYWJsZXMvY2FsYy1m bG9hdC10by1pbnQuaHRtbDogQWRkZWQuCisKIDIwMTctMDUtMDQgIENhcmxvcyBHYXJjaWEgQ2Ft cG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29tPgogCiAgICAgICAgIFVucmV2aWV3ZWQgZ2FyZGVuaW5n LiBVcGRhdGUgbWFpbiBleHBlY3RhdGlvbnMgb2YgaHR0cC90ZXN0cy9pbnNwZWN0b3IvbmV0d29y ay9yZXNvdXJjZS1yZXF1ZXN0LWhlYWRlcnMuaHRtbC4KSW5kZXg6IExheW91dFRlc3RzL2Zhc3Qv Y3NzL3ZhcmlhYmxlcy9jYWxjLWZsb2F0LXRvLWludC1leHBlY3RlZC5odG1sCj09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K LS0tIExheW91dFRlc3RzL2Zhc3QvY3NzL3ZhcmlhYmxlcy9jYWxjLWZsb2F0LXRvLWludC1leHBl Y3RlZC5odG1sCShub25leGlzdGVudCkKKysrIExheW91dFRlc3RzL2Zhc3QvY3NzL3ZhcmlhYmxl cy9jYWxjLWZsb2F0LXRvLWludC1leHBlY3RlZC5odG1sCSh3b3JraW5nIGNvcHkpCkBAIC0wLDAg KzEsMTMgQEAKKzxodG1sPgorICAgIDxoZWFkPgorICAgICAgICA8c3R5bGU+Citib2R5IHsKK2Jh Y2tncm91bmQ6IHJnYigwLCAyNTUsIDApOworaGVpZ2h0OiAxMDB2aDsKK21hcmdpbjogMDsKK30K Kzwvc3R5bGU+Cis8L2hlYWQ+Cis8Ym9keT4KKzwvYm9keT4KKzwvaHRtbD4KSW5kZXg6IExheW91 dFRlc3RzL2Zhc3QvY3NzL3ZhcmlhYmxlcy9jYWxjLWZsb2F0LXRvLWludC5odG1sCj09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvY3NzL3ZhcmlhYmxlcy9jYWxjLWZsb2F0LXRvLWludC5o dG1sCShub25leGlzdGVudCkKKysrIExheW91dFRlc3RzL2Zhc3QvY3NzL3ZhcmlhYmxlcy9jYWxj LWZsb2F0LXRvLWludC5odG1sCSh3b3JraW5nIGNvcHkpCkBAIC0wLDAgKzEsMTkgQEAKKzxodG1s PgorPGhlYWQ+Cis8c3R5bGU+CitodG1sIHsKK2JhY2tncm91bmQ6IHJlZDsKK30KKworYm9keSB7 CistLWNvbG9yOiAxLjA7CitiYWNrZ3JvdW5kOiByZWQ7CitiYWNrZ3JvdW5kOiByZ2IoMCwgY2Fs YygyNTUgKiB2YXIoLS1jb2xvcikpLCAwKTsKK2hlaWdodDogMTAwdmg7CittYXJnaW46IDA7Cit9 Cis8L3N0eWxlPgorPC9oZWFkPgorPGJvZHk+Cis8L2JvZHk+Cis8L2h0bWw+Cg==