diff options
| author | Brad Ebinger <breadley@google.com> | 2016-11-01 14:11:38 -0700 |
|---|---|---|
| committer | Brad Ebinger <breadley@google.com> | 2016-11-16 11:05:15 -0800 |
| commit | 0c3541be65fa87519a879c053a7cf4b4526be5db (patch) | |
| tree | 7ea3e6055b8cb8668db81449004ede5c730f54ec /telecomm/java/android/telecom/Logging/SessionManager.java | |
| parent | 8631e19d3883ce6887c09f72eb394f2e22c9cb64 (diff) | |
Readability improvements in Logging
We now truncate Session method paths and IDs based on the Logging level
to improve Session Logging readability in logcat. If another external
session is started off of an existing external Session, the preceding
histories are replaced with "..." so that the Session information is not
overwhelming.
Bug: 26571395
Test: Unit Tests and manual tests pass
Change-Id: I9ffda3d64f1072fa6228a82a86116a5e47d18c96
Diffstat (limited to 'telecomm/java/android/telecom/Logging/SessionManager.java')
| -rw-r--r-- | telecomm/java/android/telecom/Logging/SessionManager.java | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/telecomm/java/android/telecom/Logging/SessionManager.java b/telecomm/java/android/telecom/Logging/SessionManager.java index 8ced7f8181c5..949f7b7a89ae 100644 --- a/telecomm/java/android/telecom/Logging/SessionManager.java +++ b/telecomm/java/android/telecom/Logging/SessionManager.java @@ -177,8 +177,9 @@ public class SessionManager { } // Create Session from Info and add to the sessionMapper under this ID. + Log.d(LOGGING_TAG, Session.START_EXTERNAL_SESSION); Session externalSession = new Session(Session.EXTERNAL_INDICATOR + sessionInfo.sessionId, - sessionInfo.shortMethodName, System.currentTimeMillis(), + sessionInfo.methodPath, System.currentTimeMillis(), false /*isStartedFromActiveSession*/, null); externalSession.setIsExternal(true); // Mark the external session as already completed, since we have no way of knowing when @@ -190,8 +191,6 @@ public class SessionManager { // Create a subsession from this external Session parent node Session childSession = createSubsession(); continueSession(childSession, shortMethodName); - - Log.d(LOGGING_TAG, Session.START_SESSION); } /** |