Remove dom/base/test/test_intersectionobserver_no_root.html

RESOLVED FIXED in Firefox 56

Status

()

Core
Layout
P3
normal
RESOLVED FIXED
9 months ago
9 months ago

People

(Reporter: tobytailor, Unassigned)

Tracking

unspecified
mozilla56
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(firefox56 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

9 months ago
dom/base/test/test_intersectionobserver_no_root.html was disabled in Bug 1369253 due to its instability. We do not plan to re-enable it, therefore it should be removed entirely.
(Reporter)

Comment 1

9 months ago
Created attachment 8886829 [details] [diff] [review]
Remove test
(Reporter)

Comment 2

9 months ago
Created attachment 8886830 [details] [diff] [review]
Remove test
Attachment #8886829 - Attachment is obsolete: true

Updated

9 months ago
Priority: -- → P3
(Reporter)

Comment 4

9 months ago
Comment on attachment 8886830 [details] [diff] [review]
Remove test

Blake, as discussed this is the patch that removes the test after being disabled in Bug 1369253.
Attachment #8886830 - Flags: review?(mrbkap)

Updated

9 months ago
Attachment #8886830 - Flags: review?(mrbkap) → review+

Comment 5

9 months ago
Pushed by tschneider@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e23128fe85f1
Remove dom/base/test/test_intersectionobserver_no_root.html. r=mrbkap

Comment 6

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/e23128fe85f1
Status: NEW → RESOLVED
Last Resolved: 9 months ago
status-firefox56: --- → fixed
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla56

Comment 7

9 months ago
(In reply to Tobias Schneider [:tobytailor] from comment #0)
> dom/base/test/test_intersectionobserver_no_root.html was disabled in Bug
> 1369253 due to its instability. We do not plan to re-enable it, therefore it
> should be removed entirely.

Which of the tests from bug 1358666 replaces this one?
(Reporter)

Comment 8

9 months ago
Every single tests tests for the root bounds to be correct, but especially iframe-no-root.html. Notice that the really only reason we loaded this test in a separate window was to make sure the mochitest harness doesn't run it in a nested iframe.
You need to log in before you can comment on or make changes to this bug.