Merge \"Reduce shell power over user management.\" into nyc-dev
am: 1b4124fc95

Change-Id: Ia54f3ad9a8594e101407db6b52b378ae8bec72a2

Merge "Reduce shell power over user management." into nyc-dev

Merge \"Provide default fallback in theme\" into nyc-dev
am: 7c8360980c

Change-Id: I70328da5ce9ae9988cb58f121caf82c44a16fcec

Merge "Provide default fallback in theme" into nyc-dev

Merge "Merge Theme.Micro and Theme.Material." into nyc-mr1-dev

Merge \"Recents TV: Add a check for touch enabled mode for starting dismiss animation\" into nyc-dev
am: 7476aaa058

Change-Id: I832256859deb5ce9a15d8742cb3a9e21b0e98db9

Merge "Recents TV: Add a check for touch enabled mode for starting dismiss animation" into nyc-dev

Merge \"Fixed check for volume name.\" into nyc-dev
am: cbfc6eed32

Change-Id: Ieba9951e1899abca040e1ca456f911dc76412037

Merge "Fixed check for volume name." into nyc-dev

Reduce shell power over user management.

Remove MANAGE_USERS permission from shell and whitelist it for
some specific functionality.

Bug: 29189712
Change-Id: Ifb37448c091af91991964511e3efb1bb4dea1ff3

Merge \"Blame only location observers that actually receive updates\" into nyc-dev
am: 4e85376fe1

Change-Id: I058f0c336974a9b485c674189c97d36f13ec3df1

Merge changes I069e2b7e,Ic5e9e58a into nyc-dev
am: f4de8dd77c

Change-Id: I439643be2fd5699f1a98d8ca15d14167ea3ffde1

Merge \"Check for valid paths when getVolumeList() returns invisible volumes.\" into nyc-dev
am: 6da2fb61ce

Change-Id: I0f6c6e36c50b56036b43c32680c8abde2db61c7a

Merge "Blame only location observers that actually receive updates" into nyc-dev

Provide default fallback in theme

This reverts a part of Ifb724b5c5feea3ed70d68e5bd7615fa91185e743:
Since we enforce the background to be non-translucent, this part
can be reverted and only causes unnecessary regressions.

Change-Id: Ide7b60ee05eb275344fee2f4370f0e8461ef2783
Fixes: 29192742

Merge Theme.Micro and Theme.Material.

Bug: 29248638
Change-Id: I1f6a419de89f89e4a105e244c6495e2df29c120c

Merge changes I069e2b7e,Ic5e9e58a into nyc-dev

* changes:
  Camera2: Fix error codes for CameraDevice.StateCallback.onError()
  Camera2: Protect MarshalRegistry against concurrent access

Merge "Clean up PCO plumbing." into nyc-mr1-dev

Merge "Check for valid paths when getVolumeList() returns invisible volumes." into nyc-dev

Check for valid paths when getVolumeList() returns invisible volumes.

The FLAG_INCLUDE_INVISIBLE was created to let SM.getStorageVolumes()
return mounted volumes (like USB drives), but in the current form it
might return invalid volumes. For example, when a SD Card is adopted as
an internal storage, it will return 2 volumes instead of one, the "bad"
one being:

VolumeInfo{emulated}:
     type=EMULATED diskId=null partGuid=null mountFlags=0 mountUserId=-1
     state=UNMOUNTED
     fsType=null fsUuid=null fsLabel=null
     path=null internalPath=null

As such, we need to filter out those volumes that don't have a path.

BUG: 29250241
Change-Id: I3f84b53eac98f31f918d843c70f9d42983e2a438

Fixed check for volume name.

There are scenarios - like when an SD Card is adopted as internal
storage - where the getBestVolumeDescription() returns null, in which
case we should use the StorageVolume description instead (or a default
value if such description is also null)>

Change-Id: Ia11c3f4069dfbe252c568210ee60b5d8c384d863
Fixes: 29198046

Merge \"Don\'t call surfaceCreated for just changed surfaces\" into nyc-dev
am: e4b1fb94e3

