If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Convert more tests within layout/ to comply with new data: URI inheritance model

RESOLVED FIXED in Firefox 56

Status

()

Core
DOM: Security
P2
normal
RESOLVED FIXED
2 months ago
2 months ago

People

(Reporter: ckerschb, Assigned: ckerschb)

Tracking

unspecified
mozilla56
Points:
---

Firefox Tracking Flags

(firefox56 fixed)

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Updated

2 months ago
Assignee: nobody → ckerschb
Blocks: 1324406
Status: NEW → ASSIGNED
Priority: -- → P2
See Also: → bug 1337270
Whiteboard: [domsecurity-active]
(Assignee)

Comment 1

2 months ago
Created attachment 8889343 [details] [diff] [review]
bug_1383649_layout_tests.patch

Hey Olli, it seems the following tests within layout/ can all be fixed by using SpecialPowers:
1: layout/base/tests/test_bug399284.html
2: layout/style/test/test_media_queries.html
3: layout/style/test/test_namespace_rule.html
4: layout/style/test/test_property_syntax_errors.html
5: layout/style/test/test_selectors.html
6: layout/style/test/test_value_cloning.html

There setup is very identical and I don't think it makes sense to rewrite them other than using SpecialPowers. Agreed?
Attachment #8889343 - Flags: review?(bugs)

Updated

2 months ago
Attachment #8889343 - Flags: review?(bugs) → review+

Comment 2

2 months ago
Pushed by mozilla@christophkerschbaumer.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/57601415b9a2
Convert more tests within layout/ to comply with new data: URI inheritance model. r=smaug

Comment 3

2 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/57601415b9a2
Status: ASSIGNED → RESOLVED
Last Resolved: 2 months ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.