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

Unified Diff: components/cronet/android/java/src/org/chromium/net/HttpUrlRequestFactoryConfig.java

Issue 652603004: Fix Java Checkstyle issues. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rename R to matrixR in DeviceSensors Created 6 years, 2 months 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: components/cronet/android/java/src/org/chromium/net/HttpUrlRequestFactoryConfig.java
diff --git a/components/cronet/android/java/src/org/chromium/net/HttpUrlRequestFactoryConfig.java b/components/cronet/android/java/src/org/chromium/net/HttpUrlRequestFactoryConfig.java
index 6c59ae18da42b31e8bf50bed595c41bcd85e309e..39c737aed740aa778f217bbb7b1cb2f3a82228d2 100644
--- a/components/cronet/android/java/src/org/chromium/net/HttpUrlRequestFactoryConfig.java
+++ b/components/cronet/android/java/src/org/chromium/net/HttpUrlRequestFactoryConfig.java
@@ -125,7 +125,7 @@ public class HttpUrlRequestFactoryConfig {
hint.put(UrlRequestContextConfig.QUIC_HINT_ALT_PORT, alternatePort);
quicHints.put(hint);
} catch (JSONException e) {
- ;
+ // Intentionally do nothing.
}
return this;
}
@@ -133,6 +133,7 @@ public class HttpUrlRequestFactoryConfig {
/**
* Get JSON string representation of the config.
*/
+ @Override
public String toString() {
return mConfig.toString();
}
@@ -145,7 +146,7 @@ public class HttpUrlRequestFactoryConfig {
try {
mConfig.put(key, value);
} catch (JSONException e) {
- ;
+ // Intentionally do nothing.
}
return this;
}
@@ -158,7 +159,7 @@ public class HttpUrlRequestFactoryConfig {
try {
mConfig.put(key, value);
} catch (JSONException e) {
- ;
+ // Intentionally do nothing.
}
return this;
}
@@ -171,7 +172,7 @@ public class HttpUrlRequestFactoryConfig {
try {
mConfig.put(key, value);
} catch (JSONException e) {
- ;
+ // Intentionally do nothing.
}
return this;
}

Powered by Google App Engine
This is Rietveld 408576698