Bug 992333 - Skip custom properties when looking for properties to transition. r=dbaron

This commit is contained in:
Cameron McCormack 2014-04-13 11:44:31 +10:00
parent 59a4519586
commit dfc3049e3e
3 changed files with 13 additions and 0 deletions

View File

@ -0,0 +1,10 @@
<!DOCTYPE html>
<style>
p { --variable: value; transition: 1s --variable; }
</style>
<p>Hello.</p>
<script>
window.onload = function() {
document.querySelector("p").style.color = "green";
};
</script>

View File

@ -103,5 +103,6 @@ load 930270-2.html
load 945048-1.html
load 972199-1.html
load 989965-1.html
load 992333-1.html
load large_border_image_width.html
load border-image-visited-link.html

View File

@ -389,6 +389,7 @@ nsTransitionManager::StyleContextChanged(dom::Element *aElement,
// interpretation of transition-property and the one below.
nsCSSProperty property = t.GetProperty();
if (property == eCSSPropertyExtra_no_properties ||
property == eCSSPropertyExtra_variable ||
property == eCSSProperty_UNKNOWN) {
// Nothing to do, but need to exclude this from cases below.
} else if (property == eCSSPropertyExtra_all_properties) {
@ -429,6 +430,7 @@ nsTransitionManager::StyleContextChanged(dom::Element *aElement,
// interpretation of transition-property and the one above.
nsCSSProperty property = t.GetProperty();
if (property == eCSSPropertyExtra_no_properties ||
property == eCSSPropertyExtra_variable ||
property == eCSSProperty_UNKNOWN) {
// Nothing to do, but need to exclude this from cases below.
} else if (property == eCSSPropertyExtra_all_properties) {