Bug 54006 - [Qt] Patch for compiling QtWebKit with WebGL (--3d-canvas) support
Summary: [Qt] Patch for compiling QtWebKit with WebGL (--3d-canvas) support
Status: RESOLVED DUPLICATE of bug 53431
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P3 Normal
Assignee: Nobody
URL:
Keywords: Qt
Depends on:
Blocks:
 
Reported: 2011-02-08 08:26 PST by Dawit A.
Modified: 2011-03-08 09:43 PST (History)
1 user (show)

See Also:


Attachments
proposed patch v1 (15.44 KB, patch)
2011-02-08 08:45 PST, Dawit A.
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dawit A. 2011-02-08 08:26:56 PST
The attached patch fixes compilation of QtWebKit with WebGL support. Everything is straight forward except for the new file that was added to implement the draw buffering (platform/graphics/qt/DrawingBufferQt.cpp). Due to my lack of knowledge of 3D graphics programming, this newly added file, based on the chromium and mac ports, is there to simply ensure proper compilation and not proper functionality. More specifically the platform (Qt) specific layer support needs to be implemented for this class.
Comment 1 Dawit A. 2011-02-08 08:45:22 PST
Created attachment 81640 [details]
proposed patch v1

Important note I forgot to mention was that the need to replace the function named "emit" with "emitChar" in "html/canvas/WebGLRenderingContext.cpp" arose from compilation failure due to conflict with Qt's signal/slot keyword.
Comment 2 Benjamin Poulain 2011-02-09 03:10:22 PST
Andrew Wason already started working on that on https://bugs.webkit.org/show_bug.cgi?id=53431

Dawit, it would be nice if you could sync with him.

*** This bug has been marked as a duplicate of bug 53431 ***
Comment 3 Eric Seidel (no email) 2011-03-08 09:43:36 PST
Comment on attachment 81640 [details]
proposed patch v1

Cleared review? from attachment 81640 [details] so that this bug does not appear in http://webkit.org/pending-review.  If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).