Bug 104705 - [Texmap] Consolidate redundant code for ARB_RECT texture
Summary: [Texmap] Consolidate redundant code for ARB_RECT texture
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Noam Rosenthal
URL:
Keywords:
Depends on: 104776
Blocks:
  Show dependency treegraph
 
Reported: 2012-12-11 12:56 PST by Noam Rosenthal
Modified: 2012-12-12 12:12 PST (History)
13 users (show)

See Also:


Attachments
Patch (19.98 KB, patch)
2012-12-11 13:02 PST, Noam Rosenthal
no flags Details | Formatted Diff | Diff
Patch for landing (20.01 KB, patch)
2012-12-11 16:48 PST, Noam Rosenthal
no flags Details | Formatted Diff | Diff
Patch (20.07 KB, patch)
2012-12-12 07:10 PST, Noam Rosenthal
no flags Details | Formatted Diff | Diff
Patch (21.13 KB, patch)
2012-12-12 07:26 PST, Noam Rosenthal
no flags Details | Formatted Diff | Diff
Patch (21.15 KB, patch)
2012-12-12 07:47 PST, Noam Rosenthal
no flags Details | Formatted Diff | Diff
Patch (22.21 KB, patch)
2012-12-12 08:31 PST, Noam Rosenthal
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Noam Rosenthal 2012-12-11 12:56:15 PST
[Texmap] Consolidte redundant code for ARB_RECT texture
Comment 1 Noam Rosenthal 2012-12-11 13:02:23 PST
Created attachment 178857 [details]
Patch
Comment 2 Noam Rosenthal 2012-12-11 13:04:04 PST
Thanks for fixing the typo... was just about to do it myself.
Comment 3 Kenneth Rohde Christiansen 2012-12-11 16:36:43 PST
Comment on attachment 178857 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=178857&action=review

> Source/WebCore/platform/graphics/texmap/TextureMapperGL.h:48
> +        ShouldUseRectTexture = 0x04

ShouldUseARBTextureRect seems more correct.
Comment 4 Noam Rosenthal 2012-12-11 16:48:02 PST
Created attachment 178919 [details]
Patch for landing
Comment 5 WebKit Review Bot 2012-12-11 17:55:36 PST
Comment on attachment 178919 [details]
Patch for landing

Rejecting attachment 178919 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
m/graphics/texmap/TextureMapperGL.cpp
patching file Source/WebCore/platform/graphics/texmap/TextureMapperGL.h
patching file Source/WebCore/platform/graphics/texmap/TextureMapperShaderManager.cpp
Hunk #2 succeeded at 243 (offset 18 lines).
Hunk #3 succeeded at 256 (offset 18 lines).
patching file Source/WebCore/platform/graphics/texmap/TextureMapperShaderManager.h

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue

