stylo: Drop the comment about the transition in nsCSSFrameConstructor::ResolveStyleContext

RESOLVED FIXED

Status

()

P4
normal
RESOLVED FIXED
a year ago
2 months ago

People

(Reporter: hiro, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox56 unaffected, firefox57 wontfix, firefox58 affected)

Details

(Reporter)

Description

a year ago
We have below comment in nsCSSFrameConstructor::ResolveStyleContext [1], but I think we can drop this comment now because test cases that were added when the code for transition was added have been already passed on stylo. (but I'd like someone who is familiar with transition and frame construction to check it)

  // ServoRestyleManager does not handle transitions yet, and when it does      
  // it probably won't need to track reframed style contexts to start           
  // transitions correctly. 

https://hg.mozilla.org/mozilla-central/file/3bfcbdf5c6c3/layout/base/nsCSSFrameConstructor.cpp#l5242

[1] https://hg.mozilla.org/mozilla-central/rev/51037224ba03
status-firefox56: --- → unaffected
status-firefox57: affected → wontfix
status-firefox58: --- → affected
status-firefox-esr52: --- → unaffected
(Reporter)

Comment 1

2 months ago
The comment had been dropped in bug 1447476.
Status: NEW → RESOLVED
Last Resolved: 2 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.