Automated g4 rollback of changelist 156712385.

*** Reason for rollback ***

This change may silently introduce bugs where both parameters are acceptable in both places. It's decided that the gain isn't worth the risk.

*** Original change description ***

Make the error messages the first parameter in Assertions methods to match JUnit methods

Reverse parameter order creates a slight confusion.

***

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=156844728

demo/src/main/java/com/google/android/exoplayer2/demo/SampleChooserActivity.java

@@ -200,17 +200,17 @@ public class SampleChooserActivity extends Activity {
             extension = reader.nextString();
             break;
           case "drm_scheme":
-            Assertions.checkState("Invalid attribute on nested item: drm_scheme", !insidePlaylist);
+            Assertions.checkState(!insidePlaylist, "Invalid attribute on nested item: drm_scheme");
             drmUuid = getDrmUuid(reader.nextString());
             break;
           case "drm_license_url":
-            Assertions.checkState("Invalid attribute on nested item: drm_license_url",
-                !insidePlaylist);
+            Assertions.checkState(!insidePlaylist,
+                "Invalid attribute on nested item: drm_license_url");
             drmLicenseUrl = reader.nextString();
             break;
           case "drm_key_request_properties":
-            Assertions.checkState("Invalid attribute on nested item: drm_key_request_properties",
-                !insidePlaylist);
+            Assertions.checkState(!insidePlaylist,
+                "Invalid attribute on nested item: drm_key_request_properties");
             ArrayList<String> drmKeyRequestPropertiesList = new ArrayList<>();
             reader.beginObject();
             while (reader.hasNext()) {
@@ -221,12 +221,12 @@ public class SampleChooserActivity extends Activity {
             drmKeyRequestProperties = drmKeyRequestPropertiesList.toArray(new String[0]);
             break;
           case "prefer_extension_decoders":
-            Assertions.checkState("Invalid attribute on nested item: prefer_extension_decoders",
-                !insidePlaylist);
+            Assertions.checkState(!insidePlaylist,
+                "Invalid attribute on nested item: prefer_extension_decoders");
             preferExtensionDecoders = reader.nextBoolean();
             break;
           case "playlist":
-            Assertions.checkState("Invalid nesting of playlists", !insidePlaylist);
+            Assertions.checkState(!insidePlaylist, "Invalid nesting of playlists");
             playlistSamples = new ArrayList<>();
             reader.beginArray();
             while (reader.hasNext()) {

library/core/src/main/java/com/google/android/exoplayer2/extractor/mp4/AtomParsers.java

@@ -593,7 +593,7 @@ import java.util.List;
     for (int i = 0; i < numberOfEntries; i++) {
       int childStartPosition = stsd.getPosition();
       int childAtomSize = stsd.readInt();
-      Assertions.checkArgument("childAtomSize should be positive", childAtomSize > 0);
+      Assertions.checkArgument(childAtomSize > 0, "childAtomSize should be positive");
       int childAtomType = stsd.readInt();
       if (childAtomType == Atom.TYPE_avc1 || childAtomType == Atom.TYPE_avc3
           || childAtomType == Atom.TYPE_encv || childAtomType == Atom.TYPE_mp4v
@@ -693,7 +693,7 @@ import java.util.List;
         // Handle optional terminating four zero bytes in MOV files.
         break;
       }
-      Assertions.checkArgument("childAtomSize should be positive", childAtomSize > 0);
+      Assertions.checkArgument(childAtomSize > 0, "childAtomSize should be positive");
       int childAtomType = parent.readInt();
       if (childAtomType == Atom.TYPE_avcC) {
         Assertions.checkState(mimeType == null);
@@ -877,7 +877,7 @@ import java.util.List;
     while (childPosition - position < size) {
       parent.setPosition(childPosition);
       int childAtomSize = parent.readInt();
-      Assertions.checkArgument("childAtomSize should be positive", childAtomSize > 0);
+      Assertions.checkArgument(childAtomSize > 0, "childAtomSize should be positive");
       int childAtomType = parent.readInt();
       if (childAtomType == Atom.TYPE_esds || (isQuickTime && childAtomType == Atom.TYPE_wave)) {
         int esdsAtomPosition = childAtomType == Atom.TYPE_esds ? childPosition
@@ -936,7 +936,7 @@ import java.util.List;
     while (childAtomPosition - position < size) {
       parent.setPosition(childAtomPosition);
       int childAtomSize = parent.readInt();
-      Assertions.checkArgument("childAtomSize should be positive", childAtomSize > 0);
+      Assertions.checkArgument(childAtomSize > 0, "childAtomSize should be positive");
       int childType = parent.readInt();
       if (childType == Atom.TYPE_esds) {
         return childAtomPosition;
@@ -1032,7 +1032,7 @@ import java.util.List;
     while (childPosition - position < size) {
       parent.setPosition(childPosition);
       int childAtomSize = parent.readInt();
-      Assertions.checkArgument("childAtomSize should be positive", childAtomSize > 0);
+      Assertions.checkArgument(childAtomSize > 0, "childAtomSize should be positive");
       int childAtomType = parent.readInt();
       if (childAtomType == Atom.TYPE_sinf) {
         Pair<Integer, TrackEncryptionBox> result = parseSinfFromParent(parent, childPosition,
@@ -1071,8 +1071,8 @@ import java.util.List;
     }
 
     if (isCencScheme) {
-      Assertions.checkArgument("frma atom is mandatory", dataFormat != null);
-      Assertions.checkArgument("schi->tenc atom is mandatory", trackEncryptionBox != null);
+      Assertions.checkArgument(dataFormat != null, "frma atom is mandatory");
+      Assertions.checkArgument(trackEncryptionBox != null, "schi->tenc atom is mandatory");
       return Pair.create(dataFormat, trackEncryptionBox);
     } else {
       return null;
@@ -1157,7 +1157,7 @@ import java.util.List;
       length = chunkOffsets.readUnsignedIntToInt();
       stsc.setPosition(Atom.FULL_HEADER_SIZE);
       remainingSamplesPerChunkChanges = stsc.readUnsignedIntToInt();
-      Assertions.checkState("first_chunk must be 1", stsc.readInt() == 1);
+      Assertions.checkState(stsc.readInt() == 1, "first_chunk must be 1");
       index = C.INDEX_UNSET;
     }
 

library/core/src/main/java/com/google/android/exoplayer2/extractor/mp4/FragmentedMp4Extractor.java

@@ -388,7 +388,7 @@ public final class FragmentedMp4Extractor implements Extractor {
   }
 
   private void onMoovContainerAtomRead(ContainerAtom moov) throws ParserException {
-    Assertions.checkState("Unexpected moov box.", sideloadedTrack == null);
+    Assertions.checkState(sideloadedTrack == null, "Unexpected moov box.");
 
     DrmInitData drmInitData = getDrmInitDataFromAtoms(moov.leafChildren);
 

library/core/src/main/java/com/google/android/exoplayer2/extractor/ts/SeiReader.java

@@ -48,9 +48,9 @@ import java.util.List;
       TrackOutput output = extractorOutput.track(idGenerator.getTrackId(), C.TRACK_TYPE_TEXT);
       Format channelFormat = closedCaptionFormats.get(i);
       String channelMimeType = channelFormat.sampleMimeType;
-      Assertions.checkArgument("Invalid closed caption mime type provided: " + channelMimeType,
-          MimeTypes.APPLICATION_CEA608.equals(channelMimeType)
-          || MimeTypes.APPLICATION_CEA708.equals(channelMimeType));
+      Assertions.checkArgument(MimeTypes.APPLICATION_CEA608.equals(channelMimeType)
+          || MimeTypes.APPLICATION_CEA708.equals(channelMimeType),
+          "Invalid closed caption mime type provided: " + channelMimeType);
       output.format(Format.createTextSampleFormat(idGenerator.getFormatId(), channelMimeType, null,
           Format.NO_VALUE, channelFormat.selectionFlags, channelFormat.language,
           channelFormat.accessibilityChannel, null));

library/core/src/main/java/com/google/android/exoplayer2/source/ConcatenatingMediaSource.java

@@ -170,8 +170,8 @@ public final class ConcatenatingMediaSource implements MediaSource {
       for (int i = 0; i < timelines.length; i++) {
         Timeline timeline = timelines[i];
         periodCount += timeline.getPeriodCount();
-        Assertions.checkState("ConcatenatingMediaSource children contain too many periods",
-            periodCount <= Integer.MAX_VALUE);
+        Assertions.checkState(periodCount <= Integer.MAX_VALUE,
+            "ConcatenatingMediaSource children contain too many periods");
         sourcePeriodOffsets[i] = (int) periodCount;
         windowCount += timeline.getWindowCount();
         sourceWindowOffsets[i] = windowCount;

library/core/src/main/java/com/google/android/exoplayer2/source/LoopingMediaSource.java

@@ -102,8 +102,8 @@ public final class LoopingMediaSource implements MediaSource {
       childPeriodCount = childTimeline.getPeriodCount();
       childWindowCount = childTimeline.getWindowCount();
       this.loopCount = loopCount;
-      Assertions.checkState("LoopingMediaSource contains too many periods",
-          loopCount <= Integer.MAX_VALUE / childPeriodCount);
+      Assertions.checkState(loopCount <= Integer.MAX_VALUE / childPeriodCount,
+          "LoopingMediaSource contains too many periods");
     }
 
     @Override

library/core/src/main/java/com/google/android/exoplayer2/util/Assertions.java

@@ -41,12 +41,12 @@ public final class Assertions {
   /**
    * Throws {@link IllegalArgumentException} if {@code expression} evaluates to false.
    *
+   * @param expression The expression to evaluate.
    * @param errorMessage The exception message if an exception is thrown. The message is converted
    *     to a {@link String} using {@link String#valueOf(Object)}.
-   * @param expression The expression to evaluate.
    * @throws IllegalArgumentException If {@code expression} is false.
    */
-  public static void checkArgument(Object errorMessage, boolean expression) {
+  public static void checkArgument(boolean expression, Object errorMessage) {
     if (ExoPlayerLibraryInfo.ASSERTIONS_ENABLED && !expression) {
       throw new IllegalArgumentException(String.valueOf(errorMessage));
     }
@@ -83,12 +83,12 @@ public final class Assertions {
   /**
    * Throws {@link IllegalStateException} if {@code expression} evaluates to false.
    *
+   * @param expression The expression to evaluate.
    * @param errorMessage The exception message if an exception is thrown. The message is converted
    *     to a {@link String} using {@link String#valueOf(Object)}.
-   * @param expression The expression to evaluate.
    * @throws IllegalStateException If {@code expression} is false.
    */
-  public static void checkState(Object errorMessage, boolean expression) {
+  public static void checkState(boolean expression, Object errorMessage) {
     if (ExoPlayerLibraryInfo.ASSERTIONS_ENABLED && !expression) {
       throw new IllegalStateException(String.valueOf(errorMessage));
     }
@@ -113,13 +113,13 @@ public final class Assertions {
    * Throws {@link NullPointerException} if {@code reference} is null.
    *
    * @param <T> The type of the reference.
+   * @param reference The reference.
    * @param errorMessage The exception message to use if the check fails. The message is converted
    *     to a string using {@link String#valueOf(Object)}.
-   * @param reference The reference.
    * @return The non-null reference that was validated.
    * @throws NullPointerException If {@code reference} is null.
    */
-  public static <T> T checkNotNull(Object errorMessage, T reference) {
+  public static <T> T checkNotNull(T reference, Object errorMessage) {
     if (ExoPlayerLibraryInfo.ASSERTIONS_ENABLED && reference == null) {
       throw new NullPointerException(String.valueOf(errorMessage));
     }
@@ -143,13 +143,13 @@ public final class Assertions {
   /**
    * Throws {@link IllegalArgumentException} if {@code string} is null or zero length.
    *
+   * @param string The string to check.
    * @param errorMessage The exception message to use if the check fails. The message is converted
    *     to a string using {@link String#valueOf(Object)}.
-   * @param string The string to check.
    * @return The non-null, non-empty string that was validated.
    * @throws IllegalArgumentException If {@code string} is null or 0-length.
    */
-  public static String checkNotEmpty(Object errorMessage, String string) {
+  public static String checkNotEmpty(String string, Object errorMessage) {
     if (ExoPlayerLibraryInfo.ASSERTIONS_ENABLED && TextUtils.isEmpty(string)) {
       throw new IllegalArgumentException(String.valueOf(errorMessage));
     }

library/core/src/main/java/com/google/android/exoplayer2/util/LibraryLoader.java

@@ -36,7 +36,7 @@ public final class LibraryLoader {
    * {@link #isAvailable()}.
    */
   public synchronized void setLibraries(String... libraries) {
-    Assertions.checkState("Cannot set libraries after loading", !loadAttempted);
+    Assertions.checkState(!loadAttempted, "Cannot set libraries after loading");
     nativeLibraries = libraries;
   }
 

library/core/src/main/java/com/google/android/exoplayer2/video/DummySurface.java

@@ -226,11 +226,11 @@ public final class DummySurface extends Surface {
 
     private void initInternal(boolean secure) {
       EGLDisplay display = eglGetDisplay(EGL_DEFAULT_DISPLAY);
-      Assertions.checkState("eglGetDisplay failed", display != null);
+      Assertions.checkState(display != null, "eglGetDisplay failed");
 
       int[] version = new int[2];
       boolean eglInitialized = eglInitialize(display, version, 0, version, 1);
-      Assertions.checkState("eglInitialize failed", eglInitialized);
+      Assertions.checkState(eglInitialized, "eglInitialize failed");
 
       int[] eglAttributes = new int[] {
           EGL_RENDERABLE_TYPE, EGL_OPENGL_ES2_BIT,
@@ -247,8 +247,8 @@ public final class DummySurface extends Surface {
       int[] numConfigs = new int[1];
       boolean eglChooseConfigSuccess = eglChooseConfig(display, eglAttributes, 0, configs, 0, 1,
           numConfigs, 0);
-      Assertions.checkState("eglChooseConfig failed",
-          eglChooseConfigSuccess && numConfigs[0] > 0 && configs[0] != null);
+      Assertions.checkState(eglChooseConfigSuccess && numConfigs[0] > 0 && configs[0] != null,
+          "eglChooseConfig failed");
 
       EGLConfig config = configs[0];
       int[] glAttributes;
@@ -264,7 +264,7 @@ public final class DummySurface extends Surface {
       }
       EGLContext context = eglCreateContext(display, config, android.opengl.EGL14.EGL_NO_CONTEXT,
           glAttributes, 0);
-      Assertions.checkState("eglCreateContext failed", context != null);
+      Assertions.checkState(context != null, "eglCreateContext failed");
 
       int[] pbufferAttributes;
       if (secure) {
@@ -280,10 +280,10 @@ public final class DummySurface extends Surface {
             EGL_NONE};
       }
       EGLSurface pbuffer = eglCreatePbufferSurface(display, config, pbufferAttributes, 0);
-      Assertions.checkState("eglCreatePbufferSurface failed", pbuffer != null);
+      Assertions.checkState(pbuffer != null, "eglCreatePbufferSurface failed");
 
       boolean eglMadeCurrent = eglMakeCurrent(display, pbuffer, pbuffer, context);
-      Assertions.checkState("eglMakeCurrent failed", eglMadeCurrent);
+      Assertions.checkState(eglMadeCurrent, "eglMakeCurrent failed");
 
       glGenTextures(1, textureIdHolder, 0);
       surfaceTexture = new SurfaceTexture(textureIdHolder[0]);