RESOLVED FIXED 118302
[Qt][WK1] Support direct painting without GraphicsSurface
https://bugs.webkit.org/show_bug.cgi?id=118302
Summary [Qt][WK1] Support direct painting without GraphicsSurface
Allan Sandfeld Jensen
Reported 2013-07-02 06:53:53 PDT
In bug #118251 I reintroduced WebGL without graphics surfaces. It does however do inefficient pixel copies from one OpenGL context to the other. A few simple changes should allow us to do more effective texture copy.
Attachments
Patch (3.87 KB, patch)
2013-07-02 06:58 PDT, Allan Sandfeld Jensen
no flags
Patch (4.98 KB, patch)
2013-07-22 06:30 PDT, Allan Sandfeld Jensen
jturcotte: review+
Allan Sandfeld Jensen
Comment 1 2013-07-02 06:58:02 PDT
Zeno Albisser
Comment 2 2013-07-11 07:06:02 PDT
Comment on attachment 205908 [details] Patch LGTM
Jocelyn Turcotte
Comment 3 2013-07-18 05:40:33 PDT
Comment on attachment 205908 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=205908&action=review > Source/WebCore/platform/graphics/qt/GraphicsContext3DQt.cpp:257 > +#else What happens if we're running WebKit2 in a !USE(GRAPHICS_SURFACE) build? Maybe we also need runtime detection for WK1/2. > Source/WebCore/platform/graphics/qt/GraphicsContext3DQt.cpp:263 > + Not needed.
Allan Sandfeld Jensen
Comment 4 2013-07-22 06:26:46 PDT
(In reply to comment #3) > (From update of attachment 205908 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=205908&action=review > > > Source/WebCore/platform/graphics/qt/GraphicsContext3DQt.cpp:257 > > +#else > > What happens if we're running WebKit2 in a !USE(GRAPHICS_SURFACE) build? Maybe we also need runtime detection for WK1/2. > There was a crash I have fixed, but we do not support canvas content in coordinated graphics without graphics-surfaces.
Allan Sandfeld Jensen
Comment 5 2013-07-22 06:30:48 PDT
Jocelyn Turcotte
Comment 6 2013-07-22 06:58:10 PDT
Comment on attachment 205908 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=205908&action=review >>> Source/WebCore/platform/graphics/qt/GraphicsContext3DQt.cpp:257 >>> +#else >> >> What happens if we're running WebKit2 in a !USE(GRAPHICS_SURFACE) build? Maybe we also need runtime detection for WK1/2. > > There was a crash I have fixed, but we do not support canvas content in coordinated graphics without graphics-surfaces. Ok I was wrong, paintToTextureMapper will never be called for WK2 and "textureMapper->accelerationMode() == TextureMapper::OpenGLMode" will be enough to pick the right code path.
Jocelyn Turcotte
Comment 7 2013-07-22 06:59:46 PDT
Comment on attachment 207244 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=207244&action=review > Source/WebCore/platform/graphics/qt/GraphicsContext3DQt.cpp:98 > + bool m_canPaintDirectly; r=me if you remove this member.
Allan Sandfeld Jensen
Comment 8 2013-07-22 07:03:41 PDT
Note You need to log in before you can comment on or make changes to this bug.