From 83df8a8621287e548ee771599c9c98b61b9971a3 Mon Sep 17 00:00:00 2001
From: Tobrun <tobrun@mapbox.com>
Date: Wed, 31 May 2017 09:46:12 +0200
Subject: [android] - javadoc update for 5.1.0 release (#9138)

* [android] - javadoc update for 5.1.0 release

* oxford comma

* Grammar/spelling tweaks

* grammar tweak

* grammar tweak

* Grammar tweaks

* Grammar tweaks
---
 .../src/main/java/com/mapbox/mapboxsdk/Mapbox.java | 33 ++++++++----
 .../mapbox/mapboxsdk/camera/CameraPosition.java    |  4 +-
 .../mapboxsdk/camera/CameraUpdateFactory.java      |  2 +-
 .../mapboxsdk/constants/MyBearingTracking.java     |  4 +-
 .../mapboxsdk/constants/MyLocationTracking.java    |  4 +-
 .../java/com/mapbox/mapboxsdk/constants/Style.java |  2 +-
 .../mapbox/mapboxsdk/location/LocationSource.java  | 53 +++++++++++++++++--
 .../mapboxsdk/maps/FocalPointChangeListener.java   |  3 ++
 .../java/com/mapbox/mapboxsdk/maps/MapboxMap.java  |  6 +--
 .../mapbox/mapboxsdk/offline/OfflineManager.java   |  4 +-
 .../mapbox/mapboxsdk/offline/OfflineRegion.java    |  2 +-
 .../com/mapbox/mapboxsdk/storage/Resource.java     |  6 +++
 .../style/functions/CompositeFunction.java         |  2 +-
 .../com/mapbox/mapboxsdk/style/layers/Filter.java  |  2 +-
 .../mapboxsdk/style/layers/TransitionOptions.java  | 28 ++++++++++
 .../mapbox/mapboxsdk/style/light/package-info.java |  4 ++
 .../com/mapbox/mapboxsdk/utils/AnimatorUtils.java  | 59 ++++++++++++++++++++++
 .../com/mapbox/mapboxsdk/utils/ColorUtils.java     |  3 ++
 18 files changed, 188 insertions(+), 33 deletions(-)
 create mode 100644 platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/light/package-info.java

diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/Mapbox.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/Mapbox.java
index 8098ee4d86..f75d7ed90d 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/Mapbox.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/Mapbox.java
@@ -1,5 +1,6 @@
 package com.mapbox.mapboxsdk;
 
+import android.app.Application;
 import android.content.Context;
 import android.net.ConnectivityManager;
 import android.net.NetworkInfo;
@@ -15,6 +16,14 @@ import com.mapbox.services.android.telemetry.MapboxTelemetry;
 import com.mapbox.services.android.telemetry.location.LocationEngine;
 import com.mapbox.services.android.telemetry.location.LocationEnginePriority;
 
+/**
+ * The entry point of the Mapbox Android SDK.
+ * <p>
+ * Obtain a reference by calling {@link #getInstance(Context, String)}. Usually this class is configured in
+ * {@link Application#onCreate()} and is responsible for the active access token, application context, and
+ * connectivity state.
+ * </p>
+ */
 public final class Mapbox {
 
   private static Mapbox INSTANCE;
@@ -25,7 +34,7 @@ public final class Mapbox {
   /**
    * Get an instance of Mapbox.
    * <p>
-   * This class manages the active access token, application context and connectivity state.
+   * This class manages the active access token, application context, and connectivity state.
    * </p>
    *
    * @param context     Android context which holds or is an application context
@@ -52,9 +61,9 @@ public final class Mapbox {
   }
 
   /**
-   * Access Token for this application.
+   * Access token for this application.
    *
-   * @return Mapbox Access Token
+   * @return Mapbox access token
    */
   public static String getAccessToken() {
     validateMapbox();
@@ -72,12 +81,14 @@ public final class Mapbox {
   }
 
   /**
-   * Runtime validation of Access Token.
+   * Runtime validation of access token.
    *
    * @throws MapboxConfigurationException exception thrown when not using a valid accessToken
    */
   private static void validateAccessToken() throws MapboxConfigurationException {
-    String accessToken = INSTANCE.accessToken;
+    String 
+      
+      = INSTANCE.accessToken;
     if (TextUtils.isEmpty(accessToken) || (!accessToken.toLowerCase(MapboxConstants.MAPBOX_LOCALE).startsWith("pk.")
       && !accessToken.toLowerCase(MapboxConstants.MAPBOX_LOCALE).startsWith("sk."))) {
       throw new MapboxConfigurationException();
@@ -92,11 +103,11 @@ public final class Mapbox {
   }
 
   /**
-   * Manually sets the connectivity state of the app. This is useful for apps that control their
+   * Manually sets the connectivity state of the app. This is useful for apps which control their
    * own connectivity state and want to bypass any checks to the ConnectivityManager.
    *
    * @param connected flag to determine the connectivity state, true for connected, false for
-   *                  disconnected, null for ConnectivityManager to determine.
+   *                  disconnected, and null for ConnectivityManager to determine.
    */
   public static synchronized void setConnected(Boolean connected) {
     // Connectivity state overridden by app
@@ -104,10 +115,10 @@ public final class Mapbox {
   }
 
   /**
-   * Determines whether we have an Internet connection available. Please do not rely on this
-   * method in your apps, this method is used internally by the SDK.
+   * Determines whether we have an internet connection available. Please do not rely on this
+   * method in your apps. This method is used internally by the SDK.
    *
-   * @return true if there is an Internet connection, false otherwise
+   * @return true if there is an internet connection, false otherwise
    */
   public static synchronized Boolean isConnected() {
     if (INSTANCE.connected != null) {
@@ -119,4 +130,4 @@ public final class Mapbox {
     NetworkInfo activeNetwork = cm.getActiveNetworkInfo();
     return (activeNetwork != null && activeNetwork.isConnected());
   }
-}
\ No newline at end of file
+}
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/camera/CameraPosition.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/camera/CameraPosition.java
index 5e958ff565..66c261f1d0 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/camera/CameraPosition.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/camera/CameraPosition.java
@@ -156,7 +156,7 @@ public final class CameraPosition implements Parcelable {
   }
 
   /**
-   * Builder for composing {@link CameraPosition} objects.
+   * Builder for composing CameraPosition objects.
    */
   public static final class Builder {
 
@@ -190,7 +190,7 @@ public final class CameraPosition implements Parcelable {
     /**
      * Create Builder with an existing CameraPosition data.
      *
-     * @param typedArray TypedArray containgin attribute values
+     * @param typedArray TypedArray containing attribute values
      */
     public Builder(TypedArray typedArray) {
       super();
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/camera/CameraUpdateFactory.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/camera/CameraUpdateFactory.java
index 64b86054a0..8e1411e273 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/camera/CameraUpdateFactory.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/camera/CameraUpdateFactory.java
@@ -15,7 +15,7 @@ import java.lang.annotation.Retention;
 import java.lang.annotation.RetentionPolicy;
 
 /**
- * Factory for creating {@link CameraUpdate} objects.
+ * Factory for creating CameraUpdate objects.
  */
 public final class CameraUpdateFactory {
 
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/constants/MyBearingTracking.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/constants/MyBearingTracking.java
index f53c65d055..88c3bef673 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/constants/MyBearingTracking.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/constants/MyBearingTracking.java
@@ -16,7 +16,7 @@ import java.lang.annotation.RetentionPolicy;
 public class MyBearingTracking {
 
   /**
-   * Indicates the parameter accepts one of the values from {@link MyBearingTracking}.
+   * Indicates that the parameter accepts one of the values from MyBearingTracking.
    */
   @IntDef( {NONE, COMPASS, GPS, /**COMBINED**/})
   @Retention(RetentionPolicy.SOURCE)
@@ -38,6 +38,4 @@ public class MyBearingTracking {
    */
   public static final int GPS = 0x00000008;
 
-  // public static final int COMBINED = 0x00000012;
-
 }
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/constants/MyLocationTracking.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/constants/MyLocationTracking.java
index 39f653596f..a1744d701f 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/constants/MyLocationTracking.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/constants/MyLocationTracking.java
@@ -17,7 +17,7 @@ import java.lang.annotation.RetentionPolicy;
 public class MyLocationTracking {
 
   /**
-   * Indicates the parameter accepts one of the values from {@link MyLocationTracking}.
+   * Indicates the parameter accepts one of the values from MyLocationTracking.
    */
   @IntDef( {TRACKING_NONE, TRACKING_FOLLOW})
   @Retention(RetentionPolicy.SOURCE)
@@ -30,7 +30,7 @@ public class MyLocationTracking {
   public static final int TRACKING_NONE = 0x00000000;
 
   /**
-   * Tracking the location of the user, {@link MapView} will reposition to center of {@link MyLocationView}
+   * Tracking the location of the user. {@link MapView} will reposition to center of {@link MyLocationView}
    */
   public static final int TRACKING_FOLLOW = 0x00000004;
 
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/constants/Style.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/constants/Style.java
index 77d0929df3..31e6313509 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/constants/Style.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/constants/Style.java
@@ -19,7 +19,7 @@ public class Style {
 
 
   /**
-   * Indicates the parameter accepts one of the values from {@link Style}. Using one of these
+   * Indicates the parameter accepts one of the values from Style. Using one of these
    * constants means your map style will always use the latest version and may change as we
    * improve the style
    */
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationSource.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationSource.java
index c4bdb4a17d..4e934fa3cc 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationSource.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationSource.java
@@ -2,7 +2,7 @@ package com.mapbox.mapboxsdk.location;
 
 import android.content.Context;
 import android.location.Location;
-import android.util.Log;
+import android.support.annotation.NonNull;
 
 import com.mapbox.services.android.telemetry.location.LocationEngine;
 import com.mapbox.services.android.telemetry.location.LocationEngineListener;
@@ -15,6 +15,8 @@ import com.mapzen.android.lost.api.LostApiClient;
 
 import java.lang.ref.WeakReference;
 
+import timber.log.Timber;
+
 /**
  * Manages locational updates. Contains methods to register and unregister location listeners.
  * <ul>
@@ -33,8 +35,6 @@ import java.lang.ref.WeakReference;
 public class LocationSource extends LocationEngine implements
   LostApiClient.ConnectionCallbacks, LocationListener {
 
-  private static final String LOG_TAG = LocationSource.class.getSimpleName();
-
   private static LocationEngine instance;
 
   private WeakReference<Context> context;
@@ -48,7 +48,13 @@ public class LocationSource extends LocationEngine implements
       .build();
   }
 
-  public static synchronized LocationEngine getLocationEngine(Context context) {
+  /**
+   * Get the LocationEngine instance.
+   *
+   * @param context a Context from which the application context is derived
+   * @return the LocationEngine instance
+   */
+  public static synchronized LocationEngine getLocationEngine(@NonNull Context context) {
     if (instance == null) {
       instance = new LocationSource(context.getApplicationContext());
     }
@@ -56,6 +62,10 @@ public class LocationSource extends LocationEngine implements
     return instance;
   }
 
+  /**
+   * Activate the location engine which will connect whichever location provider you are using. You'll need to call
+   * this before requesting user location updates using {@link LocationEngine#requestLocationUpdates()}.
+   */
   @Override
   public void activate() {
     if (lostApiClient != null && !lostApiClient.isConnected()) {
@@ -63,6 +73,11 @@ public class LocationSource extends LocationEngine implements
     }
   }
 
+  /**
+   * Disconnect the location engine which is useful when you no longer need location updates or requesting the users
+   * {@link LocationEngine#getLastLocation()}. Before deactivating, you'll need to stop request user location updates
+   * using {@link LocationEngine#removeLocationUpdates()}.
+   */
   @Override
   public void deactivate() {
     if (lostApiClient != null && lostApiClient.isConnected()) {
@@ -70,11 +85,20 @@ public class LocationSource extends LocationEngine implements
     }
   }
 
+  /**
+   * Check if your location provider has been activated/connected. This is mainly used internally but is also useful in
+   * the rare case when you'd like to know if your location engine is connected or not.
+   *
+   * @return boolean true if the location engine has been activated/connected, else false.
+   */
   @Override
   public boolean isConnected() {
     return lostApiClient.isConnected();
   }
 
+  /**
+   * Invoked when the location provider has connected.
+   */
   @Override
   public void onConnected() {
     for (LocationEngineListener listener : locationListeners) {
@@ -82,11 +106,19 @@ public class LocationSource extends LocationEngine implements
     }
   }
 
+  /**
+   * Invoked when the location provider connection has been suspended.
+   */
   @Override
   public void onConnectionSuspended() {
-    Log.d(LOG_TAG, "Connection suspended.");
+    Timber.d("Connection suspended.");
   }
 
+  /**
+   * Returns the Last known location is the location provider is connected and location permissions are granted.
+   *
+   * @return the last known location
+   */
   @Override
   public Location getLastLocation() {
     if (lostApiClient.isConnected() && PermissionsManager.areLocationPermissionsGranted(context.get())) {
@@ -97,6 +129,9 @@ public class LocationSource extends LocationEngine implements
     return null;
   }
 
+  /**
+   * Request location updates to the location provider.
+   */
   @Override
   public void requestLocationUpdates() {
     // Common params
@@ -122,6 +157,9 @@ public class LocationSource extends LocationEngine implements
     }
   }
 
+  /**
+   * Dismiss ongoing location update to the location provider.
+   */
   @Override
   public void removeLocationUpdates() {
     if (lostApiClient.isConnected()) {
@@ -129,6 +167,11 @@ public class LocationSource extends LocationEngine implements
     }
   }
 
+  /**
+   * Invoked when the Location has changed.
+   *
+   * @param location the new location
+   */
   @Override
   public void onLocationChanged(Location location) {
     for (LocationEngineListener listener : locationListeners) {
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/FocalPointChangeListener.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/FocalPointChangeListener.java
index 006122a4e2..aec9a848b7 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/FocalPointChangeListener.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/FocalPointChangeListener.java
@@ -2,6 +2,9 @@ package com.mapbox.mapboxsdk.maps;
 
 import android.graphics.PointF;
 
+/**
+ * Interface definition of a callback that is invoked when the focal point will change.
+ */
 public interface FocalPointChangeListener {
 
   void onFocalPointChanged(PointF pointF);
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMap.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMap.java
index 0bfa74d1c0..0ba6c10831 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMap.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMap.java
@@ -1972,9 +1972,9 @@ public final class MapboxMap {
   /**
    * Interface definition for a callback to be invoked when the camera changes position.
    *
-   * @deprecated Replaced by MapboxMap.OnCameraMoveStartedListener, MapboxMap.OnCameraMoveListener and
-   * MapboxMap.OnCameraIdleListener. The order in which the deprecated onCameraChange method will be called in relation
-   * to the methods in the new camera change listeners is undefined.
+   * @deprecated Replaced by {@link MapboxMap.OnCameraMoveStartedListener}, {@link MapboxMap.OnCameraMoveListener} and
+   * {@link MapboxMap.OnCameraIdleListener}. The order in which the deprecated onCameraChange method will be called in
+   * relation to the methods in the new camera change listeners is undefined.
    */
   @Deprecated
   public interface OnCameraChangeListener {
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/offline/OfflineManager.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/offline/OfflineManager.java
index 225278b17d..95cb7d66c4 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/offline/OfflineManager.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/offline/OfflineManager.java
@@ -46,7 +46,7 @@ public class OfflineManager {
 
   /**
    * This callback receives an asynchronous response containing a list of all
-   * {@link OfflineRegion} in the database, or an error message otherwise.
+   * OfflineRegion in the database or an error message otherwise.
    */
   public interface ListOfflineRegionsCallback {
     /**
@@ -66,7 +66,7 @@ public class OfflineManager {
 
   /**
    * This callback receives an asynchronous response containing the newly created
-   * {@link OfflineRegion} in the database, or an error message otherwise.
+   * OfflineRegion in the database or an error message otherwise.
    */
   public interface CreateOfflineRegionCallback {
     /**
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/offline/OfflineRegion.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/offline/OfflineRegion.java
index 44bb2a05ee..baa815491f 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/offline/OfflineRegion.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/offline/OfflineRegion.java
@@ -98,7 +98,7 @@ public class OfflineRegion {
   }
 
   /**
-   * This callback receives an asynchronous response containing the {@link OfflineRegionStatus}
+   * This callback receives an asynchronous response containing the OfflineRegionStatus
    * of the offline region, or a {@link String} error message otherwise.
    */
   public interface OfflineRegionStatusCallback {
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/storage/Resource.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/storage/Resource.java
index af98a46a9b..eae83e8c1f 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/storage/Resource.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/storage/Resource.java
@@ -5,9 +5,15 @@ import android.support.annotation.IntDef;
 import java.lang.annotation.Retention;
 import java.lang.annotation.RetentionPolicy;
 
+/**
+ * Resource provides access to resource types.
+ */
 public final class Resource {
   // Note: Keep this in sync with include/mbgl/storage/resource.hpp
 
+  /**
+   * Resource type variants.
+   */
   @IntDef( {UNKNOWN, STYLE, SOURCE, TILE, GLYPHS, SPRITE_IMAGE, SPRITE_JSON})
   @Retention(RetentionPolicy.SOURCE)
   public @interface Kind {
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/functions/CompositeFunction.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/functions/CompositeFunction.java
index 8ded7ecd34..15e4474105 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/functions/CompositeFunction.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/functions/CompositeFunction.java
@@ -13,7 +13,7 @@ import com.mapbox.mapboxsdk.style.layers.PropertyValue;
 import java.util.Map;
 
 /**
- * Composite functions combine {@link android.graphics.Camera} and {@link SourceFunction}s.
+ * Composite functions combine Camera and SourceFunctions.
  * <p>
  * Composite functions allow the appearance of a map feature to change with both its
  * properties and zoom. Each stop is an array with two elements, the first is an object
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/layers/Filter.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/layers/Filter.java
index 643a126388..4dbb461e4c 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/layers/Filter.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/layers/Filter.java
@@ -11,7 +11,7 @@ import java.util.Collections;
 public class Filter {
 
   /**
-   * Base {@link Filter} statement. Subclassed to provide concrete statements.
+   * Base Filter statement. Subclassed to provide concrete statements.
    */
   public abstract static class Statement {
     protected final String operator;
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/layers/TransitionOptions.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/layers/TransitionOptions.java
index a46c11b35c..6e6e4ca613 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/layers/TransitionOptions.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/layers/TransitionOptions.java
@@ -1,23 +1,51 @@
 package com.mapbox.mapboxsdk.style.layers;
 
+/**
+ * Resembles transition property from the style specification.
+ *
+ * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#transition">Transition documentation</a>
+ */
 public class TransitionOptions {
 
   private long duration;
   private long delay;
 
+  /**
+   * Create a transition property based on duration and a delay.
+   *
+   * @param duration the duration of the transition
+   * @param delay    the delay to start the transition
+   */
   public TransitionOptions(long duration, long delay) {
     this.duration = duration;
     this.delay = delay;
   }
 
+  /**
+   * Create a transition property based on duration and a delay.
+   *
+   * @param duration the duration of the transition
+   * @param delay    the delay to start the transition
+   * @return a new transition property object
+   */
   public static TransitionOptions fromTransitionOptions(long duration, long delay) {
     return new TransitionOptions(duration, delay);
   }
 
+  /**
+   * Get the transition duration.
+   *
+   * @return the transition duration
+   */
   public long getDuration() {
     return duration;
   }
 
+  /**
+   * Get the transition delay.
+   *
+   * @return the transition delay
+   */
   public long getDelay() {
     return delay;
   }
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/light/package-info.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/light/package-info.java
new file mode 100644
index 0000000000..a613bf9587
--- /dev/null
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/light/package-info.java
@@ -0,0 +1,4 @@
+/**
+ * Contains the Mapbox Maps Android Style Light API classes.
+ */
+package com.mapbox.mapboxsdk.style.light;
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/utils/AnimatorUtils.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/utils/AnimatorUtils.java
index 7694604d9f..34c52c829b 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/utils/AnimatorUtils.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/utils/AnimatorUtils.java
@@ -10,13 +10,31 @@ import android.support.annotation.Nullable;
 import android.support.v4.view.animation.FastOutSlowInInterpolator;
 import android.view.View;
 
+/**
+ * Animator utility class.
+ */
 public class AnimatorUtils {
 
+  /**
+   * Animate a view from an animator resource.
+   *
+   * @param view        the view to be animated
+   * @param animatorRes the animator resource to be loaded
+   * @param listener    the animator end listener
+   */
   public static void animate(@NonNull final View view, @AnimatorRes int animatorRes,
                              @Nullable OnAnimationEndListener listener) {
     animate(view, animatorRes, -1, listener);
   }
 
+  /**
+   * Animate a view from an animator resource.
+   *
+   * @param view        the view to be animated
+   * @param animatorRes the animator resource to be loaded
+   * @param duration    the duration of the animator
+   * @param listener    the animator end listener
+   */
   public static void animate(final View view, @AnimatorRes int animatorRes, int duration,
                              @Nullable final OnAnimationEndListener listener) {
     if (view == null) {
@@ -43,14 +61,33 @@ public class AnimatorUtils {
     animator.start();
   }
 
+  /**
+   * Animate a view from an animator resource.
+   *
+   * @param view        the view to be animated
+   * @param animatorRes the animator resource to be loaded
+   */
   public static void animate(@NonNull final View view, @AnimatorRes int animatorRes) {
     animate(view, animatorRes, -1);
   }
 
+  /**
+   * Animate a view from an animator resource.
+   *
+   * @param view        the view to be animated
+   * @param animatorRes the animator resource to be loaded
+   * @param duration    the duration of the animator
+   */
   public static void animate(@NonNull final View view, @AnimatorRes int animatorRes, int duration) {
     animate(view, animatorRes, duration, null);
   }
 
+  /**
+   * Animate a view rotation property to a value.
+   *
+   * @param view     the view to be rotated
+   * @param rotation the value to animate to
+   */
   public static void rotate(@NonNull final View view, float rotation) {
     view.setLayerType(View.LAYER_TYPE_HARDWARE, null);
     ObjectAnimator rotateAnimator = ObjectAnimator.ofFloat(view, View.ROTATION, view.getRotation(), rotation);
@@ -64,6 +101,12 @@ public class AnimatorUtils {
     rotateAnimator.start();
   }
 
+  /**
+   * Animate a view rotation property by a value.
+   *
+   * @param view       the view to be rotated
+   * @param rotationBy the value to animate by
+   */
   public static void rotateBy(@NonNull final View view, float rotationBy) {
     view.setLayerType(View.LAYER_TYPE_HARDWARE, null);
     view.animate().rotationBy(rotationBy).setInterpolator(new FastOutSlowInInterpolator()).setListener(
@@ -76,6 +119,13 @@ public class AnimatorUtils {
       });
   }
 
+  /**
+   * Animate a view alpha property to a value.
+   *
+   * @param convertView the view to be animated
+   * @param alpha       the value to animate to
+   * @param listener    the animator end listener
+   */
   public static void alpha(@NonNull final View convertView, float alpha,
                            @Nullable final OnAnimationEndListener listener) {
     convertView.setLayerType(View.LAYER_TYPE_HARDWARE, null);
@@ -99,10 +149,19 @@ public class AnimatorUtils {
     rotateAnimator.start();
   }
 
+  /**
+   * Animate a view alpha property to a value.
+   *
+   * @param convertView the view to be animated
+   * @param alpha       the value to animate to
+   */
   public static void alpha(@NonNull final View convertView, float alpha) {
     alpha(convertView, alpha, null);
   }
 
+  /**
+   * An interface definition that is invoked when an animation ends.
+   */
   public interface OnAnimationEndListener {
     void onAnimationEnd();
   }
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/utils/ColorUtils.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/utils/ColorUtils.java
index 2da2472d69..24c76243d9 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/utils/ColorUtils.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/utils/ColorUtils.java
@@ -17,6 +17,9 @@ import com.mapbox.mapboxsdk.exceptions.ConversionException;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
+/**
+ * Color utility class.
+ */
 public class ColorUtils {
 
   /**
-- 
cgit v1.2.1