Closed
Bug 1353299
Opened 8 years ago
Closed 8 years ago
make sure to invalidate when composited frame becomes valid
Categories
(Core :: Graphics: ImageLib, enhancement)
Core
Graphics: ImageLib
Tracking
()
RESOLVED
FIXED
mozilla55
| Tracking | Status | |
|---|---|---|
| firefox55 | --- | fixed |
People
(Reporter: tnikkel, Assigned: tnikkel)
References
Details
Attachments
(1 file, 1 obsolete file)
|
9.37 KB,
patch
|
aosmond
:
review+
|
Details | Diff | Splinter Review |
If the composited frame goes from being invalid (we draw nothing) to valid (we draw something) we should invalidate. Most cases there will be an invalidate for whatever change caused the composited frame to become valid, but not always.
|
Assignee | |
Comment 1•8 years ago
|
||
Attachment #8854361 -
Flags: review?(aosmond)
|
|
Assignee | |
Comment 2•8 years ago
|
||
Attachment #8854361 -
Attachment is obsolete: true
Attachment #8854361 -
Flags: review?(aosmond)
Attachment #8854779 -
Flags: review?(aosmond)
|
||
Comment 3•8 years ago
|
||
Comment on attachment 8854779 [details] [diff] [review]
patch v2
Review of attachment 8854779 [details] [diff] [review]:
-----------------------------------------------------------------
::: image/RasterImage.cpp
@@ +1730,3 @@
> }
>
> +
nit: remove extra blank line
Attachment #8854779 -
Flags: review?(aosmond) → review+
Pushed by tnikkel@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8b4e96304c8d
Make sure to invalidate when composited frame becomes valid. r=aosmond
|
||
Comment 5•8 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•