Change-Id: I488da6f9f779ddf64beb6ac12d1481d2bfd69821

Recents TV: Add a check for touch enabled mode for starting dismiss animation

Fix until we figure out if Gamepad not eating key events during talkback is
inteneded behavior.

BUG: 29208918
Change-Id: Iffe652f10e88d0218875264f31e9d7658d75e71e

Merge \"Fix issue #29252997: Should be able to dump only broadcast stats as checkin format\" into nyc-dev
am: 9303ecb317

Change-Id: Ib7479599ba70cf05dec9b323d62149670612abaa

Merge "Don't call surfaceCreated for just changed surfaces" into nyc-dev

Merge "Fix issue #29252997: Should be able to dump only broadcast stats as checkin format" into nyc-dev

Camera2: Fix error codes for CameraDevice.StateCallback.onError()

Existing code assumed that ICameraDeviceCallbacks and
CameraDevice.StateCallback have the same error code values for matching
errors.

They do not.

Also remove duplicate error code definitions now present in the AIDL
file for ICameraDeviceCallbacks.

Bug: 29248704
Change-Id: I069e2b7ef3be7887634e128f1accb50b7558f3fd

Camera2: Protect MarshalRegistry against concurrent access

Multiple CameraMetadataNative objects could be reading and writing
to the metadata marshaler registry simultaneously.

This can lead to an infinite loop in the HashMap in the worst case,
so add synchronization against this.

Bug: 29043079
Change-Id: Ic5e9e58a9333b99b4bea87bf790c9fbfadfbbea9

Merge \"Collect statistics of source build version\" into nyc-dev
am: 8872a604b9

Change-Id: Ic920aced4f17c31a09bea55480f92252f20d728d

Merge \"Use user id 0 to get missing appId for debugger\" into nyc-dev
am: 792918e5fd

Change-Id: I8ccb08fe3aa9c00b715078086b233a55a338eb94

Merge "Collect statistics of source build version" into nyc-dev

Merge "Use user id 0 to get missing appId for debugger" into nyc-dev

Merge \"Fix framebuffer incomplete errors\" into nyc-dev
am: 4bcf66030d

Change-Id: If578de7d7ea10cceb0e19eb68e305432e4177d04

Merge "Fix framebuffer incomplete errors" into nyc-dev

Merge "fix prefix generation" into nyc-mr1-dev

Blame only location observers that actually receive updates

The location manager service keeps update records for all apps
registered to receive location updates. If an app does not have
the location permission then its update record is skipped when
dispatching location updates. However, the battery blame is spread
across all apps requesting location updates even if they don't
receive such because of lacking permission.

bug:28804546

Change-Id: Iee96fd6f304c7a49f9c70484e52abc0ed1538a29

Merge \"BatteryStats: Record fine grained battery discharge\" into nyc-dev
am: 094c40b2cc

Change-Id: Ie85fcfc2fa7aec5e7c15f57d605840c563c0e993

Merge "BatteryStats: Record fine grained battery discharge" into nyc-dev

Merge \"BatteryStats: Bump checkin format version to 18\" into nyc-dev
am: d75bc225ac

Change-Id: I70dbac4cb8002d8f216991ca78b18e9c1612eb89

Merge "BatteryStats: Bump checkin format version to 18" into nyc-dev

BatteryStats: Bump checkin format version to 18

Bug:28811636
Change-Id: If2229c438e60f20746c91bccbcaa6d72ca5f0802

BatteryStats: Record fine grained battery discharge

For devices that report battery discharge via a coulomb counter,
record how much of the battery was discharged while the screen
was on/off.

Bug:28743761
Change-Id: Ie2d1708864352029ff466c1fed14fc057e19b93b

Merge \"Stop magnifying surface insets.\" into nyc-dev
am: 4663b4e806

Change-Id: Id98ea0c13c325d79e5c2ae88184351e75ad7db46

Merge "Stop magnifying surface insets." into nyc-dev

Fix framebuffer incomplete errors

bug:29127615

