Minor cleanup

library/core/src/androidTest/assets/ts/sample.ps.0.dump

@@ -69,7 +69,7 @@ track 224:
   sample 0:
     time = 40000
     flags = 1
-    data = length 20616, hash CA38A5B5
+    data = length 20646, hash 576390B
   sample 1:
     time = 80000
     flags = 0

library/core/src/androidTest/assets/ts/sample.ts.0.dump

@@ -28,9 +28,9 @@ track 256:
       data = length 22, hash CE183139
   sample count = 2
   sample 0:
-    time = 33366
+    time = 0
     flags = 1
-    data = length 20669, hash 26DABA0F
+    data = length 20711, hash 34341E8
   sample 1:
     time = 66733
     flags = 0

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

@@ -137,9 +137,10 @@ public final class H262Reader implements ElementaryStreamReader {
         }
       }
 
-      if (hasOutputFormat && (startCodeValue == START_PICTURE || startCodeValue == START_SEQUENCE_HEADER)) {
+      if (hasOutputFormat
+          && (startCodeValue == START_PICTURE || startCodeValue == START_SEQUENCE_HEADER)) {
         int bytesWrittenPastStartCode = limit - startCodeOffset;
-        boolean resetSample = (samplePosition == C.POSITION_UNSET);
+        boolean resetSample = samplePosition == C.POSITION_UNSET;
         if (foundPicture) {
           @C.BufferFlags int flags = isKeyframe ? C.BUFFER_FLAG_KEY_FRAME : 0;
           int size = (int) (totalBytesWritten - samplePosition) - bytesWrittenPastStartCode;
@@ -147,7 +148,7 @@ public final class H262Reader implements ElementaryStreamReader {
           isKeyframe = false;
           resetSample = true;
         }
-        foundPicture = (startCodeValue == START_PICTURE);
+        foundPicture = startCodeValue == START_PICTURE;
         if (resetSample) {
           samplePosition = totalBytesWritten - bytesWrittenPastStartCode;
           sampleTimeUs = (pesPtsUsAvailable ? pesTimeUs : sampleTimeUs + frameDurationUs);

library/ui/src/main/java/com/google/android/exoplayer2/ui/SimpleExoPlayerView.java

@@ -126,7 +126,8 @@ import java.util.List;
  *         <li>Default: {@code R.id.exo_playback_control_view}</li>
  *       </ul>
  *   <li>All attributes that can be set on a {@link PlaybackControlView} can also be set on a
- *       SimpleExoPlayerView, and will be propagated to the inflated {@link PlaybackControlView}.
+ *       SimpleExoPlayerView, and will be propagated to the inflated {@link PlaybackControlView}
+ *       unless the layout is overridden to specify a custom {@code exo_controller} (see below).
  *   </li>
  * </ul>
  *
@@ -163,18 +164,17 @@ import java.util.List;
  *       </ul>
  *   </li>
  *   <li><b>{@code exo_controller_placeholder}</b> - A placeholder that's replaced with the inflated
- *       {@link PlaybackControlView}.
+ *       {@link PlaybackControlView}. Ignored if an {@code exo_controller} view exists.
  *       <ul>
  *        <li>Type: {@link View}</li>
  *       </ul>
  *   </li>
- *   <li><b>{@code exo_controller}</b> - An already inflated instance of
- *       {@link PlaybackControlView}. Allows you to use your own {@link PlaybackControlView} instead
- *       of default. Note: attrs such as rewind_increment will not be passed through to this
- *       instance and should be set at creation. {@code exo_controller_placeholder} will be ignored
- *       if this is set.
+ *   <li><b>{@code exo_controller}</b> - An already inflated {@link PlaybackControlView}. Allows use
+ *       of a custom extension of {@link PlaybackControlView}. Note that attributes such as
+ *       {@code rewind_increment} will not be automatically propagated through to this instance. If
+ *       a view exists with this id, any {@code exo_controller_placeholder} view will be ignored.
  *       <ul>
- *        <li>Type: {@link View}</li>
+ *        <li>Type: {@link PlaybackControlView}</li>
  *       </ul>
  *   </li>
  *   <li><b>{@code exo_overlay}</b> - A {@link FrameLayout} positioned on top of the player which