diff options
author | Hung-ying Tyan <tyanh@google.com> | 2010-08-09 17:15:35 +0800 |
---|---|---|
committer | Hung-ying Tyan <tyanh@google.com> | 2010-08-09 17:15:35 +0800 |
commit | 8fb2e6e4720385961083a150a3e848ccaef544ae (patch) | |
tree | 91cf07d3fef33ef26ea13551746160f7aaa44cd1 /telephony | |
parent | d122805bbfcb770f941cac259bcefafdb2d3ad56 (diff) | |
download | frameworks_base-8fb2e6e4720385961083a150a3e848ccaef544ae.zip frameworks_base-8fb2e6e4720385961083a150a3e848ccaef544ae.tar.gz frameworks_base-8fb2e6e4720385961083a150a3e848ccaef544ae.tar.bz2 |
PhoneFactory: add makeSipPhone()
Change-Id: I8e7abb0623724d24c6f82829ff06663a31ba32fb
Diffstat (limited to 'telephony')
-rw-r--r-- | telephony/java/com/android/internal/telephony/PhoneFactory.java | 11 | ||||
-rw-r--r-- | telephony/java/com/android/internal/telephony/sip/SipPhoneFactory.java | 46 |
2 files changed, 25 insertions, 32 deletions
diff --git a/telephony/java/com/android/internal/telephony/PhoneFactory.java b/telephony/java/com/android/internal/telephony/PhoneFactory.java index 803b736..2e391cb 100644 --- a/telephony/java/com/android/internal/telephony/PhoneFactory.java +++ b/telephony/java/com/android/internal/telephony/PhoneFactory.java @@ -24,6 +24,8 @@ import android.util.Log; import com.android.internal.telephony.cdma.CDMAPhone; import com.android.internal.telephony.gsm.GSMPhone; +import com.android.internal.telephony.sip.SipPhone; +import com.android.internal.telephony.sip.SipPhoneFactory; /** * {@hide} @@ -175,4 +177,13 @@ public class PhoneFactory { return phone; } } + + /** + * Makes a {@link SipPhone} object. + * @param sipUri the local SIP URI the phone runs on + * @return the {@code SipPhone} object or null if the SIP URI is not valid + */ + public static SipPhone makeSipPhone(String sipUri) { + return SipPhoneFactory.makePhone(sipUri, sContext, sPhoneNotifier); + } } diff --git a/telephony/java/com/android/internal/telephony/sip/SipPhoneFactory.java b/telephony/java/com/android/internal/telephony/sip/SipPhoneFactory.java index c9e9762..a1bdd2f 100644 --- a/telephony/java/com/android/internal/telephony/sip/SipPhoneFactory.java +++ b/telephony/java/com/android/internal/telephony/sip/SipPhoneFactory.java @@ -16,7 +16,6 @@ package com.android.internal.telephony.sip; -import com.android.internal.telephony.Phone; import com.android.internal.telephony.PhoneNotifier; import android.content.Context; @@ -26,42 +25,25 @@ import android.util.Log; import java.text.ParseException; /** - * @hide + * {@hide} */ public class SipPhoneFactory { - private static PhoneNotifier sPhoneNotifier = makeDefaultPhoneNotifier(); - private static Context sContext; - - public static void makeDefaultPhones(Context context) { - makeDefaultPhone(context); - } - - public static void makeDefaultPhone(Context context) { - sContext = context; - SipPhoneProxy.getInstance().setPhone( - makePhone("sip:anonymous@localhost")); - } - - public static Phone getDefaultPhone() { - return SipPhoneProxy.getInstance(); - } - - public static SipPhone makePhone(String sipProfileUri) { + /** + * Makes a {@link SipPhone} object. + * @param sipUri the local SIP URI the phone runs on + * @param context {@code Context} needed to create a Phone object + * @param phoneNotifier {@code PhoneNotifier} needed to create a Phone + * object + * @return the {@code SipPhone} object or null if the SIP URI is not valid + */ + public static SipPhone makePhone(String sipUri, Context context, + PhoneNotifier phoneNotifier) { try { - SipProfile profile = new SipProfile.Builder(sipProfileUri).build(); - return new SipPhone(sContext, sPhoneNotifier, profile); + SipProfile profile = new SipProfile.Builder(sipUri).build(); + return new SipPhone(context, phoneNotifier, profile); } catch (ParseException e) { - Log.v("SipPhoneProxy", "setPhone", e); + Log.w("SipPhoneProxy", "setPhone", e); return null; } } - - private static PhoneNotifier makeDefaultPhoneNotifier() { - try { - return new com.android.internal.telephony.SipPhoneNotifier(); - } catch (Error e) { - Log.e("SipPhoneProxy", "makeDefaultPhoneNotifier", e); - throw e; - } - } } |