RESOLVED FIXED 75462
Split GraphicsContext3DOpenGL into several files 
https://bugs.webkit.org/show_bug.cgi?id=75462
Summary Split GraphicsContext3DOpenGL into several files
ChangSeok Oh
Reported 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
Attachments
Patch (87.46 KB, patch)
2012-01-10 08:41 PST, ChangSeok Oh 		no flags
DetailsFormatted DiffDiff
Patch (87.40 KB, patch)
2012-01-10 09:59 PST, ChangSeok Oh 		no flags
DetailsFormatted DiffDiff
Patch (87.57 KB, patch)
2012-01-10 10:37 PST, ChangSeok Oh 		no flags
DetailsFormatted DiffDiff
Patch (87.65 KB, patch)
2012-01-12 08:37 PST, ChangSeok Oh 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
ChangSeok Oh
Comment 1 2012-01-10 08:41:25 PST
Created attachment 121852 [details] Patch
ChangSeok Oh
Comment 2 2012-01-10 09:59:47 PST
Created attachment 121862 [details] Patch
ChangSeok Oh
Comment 3 2012-01-10 10:37:17 PST
Created attachment 121866 [details] Patch
Kenneth Rohde Christiansen
Comment 4 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)
ChangSeok Oh
Comment 5 2012-01-12 08:37:16 PST
Created attachment 122251 [details] Patch
ChangSeok Oh
Comment 6 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.
Kenneth Russell
Comment 7 2012-01-12 19:27:25 PST
Comment on attachment 122251 [details] Patch Nice refactoring. r=me
WebKit Review Bot
Comment 8 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>
WebKit Review Bot
Comment 9 2012-01-12 20:51:42 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.