summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2017-09-22Adding some contacts permission guards.calderwoodra
Bug: 64302851 Test: manual PiperOrigin-RevId: 169600984 Change-Id: I65376894012f2d570b6c918f3f5575e7f45bf938
2017-09-22Better handling country iso when getting geo location of phone number.wangqi
1. Use existing country iso info instead of guessing current country iso from sim and locale for blocked number. 2. Use country iso of current sim instead of default sim for incall location info. This fixes bug in multi sim device when default sim is out of service and making call with the other sim. Bug: 65399976 Test: manual PiperOrigin-RevId: 169554641 Change-Id: I416d7e2d6bc3c872bfec3eda4005a5a1684f0e40
2017-09-22Dialer integration of contacts suggestions promoAndroid Dialer
Illustration icon TBD https://photos.app.goo.gl/ZDLNLlXg9VBKNHGx1 Manual testing of duplicates promo end to end flow accept/dismiss states Bug: 64491095 Test: http://sponge/db7bd5c3-ec97-46c8-a058-ed7f46a5e4ce PiperOrigin-RevId: 169480950 Change-Id: Ia553e87ce72d84a001769851abb925da1ab31756
2017-09-22Updated ongoing call notification to reflect video call state.calderwoodra
screenshots: ongoing: http://screen/FkAM9hOU5Rq paused: http://screen/mrp6UJCoWAu Bug: 63274792,63271690 Test: StatusBarNotifierTest PiperOrigin-RevId: 169469414 Change-Id: I442e21d93b7e11dea191311acd28846ea257c156
2017-09-22Added dismiss button to location permission request.calderwoodra
This change adds a dismiss button to the location permission prompt shown in the search UI. A few other minor changes: - Permission prompt is always the first element in the list - Adjusted a few dimensions and strings screenshot: http://screen/9CkRPUQ5dwh Bug: 65858857 Test: SAT, NSFT PiperOrigin-RevId: 169466585 Change-Id: I8177c23fe3f635fec5fdcdbe6675428cebb19f5d
2017-09-22Refactor CallList to call onUpdateCall and notifyGenericListeners directly.wangqi
Bug: 62426694 Test: none PiperOrigin-RevId: 169447674 Change-Id: I3014b289c26d24ac571421bc54fcacc303278b41
2017-09-22Added location permission request inline the search fragment.calderwoodra
This change adds a prompt inline the search fragment list view to request the location permission (if they haven't already granted it). screenshot: http://screen/upu9t55mghq Bug: 65858857 Test: NSFT, SAT, SCMT PiperOrigin-RevId: 169447095 Change-Id: I6c312057ff3c4e2362ce21b0c57e1e5de7b25ce0
2017-09-22Update package assets and strings.Eric Erfanian
Test: TH Change-Id: I48e807f3eafbc82401b2674a2610b20bf118c275
2017-09-20Merge changes I0d53a576,I0baf31c3,I297b945c,I63c9165fTreehugger Robot
* changes: Forgot to keep this file in sync with //google/internal/communications/voicemailtranscription/v1/voicemail_transcription.proto Fixed issue where all contacts weren't beign shown. Add more logging when video surface texture is destroyed. Don't truncate long voicemail transcriptions
2017-09-20Merge "Added some bug fixes in call details."Treehugger Robot
2017-09-20Forgot to keep this file in sync with ↵Android Dialer
//google/internal/communications/voicemailtranscription/v1/voicemail_transcription.proto Test: n/a PiperOrigin-RevId: 169425975 Change-Id: I0d53a576e1245cc1c4d7c8e36df3a510d8c7bdbe
2017-09-20Fixed issue where all contacts weren't beign shown.calderwoodra
The underlying issue was that if a query was input that filtered out all of the on device contacts, then the device rotated, the cursor would be reloaded with all contacts filtered and getCount == 0. This would cause the cursor to never be set in SearchCursorManager and on device contacts would no longer be shown because the cursor wasn't being saved. The reason this check was here, was because cursor headers were being managed by SearchCursorManager. Since headers were moved into the cursors we no longer need to set cursors to null if they are empty. Bug: 65859191 Test: existing PiperOrigin-RevId: 169425922 Change-Id: I0baf31c3747d679f394f3746f609b9487e52472d
2017-09-20Add more logging when video surface texture is destroyed.wangqi
Bug: 63608380 Test: none PiperOrigin-RevId: 169425768 Change-Id: I297b945c9601bb070f3ef08d7ca34a76629b42c7
2017-09-20Don't truncate long voicemail transcriptionsmdooley
Also repostion contact icon (for allen, is there a bug for that?) Originally i tried wrapping the transcription in a ScrollView, but that didn't really work since there's no easy way to set a max-height for it. Since most voicemails transcriptions are not that long, an easier solution was just to not limit the number of lines for the transcription. screen shots: https://drive.google.com/open?id=0B9o_KvtLkcuINkdXeVV0U0dOZVNfdExzSDNsai1Ld2F3ZlBN https://drive.google.com/open?id=0B9o_KvtLkcuIZTZFQzRQaW1nblBPN2JkVW1hMkR3YWl1b2JV Bug: 64211399 Test: manual PiperOrigin-RevId: 169425482 Change-Id: I63c9165f48d902c06e288280116cd9b95d67852c
2017-09-20Added some bug fixes in call details.calderwoodra
When a user dials a number with post dial digits, they are now carried to contact details and used when they redial that number. Additionally, when a user clicks on a photo in call details, they will be taken to Bugle now. This behavior already exists with messages in CD. Bug: 62236726,62230491 Test: CDEVHT PiperOrigin-RevId: 169422921 Change-Id: I1507cb64876540cfc9a683a5a1ea01910443f77b
2017-09-20Merge "Enable Orange FR/BE visual voicemail"Treehugger Robot
2017-09-20Merge "Show legacy voicemail notification under FBE"Treehugger Robot
2017-09-19Enable Orange FR/BE visual voicemailtwyen
Bug: 65980572 Test: manual PiperOrigin-RevId: 169336670 Change-Id: I6acd6226c1a03b8f0c12309bc84ae7c2bdad5f30
2017-09-19Show legacy voicemail notification under FBEtwyen
This CL partially revert cl/159492498, since data SMS cannot be properly processed under FBE and visual voicemail notification won't be received. Bug: 65555345 Test: LegacyVoicemailNotificationReceiverTest PiperOrigin-RevId: 169321096 Change-Id: I7134d80479efef67de8d5d44131eea30f82297a0
2017-09-20Merge "Fix notification update bug"Treehugger Robot
2017-09-19Merge "Refactor second call logging"Treehugger Robot
2017-09-19Fix notification update bugmdooley
The problem was that the query for looking up a voicemail by URI was broken which was causing the notifications to be cancelled then when they were supposed to be updated they were actually getting re-added thus causing the notification sound. Bug: 65692173 Test: manual and unit test PiperOrigin-RevId: 169304391 Change-Id: I1e8f648231b6f0ab984a5b0e4facd8351b01391a
2017-09-19Refactor second call logginguabdullah
Just easier readability since getFirstCall() is null until it's added to the call map. Test: N/A PiperOrigin-RevId: 169287247 Change-Id: Ifda5ead27d987e17f259da761a0109d930efa6c3
2017-09-19Merge changes I3f26d915,I59500877,Ia08ce56fTreehugger Robot
* changes: Perform remote reachability query if caller is not in contacts Only enable voicemail transcription on android O and later Removed zero suggest from dialpad search in new search fragment.
2017-09-19Merge "Tweaking emergency location text"Treehugger Robot
2017-09-19Perform remote reachability query if caller is not in contactstwyen
Lightbringer.supportsUpgrade() is changed to return absent optional when it does not have data for the number. Seeing this LightbringerTech will use Lightbringer.updateReachability() to retrieve it from the remote package, which will store it in the reachability cache. Bug: 63601277 Test: RemoteReachabillityQueryHandlerTest, LightbringerTechTest PiperOrigin-RevId: 169283953 Change-Id: I3f26d9158fc6cfed196fd533da2aad598c8e6a7a
2017-09-19Only enable voicemail transcription on android O and latermdooley
Bug: 65648343 Test: updated unit tests (already submitted in previous cl) PiperOrigin-RevId: 169280819 Change-Id: I5950087765ffd038c31b872c6468951472980407
2017-09-19Removed zero suggest from dialpad search in new search fragment.calderwoodra
Zero suggest in dialpad search clutters the UI and we shouldn't show and search results until the user has input something into the dialpad. Additionally, this change ensures the fab stays hidden through device rotation in regular search. Bug: 65858825 Test: implemented PiperOrigin-RevId: 169270161 Change-Id: Ia08ce56ffd02f74ea9e4d9d8a1efc3726fcc66d9
2017-09-19Tweaking emergency location textmdooley
Bug: 37242283 Test: none PiperOrigin-RevId: 169270157 Change-Id: Ic07df7a1611ebf1b997f1a908fa341f9bf542cbd
2017-09-19Merge "Add "Turn speaker on"/"Turn speaker off" action button in notification."Treehugger Robot
2017-09-19Add "Turn speaker on"/"Turn speaker off" action button in notification.yueg
Add action button when create in call notification and when audio state changes to ROUTE_SPEAKER or ROUTE_WIRED_OR_EARPIECE. Ignore when audio state is ROUTE_BLUETOOTH. Screenshot: Speaker on: https://screenshot.googleplex.com/hnsQL0YcFJj Speaker off: https://screenshot.googleplex.com/oFrbvbs9gVG Test: StatusBarNotifierTest PiperOrigin-RevId: 169169372 Change-Id: I2f96f20170dd174b35dfd3f7578fe5b9450391ab
2017-09-19Merge "Log if a call is IMS reachable."Treehugger Robot
2017-09-19Merge "Use logCallImpression for bubble expand/collapse if possible."Treehugger Robot
2017-09-19Merge "Always show line1 number in emergency call."Treehugger Robot
2017-09-19Merge "Now persist state through configuration changes in Search."Treehugger Robot
2017-09-18Log if a call is IMS reachable.yueg
And fix a related test in ImsVideoTechTest. LOG_STORAGE_INCREASE(GB/week): 0.7 5M active block user/day * 10 events/active = 350M event/wk 350M event/wk * (2 [bool])/event = 0.7GB/wk Test: LoggerUtilsTest PiperOrigin-RevId: 169153878 Change-Id: Ia8038a5b0961f1a5572541f21df7a2680d85bbbb
2017-09-18Use logCallImpression for bubble expand/collapse if possible.yueg
So we can associate all bubble actions with call id, and analyze bubble actions per call session. Also move code for sending primary action back to primaryButtonClick(), which is introduced in cl/167347311. Test: manual PiperOrigin-RevId: 169152088 Change-Id: I97f0971f162684d882f6addbc012f6e8552216b7
2017-09-18Always show line1 number in emergency call.wangqi
The subcription number might be different than the line1 number (sim number), which could be confusion to user during emergency call when it's not same as the sim number the user knows of. Bug: 65685277 Test: manual PiperOrigin-RevId: 169147724 Change-Id: I2f89559d10a188b13937e80528b756cc9b11a02e
2017-09-18Now persist state through configuration changes in Search.calderwoodra
This CL adds logic to persist the search query and call initiation type through configuration changes like device rotation. list of bugs: https://docs.google.com/document/d/1GajzkAqXXRqbj_-qeYeDxIKr0fPsSZC3nXhRkIQg5tE/edit#heading=h.bopot1qivvxg Bug: 65751247 Test: NSFT PiperOrigin-RevId: 169146766 Change-Id: I7a68c01d5698c07a362ad621642a9138fd9a9aaa
2017-09-18Merge "Bump versions for v14"Treehugger Robot
2017-09-18Bump versions for v14maxwelb
v13 is now at go/dialer-v13, I've set it's max version to 1899999. Test: N/A PiperOrigin-RevId: 169123311 Change-Id: Ib8304953d0ad974a2f22c1b390ff3bb180360244
2017-09-18Merge changes I6443a05c,Ie840acd6Treehugger Robot
* changes: Fixed crash when users press back twice in call composer. Pressing back when the dialpad is empty now exist the search UI.
2017-09-18Merge "Dialpad VM and + are now only accessible w/ double-tap instead of ↵Treehugger Robot
long hover."
2017-09-18Fixed crash when users press back twice in call composer.calderwoodra
Bug: 65686665 Test: manual PiperOrigin-RevId: 168983351 Change-Id: I6443a05c8435e777c52ed3586dfc228a0fb0f51f
2017-09-18Pressing back when the dialpad is empty now exist the search UI.calderwoodra
Bug: 65687672 Test: existing PiperOrigin-RevId: 168951800 Change-Id: Ie840acd608424551198ba3440d6060a7cd7a629f
2017-09-18Dialpad VM and + are now only accessible w/ double-tap instead of long hover.calderwoodra
Users using touch to explore with TalkBack on the dialpad would occasionally call voicemail if they hovered over 1 for a long period of time. This functionality doesn't align with keyboards, where a long hover still acts as a normal click. This CL updates the Dialpad to align better with a keyboard and now users can only speeddial voicemail and type a 0 with double-tap activation. Bug: 62712616 Test: manual PiperOrigin-RevId: 168909308 Change-Id: Ib46342f173aef7c2ef64c2b42745966f83266b98
2017-09-15Merge "Make voicemail transcription text selectable and phone numbers clickable"Treehugger Robot
2017-09-15Merge "Updating voicemail TOS UI to match mocks"Treehugger Robot
2017-09-15Make voicemail transcription text selectable and phone numbers clickablemdooley
Also replace deprecate android:singleLine=true with android:maxLines=1 video: https://drive.google.com/open?id=0B9o_KvtLkcuIWGVBOElWdFNKa1E Bug: 65678844 Test: manual PiperOrigin-RevId: 168858718 Change-Id: I8be70e8496a8f0c562ca09eb315bb4e28b3e9c00
2017-09-15Updating voicemail TOS UI to match mocksmdooley
verizon TOS screen shot 1: https://drive.google.com/open?id=0B9o_KvtLkcuIU3VpLUpqcXBUYy1QY19UUk81a29sdGQyNEZZ verizon TOS screen shot 2: https://drive.google.com/open?id=0B9o_KvtLkcuIN3R6SUFjS3RVTWhZQUM1UzhoajhNUF9ZbzJJ non-verizon TOS screen shot: https://drive.google.com/open?id=0B9o_KvtLkcuIeVFYaWhYZmRzYVctU2F6N1VqazcycGpmNmRZ Bug: 37340510 Test: manual and update unit test PiperOrigin-RevId: 168851805 Change-Id: I7be27360093cae315b7249152e10d1beae5c5966
2017-09-15Merge "Dialpad chooser now properly shows when in a call."Treehugger Robot
2017-09-15Dialpad chooser now properly shows when in a call.calderwoodra
Bug: 65212837 Test: DialpadFragmentTest PiperOrigin-RevId: 168745371 Change-Id: I62da45cdac58e3387343589b952e5d6bce7a4ee2
2017-09-14Merge "Don't use guava prebuilts outside of annotations processing."Treehugger Robot
2017-09-14Don't use guava prebuilts outside of annotations processing.Eric Erfanian
This results in class incompatiblity with other dependencies that are transitively dependent on the same library. Test: make, flash, th Bug: 65254753 Change-Id: Ib32b088df60582df7559c9be49b0fa1b1f1a4fef Merged-In: Idfbae76a38e64bb96eb824cda3d201ae63a13e90
2017-09-14Merge "Fixed an animation bug in new search fragment that was clipping results."Treehugger Robot
2017-09-14Merge "Setup SMS filter even if VVM is already activated"Treehugger Robot
2017-09-13Fixed an animation bug in new search fragment that was clipping results.calderwoodra
Test: manual PiperOrigin-RevId: 168612783 Change-Id: I811133aee3301bac6936d5ba101719c30322d3be
2017-09-13Setup SMS filter even if VVM is already activatedtwyen
VVM can be "activated" through restoring shared preferences from another device. In this case the SMS filter in telephony is still not configured, and needs to be set up. Bug: 65542413 Test: ActivationTaskTest PiperOrigin-RevId: 168560153 Change-Id: I46dd9b31e43899d8d567e7e6baebf06559548525
2017-09-13Merge "Log opening dialpad."Treehugger Robot
2017-09-13Merge changes Ie58920b7,Iaaf84ab9Treehugger Robot
* changes: Fixed crash in RemoteContactCusror. Log swiping and clicking for switching tabs.
2017-09-13Log opening dialpad.yueg
Bug: 64399026 Test: None. PiperOrigin-RevId: 168456040 Change-Id: I65fdae8dff1c9bb8e05e79f3ece2480b92f08c69
2017-09-13Fixed crash in RemoteContactCusror.calderwoodra
Test: manual PiperOrigin-RevId: 168427349 Change-Id: Ie58920b76266ebb298210b5faac8fcbda4dbcd15
2017-09-13Log swiping and clicking for switching tabs.yueg
If it's a swipe, onPageScrolled() is called several times before onPageScrollStateChanged(SCROLL_STATE_SETTLING) and onPageSelected(). If it's a click, only onPageScrollStateChanged(SCROLL_STATE_SETTLING) is called before onPageSelected(). And onPageScrollStateChanged(SCROLL_STATE_SETTLING) will not be called if user don't switch to a new tab. We use the difference to tell if user switching tabs by swiping or clicking. Test: DialtactsActivityTest PiperOrigin-RevId: 168403148 Change-Id: Iaaf84ab9c4955d0bc2c1e9857ba59fd37b3984af
2017-09-11Merge changes I8578dd2c,Ia8e581f1Treehugger Robot
* changes: Send PhoneAccountHandle with ACTION_CONFIGURE_VOICEMAIL Fix NPE in contacts fragment.
2017-09-11Merge changes Ia566fce6,I36f03c07,I81492f7fTreehugger Robot
* changes: This is a rollforward of cl/167332236 Updating voicemail notifications Updating voicemail notifications for transcription, part 2
2017-09-11Send PhoneAccountHandle with ACTION_CONFIGURE_VOICEMAILtwyen
When opening a voicemail setting from Dialer with ACTION_CONFIGURE_VOICEMAIL, no subId is passed. So on a multi-SIM device, it is always configuration on the default phone regardless of SIM. To solve this, set the PhoneAccountHandle to intent extra for ACTION_CONFIGURE_VOICEMAIL. In the advanced settings, obtains subId from the PhoneAccountHandle. Test: manual - Checked that correct PhoneAccountHandle will be passed with ACTION_CONFIGURE_VOICEMAIL intent Bug: 65001008 Author: xuegang.x.liu <xuegang.x.liu@sony.com> Bug: 65001008 Test: manual - Checked that correct PhoneAccountHandle will be passed PiperOrigin-RevId: 168019873 Change-Id: I8578dd2c4ac24e4af121a90b2b175baae2832461
2017-09-11Fix NPE in contacts fragment.calderwoodra
Bug: 64716624 Test: manual PiperOrigin-RevId: 168017541 Change-Id: Ia8e581f19a0a9c4d45bacc4ab94e54131763640a
2017-09-11This is a rollforward of cl/167332236calderwoodra
Improved behavior of back button in search ui. Pressing the back button in the search UI now functions as follows: - If the keyboard is opened, the keyboard is minimized - If the dialpad is opened, the dialpad is closed - If the keyboard and dialpad is closed, the search ui is closed Our existing behavior was dependent on whether a query had been built yet. basically, if the user pressed back with no query selected, the search ui was closed. From the bugbash: 7. No scroll bar in the search results if the results do not fill the entire view. The keyboard overlaps the results and there is no way to get to the enter list without dismissing the keyboard. 10. Dismiss the keyboard on tapping the down arrow on the keyboard. Arrow points down but works like back button Bug: 62685859,63691995,63939331,64137632,64902476 Test: manual PiperOrigin-RevId: 168004913 Change-Id: Ia566fce6d90454a3eae6ecccfa81ce7e909e878f
2017-09-11Updating voicemail notificationsmdooley
This is mostly just a revived version of cl/158901400. It adds a job service that is triggered by changes to the voicemail database. The job updates voicemail notifications, as necessary. video of notification update: https://drive.google.com/open?id=0B9o_KvtLkcuId1ptNk1EbGotWFU Bug: 37340510,27535759 Test: manual and unit test PiperOrigin-RevId: 167934550 Change-Id: I36f03c0769645f7a0cb478172171f1079eca2108
2017-09-11Updating voicemail notifications for transcription, part 2mdooley
Changing voicemail notifications to match the latest mocks, (replacing the 'Voicemail' title with what was the notification content, and putting the voicemail status or transcription in the notification content). The notifications will be updated in a following cl. screen shot: https://drive.google.com/open?id=0B9o_KvtLkcuIbDdjaTNHTFFZLW9YZEo1SG5scDR6SzloaWc4 Bug: 37340510 Test: manual PiperOrigin-RevId: 167928866 Change-Id: I81492f7fbf1cc398401a948e036854d6a8ff878b
2017-09-11Merge changes I6af1fc7d,I0627d54aTreehugger Robot
* changes: Automated g4 rollback of changelist 167310802. Automated g4 rollback of changelist 167332236.
2017-09-11Merge "Updating transcription UI to match latest mocks, part 1"Treehugger Robot
2017-09-11Merge changes Icc92997c,I70c032fd,Ibafdaf5f,Ifd2efcc9,Iba5609acTreehugger Robot
* changes: Fixing transcription crashes caused by job stoppage Strip trace info for release build. Load VVM config override before loading anything else Fix bubble crash when dismiss audio route selector dialog. Ignore read missed calls and voicemails for notification
2017-09-11Automated g4 rollback of changelist 167310802.calderwoodra
*** Reason for rollback *** Turns out blocked number activity is actually used for devices on M. *** Original change description *** Removes the unused dialer/app/filterednumber package. *** Test: n/a PiperOrigin-RevId: 167663280 Change-Id: I6af1fc7d6eb61f946d4fba255fe7374bcdee0e72
2017-09-11Automated g4 rollback of changelist 167332236.calderwoodra
*** Reason for rollback *** Based off a CL that needs to be rolled back. *** Original change description *** Improved behavior of back button in search ui. Pressing the back button in the search UI now functions as follows: - If the keyboard is opened, the keyboard is minimized - If the dialpad is opened, the dialpad is closed - If the keyboard and dialpad is closed, the search ui is closed Our existing behavior was dependent on whether a query had been built yet. basically, if the user pressed back with no query selected, the search ui was closed. From the bugbash: 7. No scroll bar in the search... *** Bug: 64902476,64137632,62685859,63691995,63939331 Test: n/a PiperOrigin-RevId: 167661409 Change-Id: I0627d54af33fb55c64b5edcd459dde6a73d93266
2017-09-11Updating transcription UI to match latest mocks, part 1mdooley
-add branding 'Google is transcribing' -add branding 'Transcribed by Google' in-progress screenshot: https://drive.google.com/open?id=0B9o_KvtLkcuIdVVTX3BBZkNzV180alZPQ3EtR3gzRFNoamhV transcribed by screenshot: https://drive.google.com/open?id=0B9o_KvtLkcuISUdaRjBlUFpjYTBWWTg1STVHS0oyQmNiSUVR Bug: 37340510 Test: manual PiperOrigin-RevId: 167620447 Change-Id: I5526bebb6999cf77b0902ae50ca72e380538a6e7
2017-09-11Fixing transcription crashes caused by job stoppagemdooley
Apparently, scheduling a new job when one is already running (even using the enqueue api) causes the running job to be stopped. We weren't handling that case correctly. This cl makes sure no more work is attempted after a job is stopped by cancelling any active transcription task. We request that stopped task be rescheduled by the job scheduler, so it will get run eventually. I was able to verify this fix by sending a new voicemail while backfill old transcription tasks were running. Bug: 64908823,63524274,65129734,63803709 Test: manual and unit tests PiperOrigin-RevId: 167617191 Change-Id: Icc92997c2687e61bef9b3b7f9ff572da2cb4ed2e
2017-09-11Strip trace info for release build.wangqi
Each trace info will add several milliseconds overhead and it's not used by end user. Bug: 64542087 Test: none. PiperOrigin-RevId: 167614347 Change-Id: I70c032fde162d6c8aa46be72c9ece3b7c3249fe8
2017-09-11Load VVM config override before loading anything elsetwyen
The override config is used by carriers to test out different configs using a prod dialer. Previously the type was loaded before the override config, causing carriers that is not currently supported never able to force enable VVM. This CL also fixes getCarrierPackageNames() and getDisabledCapabilities() not loading the override. Bug: 65370543 Test: revived OmtpVvmCarrierConfigHelperTest PiperOrigin-RevId: 167614182 Change-Id: Ibafdaf5fdc3e948bc65656f94c4bdc7d6e97046c
2017-09-11Fix bubble crash when dismiss audio route selector dialog.yueg
When AudioRouteSelectorDialogFragment tries to call onAudioRouteSelectorDismiss() on its parent AudioRouteSelectorActivity, the parent might already finish, which causes NPE. We should make sure the fragment is dismissed and removed before the activity finished. We do it when activity onPause because we don't expect it to resume. Test: AudioRouteSelectorDialogFragmentTest PiperOrigin-RevId: 167607068 Change-Id: Ifd2efcc92eb45262da2c6441bfac8119799d78f2
2017-09-11Ignore read missed calls and voicemails for notificationtwyen
When the call log is restored by the system, all items are marked as new even though they could already been read. This causes a torrent of "missed" calls after restoring call log with the setup wizard. This CL ignore read entities in the query. Bug: 62871863 Test: CallLogNotificationQueryHelperTest PiperOrigin-RevId: 167602820 Change-Id: Iba5609ace895a309685bfcd61eae85e080562ec5
2017-09-11Merge "Switch Dialer to use new third_party bubble library"Treehugger Robot
2017-09-11Switch Dialer to use new third_party bubble librarysail
This CL switches Dialer to use the new common bubble libary. It also moves the integration tests into the bubble libary. Bug: 64797730 Test: BubbleIntegrationTest PiperOrigin-RevId: 167439680 Change-Id: Ie2e9367cb6a6561efb8abd425b6a12f8c1e78138
2017-09-07Merge changes I2f94d863,Id7f5ce4b,I1c0b5b42,I4d8d8de8,I3765a9ae, ...Treehugger Robot
* changes: Added search actions to the end of the dialpad search results. Disable action on bubble when hiding, and reable it when showing. Improved behavior of back button in search ui. Fix crash when Context is null in InCallPresenter Bump version code and name to Dialer v12.1 and Dialer v13 (again). Adding dialer specific transcription error codes Removes the unused dialer/app/filterednumber package. Ignore invalid action for LegacyVociemailNotificationReceiver
2017-09-07Added search actions to the end of the dialpad search results.calderwoodra
Users can now create new contacts, add to existing contacts, send sms and make ViLTE calls from dialpad search results. screenshot: http://screen/7iR038nUvmh from the bugbash: 11. Missing “Create new contact” “Add to a contact” “Send SMS” from search results with a phone number search Bug: 64902476 Test: many PiperOrigin-RevId: 167362073 Change-Id: I2f94d863035c119ec526e02e088992c618a858a9
2017-09-07Disable action on bubble when hiding, and reable it when showing.yueg
Allowing click or moving bubble when hiding bubble makes no sense. Also, hide() and swapViewHolders() (called when expand/collapse) may add/remove the same view to windowManager and cause crash. Before: https://drive.google.com/a/google.com/file/d/0Bz1rQbdSCWSKRDEyT2JDdk9LVDA After: https://drive.google.com/a/google.com/file/d/0Bz1rQbdSCWSKUE53aldna0RFVFE Test: BubbleTest PiperOrigin-RevId: 167334496 Change-Id: Id7f5ce4bdfa074840c1e9fac981abad59e261c0e
2017-09-07Improved behavior of back button in search ui.calderwoodra
Pressing the back button in the search UI now functions as follows: - If the keyboard is opened, the keyboard is minimized - If the dialpad is opened, the dialpad is closed - If the keyboard and dialpad is closed, the search ui is closed Our existing behavior was dependent on whether a query had been built yet. basically, if the user pressed back with no query selected, the search ui was closed. From the bugbash: 7. No scroll bar in the search results if the results do not fill the entire view. The keyboard overlaps the results and there is no way to get to the enter list without dismissing the keyboard. 10. Dismiss the keyboard on tapping the down arrow on the keyboard. Arrow points down but works like back button Bug: 64902476,64137632,62685859,63691995,63939331 Test: manual PiperOrigin-RevId: 167332236 Change-Id: I1c0b5b429316dae119b5fb21be4303d7fe052e35
2017-09-07Fix crash when Context is null in InCallPresenterroldenburg
We need to use the Context to check if the device is a ODR device for a workaround (http://cl/165734624). It turns out the Context can be null (rarely). In those cases, we should not crash. Bug: 64954483 Test: manual PiperOrigin-RevId: 167326838 Change-Id: I4d8d8de86b5e3a4eca165a562f2ddc6106052c6e
2017-09-07Bump version code and name to Dialer v12.1 and Dialer v13 (again).Android Dialer
This CL updates version codes and name for Dialer v12.1 to 17*****. Doing so also bumps v13 to 18*****. Test: N/A PiperOrigin-RevId: 167326768 Change-Id: I3765a9ae8b06669a3eb96fef03a8a1432b3fc278
2017-09-07Adding dialer specific transcription error codesmdooley
These new states will be used to show descriptive failure messages to the user. Also moved the VoicemailCompat class to avoid duplicate code. Bug: 37340510 Test: none PiperOrigin-RevId: 167314611 Change-Id: I618d7f196ea2b0161cbd33479553cae3521328d7
2017-09-07Removes the unused dialer/app/filterednumber package.calderwoodra
Test: manual PiperOrigin-RevId: 167310802 Change-Id: Ie27a1b2f4daf73133edfb9de7f4e4aff1d87b6c8
2017-09-07Ignore invalid action for LegacyVociemailNotificationReceivertwyen
Somehow a null action will be sent to the receiver. Bug: 64934925 Test: LegacyVoicemailNotificationReceiveverTest PiperOrigin-RevId: 167300542 Change-Id: Ifa001d102d265e1dad42806b48f0a1710d05b195
2017-09-07Merge changes Ib013b10f,I07563fd2Treehugger Robot
* changes: Use simulator to add in-call UI integration tests Adjusted layout dimensions in new search UI.
2017-09-07Merge changes I0e8549cf,I12ac7b89,I2c77f0c6,I3ba5795bTreehugger Robot
* changes: Lazy loading animation of hd icon. Use TelecomManager#isInManagedCall starting from O. Setup SMS filter during activation if legacy mode is used Last contact in search is now visible when dialpad is closed.
2017-09-07Merge changes I4b5a494b,I4b6f7ca8Treehugger Robot
* changes: Added logging to aid debugging of issue where dialer shows blank screen. NewSearchFragment contact photos now properly open quick contact cards.
2017-09-07Merge "Added presubmit check to remove static imports from third_party java/"Treehugger Robot
2017-09-07Use simulator to add in-call UI integration testssail
This CL uses the simulator connection service to perform integration tests for incallui. The main pieces of this CL are: - DialerCallEvent - this is how we track changes to the incallui calls - Simulator.Event - this is how we track changes to a simulator connection With the above two we can do things like: - block until a DialerCall switches from ACTIVE TO ONHOLD: - DialerCallEspresso.waitForNextEvent(tracker, call, new DialerCallEvent(STATE_CHANGE, "ACTIVE", "ONHOLD") - block for a connection to recive a particular DTMF code: - SimulatorConnectionEspresso.waitForNextEvent(call, Event.DTMF) Future CLs will include things like: - fling to answer / reject - conference calls - screenshot diffing - video calling Test: InCallActivityTest PiperOrigin-RevId: 167211015 Change-Id: Ib013b10fe963092fad0816b07b1659efd69d9468
2017-09-06Adjusted layout dimensions in new search UI.calderwoodra
screenshots: default: http://screen/qEdB51A3PDn zero-query: http://screen/qEBDg0auGA8 query: http://screen/SuLp2GwyvYr Bug: 64806278 Test: manual PiperOrigin-RevId: 167205946 Change-Id: I07563fd23801da8d9d2eb95f6d09a079474403eb
2017-09-06Lazy loading animation of hd icon.wangqi
This save us ~180ms on Go device. (analyzed by systrace) This change also adds some trace information on incallui. Bug: 64542087 Test: manual PiperOrigin-RevId: 167201182 Change-Id: I0e8549cfb3534d518e3990696616855bcf7733f2
2017-09-06Use TelecomManager#isInManagedCall starting from O.wangqi
The TelecomManager#isInCall method returns true anytime the user is in a call. Starting in O, the APIs include support for self-managed ConnectionServices so that other apps like Duo can tell Telecom about its calls. So, if the user is in a Duo call, isInCall would return true. Dialer uses this to determine whether to show the "return to call in progress" when Dialer is launched. Instead, Dialer should use TelecomManager#isInManagedCall, which only returns true if the device is in a managed call which Dialer would know about. Bug: 36991070 Test: none PiperOrigin-RevId: 167200903 Change-Id: I12ac7b893dcbfa2fc842ca5ab356fbbc490a098b