Looks good to me. I usually wrap my #requestLocationUpdates() in conditionals that check for availability first.
if (this.mLocationManager.isProviderEnabled(LocationManager.GPS_PROVIDER)) {
this.mLocationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER, 0, 0, this);
}
if (this.mLocationManager.isProviderEnabled(LocationManager.NETWORK_PROVIDER)) {
this.mLocationManager.requestLocationUpdates(LocationManager.NETWORK_PROVIDER, 0, 0, this);
}