corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1058136 - Fix debug logging crash; r=cpeterson

Browse Source
This commit is contained in:
Jim Chen
2014-09-04 14:38:17 -04:00
parent 4a7625bd8a
commit f12f4ca421
2 changed files with 23 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
mobile/android/base/GeckoEditable.java
View File

@@ -1039,12 +1039,14 @@ final class GeckoEditable
if (DEBUG) { if (DEBUG) {
StringBuilder log = new StringBuilder(method.getName()); StringBuilder log = new StringBuilder(method.getName());
log.append("("); log.append("(");
if (args != null) {
for (Object arg : args) { for (Object arg : args) {
debugAppend(log, arg).append(", "); debugAppend(log, arg).append(", ");
} }
if (args.length > 0) { if (args.length > 0) {
log.setLength(log.length() - 2); log.setLength(log.length() - 2);
} }
}
if (method.getReturnType().equals(Void.TYPE)) { if (method.getReturnType().equals(Void.TYPE)) {
log.append(")"); log.append(")");
} else { } else {

2
mobile/android/base/GeckoInputConnection.java
View File

@@ -1024,6 +1024,7 @@ final class DebugGeckoInputConnection
StringBuilder log = new StringBuilder(mCallLevel); StringBuilder log = new StringBuilder(mCallLevel);
log.append("> ").append(method.getName()).append("("); log.append("> ").append(method.getName()).append("(");
if (args != null) {
for (Object arg : args) { for (Object arg : args) {
// translate argument values to constant names // translate argument values to constant names
if ("notifyIME".equals(method.getName()) && arg == args[0]) { if ("notifyIME".equals(method.getName()) && arg == args[0]) {
@@ -1040,6 +1041,7 @@ final class DebugGeckoInputConnection
if (args.length > 0) { if (args.length > 0) {
log.setLength(log.length() - 2); log.setLength(log.length() - 2);
} }
}
log.append(")"); log.append(")");
Log.d(LOGTAG, log.toString()); Log.d(LOGTAG, log.toString());