Make use of new JsConstructor available in GWT 2.8.1

Author: Paul Stockley

DOCUMENTATION.md

@@ -133,7 +133,7 @@ references will result in slightly less code being generated by the GWT compiler
     }
 ```
 
-<p>To use one of these components, the easiest way is to pass it's class into <code>React.createElement</code> e.g.</p>
+<p>To use one of these components, pass it's class into <code>React.createElement</code> e.g.</p>
 
 ```java
     //The above StatefulExample component could then be used as follows:
@@ -143,33 +143,6 @@ references will result in slightly less code being generated by the GWT compiler
     React.createElement(StatefulExample.class, props);
 ```
 
-<p>There are a couple of restrictions when using this approach. It will not work if you have disabled class meta data
-when compiling your code. Also if you specifiy an altenative name or namespace in the JsType annotation it will not work.
-In these cases you need to make use of the <code>ComponentUtils.getCtorFn</code> method and pass in the fully qualified 
-class name e.g.</p>
-
-```java
-    // Assuming the StatefulExample class was defined in the package com.react.example, you can 
-    // use ComponentUtils.getCtorFn to reference it as follows:
-    
-    StatefulExample.Props props = new StatefulExample.Props();
-    props.aProp = "Some value";
-    
-    React.createElement(ComponentUtils.getCtorFn("com.react.example.StatefulExample"), props);
-    
-    //Assuming you defined StatefulExample as
-    
-    @JsType(name="RenamedStatefullExample", namespace=JsPackage.GLOBAL)
-    class StatefulExample extends Component<StatefulExample.Props, StatefulExample.State> {
-    	.
-    	.
-    }
-    
-    //You would refer to this in React.createElement using
-    React.createElement(ComponentUtils.getCtorFn("RenamedStatefulExample"), props);
-```
-
-
 ## 3. Rendering React Elements
 
 <p>The majority of javascript React code you will see uses something called JSX. This is just a preprocessor that allows you

src/gwt/react/client/components/ComponentUtils.java

@@ -1,8 +1,6 @@
 package gwt.react.client.components;
 
-import gwt.interop.utils.client.collections.JsStringMap;
 import gwt.interop.utils.client.plainobjects.JsPlainObj;
-import gwt.interop.utils.shared.collections.StringMap;
 import gwt.react.client.proptypes.BaseProps;
 
 /**
@@ -12,8 +10,6 @@ public class ComponentUtils {
     private ComponentUtils() {
     }
 
-    private static StringMap<ComponentConstructorFn> constructorLookup = JsStringMap.create() ;
-
     /**
      * Given the Class of a JsType annotated {@link Component} class, return the constructor function to use in Javascript
      * @param cls The Class
@@ -22,39 +18,7 @@ private ComponentUtils() {
      * @param <T> The type of {@link Component}
      * @return The constructor function
      */
-    public static <P extends BaseProps, S extends JsPlainObj, T extends Component<P, S>> ComponentConstructorFn<P> getCtorFn(Class<T> cls) {
-        return getCtorFn(cls, cls.getName());
-    }
-
-    /**
-     * Given the fully qualified name of a JsType annotated {@link Component} class, return the constructor function to use in Javascript
-     * @param cls The Class
-     * @param actualClassName The fully qualified name. This is useful if it does not match the className of cls.
-     * @param <P>
-     * @return The constructor function
-     */
-    public static <P extends BaseProps, S extends JsPlainObj, T extends Component<P, S>> ComponentConstructorFn<P> getCtorFn(Class<T> cls, String actualClassName) {
-        ComponentConstructorFn<P> fn = constructorLookup.get(actualClassName);
-
-        if (fn == null) {
-            fn = getJSConstructorFn(actualClassName);
-            assert(fn != null);
-
-            constructorLookup.put(actualClassName, fn);
-        }
-        return fn;
-    }
-
-    /**
-     * TODO find a more efficient way of obtaining the constructor function
-     */
-    private static native <P extends BaseProps> ComponentConstructorFn<P> getJSConstructorFn(String className) /*-{
-        var namespaces = className.split(".");
-        var context = $wnd;
-
-        for (var i = 0; i < namespaces.length; i++) {
-            context = context[namespaces[i]];
-        }
-        return context;
-	}-*/;
+    public static native <P extends BaseProps, S extends JsPlainObj, T extends Component<P, S>> ComponentConstructorFn<P> getCtorFn(Class<T> cls) /*-{
+        return [email protected]::jsConstructor;
+    }-*/;
 }