1854809 - Include quotes in AtomString CSS serialization

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, task)

Description

[Oliver Medhurst [:canadahonk]]

The serialization of AtomString should include quotes inside itself rather than users of it having to do it themselves.

Idea from :emilio in this patch comment: https://phabricator.services.mozilla.com/D176801#inline-1044682

Comment 1

[Oliver Medhurst [:canadahonk]]

Attachment: Bug 1854809 - Include quotes in AtomString CSS serialization

The serialization of AtomString should include quotes itself
rather than users of it having to do so themselves.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f4a2f296a2ea
Include quotes in AtomString CSS serialization r=emilio

Comment 3

[Atila Butkovits]

Backed out for causing build bustages.

Backout link: https://hg.mozilla.org/integration/autoland/rev/fada0a57f99d5b5fc87e6e746df5d5b55781f57d

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=f4a2f296a2ea585882f2a2a627c036538eb875b0&selectedTaskRun=dpGgDDPUSsGAYR-TbyVgpw.0

Failure log:
https://treeherder.mozilla.org/logviewer?job_id=430131449&repo=autoland&lineNumber=12296
https://treeherder.mozilla.org/logviewer?job_id=430131443&repo=autoland&lineNumber=15107

Comment 4

[Natalia Csoregi [:nataliaCs]]

• Backout merged to mozilla-central: https://hg.mozilla.org/mozilla-central/rev/fada0a57f99d5b5fc87e6e746df5d5b55781f57d

Comment 5

[Oliver Medhurst [:canadahonk]]

Apologies, missed internal rust tests failing. Fixed in new patch revision.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a49b5227bc93
Include quotes in AtomString CSS serialization r=emilio

Comment 7

[Sandor Molnar[:smolnar]]

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