1293164 - Add some specific tests for fragment url serialization.

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[u459114]

According to bug 1288812 comment 6, file this one to create new test cases.

Comment 1

[u459114]

Create test cases in layout/style/test/test_computed_style.html.
1. computed style value of local-ref url, such as url(#foo), should be a local-ref.
2. computed style value of non-local-ref url, such as url(foo.svg#foo), should be absolute url.

Comment 2

[u459114]

Attachment: Bug 1293164 - Create serialization test for local-ref URL.

Review commit: https://reviewboard.mozilla.org/r/72510/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/72510/

Comment 3

[u459114]

Comment on attachment 8782302 [details]
Bug 1293164 - Create serialization test for local-ref URL.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/72510/diff/1-2/

Comment 4

[Cameron McCormack (:heycam)]

Comment on attachment 8782302 [details]
Bug 1293164 - Create serialization test for local-ref URL.

https://reviewboard.mozilla.org/r/72510/#review70136

::: layout/style/test/test_computed_style.html:370
(Diff revision 2)
> +
> +  for (var prop in postFix) {
> +    p.style[prop] = localURL;
> +    is(cs[prop], localURL + testStyles[prop], "computed value of " + prop);
> +    p.style[prop] = nonLocalURL;
> +    is(cs[prop], resolvedNonLocalURL + testStyles[prop], "computed value of "  + prop);

Nit: remove one space before the last "+".

Comment 5

[u459114]

Comment on attachment 8782302 [details]
Bug 1293164 - Create serialization test for local-ref URL.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/72510/diff/2-3/

Comment 6

[u459114]

Comment on attachment 8782302 [details]
Bug 1293164 - Create serialization test for local-ref URL.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/72510/diff/3-4/

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e6b28b84c2cb
Create serialization test for local-ref URL. r=heycam

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/e6b28b84c2cb