Primarily fixes case where 0 dimensioned layers could be
created/updated. Additionally, adds more logging in incomplete
framebuffer cases, if they still occur.

Change-Id: Ib90dbbafd6905aca3c8f46e64064e13a308f713d

Fix issue #29252997: Should be able to dump only broadcast stats as checkin format

Change-Id: I36d4e5e20ff2ed1b4d02a04390e0415b9b2394a3

Merge "Fix race conditions between Tethering and TetherInterfaceStateMachine" into nyc-mr1-dev

Merge \"Don\'t allow restore sessions during backups\" into nyc-dev
am: 9faaf5418a

Change-Id: I68e844ee0710bc4125da7789c2059ab23262ba76

Merge \"Improve accessibility window title behavior\" into nyc-dev
am: f8af34365c

Change-Id: I620cb3dc270a8ad409ec1760cedd778821fd8d98

Use user id 0 to get missing appId for debugger

When forceStopPackageLocked is used to kill existing app process for
relaunching with debugger, user id could be -1 and getPackageUid will
already return -1. This results in killPackageProcessesLocked() not
able to find and kill the app, and wait for debugger will.

This also revert commit 55c9d739 (ag/969408) which tried to fix the
same problem but got it wrong. It's valid to -D when the process is
already running, as we're supposed to kill it. The really problem is
that the killing fails because of bad appId.

bug: 28980761
Change-Id: Ibebdcc26865464216a1f4b6773e540921cdad7d2

Merge "Ephemeral changes for debug builds" into nyc-mr1-dev

Merge "Don't allow restore sessions during backups" into nyc-dev

Merge changes from topic 'VectorDrawable polishing cherrypicks from master' into nyc-mr1-dev

* changes:
  Support Keyframe definition for AVD on RT
  Throw Exception for wrong valueType with API guard

Merge "Improve accessibility window title behavior" into nyc-dev

Don't allow restore sessions during backups

Gracefully no-op if apps attempt to restore themselves while there is
a backup pass in flight.

Bug 29135379

Change-Id: I8f0b5cd9d149b703e1de7a3a0b4b54c3aff766b6

Don't call surfaceCreated for just changed surfaces

Fixes: 29223302

If the surface only changed we want to just call onSurfaceChanged,
not onSurfaceCreated.

Change-Id: I65c8b210a7e34eaa2862020335ee7f6ef56eeac2

Collect statistics of source build version

Parse the last_install and report the source build version of an ota
update.
Related CL in: ag/1121141

Bug: 28658632
Change-Id: I5e835c144aabe97fda681f60397ebf4416f7bd4f

Merge \"Protect the android.intent.action.CALL_PRIVILEGED broadcast\" into nyc-dev
am: c94872e940

Change-Id: I722d5bfbed781c28db5da59f303c2579123d8e6e

Merge "Protect the android.intent.action.CALL_PRIVILEGED broadcast" into nyc-dev

Merge \"Fix more of issue #28689719: Runtime restart\" into nyc-dev
am: 4781525399

Change-Id: I03d781408ef7b0c7c6156d6b63ca0ab4ee457ae1

fix prefix generation

when performing bitwise operations, java promotes byte's to int's.
this is normally okay except when the value is negative. in that
case, java extends the sign bits. this means that after shifting,
all of the upper bits are most likely going to be all f's. instead,
undo the sign extension by pulling out just the 8 bits we're
interested in.

Change-Id: I89f7244f6b675be2cca4a185a7002b5f8a6eece1

Ephemeral changes for debug builds

The installer/resolver must typically be specified in a baked in XML
file. But, to aid development, allow the installer/resolver to be
added dynamically on debug builds.

Change-Id: I8617bb2a33a51fc596579bdaadb27a11c3740c14

Merge "Fix more of issue #28689719: Runtime restart" into nyc-dev

Merge \"Turn down factory-default lock wallpaper support\" into nyc-dev
am: 52e323bd38

Change-Id: I71646e6fb8a6996c17a9d2760aca88d6927d33ec

Merge "Turn down factory-default lock wallpaper support" into nyc-dev

