Closed
Bug 981430
Opened 11 years ago
Closed 11 years ago
Convert WinUtils consumer of imgIContainer::GetFrame to act on a Moz2D SourceSurface
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla30
People
(Reporter: jwatt, Assigned: jwatt)
References
Details
Attachments
(1 file)
5.02 KB,
patch
|
mattwoodrow
:
review+
|
Details | Diff | Splinter Review |
We should convert the WinUtils consumer of imgIContainer::GetFrame to act on a Moz2D SourceSurface.
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8388233 -
Flags: review?(matt.woodrow)
Comment 2•11 years ago
|
||
Comment on attachment 8388233 [details] [diff] [review]
patch
Review of attachment 8388233 [details] [diff] [review]:
-----------------------------------------------------------------
::: widget/windows/WinUtils.cpp
@@ +763,5 @@
> + ColorPattern(Color(1.0f, 1.0f, 1.0f, 1.0f)));
> + dt->DrawSurface(surface,
> + Rect(16, 16, 16, 16),
> + Rect(Point(0, 0),
> + Size(surface->GetSize().width, surface->GetSize().height)));
This will scale the input surface to 16x16, the old code just drew 16x16 pixels from the top left of it.
See http://mxr.mozilla.org/mozilla-central/source/gfx/thebes/gfxContext.cpp#595
@@ +778,5 @@
> }
>
> + // Allocate a new buffer that we own and can use out of line in
> + // another thread.
> + uint8_t *data = SurfaceToPackedBGRA(dataSurface);
Still an unnecessary copy here in the mURLShortcut case, but not a regression so I guess nobody cares too much.
Attachment #8388233 -
Flags: review?(matt.woodrow) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Comment 4•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in
before you can comment on or make changes to this bug.
Description
•