Don't use window object when using getComputedStyle in dom/animation/test.

RESOLVED FIXED in Firefox 55

Status

()

RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: mantaroh, Assigned: mantaroh)

Tracking

54 Branch
mozilla55
Points:
---

Firefox Tracking Flags

(firefox54 wontfix, firefox55 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
It's small change.

In bug 1302648, we didn't specify the 'window' object when using getComputedStyle.
For consistency, I drop 'window' object when using getComputedStyle in the animation tests.
Comment hidden (mozreview-request)
I'm not completely sure this is worth doing, at least not for now. There are various upstream changes happening and a big cross-cutting change like this here is going to make merging those really hard work for James. We should probably hold off a couple of weeks.

Also, we need to update the checksums of all these files we're changing by running 'mach web-platform-tests --manifest-update yer' otherwise anyone else running that command on m-c will get a bunch of unrelated changes.
I haven't forgotten about this. I'm just waiting for the latest web-platform-tests sync to hit mozilla central (should be next 1~2 days).

Comment 4

2 years ago
mozreview-review
Comment on attachment 8835333 [details]
Bug 1338069 - Don't use 'window' object when using getComputedStyle in tests.

https://reviewboard.mozilla.org/r/111024/#review115298

web-platform-tests has finally been synced so I think this is ok to land.
Attachment #8835333 - Flags: review?(bbirtles) → review+
Mantaroh, are you going to land this?
Flags: needinfo?(mantaroh)
Comment hidden (mozreview-request)
(Assignee)

Comment 7

2 years ago
(In reply to Brian Birtles (:birtles) from comment #5)
> Mantaroh, are you going to land this?

Sorry, I forgot to land it.

Try :
https://treeherder.mozilla.org/#/jobs?repo=try&revision=eba443758b0896ec2f0f79c7058cfbb287a545cc
You can specify "--artifact" syntax for test file only changes.  It does not work on stylo for now though.
(Assignee)

Updated

2 years ago
Assignee: nobody → mantaroh
Flags: needinfo?(mantaroh)

Comment 9

2 years ago
Pushed by mantaroh@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/c80974470261
Don't use 'window' object when using getComputedStyle in tests. r=birtles

Comment 10

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c80974470261
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
status-firefox54: affected → wontfix
You need to log in before you can comment on or make changes to this bug.