Closed Bug 1288812 Opened 8 years ago Closed 8 years ago

When serializing a url() with the local url flag set, it must serialize as just the fragment.

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: u459114, Assigned: u459114)

References

Details

Attachments

(2 files)

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
Depends on: 1291280
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/
Attachment #8778477 - Flags: review?(cam)
Attachment #8778478 - Flags: review?(cam)
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.
Attachment #8778477 - Flags: review?(cam) → review+
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?
Attachment #8778478 - Flags: review?(cam) → review+
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
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
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
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: