Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Intermittent pass and fail in /css/css-transitions/properties-value-001.html, /css/css-transitions/properties-value-inherit-001.html, /css/css-transitions/properties-value-implicit-001.html, /css/css-transitions/properties-value-002.html, /css/css-transitions/properties-value-inherit-002.html, /css/css-transitions/properties-value-003.html #21486

Closed
jdm opened this issue Aug 22, 2018 · 1 comment · Fixed by #29664
Labels
I-intermittent Problem reproduces intermittently.

Comments

@jdm
Copy link
Member

jdm commented Aug 22, 2018

  ▶ Unexpected subtest result in /css/css-transitions/properties-value-001.html:
  └ PASS [expected FAIL] text-shadow shadow(shadow) / values
@jdm jdm added the I-intermittent Problem reproduces intermittently. label Aug 22, 2018
@mrobinson mrobinson changed the title Intermittent pass in /css/css-transitions/properties-value-001.html Intermittent pass and fail in /css/css-transitions/properties-value-001.html, /css/css-transitions/properties-value-inherit-001.html, /css/css-transitions/properties-value-002.html, /css/css-transitions/properties-value-inherit-002.html Feb 15, 2023
@mrobinson
Copy link
Member

mrobinson commented Feb 15, 2023

Combining these issues into one because they are all failing due to instability in the timing of the delivery of animation/transition related events.

@mrobinson mrobinson changed the title Intermittent pass and fail in /css/css-transitions/properties-value-001.html, /css/css-transitions/properties-value-inherit-001.html, /css/css-transitions/properties-value-002.html, /css/css-transitions/properties-value-inherit-002.html Intermittent pass and fail in /css/css-transitions/properties-value-001.html, /css/css-transitions/properties-value-inherit-001.html, /css/css-transitions/properties-value-implicit-001.html, /css/css-transitions/properties-value-002.html, /css/css-transitions/properties-value-inherit-002.html Feb 15, 2023
@mrobinson mrobinson changed the title Intermittent pass and fail in /css/css-transitions/properties-value-001.html, /css/css-transitions/properties-value-inherit-001.html, /css/css-transitions/properties-value-implicit-001.html, /css/css-transitions/properties-value-002.html, /css/css-transitions/properties-value-inherit-002.html Intermittent pass and fail in /css/css-transitions/properties-value-001.html, /css/css-transitions/properties-value-inherit-001.html, /css/css-transitions/properties-value-implicit-001.html, /css/css-transitions/properties-value-002.html, /css/css-transitions/properties-value-inherit-002.html, /css/css-transitions/properties-value-003.html Feb 15, 2023
mrobinson added a commit to mrobinson/servo that referenced this issue Apr 26, 2023
1. When updating the animation timeline, ensure that nodes that are
   animating are marked dirty, if necessary, so any style queries will
   force an layout flush.
2. Disable the problematic transition test suites, as they are in Gecko.
   These suites often fail when Servo is so overloaded that it cannot
   deliver frames fast enough to get more than two samples during the
   animation lifecycle.

Fixes servo#28334.
Fixes servo#26435.
Fixes servo#21486.
mrobinson added a commit to mrobinson/servo that referenced this issue Apr 28, 2023
1. When updating the animation timeline, ensure that nodes that are
   animating are marked dirty, if necessary, so any style queries will
   force an layout flush.
2. Disable the problematic transition test suites, as they are in Gecko.
   These suites often fail when Servo is so overloaded that it cannot
   deliver frames fast enough to get more than two samples during the
   animation lifecycle.

Fixes servo#28334.
Fixes servo#26435.
Fixes servo#21486.
bors-servo added a commit that referenced this issue Apr 28, 2023
Fix flakiness in animation tests

1. When updating the animation timeline, ensure that nodes that are
    animating are marked dirty, if necessary, so any style queries will
    force a layout flush.
2. Disable the problematic transition test suites, as they are in Gecko.
    These suites often fail when Servo is so overloaded that it cannot
    deliver frames fast enough to get more than two samples during the
    animation lifecycle.

Fixes #28334.
Fixes #26435.
Fixes #21486.

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #28334, #26435, #21486.

<!-- Either: -->
- [x] There are tests for these changes

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
mrobinson added a commit to mrobinson/servo that referenced this issue May 1, 2023
1. When updating the animation timeline, ensure that nodes that are
   animating are marked dirty, if necessary, so any style queries will
   force an layout flush.
2. Disable the problematic transition test suites, as they are in Gecko.
   These suites often fail when Servo is so overloaded that it cannot
   deliver frames fast enough to get more than two samples during the
   animation lifecycle.

Fixes servo#28334.
Fixes servo#26435.
Fixes servo#21486.
bors-servo added a commit that referenced this issue May 1, 2023
Fix flakiness in animation tests

