Mark missed calls as read in calllog restore(2/2)

When missed calls are restored by CalllogBackupAgent using the addCall
API of CallLog provider, we were setting the IS_READ flag to 0 which
resulted in user getting a missed call notifications for all missed
calls after restore.
Adding a new addCall API signature which can specify the value to be put in the
IS_READ field when a call is added to the calllog provider. This will be
used by the CallLogBackupAgent to mark all restored missed calls as
read.

BUG: 22908952
Change-Id: Iee3bea09fbdefef09c7eb6c6c61728cb4d94d9e7

Merge "AudioService: check for audio device connection success" into mnc-dev

Merge "Ensure per package and per UID state share same package settings." into mnc-dev

Merge "Revert "Do not forceLayout when window is resized"" into mnc-dev

Merge "UsbDeviceManager: fix b/21429947 regression (try #2)" into mnc-dev

Merge "Fix crash in kModeProcessNoContext" into mnc-dev

Merge "Include dnd status in bugreports." into mnc-dev

Fix crash in kModeProcessNoContext

Bug: 22931143
Change-Id: I9897a28b1edd006aee67ae2343874ad92bbd15a0

Ensure per package and per UID state share same package settings.

When renaming a package during an OTA we were getting in a state
where the package setting mapped to the package UID was not the
same instance as the one we create for the new package mapped.
This leads to a drift between the permissions state for the package
and that state for the UID, resulting in broken for UID permission
checks as granted permissions were never appearing in the per UID
package setting.

bug:22928831

Change-Id: Ib0372632ec84a917304561fd94032cd09bb4c12f

UsbDeviceManager: fix b/21429947 regression (try #2)

As discussed in b/21429947 (commit
674019065bceb4150190bfb1aa63cda9de0a8560), MTP must always be
enabled, even if access to the underlying MTP data is disabled.
Otherwise, Android will not enumerate on the USB bus, and won't
receive notifications from the kernel about USB state changes. This
effectively prevents using MTP functionality on user builds, or
on userdebug/eng builds with adb turned off.

Always ensure that MTP is the default driver mode.

Move the DISALLOW_USB_FILE_TRANSFER filtering of mUsbDataUnlocked from
setting time to the time we post the sticky broadcast.

Remove isUsbDataUnlocked(). It essentially duplicates data in the sticky
broadcast.

Bug: 22447614
Bug: 21429947
Change-Id: I9d0d94cadbf6db6281ebd77bfb7162f9d06520c2

Merge "Remove spurious syncmanager wake-ups" into mnc-dev

Revert "Permissions: GET_ACCOUNTS permission cleanup"

Temporarily revert ag/735253 until b/22902898 can be resolved with a
proper DMAgent prebuilt drop.

This reverts commit e7ed827a104ba005b93faa2edb3bc77f72b240ec.

Bug: 22902898

Merge "Fail silently if a process trys to launch an acitvity for background user" into mnc-dev

Merge "QS: Allow Flashlight Tile to handle reset state" into mnc-dev

Reset the pointer to the Bluetooth GATT service

During error recovery, if the mBluetooth pointer is reset to null,
reset the mBluetoothGatt pointer as well.

Bug: 21756298
Change-Id: I26204ba47dd3c5465bb7de30cfa5dc0f07eee2fd

Merge "BatteryStats: Add debug logging to find source of b/22716723" into mnc-dev

Remove spurious syncmanager wake-ups

BUG: 22287469
1) Fix SyncManager waking up every 2 hrs if there is nothing
to do.
2) Fix sync wake-up alarm not being properly updated if the new
alarm was in the future.
2a) Due to staus bar sync signal that were removed post-K
one of the wake-ups was for 30s in the future, removed this
@hide intent completely.
3) The SyncManager will still set a timeout alarm for 5mins after
the start of a sync. Leaving this in as to post to a handler is
less expensive but more complex, and the alarm update is
correctly working now.

Change-Id: If51c9dd68391ccaeb480a17eb5a1364c4afe4c2a

Merge "Fix issue #22860466: viapi security bug - rubber stamping in nested VIs" into mnc-dev

Merge "Make default selection for contact sharing to "Checked"." into mnc-dev

