In nsComputedDOMStyle::SetValueToStyleImage, use the URL stored in the nsStyleImage's mImageValue if we had some problem resolving the imgRequestProxy
RESOLVED
FIXED
in Firefox 55
Status
()
People
(Reporter: u459114, Assigned: u459114)
Tracking
Firefox Tracking Flags
(firefox55 fixed)
Details
MozReview Requests
()
| Submitter | Diff | Changes | Open Issues | Last Updated |
|---|---|---|---|---|
| Loading... | ||||
| Error loading review requests: | ||||
Attachments
(1 attachment)
While working on 1301245, I found these comments [1]/[2]. So, file this bug to fix these XXX [1] https://hg.mozilla.org/mozilla-central/file/9f17f5e31f3d/layout/style/nsComputedDOMStyle.cpp#l2268 [2] https://hg.mozilla.org/mozilla-central/file/9f17f5e31f3d/layout/style/nsComputedDOMStyle.cpp#l3697
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
Attachment #8845238 -
Flags: review?(cam)
|
||
Comment 3•a year ago
|
||
| mozreview-review | ||
Comment on attachment 8845238 [details] Bug 1345739 - Implement nsStyleImageRequest::GetImageURI. https://reviewboard.mozilla.org/r/118426/#review120380 Thanks for fixing this. ::: layout/style/nsStyleStruct.cpp:2269 (Diff revision 2) > + return uri.forget(); > + } > + } > + > + // If we had some problem resolving the mRequestProxy, use the URL stored > + // in the mImageValue. Nit: remove one of the spaces after "the".
Attachment #8845238 -
Flags: review?(cam) → review+
| Comment hidden (mozreview-request) |
Pushed by cku@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7368729b7ba0 Implement nsStyleImageRequest::GetImageURI. r=heycam
|
||
Comment 6•a year ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/7368729b7ba0
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•