Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1053)

Unified Diff: media/base/android/java/src/org/chromium/media/MediaCodecBridge.java

Issue 744453002: Fix a bunch of Java Checkstyle issues. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: NeedsBraces to info Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/base/android/java/src/org/chromium/media/MediaCodecBridge.java
diff --git a/media/base/android/java/src/org/chromium/media/MediaCodecBridge.java b/media/base/android/java/src/org/chromium/media/MediaCodecBridge.java
index 399f355f8735a097267929ea42eb81ddb41ffda6..11cff4baf9cb6e8bc28fc97b11f84b5b1994bbf5 100644
--- a/media/base/android/java/src/org/chromium/media/MediaCodecBridge.java
+++ b/media/base/android/java/src/org/chromium/media/MediaCodecBridge.java
@@ -227,8 +227,8 @@ class MediaCodecBridge {
codecName = mediaCodec.getName();
mediaCodec.release();
} catch (Exception e) {
- Log.w(TAG, "getDefaultCodecName: Failed to create MediaCodec: " +
- mime + ", direction: " + direction, e);
+ Log.w(TAG, "getDefaultCodecName: Failed to create MediaCodec: "
+ + mime + ", direction: " + direction, e);
}
}
return codecName;
@@ -688,8 +688,8 @@ class MediaCodecBridge {
}
int size = mAudioTrack.write(buf, 0, buf.length);
if (buf.length != size) {
- Log.i(TAG, "Failed to send all data to audio output, expected size: " +
- buf.length + ", actual size: " + size);
+ Log.i(TAG, "Failed to send all data to audio output, expected size: "
+ + buf.length + ", actual size: " + size);
}
// TODO(qinmin): Returning the head position allows us to estimate
// the current presentation time in native code. However, it is

Powered by Google App Engine
This is Rietveld 408576698