Move UIManager annotations to separate package

Summary:
Moves ReactProp and ReactPropGroup to `com.facebook.react.uimanager.annotations`. This is needed
so that future annotation processor can run on code inside the com.facebook.react.uimanager package.

public

Reviewed By: astreet

Differential Revision: D2754842

fb-gh-sync-id: 3808c3e3f6c45e5897ecab88cd4175573c0c24e6

Author: lexs

ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java

@@ -7,6 +7,7 @@
 import android.view.View;
 
 import com.facebook.react.bridge.ReadableMap;
+import com.facebook.react.uimanager.annotations.ReactProp;
 
 /**
  * Base class that should be suitable for the majority of subclasses of {@link ViewManager}.

ReactAndroid/src/main/java/com/facebook/react/uimanager/LayoutShadowNode.java

@@ -12,6 +12,8 @@
 import com.facebook.csslayout.CSSJustify;
 import com.facebook.csslayout.CSSPositionType;
 import com.facebook.csslayout.CSSWrap;
+import com.facebook.react.uimanager.annotations.ReactProp;
+import com.facebook.react.uimanager.annotations.ReactPropGroup;
 
 /**
  * Supply setters for base view layout properties such as width, height, flex properties,

ReactAndroid/src/main/java/com/facebook/react/uimanager/ViewManager.java

@@ -20,6 +20,8 @@
 import com.facebook.react.bridge.ReadableMapKeySetIterator;
 import com.facebook.react.touch.CatalystInterceptingViewGroup;
 import com.facebook.react.touch.JSResponderHandler;
+import com.facebook.react.uimanager.annotations.ReactProp;
+import com.facebook.react.uimanager.annotations.ReactPropGroup;
 
 /**
  * Class responsible for knowing how to create and update catalyst Views of a given type. It is also

ReactAndroid/src/main/java/com/facebook/react/uimanager/ViewManagersPropertyCache.java

@@ -15,6 +15,8 @@
 import com.facebook.react.bridge.JSApplicationIllegalArgumentException;
 import com.facebook.react.bridge.ReadableArray;
 import com.facebook.react.bridge.ReadableMap;
+import com.facebook.react.uimanager.annotations.ReactProp;
+import com.facebook.react.uimanager.annotations.ReactPropGroup;
 
 /**
  * This class is responsible for holding view manager property setters and is used in a process of

…/facebook/react/uimanager/ReactProp.java …act/uimanager/annotations/ReactProp.javaReactAndroid/src/main/java/com/facebook/react/uimanager/ReactProp.java renamed to ReactAndroid/src/main/java/com/facebook/react/uimanager/annotations/ReactProp.java ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactProp.java renamed to ReactAndroid/src/main/java/com/facebook/react/uimanager/annotations/ReactProp.java

@@ -1,6 +1,6 @@
 // Copyright 2004-present Facebook. All Rights Reserved.
 
-package com.facebook.react.uimanager;
+package com.facebook.react.uimanager.annotations;
 
 import javax.annotation.Nullable;
 
@@ -12,7 +12,8 @@
 
 /**
  * Use this annotation to annotate properties of native views that should be exposed to JS. This
- * annotation should only be used for setter methods of subclasses of {@link ViewManager}.
+ * annotation should only be used for setter methods of subclasses of
+ * {@link com.facebook.react.uimanager.ViewManager}.
  *
  * Each annotated method should return {@code void} and take exactly two arguments: first being
  * a view instance to be updated and second a value that should be set.
@@ -21,8 +22,8 @@
  *  - primitives (int, boolean, double, float)
  *  - {@link String}
  *  - {@link Boolean}
- *  - {@link ReadableArray}
- *  - {@link ReadableMap}
+ *  - {@link com.facebook.react.bridge.ReadableArray}
+ *  - {@link com.facebook.react.bridge.ReadableMap}
  *
  * When property gets removed from the corresponding component in React, annotated setter will be
  * called with {@code null} in case of non-primitive value type or with a default value in case when

…book/react/uimanager/ReactPropGroup.java …imanager/annotations/ReactPropGroup.javaReactAndroid/src/main/java/com/facebook/react/uimanager/ReactPropGroup.java renamed to ReactAndroid/src/main/java/com/facebook/react/uimanager/annotations/ReactPropGroup.java ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactPropGroup.java renamed to ReactAndroid/src/main/java/com/facebook/react/uimanager/annotations/ReactPropGroup.java

@@ -1,6 +1,6 @@
 // Copyright 2004-present Facebook. All Rights Reserved.
 
-package com.facebook.react.uimanager;
+package com.facebook.react.uimanager.annotations;
 
 import javax.annotation.Nullable;
 
@@ -12,9 +12,10 @@
 
 /**
  * Use this annotation to annotate group of properties of native views that should be exposed to JS.
- * This annotation should only be used for setter methods of subclasses of {@link ViewManager}. It's
- * a batched version of {@link ReactProp} annotation (please see documentation of {@link ReactProp}
- * for more details about how this annotation can be used).
+ * This annotation should only be used for setter methods of subclasses of
+ * {@link com.facebook.react.uimanager.ViewManager}. It's a batched version of {@link ReactProp}
+ * annotation (please see documentation of {@link ReactProp} for more details about how this
+ * annotation can be used).
  *
  * This annotation is meant to be used for a group of similar properties. That's why it support only
  * a set of properties of the same type. A good example is supporting "border", where we have 7

ReactAndroid/src/main/java/com/facebook/react/views/art/ARTShapeShadowNode.java

@@ -20,7 +20,7 @@
 import com.facebook.react.bridge.JSApplicationIllegalArgumentException;
 import com.facebook.react.bridge.ReadableArray;
 import com.facebook.react.common.ReactConstants;
-import com.facebook.react.uimanager.ReactProp;
+import com.facebook.react.uimanager.annotations.ReactProp;
 
 /**
  * Shadow node for virtual ARTShape view

ReactAndroid/src/main/java/com/facebook/react/views/art/ARTTextShadowNode.java

@@ -18,7 +18,7 @@
 
 import com.facebook.react.bridge.ReadableArray;
 import com.facebook.react.bridge.ReadableMap;
-import com.facebook.react.uimanager.ReactProp;
+import com.facebook.react.uimanager.annotations.ReactProp;
 
 /**
  * Shadow node for virtual ARTText view

ReactAndroid/src/main/java/com/facebook/react/views/art/ARTVirtualNode.java

@@ -19,7 +19,7 @@
 import com.facebook.react.bridge.ReadableArray;
 import com.facebook.react.uimanager.CatalystStylesDiffMap;
 import com.facebook.react.uimanager.DisplayMetricsHolder;
-import com.facebook.react.uimanager.ReactProp;
+import com.facebook.react.uimanager.annotations.ReactProp;
 import com.facebook.react.uimanager.ReactShadowNode;
 
 /**

ReactAndroid/src/main/java/com/facebook/react/views/drawer/ReactDrawerLayoutManager.java

@@ -22,7 +22,7 @@
 import com.facebook.react.bridge.ReadableArray;
 import com.facebook.react.common.MapBuilder;
 import com.facebook.react.uimanager.PixelUtil;
-import com.facebook.react.uimanager.ReactProp;
+import com.facebook.react.uimanager.annotations.ReactProp;
 import com.facebook.react.uimanager.ThemedReactContext;
 import com.facebook.react.uimanager.UIManagerModule;
 import com.facebook.react.uimanager.ViewGroupManager;