Make WebGL extensions get lost when context is lost.

Source/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp

Index: Source/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp
diff --git a/Source/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp b/Source/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp
index bf1689ad1af1015135078c1c0235c7a336e15863..30392d7721a91f050f7bb09df347e4bdb3e0784f 100644
--- a/Source/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp
+++ b/Source/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp
@@ -351,8 +351,8 @@ void V8WebGLRenderingContext::getExtensionMethodCustom(const v8::FunctionCallbac
         return;
     }
     V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<>, name, args[0]);
-    WebGLExtension* extension = imp->getExtension(name);
-    v8SetReturnValue(args, toV8Object(extension, args.Holder(), args.GetIsolate()));
+    RefPtr<WebGLExtension> extension(imp->getExtension(name));
+    v8SetReturnValue(args, toV8Object(extension.get(), args.Holder(), args.GetIsolate()));

[Ken Russell (switch to Gerrit), 2013/06/11 21:50:54]

This could possibly use extension.release() since the extension RefPtr isn't
used afterward.

[greggman, 2013/06/11 21:55:21]

I copied this from getAttachedShadersMethodCustom. Is that wrong too? Or is
there some difference between the 2 cases?

[Ken Russell (switch to Gerrit), 2013/06/11 22:07:31]

getAttachedShadersMethodCustom could be changed too, but it isn't important.
This code is correct as is.

 }
 
 void V8WebGLRenderingContext::getFramebufferAttachmentParameterMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& args)