1. When updating the animation timeline, ensure that nodes that are
    animating are marked dirty, if necessary, so any style queries will
    force a layout flush.
2. Disable the problematic transition test suites, as they are in Gecko.
    These suites often fail when Servo is so overloaded that it cannot
    deliver frames fast enough to get more than two samples during the
    animation lifecycle.

Fixes #28334.
Fixes #26435.
Fixes #21486.

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #28334, #26435, #21486.

<!-- Either: -->
- [x] There are tests for these changes

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
mrobinson added a commit to mrobinson/servo that referenced this issue May 1, 2023
1. When updating the animation timeline, ensure that nodes that are
   animating are marked dirty, if necessary, so any style queries will
   force an layout flush.
2. Disable the problematic transition test suites, as they are in Gecko.
   These suites often fail when Servo is so overloaded that it cannot
   deliver frames fast enough to get more than two samples during the
   animation lifecycle.

Fixes servo#28334.
Fixes servo#26435.
Fixes servo#21486.
bors-servo added a commit that referenced this issue May 1, 2023
Fix flakiness in animation tests

1. When updating the animation timeline, ensure that nodes that are
    animating are marked dirty, if necessary, so any style queries will
    force a layout flush.
2. Disable the problematic transition test suites, as they are in Gecko.
    These suites often fail when Servo is so overloaded that it cannot
    deliver frames fast enough to get more than two samples during the
    animation lifecycle.

Fixes #28334.
Fixes #26435.
Fixes #21486.

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #28334, #26435, #21486.

<!-- Either: -->
- [x] There are tests for these changes

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
bors-servo added a commit that referenced this issue May 1, 2023
Fix flakiness in animation tests

1. When updating the animation timeline, ensure that nodes that are
    animating are marked dirty, if necessary, so any style queries will
    force a layout flush.
2. Disable the problematic transition test suites, as they are in Gecko.
    These suites often fail when Servo is so overloaded that it cannot
    deliver frames fast enough to get more than two samples during the
    animation lifecycle.

Fixes #28334.
Fixes #26435.
Fixes #21486.

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #28334, #26435, #21486.

<!-- Either: -->
- [x] There are tests for these changes

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
bors-servo added a commit that referenced this issue May 1, 2023
Fix flakiness in animation tests

1. When updating the animation timeline, ensure that nodes that are
    animating are marked dirty, if necessary, so any style queries will
    force a layout flush.
2. Disable the problematic transition test suites, as they are in Gecko.
    These suites often fail when Servo is so overloaded that it cannot
    deliver frames fast enough to get more than two samples during the
    animation lifecycle.

Fixes #28334.
Fixes #26435.
Fixes #21486.

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #28334, #26435, #21486.

<!-- Either: -->
- [x] There are tests for these changes

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
bors-servo added a commit that referenced this issue May 1, 2023
Fix flakiness in animation tests

1. When updating the animation timeline, ensure that nodes that are
    animating are marked dirty, if necessary, so any style queries will
    force a layout flush.
2. Disable the problematic transition test suites, as they are in Gecko.
    These suites often fail when Servo is so overloaded that it cannot
    deliver frames fast enough to get more than two samples during the
    animation lifecycle.

Fixes #28334.
Fixes #26435.
Fixes #21486.

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #28334, #26435, #21486.

<!-- Either: -->
- [x] There are tests for these changes

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
@github-actions github-actions bot mentioned this issue May 2, 2023
5 tasks
bors-servo added a commit that referenced this issue May 2, 2023
Fix flakiness in animation tests

1. When updating the animation timeline, ensure that nodes that are
    animating are marked dirty, if necessary, so any style queries will
    force a layout flush.
2. Disable the problematic transition test suites, as they are in Gecko.
    These suites often fail when Servo is so overloaded that it cannot
    deliver frames fast enough to get more than two samples during the
    animation lifecycle.

Fixes #28334.
Fixes #26435.
Fixes #21486.

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #28334, #26435, #21486.

<!-- Either: -->
- [x] There are tests for these changes

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
bors-servo added a commit that referenced this issue May 2, 2023
Fix flakiness in animation tests

1. When updating the animation timeline, ensure that nodes that are
    animating are marked dirty, if necessary, so any style queries will
    force a layout flush.
2. Disable the problematic transition test suites, as they are in Gecko.
    These suites often fail when Servo is so overloaded that it cannot
    deliver frames fast enough to get more than two samples during the
    animation lifecycle.

Fixes #28334.
Fixes #26435.
Fixes #21486.

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #28334, #26435, #21486.

<!-- Either: -->
- [x] There are tests for these changes

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I-intermittent Problem reproduces intermittently.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants