Regression - MozBackgroundSize present in document.documentElement.style

RESOLVED INVALID

Status

()

Core
CSS Parsing and Computation
--
major
RESOLVED INVALID
8 years ago
7 years ago

People

(Reporter: mkaply, Unassigned)

Tracking

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

8 years ago
In Firefox 4, -moz-background-size was changed to background-size.

The proper way to check for this is:

if ("MozBackgroundSize" in document.documentElement.style) {

In Firefox 4 Beta 2 , this worked.

In Firefox 4 beta 3, this is returning true.
This worksforme with a trunk build.  Are you sure you were testing beta3?
(Reporter)

Comment 2

8 years ago
Yep, I was testing beta 3. I'll check again.
(Reporter)

Comment 3

8 years ago
Strange. There was a change in behavior, but it's not what I thought.

I'm doing the wrong thing in my code at some point:

(root is #main-window)

	  if (aData.backgroundSize) {
		root.style.MozBackgroundSize = aData.backgroundSize;
	  } else {
		root.style.MozBackgroundSize = "";
	  }

without regards to FF4 or previous (need to fix that).

But in the previous Firefox 3 beta, this didn't affect document.documentElement.style but in beta 3 it does.

Any ideas?

I'll fix my code, but there is still something weird going on here...
> Any ideas?

Well, the previous behavior certainly sounded buggy....

Note that expandos on .style might be gc-sensitive; I don't recall.
Yes, expandos on style are GC-sensitive.
(Reporter)

Comment 6

7 years ago
This is invalid then.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.