Merge "Set broadcast nextReceiver correctly when package is disabled" into mnc-dev

Fix issue #22860466: viapi security bug - rubber stamping in nested VIs

Add new Activity.isVoiceInteractionRoot() API that an activity can use
to determine whether it is the root activity of a voice interaction
session started by the user's designated voice interaction service.

This is a special new API that apps must explicitly check, because as
with visual activities the model behind an activity should usually be
that it accomplishes its task by interacting with the user (implicitly
getting their approval) rather than trusting that whoever invoked it
is telling it to do what the user once.  In the voice world, however,
there are some cases where quick interactions want to allow for immediate
execution without further user involvement, so this API allows for that
without opening up security holes from other applications.

Change-Id: Ie02d2458f16cb0b12af825641bcf8beaf086931b

Set broadcast nextReceiver correctly when package is disabled

When we are cleaning up broadcat receivers due to a package been
disabled, it is possible to remove enough recievers to cause the
nextReceiver index to be greater than the size of recievers list.
We now set the nextReceiver to the size of the receiver list
(which means done processing) for this case.

Bug: 22874330
Change-Id: Ie151d1b5bff4c11533b3a8635fe5ee82eb21c13c

AudioService: check for audio device connection success

When connecting/disconnecting and audio device, AudioService
  was assuming that the call was always successful through
  AudioSystem.setDeviceConnectionState(). In the case of the
  connection of a wired headset (mic + headphones), this also
  causes A2DP to not be used for media playback.
The connection can fail if the audio device being connected
  is not supported by the platform. But if this failure is
  not taken into account for DEVICE_OUT_WIRED_HEADSET or
  HEADPHONES, A2DP would still be avoided even when an A2DP
  device reconnects at boot.
The fix consists in executing the connection logic only when
  the connection was deemed successful. Nothing is altered
  on the disconnection code path, or the direct connection
  of A2DP.
The javadoc is updated in AudioSystem to indicate the return
  codes to take into account in setDeviceConnectionState().

Bug 22511833

Change-Id: I22f0d2c7d4ab4fb9ee1be2f248907f721596a16f

Merge "Disable skipping frames on empty damage" into mnc-dev

Merge "Notify settings URI change without a lock held" into mnc-dev

Merge "Fix issue #22912704: "android.process.acore has stopped" dialog keeps popping up" into mnc-dev

Notify settings URI change without a lock held

bug:22469552

Change-Id: Ie4a42ceef07e3a8e593fe2b1374420239242ce7b

Fail silently if a process trys to launch an acitvity for background user

Doing this so we don't break current apps. In the future we should
properly position the activity in the stack (i.e. behind all current
user activity/task) and not change the positioning of stacks.

Bug: 21801163
Bug: 13507605
Bug: 22929608
Change-Id: I979b6288e66f5b2ec2a6f22cb8d416e5c68109bd

Merge "Yell loudly about undefined behind in Bitmap, but work anyway" into mnc-dev

Disable skipping frames on empty damage

Bug: 22592975

Optimization added in MNC is not quite correct, occasionally
it will fail to repaint when it was supposed to leading
to buffer corruption. Disable the optimization for now.

Change-Id: I34dfdfb357eda298198043ded7335d4588a003fd

BatteryStats: Add debug logging to find source of b/22716723

Bug:22716723
Change-Id: I97b940b702666b393555622d034e1c694b1e346a

Merge "Avoid deadlock by broadcasting outside lock." into mnc-dev

Fix issue #22912704: "android.process.acore has stopped" dialog keeps popping up

Slice 'em up!

Change-Id: Ibba3af7ec5b7f92e6d5e55a57aa838a7f7f936e1

Avoid deadlock by broadcasting outside lock.

We can end up in a rare tangled mess when sending broadcasts while
PackageManager is still trying to deal with scanAvailableAsecs().

Long-term solution is to make Environment.isExternalStorageEmulated()
more robust, but for now we just send these internal broadcasts
after we drop the MountService lock.

Bug: 22858403
Change-Id: I7b43865782f48a679882fd1675b7b961e1292df4

Merge "Restore session params "no decoration" flag" into mnc-dev

