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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/PepperPluginManager.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: content/public/android/java/src/org/chromium/content/browser/PepperPluginManager.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/PepperPluginManager.java b/content/public/android/java/src/org/chromium/content/browser/PepperPluginManager.java
index 4db35fe6122214f7a3085f8dfbd037bc523b79e9..4062121cc2732b464e89cdc7e17d2e4af2899c12 100644
--- a/content/public/android/java/src/org/chromium/content/browser/PepperPluginManager.java
+++ b/content/public/android/java/src/org/chromium/content/browser/PepperPluginManager.java
@@ -96,8 +96,8 @@ public class PepperPluginManager {
for (ResolveInfo info : plugins) {
// Retrieve the plugin's service information.
ServiceInfo serviceInfo = info.serviceInfo;
- if (serviceInfo == null || serviceInfo.metaData == null ||
- serviceInfo.packageName == null) {
+ if (serviceInfo == null || serviceInfo.metaData == null
+ || serviceInfo.packageName == null) {
Log.e(LOGTAG, "Can't get service information from " + info);
continue;
}
@@ -110,8 +110,8 @@ public class PepperPluginManager {
Log.e(LOGTAG, "Can't find plugin: " + serviceInfo.packageName);
continue;
}
- if (pkgInfo == null ||
- (pkgInfo.applicationInfo.flags & ApplicationInfo.FLAG_SYSTEM) == 0) {
+ if (pkgInfo == null
+ || (pkgInfo.applicationInfo.flags & ApplicationInfo.FLAG_SYSTEM) == 0) {
continue;
}
Log.i(LOGTAG, "The given plugin package is preloaded: " + serviceInfo.packageName);

Powered by Google App Engine
This is Rietveld 408576698