Bug 114726 - [BlackBerry] GraphicsContext3D: don't initialize m_extensions in the constructor
Summary: [BlackBerry] GraphicsContext3D: don't initialize m_extensions in the constructor
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit BlackBerry (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-17 01:36 PDT by Alberto Garcia
Modified: 2013-04-17 02:17 PDT (History)
6 users (show)

See Also:

Attachments
Patch (2.72 KB, patch)
2013-04-17 01:37 PDT, Alberto Garcia 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alberto Garcia 2013-04-17 01:36:36 PDT 
m_extensions is now initialized on demand by GraphicsContext3D::getExtensions().

Doing it in the constructor produces a crash, because Extensions3DOpenGLES() calls glGetString before the WebGL context is current.
Comment 1 Alberto Garcia 2013-04-17 01:37:46 PDT 
Created attachment 198482 [details]
Patch
Comment 2 WebKit Commit Bot 2013-04-17 02:17:08 PDT 
Comment on attachment 198482 [details]
Patch

Clearing flags on attachment: 198482

Committed r148595: <http://trac.webkit.org/changeset/148595>
Comment 3 WebKit Commit Bot 2013-04-17 02:17:11 PDT 
All reviewed patches have been landed.  Closing bug.