Merge changes Ib5f53f3b,I7debf3ee,Ic494ee7f,Ia8d223ae,Iff7dbbf6 into mnc-dev

* changes:
  Revert to using AppTheme for LayoutLib tests.
  Fix Matrix_Delegate.native_getValues()
  Update test image for upated nav icons.
  Fix typo: s/FullScreen/Fullscreen/ in LayoutLib.
  Fix bugs introduced in 78aa664.

Yell loudly about undefined behind in Bitmap, but work anyway

Bug: 22214367

Previous releases would let the getters on a recycle()'d bitmap to still
work despite being firmly in undefined behavior per the documentation
on Bitmap#recycle().

As there are apps relying on this, yell very loudly about this behavior
in the log and give them a bit of time to fix it

Change-Id: I857be7e74cb217877973d9c6f03eb761d12fd056

Merge "Expand ringtone titles to support OGG and FLAC." into mnc-dev

Expand ringtone titles to support OGG and FLAC.

These file types don't have "audio/" MIME type, so check for those
MIME types explicitly.

Bug: 22803550
Change-Id: Ia62f94ad4a19409d6a50462935de005bc572f93a

Merge "Also consider task intent when trimming recent tasks" into mnc-dev

Merge "Update Material link text to be colorAccent" into mnc-dev

Merge "Don't wipe device because of short PIN, patterns or passwords" into mnc-dev

Merge "Make ResolverDrawerLayout look like a ScrollView to accessibility" into mnc-dev

Merge "Add badge content description to TargetInfo, use in ResolverActivity" into mnc-dev

Merge "Bug: 22879202 App compatibility with WRITE_SETTINGS failing" into mnc-dev

Merge "Fix NPE in AppOpsService" into mnc-dev

Merge "User correct category for finding headless sync adapters" into mnc-dev

Don't wipe device because of short PIN, patterns or passwords

This fixes a bug where we were counting short PIN, patterns and
passwords as attempts. For devices with a device policy admin,
this would cause devices to get wiped after a short amount of
interaction with the UI.

Fixes bug 22844609

Change-Id: I7616b38d954f89d4a2cee23f9aec1b898041b1f2

Revert to using AppTheme for LayoutLib tests.

Use AppTheme for all tests other than testExpand. AppTheme contains an
attribute needed by testArrayCheck. Also, since the AppTheme extends
Theme.Material.Light.DarkActionBar, other tests should be unaffected.

Also, update the golden image for the testArrayCheck.

Change-Id: Ib5f53f3b029c5aceef2b622e5ac63952d6b1972a

Merge "media: Clarify MediaCodec and CodecCapabilities documentation" into mnc-dev

Fix Matrix_Delegate.native_getValues()

The bug manifested itself by not rendering the wifi icon in the status
bar.

Change-Id: I7debf3eef387965d774bf66e856812e67c8552a7

Update Material link text to be colorAccent

Bug: 22918108
Change-Id: I29b795b4c8d2a017dcaca1a9b65705eafaf2140c

Merge "Make vendor keys visible in Camera2 java APIs." into mnc-dev

Add more dump info about emergency calls state

Bug: 21549528
Change-Id: Idbd70e2f4b1b3285af283a28dc1933c02ad8d04c

Make ResolverDrawerLayout look like a ScrollView to accessibility

Disable accessibility focus on the layout itself and expose the class
name as ScrollView so that we can get auto-scroll working until we have
first-class support for specifying automatic scrolling behavior.

Bug: 22667764
Change-Id: I9b97e40f16038046898e5b56b935a61db9073ac6

Restore session params "no decoration" flag

This seems to have removed from a previous version and the theme editor
relies on it to remove the decoration around the preview.

Change-Id: Ifab99e9c1689bfe44ca593e8c66c24e57c3e8c4c

Fix NPE in AppOpsService

bug:22802981

Change-Id: I73eee5ff9cf1603d8f30355554ef075aa14afd30

Also consider task intent when trimming recent tasks

7cbfcd88657ac7d5ab1baf8c964b669c9bd21e24 we stopped comparing
the tasks realActivity when trimming recent task. This led to
task with the same intent been duplicated in the recents list.
We now consider the task intent when deciding when to trim like
we did pre 510e554283d3dc250f836cd3e4abc36d87319333.