Support Keyframe definition for AVD on RT

BUG: 27441613
Change-Id: Iece386f65f3704d1b7caa2b3690a8d3048ccf6e2

Throw Exception for wrong valueType with API guard

Previously, wrong valueType error is swallowed in jni. As a result,
some animations are quietly skipped without letting developers know.
This CL maintains that behavior for pre-N, and throws Exception
to notify developers of the error for N and above.

Bug: 29009391
Change-Id: I3e8f003cdb97d214da72af3f93a84f64797b1c2c
(cherry picked from commit 94db09917a17976135e2c63d8e4171c54730c079)

Merge "Add UCE Service Down protected broadcast" into nyc-mr1-dev

Merge "Handle hidden RT VectorDrawable animators" into nyc-mr1-dev

Merge "Improve the property modification effiency of VectorDrawable animators" into nyc-mr1-dev

Merge \"ResourcesManager: Apply ContextThemeWrapper config\" into nyc-dev
am: 9e9c78f8ce

Change-Id: I71250c58f237135d156c64bdc08caafbe5b4a59e

Merge "ResourcesManager: Apply ContextThemeWrapper config" into nyc-dev

Merge \"Make Activity.getReferrer tolerate custom Parcelables in intents\" into nyc-dev
am: 7a37f8945f

Change-Id: I8e14173836f153d75b9c9f29f3493335af5f2763

Merge "Make Activity.getReferrer tolerate custom Parcelables in intents" into nyc-dev

Turn down factory-default lock wallpaper support

The infrastructure is here but not yet fully plumbed, so gracefully
provide the historical behavior until the rest is in place.

Bug 29220492

Change-Id: I36feaf68099d85e4ae11a5262002ee85346703ed

Improve accessibility window title behavior

Un-reverts ag/1057448, but uses a version of Resources#getText
that returns null rather than throw an exception when no title
is available. This is the same call made before, so this change
should be safer.

Bug: 28744278
Change-Id: I8c123f4f6d74f796ab9e8ffcf955aaf881770da1

Merge \"Cache resources needed to sort media sizes.\" into nyc-dev
am: 142fc425c4

Change-Id: I368ffa5269029801ed191351fb1fff9bde72778a

Merge "Cache resources needed to sort media sizes." into nyc-dev

Merge \"Don\'t update mTaskToReturnTo when task is cleared\" into nyc-dev
am: 1a77efe9ba

Change-Id: Idb9aad2fb3161552b699f271bed940cb47fd98ad

Merge "Don't update mTaskToReturnTo when task is cleared" into nyc-dev

Unlock mBluetoothLock.readLock() instead of double-locking it
am: 7ee53be300

Change-Id: I521890b2554b47f278f1e3d169b3a3cde825240f

Add UCE Service Down protected broadcast

bug:29173516
Change-Id: I1c5743962dbca9c482ce7d87f6fe570a3249f3fe

Fix more of issue #28689719: Runtime restart

Change-Id: I0cce1cef884cf1508f1e726ea3439a5ffee84993

Unlock mBluetoothLock.readLock() instead of double-locking it

Bug: 29011117
Change-Id: Ie9e359c3ee079c2cecc33f11cfff7c0dc6406be1

ResourcesManager: Apply ContextThemeWrapper config

The ContextThemeWrapper's configuration override must also
be applied to ResourcesManager during a configuration change.

Bug:27644297
Change-Id: Ic126308b753979c18801d422a8243aa4daf45147

Make Activity.getReferrer tolerate custom Parcelables in intents

Referrer information is passed as an intent extra to each Activity,
therefore if that intent contains a custom parcelable unmarshaling
the intent extras will fail in a call to getReferrer. Insulate apps
from this concern and log a warning when it happens.

Bug 28972438

Change-Id: I37dbc033bfcfe816e4b58b382c012854fbed943b

Merge \"PiP animation: Move window with resize when ending animation.\" into nyc-dev
am: 4624d687f7

Change-Id: Ib9ce0ef71a71e998e1ffe783c178b8f514ab1469

