Bug 75462 - Split GraphicsContext3DOpenGL into several files
Summary: Split GraphicsContext3DOpenGL into several files
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebGL (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 75466
Blocks: 59064 66903
  Show dependency treegraph
 
Reported: 2012-01-02 22:37 PST by ChangSeok Oh
Modified: 2012-01-12 20:51 PST (History)
2 users (show)

See Also:


Attachments
Patch (87.46 KB, patch)
2012-01-10 08:41 PST, ChangSeok Oh
no flags Details | Formatted Diff | Diff
Patch (87.40 KB, patch)
2012-01-10 09:59 PST, ChangSeok Oh
no flags Details | Formatted Diff | Diff
Patch (87.57 KB, patch)
2012-01-10 10:37 PST, ChangSeok Oh
no flags Details | Formatted Diff | Diff
Patch (87.65 KB, patch)
2012-01-12 08:37 PST, ChangSeok Oh
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description ChangSeok Oh 2012-01-02 22:37:15 PST
This bug is for spliting GraphicsContext3DOpenGL into following files as Chris Marrin mentioned.
+GraphicsContext3DOpenGLCommon - code common to any OpenGL implementation
+GraphicsContext3DOpenGL - OpenGL (not OpenGL ES) specific code)
+GraphicsContext3DGLES - OpenGL ES specific code
Comment 1 ChangSeok Oh 2012-01-10 08:41:25 PST
Created attachment 121852 [details]
Patch
Comment 2 ChangSeok Oh 2012-01-10 09:59:47 PST
Created attachment 121862 [details]
Patch
Comment 3 ChangSeok Oh 2012-01-10 10:37:17 PST
Created attachment 121866 [details]
Patch
Comment 4 Kenneth Rohde Christiansen 2012-01-11 01:44:49 PST
Comment on attachment 121866 [details]
Patch

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

> Source/WebCore/ChangeLog:13
> +
> +        Split GraphicsContext3DOpenGL.cpp into
> +        GraphicsContext3DOpenGLCommon.cpp, GraphicsContext3DOpenGL.cpp & GraphicsContext3DOpenGLES.cpp.
> +        Most of common APIs between gl and gles are in GraphicsContext3DOpenGLCommon.cpp.
> +        The other gl and gles specific APIs are placed apart in GraphicsContext3DOpenGL.cpp & GraphicsContext3DOpenGLES.cpp
> +
> +        No new tests required.

It would be good if you could write why this is an advantage. Also can you try to keep the lines shorter (around 80-100 chars)
Comment 5 ChangSeok Oh 2012-01-12 08:37:16 PST
Created attachment 122251 [details]
Patch
Comment 6 ChangSeok Oh 2012-01-12 08:38:45 PST
Comment on attachment 121866 [details]
Patch

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

Thank you for review. :)

>> Source/WebCore/ChangeLog:13
>> +        No new tests required.
> 
> It would be good if you could write why this is an advantage. Also can you try to keep the lines shorter (around 80-100 chars)

Done.
Comment 7 Kenneth Russell 2012-01-12 19:27:25 PST
Comment on attachment 122251 [details]
Patch

Nice refactoring. r=me
Comment 8 WebKit Review Bot 2012-01-12 20:51:37 PST
Comment on attachment 122251 [details]
Patch

Clearing flags on attachment: 122251

Committed r104896: <http://trac.webkit.org/changeset/104896>
Comment 9 WebKit Review Bot 2012-01-12 20:51:42 PST
All reviewed patches have been landed.  Closing bug.