Closed
Bug 1364830
Opened 7 years ago
Closed 7 years ago
Fix image flags for background image layer
Categories
(Core :: Graphics: WebRender, enhancement)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: ethlin, Assigned: ethlin)
References
Details
Attachments
(1 file, 1 obsolete file)
The wrong image flags and wrong draw result cause the image disappearing issue. I will fix it.
Assignee | ||
Comment 1•7 years ago
|
||
In this patch, I correct the draw result of background image layer and also correct the image flags. The try result[1] doesn't have image disappearing issue. I also check the original gecko try[2] without the pref on, and the result looks fine. [1] https://treeherder.mozilla.org/#/jobs?repo=try&revision=30cc3fb4f2469e2b446390841213de93777b0a9e&selectedJob=99068440 [2] https://treeherder.mozilla.org/#/jobs?repo=try&revision=de5b94a1b9e5924c333c5631f1eec29953724a8c
Attachment #8867638 -
Flags: review?(matt.woodrow)
Comment 2•7 years ago
|
||
Comment on attachment 8867638 [details] [diff] [review] Fix background image layer's draw result and image flags Review of attachment 8867638 [details] [diff] [review]: ----------------------------------------------------------------- r=me without the gradient change. ::: layout/painting/nsCSSRendering.cpp @@ +1983,5 @@ > } > } > > // We only support painting gradients and image for a single style image layer > + return //styleImage->GetType() == eStyleImageType_Gradient || Why are you disabling gradients?
Attachment #8867638 -
Flags: review?(matt.woodrow) → review+
Assignee | ||
Comment 3•7 years ago
|
||
(In reply to Matt Woodrow (:mattwoodrow) from comment #2) > Comment on attachment 8867638 [details] [diff] [review] > Fix background image layer's draw result and image flags > > Review of attachment 8867638 [details] [diff] [review]: > ----------------------------------------------------------------- > > r=me without the gradient change. > > ::: layout/painting/nsCSSRendering.cpp > @@ +1983,5 @@ > > } > > } > > > > // We only support painting gradients and image for a single style image layer > > + return //styleImage->GetType() == eStyleImageType_Gradient || > > Why are you disabling gradients? Oh...That's for test. I will remove this change.
Assignee | ||
Comment 4•7 years ago
|
||
Remove the test code.
Attachment #8867638 -
Attachment is obsolete: true
Pushed by ethlin@mozilla.com: https://hg.mozilla.org/projects/graphics/rev/65542e1d8eae Fix background image layer's draw result and image flags. r=mattwoodrow
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/65542e1d8eae
Updated•7 years ago
|
status-firefox55:
--- → fixed
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•