Full output: http://queues.webkit.org/results/15272520
Comment 6 Csaba Osztrogonác 2012-12-12 00:53:51 PST
It broke the Qt build:
cc1plus: warnings being treated as errors
/home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp: In member function 'void WebCore::TextureMapperGL::drawTexturedQuadWithProgram(WebCore::TextureMapperShaderProgram*, uint32_t, int, const WebCore::IntSize&, const WebCore::TextureMapperGL::DrawQuad&, const WebCore::TransformationMatrix&, float, const WebCore::BitmapTexture*)':
[1;31m/home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp:579: error: enumeral mismatch in conditional expression: 'WebCore::Extensions3D::ExtensionsEnumType' vs 'WebCore::GraphicsContext3D::<anonymous enum>'[0m
Comment 7 Csaba Osztrogonác 2012-12-12 01:00:14 PST
and the EFL build:

/home/buildslave-1/webkit-buildslave/efl-linux-64-release-wk2/build/Source/WebCore/platform/graphics/surfaces/glx/GraphicsSurfaceGLX.cpp: In member function ‘void WebCore::GraphicsSurface::platformPaintToTextureMapper(WebCore::TextureMapper*, const WebCore::FloatRect&, const WebCore::TransformationMatrix&, float, WebCore::BitmapTexture*)’:
/home/buildslave-1/webkit-buildslave/efl-linux-64-release-wk2/build/Source/WebCore/platform/graphics/surfaces/glx/GraphicsSurfaceGLX.cpp:442:14: error: ‘SupportsBlending’ is not a member of ‘WebCore::TextureMapperGL’
make[2]: *** [Source/WebCore/CMakeFiles/webcore_efl.dir/platform/graphics/surfaces/glx/GraphicsSurfaceGLX.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
/home/buildslave-1/webkit-buildslave/efl-linux-64-release-wk2/build/Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp: In member function ‘void WebCore::TextureMapperGL::drawTexturedQuadWithProgram(WebCore::TextureMapperShaderProgram*, uint32_t, WebCore::TextureMapperGL::Flags, const WebCore::IntSize&, const WebCore::TextureMapperGL::DrawQuad&, const WebCore::TransformationMatrix&, float, const WebCore::BitmapTexture*)’:
/home/buildslave-1/webkit-buildslave/efl-linux-64-release-wk2/build/Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp:579:114: error: enumeral mismatch in conditional expression: ‘WebCore::Extensions3D::ExtensionsEnumType’ vs ‘WebCore::GraphicsContext3D::<anonymous enum>’ [-Werror]
cc1plus: all warnings being treated as errors


and the GTK build:

../../Source/WebCore/platform/graphics/cairo/GraphicsContext3DPrivate.cpp: In member function ‘virtual void WebCore::GraphicsContext3DPrivate::paintToTextureMapper(WebCore::TextureMapper*, const WebCore::FloatRect&, const WebCore::TransformationMatrix&, float, WebCore::BitmapTexture*)’:
../../Source/WebCore/platform/graphics/cairo/GraphicsContext3DPrivate.cpp:141:101: error: ‘SupportsBlending’ is not a member of ‘WebCore::TextureMapperGL’
make[1]: *** [Source/WebCore/platform/graphics/cairo/libWebCore_la-GraphicsContext3DPrivate.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
../../Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp: In member function ‘void WebCore::TextureMapperGL::drawTexturedQuadWithProgram(WebCore::TextureMapperShaderProgram*, uint32_t, WebCore::TextureMapperGL::Flags, const WebCore::IntSize&, const WebCore::TextureMapperGL::DrawQuad&, const WebCore::TransformationMatrix&, float, const WebCore::BitmapTexture*)’:
../../Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp:579:114: warning: enumeral mismatch in conditional expression: ‘WebCore::Extensions3D::ExtensionsEnumType’ vs ‘WebCore::GraphicsContext3D::<anonymous enum>’ [-Wenum-compare]
make[1]: Leaving directory `/home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release'
Comment 8 Csaba Osztrogonác 2012-12-12 01:03:59 PST
I meant https://trac.webkit.org/changeset/137428. It's strange for me if CQ landed it but didn't comment and close the bug. How is it possible?
Comment 9 Csaba Osztrogonác 2012-12-12 01:20:30 PST
Rollout landed in http://trac.webkit.org/changeset/137431

You can simple avoid this situations if you upload appliable patches 
and wait a little bit for EWS bubbles to become green. Or land the 
patch manually and wait for the buildbots.
Comment 10 Csaba Osztrogonác 2012-12-12 01:28:50 PST
And how could CQ landed it if it wasn't appliable? ...
Comment 11 Noam Rosenthal 2012-12-12 07:10:33 PST
Created attachment 179032 [details]
Patch
Comment 12 Noam Rosenthal 2012-12-12 07:12:04 PST
(In reply to comment #10)
> And how could CQ landed it if it wasn't appliable? ...

Yes, I wonder. I would expect this kind of problem if I landed manually, not with CQ.
Comment 13 Early Warning System Bot 2012-12-12 07:19:28 PST
Comment on attachment 179032 [details]
Patch

Attachment 179032 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/15296131
Comment 14 Early Warning System Bot 2012-12-12 07:20:09 PST
Comment on attachment 179032 [details]
Patch

Attachment 179032 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/15260869
Comment 15 Noam Rosenthal 2012-12-12 07:26:42 PST
Created attachment 179037 [details]
Patch
Comment 16 Noam Rosenthal 2012-12-12 07:28:45 PST
btw this is the kind of breakage that is really easy to fix without a rollout.
Comment 17 Early Warning System Bot 2012-12-12 07:39:52 PST
Comment on attachment 179037 [details]
Patch

Attachment 179037 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/15259997
Comment 18 Early Warning System Bot 2012-12-12 07:44:15 PST
Comment on attachment 179037 [details]
Patch

Attachment 179037 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/15282523
Comment 19 Noam Rosenthal 2012-12-12 07:47:46 PST
Created attachment 179042 [details]
Patch
Comment 20 Noam Rosenthal 2012-12-12 08:31:53 PST
Created attachment 179053 [details]
Patch
Comment 21 Csaba Osztrogonác 2012-12-12 08:38:46 PST
If you don't set the r?, you should press the "Submit for EWS" 
button to make the bots test the patch. ;) (I pressed it for you now.)
Comment 22 Noam Rosenthal 2012-12-12 08:47:19 PST
(In reply to comment #16)
> btw this is the kind of breakage that is really easy to fix without a rollout.

Actually I take it back. I should have seen this rollout coming :)
Comment 23 WebKit Review Bot 2012-12-12 12:11:56 PST
Comment on attachment 179053 [details]
Patch

Clearing flags on attachment: 179053

Committed r137498: <http://trac.webkit.org/changeset/137498>
Comment 24 WebKit Review Bot 2012-12-12 12:12:03 PST
All reviewed patches have been landed.  Closing bug.