Merge \"Handling of Fuji Xerox in RecommendationService.\" into nyc-dev
am: 5398f2731b

Change-Id: I79888e8b145c93eae9bea9c7e6f2c59e3912c29f

Merge "PiP animation: Move window with resize when ending animation." into nyc-dev

Merge "Handling of Fuji Xerox in RecommendationService." into nyc-dev

Don't update mTaskToReturnTo when task is cleared

If activity launch has moved other task that was over home while
clearing task - don't override its mTaskToReturnTo not to loose
the original value of other task.

Bug: 28986980
Change-Id: Ib1680769fbb8a8d99f2b210c8d9cd8fe6fd5a196

Fix race conditions between Tethering and TetherInterfaceStateMachine

( cherry-pick of de4819dc82b734036b22416228327d2712e01884 )

Previously, Tethering would answer requests for current tethering state
by calling methods on instances of TetherInterfaceStateMachine to build
up that state. This is incorrect, since state queries can come in on an
arbitrary thread, independent of updates to the state machines
instances.

Fix this by:
  - Altering TetherInterfaceStateMachine to push consistent state
    snapshots to Tethering via notifyInterfaceStateChange()
  - Storing the last state snapshot in Tethering.mTetherStates
  - Removing public methods to query TetherInterfaceStateMachine state.
  - Consistently synchronizing access to Tethering.mTetherStates

Bug: 29009601
Test: WiFi Tethering continues to work, unittests continue to pass

Change-Id: Ied334f5e8739bc3aff1b08a1079095b9cc2a7958

Merge "Seascape Navigation Bar View" into nyc-mr1-dev

Merge \\\"docs: Added links to performance resources section\\\" into mnc-io-docs am: e4b1149320 am: 2dea7d3836
am: b1a99c46b6

Change-Id: If38d960ef06b24ea1f23c82b521a8162ca625698

Merge \\"docs: Added links to performance resources section\\" into mnc-io-docs am: e4b1149320
am: 2dea7d3836

Change-Id: I73d48da51be02656ae636686b42253c35ebc3762

Merge \"docs: Added links to performance resources section\" into mnc-io-docs
am: e4b1149320

Change-Id: Ib1fa097035dce4d6e4c55ac6d87aa26bacceade8

Merge "docs: Added links to performance resources section" into mnc-io-docs

PiP animation: Move window with resize when ending animation.

At the end of the animation (when going from larger to smaller),
we are left with a scaled surface, that we want to seamlessly
resize to an unscaled surface of the new size. Because we have scaled
the shadow region of the surface, the position of the content
will differ before and after the resize applies. We use new
SurfaceFlinger API to cause position updates to apply after
resize. Because we have to switch into SCALING_MODE_FREEZE,
we could end up prematurely cropping the window, so we
switch to using screen space crop for the pinned stack.

Bug: 28899837
Change-Id: I9b762a237413e4fa3d432e67d30c7125bfef484c

Handle hidden RT VectorDrawable animators

This CL changes the target of VD specific animators to VectorDrawable,
instead of RenderNode. The benefit of doing so is that animators can
now detect whether the animation is meaningful by checking whether
their VD target is in the display list. If not, that means the VD is
not drawing for the current frame, in which case we can be smarter
and more power efficient by removing the animator from the list and
posting a delayed onFinished listener callback.

By setting VD as the animation target, when an ImageView decides to
update its drawable from one AVD to something else, we'll be able
to detect that the previous AVD is no longer in the display list,
and stop providing animation pulse to the stale AVD, which is
something we couldn't do previously.  This change also
handles the case where one AVD instance could be drawn in two
different views.

Bug: 27441375
Change-Id: Iaad1ed09cfd526276b95db0dd695275c28e074e8

Merge \\\"Correct the size of example bitmap for xxhdpi.\\\" into mnc-io-docs am: 396ed8d2cf am: 6e70538600
am: e82ac10fa2

Change-Id: Ie1c0f2c4ff45420a4cb54f57ada28ee85db779fa