diff options
author | calderwoodra <calderwoodra@google.com> | 2018-03-12 18:30:32 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-03-12 18:32:51 -0700 |
commit | f53ba7be0531904a25c15f69c8e1bfdea88be869 (patch) | |
tree | 67ce349c892e04e817c3c96f7210560f49793e07 /java/com/android/dialer/searchfragment | |
parent | c8039e034838a030848bb040d10997835153427e (diff) |
Keyboard and search now close when appropriate.
Keyboard should close when:
- requeting permission
- placing a call
- leaving the app
Search should close when:
- placing a call
- leaving the app
Bug: 74235853
Test: search closing onPause tests already exist (MainActivityOldPeerSearchIntegrationTest#placingCallFromSearchClosesSearch)
PiperOrigin-RevId: 188812026
Change-Id: Iaee084c59b4c514d10375deff5d73d5a456ef9df
Diffstat (limited to 'java/com/android/dialer/searchfragment')
-rw-r--r-- | java/com/android/dialer/searchfragment/list/NewSearchFragment.java | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/java/com/android/dialer/searchfragment/list/NewSearchFragment.java b/java/com/android/dialer/searchfragment/list/NewSearchFragment.java index aff946206..6a60598a7 100644 --- a/java/com/android/dialer/searchfragment/list/NewSearchFragment.java +++ b/java/com/android/dialer/searchfragment/list/NewSearchFragment.java @@ -29,7 +29,6 @@ import android.preference.PreferenceManager; import android.support.annotation.NonNull; import android.support.annotation.Nullable; import android.support.annotation.VisibleForTesting; -import android.support.v13.app.FragmentCompat; import android.support.v7.widget.LinearLayoutManager; import android.support.v7.widget.RecyclerView; import android.telephony.PhoneNumberUtils; @@ -342,8 +341,8 @@ public final class NewSearchFragment extends Fragment LogUtil.i( "NewSearchFragment.onEmptyViewActionButtonClicked", "Requesting permissions: " + Arrays.toString(deniedPermissions)); - FragmentCompat.requestPermissions( - this, deniedPermissions, READ_CONTACTS_PERMISSION_REQUEST_CODE); + FragmentUtils.getParentUnsafe(this, SearchFragmentListener.class).requestingPermission(); + requestPermissions(deniedPermissions, READ_CONTACTS_PERMISSION_REQUEST_CODE); } } @@ -411,6 +410,7 @@ public final class NewSearchFragment extends Fragment String[] deniedPermissions = PermissionsUtil.getPermissionsCurrentlyDenied( getContext(), PermissionsUtil.allLocationGroupPermissionsUsedInDialer); + FragmentUtils.getParentUnsafe(this, SearchFragmentListener.class).requestingPermission(); requestPermissions(deniedPermissions, LOCATION_PERMISSION_REQUEST_CODE); } @@ -557,5 +557,8 @@ public final class NewSearchFragment extends Fragment /** Called when a call is placed from the search fragment. */ void onCallPlacedFromSearch(); + + /** Called when a permission is about to be requested. */ + void requestingPermission(); } } |