diff options
author | Zhenyao Mo <zmo@google.com> | 2011-06-30 23:24:26 +0000 |
---|---|---|
committer | Steve Kondik <shade@chemlab.org> | 2013-01-20 18:38:32 -0800 |
commit | 4da5f26c019971a319dda77e4bfe6480c269bdf1 (patch) | |
tree | 2ccf19cbab0dddbd80f45d3397526a571ce5b0d3 /Source/WebCore/bindings/v8 | |
parent | c018655c37a1da5d7e1f9dc94d0467130cf630f1 (diff) | |
download | external_webkit-4da5f26c019971a319dda77e4bfe6480c269bdf1.zip external_webkit-4da5f26c019971a319dda77e4bfe6480c269bdf1.tar.gz external_webkit-4da5f26c019971a319dda77e4bfe6480c269bdf1.tar.bz2 |
Improve WebGL object lifetime management in WebGLRenderingContext
2011-06-30 Zhenyao Mo <zmo@google.com>
Reviewed by Kenneth Russell.
Improve WebGL object lifetime management in WebGLRenderingContext
https://bugs.webkit.org/show_bug.cgi?id=63635
* fast/canvas/webgl/gl-object-get-calls-expected.txt:
* fast/canvas/webgl/gl-object-get-calls.html: Fix a bug so getFramebufferAtatchmentParameter generates an error if nothing is attached and something other than TYPE is queried.
* fast/canvas/webgl/object-deletion-behaviour-expected.txt:
* fast/canvas/webgl/object-deletion-behaviour.html: Ditto.
* fast/canvas/webgl/program-test.html: Fix the test so the order of shaders returned by getAttachedShaders doesn't matter.
2011-06-30 Zhenyao Mo <zmo@google.com>
Reviewed by Kenneth Russell.
Improve WebGL object lifetime management in WebGLRenderingContext
https://bugs.webkit.org/show_bug.cgi?id=63635
* html/canvas/WebGLRenderingContext.cpp:
(WebCore::WebGLRenderingContext::getAttachedShaders): Use cached objects instead of querying the underlying GL.
(WebCore::WebGLRenderingContext::getFramebufferAttachmentParameter): Ditto.
(WebCore::WebGLRenderingContext::detachAndRemoveAllObjects): Multiple loop because objects might be removed from the table within an iteration.
* html/canvas/WebGLRenderingContext.h:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@90180 268f45cc-cd09-0410-ab3c-d52691b4dbfc
CRs-fixed: 407009
(cherry-picked from commit 2dbba5d5facb2c0c57a37eb0cf3958bae0f697fa)
Change-Id: I2cb88cc5790452c18f6e58420cc7b063c9ffce26
Diffstat (limited to 'Source/WebCore/bindings/v8')
-rw-r--r-- | Source/WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Source/WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp b/Source/WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp index a23b707..2441dfb 100644 --- a/Source/WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp +++ b/Source/WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp @@ -164,7 +164,7 @@ static v8::Handle<v8::Value> toV8Object(WebGLExtension* extension, v8::Handle<v8 if (!extension) return v8::Null(); v8::Handle<v8::Value> extensionObject; - const char* referenceName; + const char* referenceName = 0; switch (extension->getName()) { case WebGLExtension::WebKitLoseContextName: extensionObject = toV8(static_cast<WebKitLoseContext*>(extension)); |