Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add logging to CallerInfoAsyncQuery

Browse Source 
Add logging to CallerInfoAsyncQuery to help diagnose timeout errors

Bug: 31755827
Change-Id: If7cf34baf084ca8916952842585a32b3e93eba73
This commit is contained in:
Hall Liu
2016-10-05 16:56:17 -07:00
parent e4865bcb39
commit 4c01910cb0
1 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
telephony/java/com/android/internal/telephony/CallerInfoAsyncQuery.java
View File

@@ -176,13 +176,13 @@ public class CallerInfoAsyncQuery {
// However, if there is any code that this Handler calls (such as in // However, if there is any code that this Handler calls (such as in
// super.handleMessage) that DOES place unexpected messages on the // super.handleMessage) that DOES place unexpected messages on the
// queue, then we need pass these messages on. // queue, then we need pass these messages on.
if (DBG) Rlog.d(LOG_TAG, "Unexpected command (CookieWrapper is null): " + msg.what + Rlog.i(LOG_TAG, "Unexpected command (CookieWrapper is null): " + msg.what +
" ignored by CallerInfoWorkerHandler, passing onto parent."); " ignored by CallerInfoWorkerHandler, passing onto parent.");
super.handleMessage(msg); super.handleMessage(msg);
} else { } else {
if (DBG) Rlog.d(LOG_TAG, "Processing event: " + cw.event + " token (arg1): " + msg.arg1 + Rlog.d(LOG_TAG, "Processing event: " + cw.event + " token (arg1): " + msg.arg1 +
" command: " + msg.what + " query URI: " + sanitizeUriToString(args.uri)); " command: " + msg.what + " query URI: " + sanitizeUriToString(args.uri));
switch (cw.event) { switch (cw.event) {
@@ -239,7 +239,7 @@ public class CallerInfoAsyncQuery {
*/ */
@Override @Override
protected void onQueryComplete(int token, Object cookie, Cursor cursor) { protected void onQueryComplete(int token, Object cookie, Cursor cursor) {
if (DBG) Rlog.d(LOG_TAG, "##### onQueryComplete() ##### query complete for token: " + token); Rlog.d(LOG_TAG, "##### onQueryComplete() ##### query complete for token: " + token);
//get the cookie and notify the listener. //get the cookie and notify the listener.
CookieWrapper cw = (CookieWrapper) cookie; CookieWrapper cw = (CookieWrapper) cookie;
@@ -248,7 +248,7 @@ public class CallerInfoAsyncQuery {
// from within this code. // from within this code.
// However, if there is any code that calls this method, we should // However, if there is any code that calls this method, we should
// check the parameters to make sure they're viable. // check the parameters to make sure they're viable.
if (DBG) Rlog.d(LOG_TAG, "Cookie is null, ignoring onQueryComplete() request."); Rlog.i(LOG_TAG, "Cookie is null, ignoring onQueryComplete() request.");
if (cursor != null) { if (cursor != null) {
cursor.close(); cursor.close();
} }
@@ -333,9 +333,11 @@ public class CallerInfoAsyncQuery {
//notify the listener that the query is complete. //notify the listener that the query is complete.
if (cw.listener != null) { if (cw.listener != null) {
if (DBG) Rlog.d(LOG_TAG, "notifying listener: " + cw.listener.getClass().toString() + Rlog.d(LOG_TAG, "notifying listener: " + cw.listener.getClass().toString() +
" for token: " + token + mCallerInfo); " for token: " + token + mCallerInfo);
cw.listener.onQueryComplete(token, cw.cookie, mCallerInfo); cw.listener.onQueryComplete(token, cw.cookie, mCallerInfo);
} else {
Rlog.w(LOG_TAG, "There is no listener to notify for this query.");
} }
if (cursor != null) { if (cursor != null) {