Closed Bug 1341690 Opened 5 years ago Closed 5 years ago

stylo: background-image doesn't work in inline style

Categories

(Core :: CSS Parsing and Computation, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: bzbarsky, Assigned: manishearth)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

This is what causes the really-big-background.html test failure.  Probably relevant error message:

WARN:style::values::specified::url: stylo: skipping declaration without ParserContextExtraData
Depends on: 1310886
This is getting fixed in bug 1329093
Depends on: 1329093
Can we reenable the testcase in comment 0 now?
Flags: needinfo?(manishearth)
yes
Flags: needinfo?(manishearth)
Assignee: nobody → manishearth
Status: NEW → ASSIGNED
Hm, this test still fails. Wat.
Ah, I'd missed ParseStyleAttribute in my other patch. Fixing.
Comment on attachment 8846191 [details]
Bug 1341690 - stylo: Make URLs work in inline style;

https://reviewboard.mozilla.org/r/119296/#review121220

r=me with that.

::: layout/style/nsDOMCSSDeclaration.cpp:135
(Diff revision 3)
>    mozAutoDocConditionalContentUpdateBatch autoUpdate(DocToUpdate(), true);
>  
>    RefPtr<DeclarationBlock> newdecl;
>    if (olddecl->IsServo()) {
> -    newdecl = ServoDeclarationBlock::FromCssText(aCssText);
> +    GeckoParserExtraData data(env.mBaseURI, env.mSheetURI, env.mPrincipal);
> +    nsCString baseString;

Let's move this to `FromCSSText`, where we already have the base uri in `GeckoParserExtraData` instead of duplicating it twice.
Attachment #8846191 - Flags: review?(emilio+bugs) → review+
Looks like the test should be able to be reenabled now.
Priority: -- → P1
Yep, just need to autoland this, but that's blocked on VCS sync waking up
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 15ad85dc898c -d e18199b999a3: rebasing 381624:15ad85dc898c "Bug 1341690 - stylo: Make URLs work in inline style; r=emilio" (tip)
merging dom/base/nsAttrValue.cpp
merging layout/style/ServoBindingList.h
merging layout/style/test/stylo-failures.md
warning: conflicts while merging layout/style/test/stylo-failures.md! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Pushed by cmccormack@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8949490f91a7
stylo: Make URLs work in inline style; r=emilio
https://hg.mozilla.org/mozilla-central/rev/8949490f91a7
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.