Bug: 22812470
Bug: 22564474
Bug: 18642190
Change-Id: I90b3ab9cf7a06b4691099f697e723d8a54def9fa

User correct category for finding headless sync adapters

bug:22857409

Change-Id: Icc42f1af72c59fba3c60e1bcd558c621df70940a

Bug: 22879202 App compatibility with WRITE_SETTINGS failing

Fix a bug in the app ops protected permission check that broke backwards
compatibility.

Change-Id: Ic73759edd29a479a670fb31407e36aa50d09b6c3

Merge "Add per UID control to app ops." into mnc-dev

Add per UID control to app ops.

The app ops mananger service maintains a mapping from UID to
a list of packages where each package is mapped to a list of
non-default app op states (default states are inferred and
not stored). Hence, specifying the app op state for a UID
requires setting the app op for each package in the shared
UID.

This is problematic when installing new packages if there
is a non-default app op policy set for another already
installed package in the same UID as the app op for the new
package has to be updated to be in sync. The package installer
cannot do this as it is in another process and the app op
update will not be atomic. Therefore, the app ops manager
service has to support specifying app op policy on a per
UID basis.

We now have a UID state object that contains the per package
non-default app op states as well as the per uid non-default
app op states. If there is a UID policy specified then it
takes precedence over the per package one. Even further,
changing the uid policy updates the package policies in this
UID if the state is non-default. Changing a package app op
state also updates the app op state for the whole UID if
the per UID policy for this op is non-default. Clearing the
app op state for a package, clears the policy for the UID
as well.

bug:22802981

Change-Id: I78044906d9fcc6066abf07e706c2c88f3397d293

Merge "Added logging in a few error paths for wifi config installation." into mnc-dev

Added logging in a few error paths for wifi config installation.

Bug: 22790527
Change-Id: I71b298b98da098f647f52886c3c3a7456cb24d51

Make vendor keys visible in Camera2 java APIs.

- Add vendor keys to getKeys() calls for CameraCharacteristics,
  CaptureRequest, and CaptureResult.
- Vendors can specify whether custom keys show up by listing
  visible keys in the REQUEST_AVAILABLE_RESULT_KEYS field.
- Vendor key types are always treated as a primitive (or Rational)
  array type corresponding to one of the valid types for
  a camera metadata entry.

Bug: 22067625
Change-Id: I6e7dd3db7a8bf533c2ec15ff69ca38824134e971

Merge "Instrument printing service for metrics collection." into mnc-dev

Fixed a bug where notifications shadows were blinking out

Bug: 22567770
Change-Id: I2d335241a287d0c64e659221994f19b7c13eccd3

Merge "Make enter reveal animation be zorder top." into mnc-dev

Merge "Fix issue #21895842: Add is_assist_blocked to assist.ViewNode.NodeProperties" into mnc-dev

Merge "Fix that shutter sound does not play after ringer mode changes to vibrator" into mnc-dev

media: Clarify MediaCodec and CodecCapabilities documentation

Bug: 21932760
Bug: 22847191
Change-Id: Iacd24099a98c73fe4fb50226564690aa9b01c772

Merge "Revert two changes to LinearLayout measurement" into mnc-dev

Add badge content description to TargetInfo, use in ResolverActivity

Bug: 22667344
Change-Id: Iee241b9179c74a15b1c635cf90d2f96f6f7b31cd

Update test image for upated nav icons.

The image shows the nav icons to be separated by a significant distance.
This is because the switch to wide-screen layout happens based on
shortest width. The layout has a small shortest width and hence it uses
the phone layout for a horizontal nav bar and it doesn't look nice. Real
devices will never have this config, so it's okay.

Change-Id: Ic494ee7fdfb1116db39bcab6d6ee7a1e514b23df

Fix issue #21895842: Add is_assist_blocked to assist.ViewNode.NodeProperties

Change-Id: I928882d42d0546cc6a12e803d96131beaba76d4e

Fix typo: s/FullScreen/Fullscreen/ in LayoutLib.

