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

Unified Diff: net/android/java/src/org/chromium/net/ProxyChangeListener.java

Issue 600983002: [Checkstyle] Fix misc style issues in Java files. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix build Created 6 years, 3 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: net/android/java/src/org/chromium/net/ProxyChangeListener.java
diff --git a/net/android/java/src/org/chromium/net/ProxyChangeListener.java b/net/android/java/src/org/chromium/net/ProxyChangeListener.java
index b130da06f842199480ca35268bd5be9676ed803b..51b67143b706a1e54410a14e0d7e2dcb102ddd9d 100644
--- a/net/android/java/src/org/chromium/net/ProxyChangeListener.java
+++ b/net/android/java/src/org/chromium/net/ProxyChangeListener.java
@@ -101,9 +101,9 @@ public class ProxyChangeListener {
// TODO(sgurun): once android.net.ProxyInfo is public, rewrite this.
private ProxyConfig extractNewProxy(Intent intent) {
try {
- final String GET_HOST_NAME = "getHost";
- final String GET_PORT_NAME = "getPort";
- final String GET_PAC_FILE_URL = "getPacFileUrl";
+ final String getHostName = "getHost";
Yaron 2014/09/25 00:12:05 this doesn't match item 5.2.7 of java style guide.
+ final String getPortName = "getPort";
+ final String getPacFileUrl = "getPacFileUrl";
String className;
String proxyInfo;
if (Build.VERSION.SDK_INT <= Build.VERSION_CODES.KITKAT) {
@@ -120,8 +120,8 @@ public class ProxyChangeListener {
}
Class<?> cls = Class.forName(className);
- Method getHostMethod = cls.getDeclaredMethod(GET_HOST_NAME);
- Method getPortMethod = cls.getDeclaredMethod(GET_PORT_NAME);
+ Method getHostMethod = cls.getDeclaredMethod(getHostName);
+ Method getPortMethod = cls.getDeclaredMethod(getPortName);
String host = (String) getHostMethod.invoke(props);
int port = (Integer) getPortMethod.invoke(props);
@@ -129,14 +129,14 @@ public class ProxyChangeListener {
// TODO(xunjieli): rewrite this once the API is public.
if (Build.VERSION.SDK_INT == Build.VERSION_CODES.KITKAT) {
Method getPacFileUrlMethod =
- cls.getDeclaredMethod(GET_PAC_FILE_URL);
+ cls.getDeclaredMethod(getPacFileUrl);
String pacFileUrl = (String) getPacFileUrlMethod.invoke(props);
if (!TextUtils.isEmpty(pacFileUrl)) {
return new ProxyConfig(host, port, pacFileUrl);
}
} else if (Build.VERSION.SDK_INT > Build.VERSION_CODES.KITKAT) {
Method getPacFileUrlMethod =
- cls.getDeclaredMethod(GET_PAC_FILE_URL);
+ cls.getDeclaredMethod(getPacFileUrl);
Uri pacFileUrl = (Uri) getPacFileUrlMethod.invoke(props);
if (!Uri.EMPTY.equals(pacFileUrl)) {
return new ProxyConfig(host, port, pacFileUrl.toString());

Powered by Google App Engine
This is Rietveld 408576698