According to Fragment URLs defined in [1], when serializing a url() with the local url flag set, it must serialize as just the fragment. [1]https://drafts.csswg.org/css-values/#local-urls

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

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

Comment on attachment 8778477 [details] Bug 1288812 - Part 1. Serialize just the fragment for local urls. Review request updated; see interdiff: https://reviewboard.mozilla.org/r/69782/diff/1-2/

Comment on attachment 8778478 [details] Bug 1288812 - Part 2. Correct test cases. Review request updated; see interdiff: https://reviewboard.mozilla.org/r/69784/diff/1-2/

Comment on attachment 8778477 [details] Bug 1288812 - Part 1. Serialize just the fragment for local urls. https://reviewboard.mozilla.org/r/69782/#review67008 ::: layout/style/nsComputedDOMStyle.cpp:78 (Diff revision 2) > aStyleType); > return computedStyle.forget(); > } > > +static void > +FragmentOrURLToROCSSPrimitiveValue(const FragmentOrURL* aFragmentOrURL, Some other similar functions in this file just use "Value" in their name to refer to the target nsROCSSPrimitiveValue, like SetValueToStyleImage, SetValueToPositionCoord, SetValueToPosition. So let's use similar naming here -- SetValueToFragmentOrURL.

Comment on attachment 8778478 [details] Bug 1288812 - Part 2. Correct test cases. https://reviewboard.mozilla.org/r/69784/#review67012 Can we add some specific tests for this new serialization, too? Something that checks that when we use url(#a) that it serializes like that and when we use url(name_of_test_file.html#a) that it serializes to the absolute URL?

Pushed by cku@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/279cf3d56811 Part 1. Serialize just the fragment for local urls. r=heycam https://hg.mozilla.org/integration/autoland/rev/b396323fb8a7 Part 2. Correct test cases. r=heycam

https://hg.mozilla.org/mozilla-central/rev/279cf3d56811 https://hg.mozilla.org/mozilla-central/rev/b396323fb8a7

https://hg.mozilla.org/integration/mozilla-inbound/rev/0d754bde37ab9b7d3f25bd569b77cdfd4ee7e423 Bug 1288812 - (follow-up) Part 3. Correct test_transitions_per_property.html.

https://hg.mozilla.org/integration/mozilla-inbound/rev/e84e72b0a967bcbc362e1a66685d87f49e850bdd Bug 1288812 - (follow-up) Part 3. Correct test_transitions_per_property.html. r=me

Backout by ihsiao@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/6742cb73d63f Backed out changeset 0d754bde37ab for CJKu's request