Merge "Add null check for cursor in DirectoryListLoader"
diff --git a/src/com/android/contacts/common/GeoUtil.java b/src/com/android/contacts/common/GeoUtil.java
index 4bf001a..6c41d24 100644
--- a/src/com/android/contacts/common/GeoUtil.java
+++ b/src/com/android/contacts/common/GeoUtil.java
@@ -52,8 +52,6 @@
public static String getGeocodedLocationFor(Context context, String phoneNumber) {
final PhoneNumberOfflineGeocoder geocoder = PhoneNumberOfflineGeocoder.getInstance();
final PhoneNumberUtil phoneNumberUtil = PhoneNumberUtil.getInstance();
- final CountryDetector countryDetector =
- (CountryDetector) context.getSystemService(Context.COUNTRY_DETECTOR);
try {
final Phonenumber.PhoneNumber structuredPhoneNumber =
phoneNumberUtil.parse(phoneNumber, getCurrentCountryIso(context));
diff --git a/src/com/android/contacts/common/list/ContactListItemView.java b/src/com/android/contacts/common/list/ContactListItemView.java
index 9c30cb6..9fdb24d 100644
--- a/src/com/android/contacts/common/list/ContactListItemView.java
+++ b/src/com/android/contacts/common/list/ContactListItemView.java
@@ -1536,10 +1536,15 @@
// If the touch event's coordinates are not within the view's header, then delegate
// to super.onTouchEvent so that regular view behavior is preserved. Otherwise, consume
// and ignore the touch event.
- if (mBoundsWithoutHeader.contains((int) x, (int) y) || !pointInView(x, y, 0)) {
+ if (mBoundsWithoutHeader.contains((int) x, (int) y) || !pointIsInView(x, y)) {
return super.onTouchEvent(event);
} else {
return true;
}
}
+
+ private final boolean pointIsInView(float localX, float localY) {
+ return localX >= 0 && localX < (mRight - mLeft)
+ && localY >= 0 && localY < (mBottom - mTop);
+ }
}