RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 16:27:35 +00:00
Code Issues Activity Actions

LibWeb: Remove redundant flush() call in PaintingCommandExecutorGPU

Browse source 
Since we already call `Painter::flush()` in `PageHost::paint()` we do
not need to do that again in `PaintingCommandExecutorGPU` destructor.

This makes GPU painting run noticeably faster because `flush()` does
expensive `glReadPixels()` call.
This commit is contained in:
Aliaksandr Kalenik 2023-11-12 20:16:06 +01:00 • committed by Andreas Kling
parent b532dedc91
commit f7874d03fc
1 changed files with 0 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

1
Userland/Libraries/LibWeb/Painting/PaintingCommandExecutorGPU.cpp
View file

@ -15,7 +15,6 @@ PaintingCommandExecutorGPU::PaintingCommandExecutorGPU(AccelGfx::Painter& painte
PaintingCommandExecutorGPU::~PaintingCommandExecutorGPU() PaintingCommandExecutorGPU::~PaintingCommandExecutorGPU()
{ {
m_painter.flush();
} }
CommandResult PaintingCommandExecutorGPU::draw_glyph_run(Vector<Gfx::DrawGlyphOrEmoji> const& glyph_run, Color const& color) CommandResult PaintingCommandExecutorGPU::draw_glyph_run(Vector<Gfx::DrawGlyphOrEmoji> const& glyph_run, Color const& color)