WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
51716
[GTK] Initial build support for WebGL
https://bugs.webkit.org/show_bug.cgi?id=51716
Summary
[GTK] Initial build support for WebGL
Martin Robinson
Reported
2010-12-29 11:24:57 PST
There are some very simple tasks for bringing WebGL to GTK+. This bug tracks adding the initial build support for WebGL.
Attachments
Initial WebGL build support
(8.24 KB, patch)
2010-12-29 11:43 PST
,
Martin Robinson
no flags
Details
Formatted Diff
Diff
Patch which tries to remove duplicate typedefs
(9.30 KB, patch)
2010-12-30 13:18 PST
,
Martin Robinson
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Martin Robinson
Comment 1
2010-12-29 11:43:54 PST
Created
attachment 77633
[details]
Initial WebGL build support
Eric Seidel (no email)
Comment 2
2010-12-29 18:36:04 PST
Comment on
attachment 77633
[details]
Initial WebGL build support View in context:
https://bugs.webkit.org/attachment.cgi?id=77633&action=review
In general looks fine, but needs more explanation (in code) as to what is permenant and what is temporary? (if any). In general looks fine.
> WebCore/platform/graphics/GraphicsContext3D.h:70 > +#undef VERSION > +#undef None
Huh?
> WebCore/platform/graphics/GraphicsContext3D.h:74 > +typedef unsigned int GLuint; > +typedef void* PlatformGraphicsContext3D; > +const PlatformGraphicsContext3D NullPlatformGraphicsContext3D = 0; > +typedef GLuint Platform3DObject;
Should these be shared between implementations? Seems they all define he same...
Martin Robinson
Comment 3
2010-12-30 13:18:36 PST
Created
attachment 77697
[details]
Patch which tries to remove duplicate typedefs
Martin Robinson
Comment 4
2010-12-30 13:21:50 PST
(In reply to
comment #2
) Thanks for the review!
> In general looks fine, but needs more explanation (in code) as to what is permenant and what is temporary? (if any). In general looks fine.
Everything in this patch is permanent, barring unforseen circumstances. :)
> > WebCore/platform/graphics/GraphicsContext3D.h:70 > > +#undef VERSION > > +#undef None
I've dropped a comment about this in the second version of this patch and removed the #undef None, which is not necessary yet.
> > WebCore/platform/graphics/GraphicsContext3D.h:74 > > +typedef unsigned int GLuint; > > +typedef void* PlatformGraphicsContext3D; > > +const PlatformGraphicsContext3D NullPlatformGraphicsContext3D = 0; > > +typedef GLuint Platform3DObject; > Should these be shared between implementations? Seems they all define he same...
I posted a new version of the patch, which consolidates the similar typedefs. This sacrifices a little bit of clarity for less repitition, but it's a decent tradeoff in my opinion. Let me know which version of the patch you prefer. Thanks!
Eric Seidel (no email)
Comment 5
2011-01-06 15:22:03 PST
Comment on
attachment 77697
[details]
Patch which tries to remove duplicate typedefs LGTM.
Martin Robinson
Comment 6
2011-01-06 16:24:11 PST
Comment on
attachment 77697
[details]
Patch which tries to remove duplicate typedefs Clearing flags on attachment: 77697 Committed
r75209
: <
http://trac.webkit.org/changeset/75209
>
Martin Robinson
Comment 7
2011-01-06 16:24:15 PST
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug