Merge "Fix NPE caused by misplaced parenthesis." (011d958b) · Commits · e / os / android_packages_apps_Dialer

InCallUI/src/com/android/incallui/CallCardPresenter.java

+12 −4

Original line number	Diff line number	Diff line
		@@ -249,7 +249,6 @@ public class CallCardPresenter extends Presenter<CallCardPresenter.CallCardUi>
		Call.areSameNumber(mPrimary, primary));
		final boolean secondaryChanged = !(Call.areSame(mSecondary, secondary) &&
		Call.areSameNumber(mSecondary, secondary));
		final boolean shouldShowCallSubject = shouldShowCallSubject(mPrimary);

		mSecondary = secondary;
		Call previousPrimary = mPrimary;
		@@ -262,9 +261,8 @@ public class CallCardPresenter extends Presenter<CallCardPresenter.CallCardUi>
		// Refresh primary call information if either:
		// 1. Primary call changed.
		// 2. The call's ability to manage conference has changed.
		if (mPrimary != null && (primaryChanged \|\|
		ui.isManageConferenceVisible() != shouldShowManageConference()) \|\|
		ui.isCallSubjectVisible() != shouldShowCallSubject) {
		// 3. The call subject should be shown or hidden.
		if (shouldRefreshPrimaryInfo(primaryChanged, ui, shouldShowCallSubject(mPrimary))) {
		// primary call has changed
		if (previousPrimary != null) {
		CallList.getInstance().removeCallUpdateListener(previousPrimary.getId(), this);
		@@ -396,6 +394,16 @@ public class CallCardPresenter extends Presenter<CallCardPresenter.CallCardUi>
		updatePrimaryDisplayInfo();
		}

		private boolean shouldRefreshPrimaryInfo(boolean primaryChanged, CallCardUi ui,
		boolean shouldShowCallSubject) {
		if (mPrimary == null) {
		return false;
		}
		return primaryChanged \|\|
		ui.isManageConferenceVisible() != shouldShowManageConference() \|\|
		ui.isCallSubjectVisible() != shouldShowCallSubject;
		}

		private String getSubscriptionNumber() {
		// If it's an emergency call, and they're not populating the callback number,
		// then try to fall back to the phone sub info (to hopefully get the SIM's