Change-Id: Ia8d223ae01f08df945ab05615b07075b98a54a32

Fix bugs introduced in 78aa664.

Fix a couple of bugs (ArrayIndexOutOfBoundsException due to
miscalculation and a typo - swapped src and dst) introduced in the CL:
78aa664 Poor implementation of Path.approximate()

Change-Id: Iff7dbbf66d2714df51699ddb9a4169c38feefa7b

Revert "View measurement optimization"

LinearLayout relied on multiple measurement passes to hide a
discrepancy between EXACTLY and non-EXACTLY measurements.

This reverts commit 9cefbda11ee5308145d58b0b99ced0f66a0b1cf9.

Bug: 22810327
Change-Id: I515a80749420d00efc5002fa68221b7c236f03df

Moving removeTask to separate thread.

Bug: 22760556
Change-Id: Iac6eef77e3ba5432ac1cd4aa792e7b2aadc27c01

Make enter reveal animation be zorder top.

This fixes the bug where launcher icons become temporarily visible when
home button is pressed while other activity is starting. By having the
starting activity's animation be zoder top, it will continue to be drawn
on top of the launcher until it animates away.

Bug: 22809202
Change-Id: If5e3c09b7a5df4537c355f94e986766f77ad4943

Merge "Ensuring that we don't try to use and propagate null activity labels." into mnc-dev

Fix build.

Change-Id: I82295b758e5ceb42e260d3c36ea2310a9def5190

Ensuring that we don't try to use and propagate null activity labels.

- Also fixes issue with tapping outside bounds not working to dismiss recents

Bug: 21774486
Bug: 22241587
Change-Id: Ib50f6fece8fb150929a1f8cdb01b8e8fe7b665cd

Merge "Fixed issue with artifacts during scale-up transition animation" into mnc-dev

Merge "Fix build." into mnc-dev

Fix build.

Change-Id: I9c6b6cd354039ff2914837cfe97a783564ded66f

Merge "Improve getTimestamp documentation" into mnc-dev

Fix build.

Change-Id: Icdf6fa14e53a1c030d0d90205d3bf2d7610b6196

Merge "Change MNC codename to just M." into mnc-dev

Revert two changes to LinearLayout measurement

"Include non-zero dimension views in excess space calculation" and
"Always distribute excess space in LinearLayout measurement" changed
LinearLayout behavior significantly in a way that wasn't covered by
CTS tests.

This reverts commits da2f3044092318d44173bdb9918f31c592b86a73 and
4fabc021583cfd5cc551eda7d7b9809683a6ca9a.

Bug: 22862047
Change-Id: I8d37a525ccf295445d3239b80e5cacb10bf3c947

Fix that shutter sound does not play after ringer mode changes to vibrator

Reproduce sequence:

1. Change ring volume to vibration in Settings > Sound & notification
   > Ring volume using touch
2. Launch camera app
3. Capture, but cannot hear shutter sound
4. Even change ringtone to maximum, shutter sound is not played before reboot

Bug: 22589664
Change-Id: I90eb044eaf21f3b558bf050bf1d9f0b33ea4888b

Merge "Add telecom extra keys required for DMR release." into mnc-dev

QS: Allow Flashlight Tile to handle reset state

Bug: 22815987
Change-Id: I3c22760c546c5407495e1e8f09dc402a56817f9d

Merge "Fix SysUI crash on device boot." into mnc-dev

Include dnd status in bugreports.

Bug: 22868844
Change-Id: I7b64cd8e27d6ebf254150b68ac791f299a72d4b1

Fix SysUI crash on device boot.

Bug: 22869066
Change-Id: I55ab64c430899ebc20bfc8a8e4d1e2a7cd12f0cd

Merge "Don't always transfer device owner status to other users." into mnc-dev

Don't always transfer device owner status to other users.

A device owner cannot use device or profile owner policies on
other users unless it is profile owner there. Also limit device
initializer to system apps only.

Bug: 21800830
Change-Id: Ie1abbd891945b91b17ecdf7f73ba93aaa19819be

Merge "Add missing return statement" into mnc-dev

Merge "Stop supporting legacy ConnectivityManager routing methods in M." into mnc-dev