Merge "Properly map runtime permissions to app ops" into nyc-dev
am: ba83e90e18

* commit 'ba83e90e1825f3e40a16370babc83b8ac0645cf9':
  Properly map runtime permissions to app ops

Change-Id: Ie4830b140c2589abecbe8e6f575ab939d9c988ce

Merge "Properly map runtime permissions to app ops" into nyc-dev

Merge "Fix onStop not called when activity is under translucent" into nyc-dev
am: a5aa3ecbe3

* commit 'a5aa3ecbe3426b44b6d550624e7b30f4b84c1025':
  Fix onStop not called when activity is under translucent

Change-Id: I863e28ef8884e3c6e629c1b2c30c554fb0538988

Merge "Fix onStop not called when activity is under translucent" into nyc-dev

Merge "Don\'t persist selection after restore." into nyc-dev
am: 522a437786

* commit '522a4377863339d9486c833f5a39f8047576bfee':
  Don't persist selection after restore.

Change-Id: Idee08d4397212a2a2a58b650ebcaa3b162b73aab

Merge "Don't persist selection after restore." into nyc-dev

Merge "Refactored NetworkPolicyManagerService mUidRules." into nyc-dev
am: de66540fd3

* commit 'de66540fd3f73937c3cfbe1d50e575aa69acc474':
  Refactored NetworkPolicyManagerService mUidRules.

Change-Id: I89af2823e8d791a487917daafe16be6764c231b2

Merge "Move the \'pretend idle jobs can run now\' broadcast into AMS" into nyc-dev
am: 0588954071

* commit '058895407140de0e513b82150c3acb6131e971a2':
  Move the 'pretend idle jobs can run now' broadcast into AMS

Change-Id: Ia97d382761fc8b9f2c11729cd027a2d8c6c9379b

Merge "Refactored NetworkPolicyManagerService mUidRules." into nyc-dev

Merge "Move the 'pretend idle jobs can run now' broadcast into AMS" into nyc-dev

Merge "Reset Cards as they come into view" into nyc-dev
am: 4fd98d1f09

* commit '4fd98d1f0996783ae5d9520315c02983c68c41f9':
  Reset Cards as they come into view

Change-Id: Ib659edbd4c5d7875fb1210cae895ea94c842ee63

Merge "Reset Cards as they come into view" into nyc-dev

Merge "Fix divergent equals and hashCode behavior" into nyc-dev
am: 2ff5780d90

* commit '2ff5780d903bb052fd9349b742d4d754f049202c':
  Fix divergent equals and hashCode behavior

Change-Id: I16809877a43fa3b3c70273de2a84a90e493d9caf

Fix exception when docking task.
am: 27c28f8da2

* commit '27c28f8da2925fd6fdaa91603c1740447b8fc195':
  Fix exception when docking task.

Change-Id: Iec927a1b079a7d6935b8430dca70eec760a84739

Merge "Disabling movement of entire affiliated task set." into nyc-dev
am: 7e2d589ce0

* commit '7e2d589ce053ded6d07cc3f93297d6c32128dd3d':
  Disabling movement of entire affiliated task set.

Change-Id: Ia307ac34662cf0092f3ba00e20451b312d0cd7a8

Merge "Fix divergent equals and hashCode behavior" into nyc-dev

Fix exception when docking task.

- If a task fails to dock, animate the stack back to original state so
  that the layout is not stuck in a "docked" state.

Bug: 28577229
Change-Id: If927b898a48cd5949764cb3b0c0798d22efd850a

Merge "Disabling movement of entire affiliated task set." into nyc-dev

Fix onStop not called when activity is under translucent

Activity was not stopped when device went to sleep if it was covered by
another translucent activity. The issue appeared after ag/864253 which
disabled stopping for visible activities.

Bug: 28191555
Change-Id: I39f0c5579ee3e5a16db888c72ace5dc43aa307bd

Merge "Fix: Cannot control ime_switcher and menu after resetting." into nyc-dev
am: a423e2535b

* commit 'a423e2535b2d79849465db1a24aefa6779cc695b':
  Fix: Cannot control ime_switcher and menu after resetting.

Change-Id: I5d50dd2cb84687540a556922ad4000815c8e3073

Merge "Fix: Cannot control ime_switcher and menu after resetting." into nyc-dev

Fix divergent equals and hashCode behavior

Calling Objects.hash with a byte[] will call the identity hashCode on
the byte[] and this doesn't agree with the use of Objects.deepEquals
in equals.
Bug caught by error prone.

Also, replaced usage of Objects.deepEquals(mServiceDataUuid, ...) with
Objects.equals(mServiceDataUuid, ...), because mServiceDataUuid
is an Object of type ParcelUuid.

Bug: 28585195
Change-Id: Id92734874339985fedafe1a28286a6a4dcd88d3b

Don't persist selection after restore.

Bug: 28469988
Change-Id: I1fec774d8ebdec13b33262d7fe8e17fccd84b657

Reset Cards as they come into view

This will ensure that they are in the correct configuration. Needed since
recycler view will be re-using views.

BUG:28632740
Change-Id: Icdaa48696e198cfeb8c37718fec05f7f9b195842

Disabling movement of entire affiliated task set.

- This CL removes the dependency on the last-active-time of the task
  record, since that is dependent on the current system time (which can
  be changed by the user).  Instead of working around the previous
  behavior in ActivityManager which moves all affiliated tasks to the
  top when one task is launched, we change the behavior in the AM
  directly, and prevent re-sorting the list of recent tasks improperly
  (aside from the stable sort which puts the freeform tasks first).

Bug: 27398177

Change-Id: I9fa9b3497d08082fe00aa724538255de87e746d6

Merge "QS Header: remove collapsed alarm ripple" into nyc-dev
am: 5df0ee53ac

* commit '5df0ee53acc90e174f59be6c3c0fe52fd778a66e':
  QS Header: remove collapsed alarm ripple

Change-Id: Ic26aef478324ff0a579b73ba901ed8798d814f7d

Merge "Dismiss docked stack if an activity is shown on top of the lock screen" into nyc-dev
am: 5d6a218d9e

* commit '5d6a218d9e0d21367f6365d4de51532ca7e14dc9':
  Dismiss docked stack if an activity is shown on top of the lock screen

Change-Id: Ia5f98128501fcf648da94d6393f799d83ca89656

Merge "QS Header: remove collapsed alarm ripple" into nyc-dev

Merge "Dismiss docked stack if an activity is shown on top of the lock screen" into nyc-dev

Merge "Make AccessibilityService#disableSelf atomic." into nyc-dev
am: b42afe9be0

* commit 'b42afe9be02b23c50c0479e93df2a853be7c369a':
  Make AccessibilityService#disableSelf atomic.

Change-Id: I90fc019bb38c2b8c5b71357bce60c8df298bddcc

Move the 'pretend idle jobs can run now' broadcast into AMS

It's a protected broadcast, so sending it directly from 'am' is
no longer an option.  This is needed for CTS as well as being
generally useful during app development.

Bug 28406044

Change-Id: I101915a8c6f19454330a8db2079a75023c112582

Merge "Fix NPE in KeyphraseEnrollmentInfo.toString()" into nyc-dev
am: 0f6a7abb83

* commit '0f6a7abb83687e8f800e7754cf585edca725d83e':
  Fix NPE in KeyphraseEnrollmentInfo.toString()

Change-Id: Ieeb8c8eeac89a68ffcbcaf31b7f6d2b8c60dea7d

Merge "Make AccessibilityService#disableSelf atomic." into nyc-dev

Merge "Fix NPE in KeyphraseEnrollmentInfo.toString()" into nyc-dev

Refactored NetworkPolicyManagerService mUidRules.

NetworkPolicyManagerService (NMPS) keeps an internal list of uid
rules (mUidRules) for network restrictions, and when these rules
changes it needs to notify external listeners (such as
ConnectivityService / CS).

Prior to Android N, both Data Saver mode (the feature previously known
as "Restrict Baground Data") and Battery Save mode used the same set of
firewall rules to implement their restrictions: when Battery Saver mode
NPMS would mark all networks as metered and set the proper firewall
rules externally.

Recently, these 2 modes were split in 2 distinct firewall rules and
NMPS.updateRuleForRestrictBackgroundLocked() was changed to update
the mUidRules logic based on the Data Saver firewall (since the Battery
Saver firewall changes are handled externally, on
updateRuleForRestrictPowerLocked()). As such, CS was not notified when
the power-related changes were made, which would cause apps to get a
state of CONNECTED / CONNECTED when querying its active connection.

This change refactores the mUidRules to use bitmasks, in preparation for
another change that will fix the issue.

It also fixes a minor bug that was preventing removed packages to be
removed from the whitelist.

BUG: 28521946
Change-Id: I9f0e1509a6192cad403f740c1cd76a6b7dab7d26

Fix: Cannot control ime_switcher and menu after resetting.

mButtonDispatchers contains ButtonDispatcher for ime_switcher and menu;
however, in #inflateButton, we tried to find ButtonDispatcher for
menu_ime, which is a FrameLayout containing ime_switcher and menu.
Therefore, after #clearViews and #inflateLayout, ime_switcher and menu
KeyButtonView are not in the corresponding ButtonDispatchers.
It means NavigationBarView#getImeSwitchButton and
NavigationBarView#getMenuButton returns empty ButtonDispatcher.
As a result, we can't control ime switch button and menu button via
NavigationBarView after then.  e.g. We can't control ime switch button
visibility.

This CL fixes that issue by calling #addToDispatchers recursively for
ViewGroup children if corresponding ButtonDispatcher is not found.  This
behavior is aligned with #addAll.

Bug: 28580774

Change-Id: Ibe724753390b7bbb395a6d53d00bc6d06d00aa9a

Don\'t crash the system when detecting a transient wrong state
am: f970410afe

* commit 'f970410afef518003c84eef022194848b2a4f606':
  Don't crash the system when detecting a transient wrong state

Change-Id: Ied0b4700c9c96a7ce04cce20f5da0ca62d5a1a72

Also notify divider when getting DockedTopTaskEvent
am: 3273f31c40

* commit '3273f31c40ce36010df9c6aa9324102bab9a2464':
  Also notify divider when getting DockedTopTaskEvent

Change-Id: I48c31e355586c6698523dd4748371bfaa5ab6945

Merge "Fixed bad focus stack state when focused docked stack is removed" into nyc-dev
am: 316f67bcfb

* commit '316f67bcfbc8b57a115a83d3484b8edde972570b':
  Fixed bad focus stack state when focused docked stack is removed

Change-Id: I8870b4927852b9b577116875a2b9f946a15f2513

Don't crash the system when detecting a transient wrong state

Bug: 28616824
Change-Id: I124f7196d709c33fe70137541b650788af83b3b3

Also notify divider when getting DockedTopTaskEvent

Under some circumstances we never go through the activity lifecycle
if we call startActivity before it gets stopped, so we never notified
the divider about that we have drawn.

Bug: 28531536
Change-Id: I6a13527d52bf40a87f89de4b2799bd0833c6612a

Merge "Fixed bad focus stack state when focused docked stack is removed" into nyc-dev

Dismiss docked stack if an activity is shown on top of the lock screen

It is risky/complicated to try to put the activity showing on-top of
the lock screen in the right fullscreen configuration with the current
architecture. So, we just dismiss for now.

Bug: 28195260
Change-Id: I29a0350993ce8fe548d4a465b06d877cde151c78

Make AccessibilityService#disableSelf atomic.

This API was using a oneway aidl call, which meant that the
service was disabled some time after the method returned. That
confused tests that were turning a service off during tearDown
and then turning it back on again in setUp.

Bug: 28621277

Change-Id: I75984df0613bdbb1bc876e2a15caf59106027337

Merge "Quicksettings accessibility." into nyc-dev
am: 450f91b529

* commit '450f91b52938de2a6cb9053b13b03ae75edd6be0':
  Quicksettings accessibility.

Change-Id: I0c3f246894739defc09f3e309f97ff8ccf4719c1

Fixed bad focus stack state when focused docked stack is removed

ag/990438 allowed for stack focus not to be adjusted when the currently
focused stack is removed since the caller will be moving the focus to
the right stack next. This was incorrectly enabled for the positionTask
call were the stack will not be moved to the front since the functionality
is used to move task within stacks we don't want to gain focus.

Bug: 28616824
Change-Id: Ia6aea17c0335d036a17b83fed28799428ca58eda

Merge "Quicksettings accessibility." into nyc-dev

Merge "Protect TRANSFER_COMPLETE broadcast am: 2759e429d1" into nyc-mr1-dev

Merge "Merge "Fix regression, show internal storage option..." into nyc-dev am: 9a76143ef5" into nyc-mr1-dev

Protect TRANSFER_COMPLETE broadcast
am: 2759e429d1

* commit '2759e429d1f5e18869de34e5f0e13257c269aff9':
  Protect TRANSFER_COMPLETE broadcast

Change-Id: Ibd02b250b09a6d43e1e3258e1a80a469e7f15458

Merge "Fix regression, show internal storage option..." into nyc-dev
am: 9a76143ef5

* commit '9a76143ef550a547aefed81acbad5d380cc33393':
  Fix regression, show internal storage option...

Change-Id: Iccbaa87826350a9ac10a0b852dc793bc222f20de

Merge "Fix notifications for USB PD enabled devices" into nyc-mr1-dev

Protect TRANSFER_COMPLETE broadcast

Bug: 28621801
Change-Id: Ia90abd00694ae80000f59ee0a17826592fd2159b

Merge "Fix regression, show internal storage option..." into nyc-dev

Merge "Import translations. DO NOT MERGE" into nyc-dev
am: b8dd0c3786  -s ours

* commit 'b8dd0c378657efd04557a02f8e50cfa4b11ebc49':
  Import translations. DO NOT MERGE

Change-Id: I4934f0aedb3027b13ba2ff793a39c9847fe35a40

Quicksettings accessibility.

Bug: 15696340
Change-Id: I6887e2dad4822911d3a1642aaec5703174b57330

Merge "Merge "Import translations. DO NOT MERGE" into nyc-dev am: d9a4a6df9a  -s ours" into nyc-mr1-dev

Merge "Import translations. DO NOT MERGE" into nyc-dev
am: d9a4a6df9a  -s ours

* commit 'd9a4a6df9aeb73579795adf942ab2ec72c0d3a67':
  Import translations. DO NOT MERGE

Change-Id: I6772905351b855fcc0d2b7888c0ce08250ea83f6

Merge "Import translations. DO NOT MERGE" into nyc-mr1-dev

Merge "Import translations. DO NOT MERGE" into nyc-dev

Merge "Import translations. DO NOT MERGE" into nyc-mr1-dev

Merge "Import translations. DO NOT MERGE" into nyc-dev

Merge "Import translations. DO NOT MERGE" into nyc-mr1-dev

Hide \'Optimizing apps\' dialog if not M->N upgrade
am: ee057752cc

* commit 'ee057752cce759520e3ba7a022f8616504c12fa0':
  Hide 'Optimizing apps' dialog if not M->N upgrade

Change-Id: I63e8184e42851a40ccb8d0df837dc96bf85eb8c5

Hide 'Optimizing apps' dialog if not M->N upgrade

Recent changes reduced the amount of time spent optimizing apps at
post-OTA boot. Instead of briefly showing the 'Optimizing apps'
dialog, we hide it completely.

The dialog is still shown for OTAs from a pre-N version of Android
since they can take a non-negligible amount of time (still much less
than in M).

Bug: 27350503
Change-Id: I5f6dae0c69799e1fc522f36453d2671b333bef0e

Merge "Fix enterprise contacts APIs naming and docs" into nyc-dev
am: 13532bdd2e

* commit '13532bdd2ea63c1b8a5a8d4f95346f3d1e7ecbb5':
  Fix enterprise contacts APIs naming and docs

Change-Id: I2305b72e0e3a1d5625d9016ffefed16fd7493c3e

Merge "Fix enterprise contacts APIs naming and docs" into nyc-dev

Import translations. DO NOT MERGE

Change-Id: I13bd96983d273f4a65e270689b83e5214ef0b25e
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I3ccdef79879827db018c8ac8960a7eb830607e2b
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I310e4c4effad31d6706d1a3260caa3bc2b6107f6
Auto-generated-cl: translation import

Merge "Update MIDI package summary am: 15a9a35bee am: c1b551aba6 am: fad0efcdb9" into nyc-dev
am: 06e066e9e1

* commit '06e066e9e197e192155703061b235bf167b02a90':

Change-Id: Ife3e997a25b70d5a3c93165ec70a1ac956971334

Update MIDI package summary am: 15a9a35bee am: c1b551aba6 am: fad0efcdb9
am: 9bef3417c9

* commit '9bef3417c930b773509c13008cf3b6152cc50e94':
  Update MIDI package summary

Change-Id: I1708eb74df8d9281bc74d5f8752a67269b0f82f6

Merge "Update MIDI package summary am: 15a9a35bee am: c1b551aba6 am: fad0efcdb9" into nyc-dev

Update MIDI package summary am: 15a9a35bee am: c1b551aba6
am: fad0efcdb9

* commit 'fad0efcdb96f87da9ded7474c4bfa67a6f221a88':
  Update MIDI package summary

Change-Id: I98018736d395487e3e680bef7bbb770e61b54389

Update MIDI package summary am: 15a9a35bee am: c1b551aba6
am: fad0efcdb9

* commit 'fad0efcdb96f87da9ded7474c4bfa67a6f221a88':
  Update MIDI package summary

Change-Id: Icb509b232ea816bb6305da6f38677d3cb6bea55e

Update MIDI package summary am: 15a9a35bee
am: c1b551aba6

* commit 'c1b551aba6f760ac41bcaf4391c1ff861803de27':
  Update MIDI package summary

Change-Id: Ic7a9920dc3f4a70f5f4eb718bc9f2dd00c1bbb7c

Update MIDI package summary
am: 15a9a35bee

* commit '15a9a35beeebeed42335d9f51663daad932a64b2':
  Update MIDI package summary

Change-Id: I3a86d68c14127e7969d300f77c10e0fe807cad72

Merge "allow optionally skipping of initial launch" into nyc-dev
am: 2c7d670933

* commit '2c7d670933e0e0f7f6a83c4353519511faaf4a2f':
  allow optionally skipping of initial launch

Change-Id: I9621009ce4c3774368e974494c7f2beb4b2478f3

Merge "allow optionally skipping of initial launch" into nyc-dev

Import translations. DO NOT MERGE

Change-Id: I1d4745156fb9ddbff3d6af4564fa0a1754e1a384
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I15055663e1705ae0e64ca3f4242bbad3748461e4
Auto-generated-cl: translation import

Merge "Coerce 8-bit bsic to Integer if Invalid" into nyc-dev
am: 08acd3ce28

* commit '08acd3ce28ff34f5a49d693b4c1144d967208e67':
  Coerce 8-bit bsic to Integer if Invalid

Change-Id: Ib1cf6ecce437c8198da169fd5a4f0d1301e1547e

Merge "CellInfo Parceling for Invalid Values" into nyc-dev
am: 5b45c62a1a

* commit '5b45c62a1a13801ed4c5fd228976928b30f63867':
  CellInfo Parceling for Invalid Values

Change-Id: I049eabf4ede20f591fd193e133e62444c1185fb7

Merge "Coerce 8-bit bsic to Integer if Invalid" into nyc-dev

Merge "CellInfo Parceling for Invalid Values" into nyc-dev

allow optionally skipping of initial launch

Current harness performs one round of app launch as a "warm up", and
does not include that into final app launch time calculation. This
change skips this process so that it can speed up local test run
investigation.

Change-Id: I0b1948b2247d9f7e442e799d537508fba6dfa508

Merge "Add vanity url for bg optimations." into mnc-mr-docs am: d4c3473906 am: 4521e4e605 am: 631316a832
am: 61926ad85f

* commit '61926ad85fdd9b3cb6444f2ac41a14b9680f98c6':
  Add vanity url for bg optimations.

Change-Id: Ieca598b5a6260a9ec6fe03b13d32492230a4207c

Merge "Add vanity url for bg optimations." into mnc-mr-docs am: d4c3473906 am: 4521e4e605
am: 631316a832

* commit '631316a83231bdd1089261c165b87a9fff08822a':
  Add vanity url for bg optimations.

Change-Id: Iec34ba687fa442be88710e3f918be9fb721a9f11

Merge "Add vanity url for bg optimations." into mnc-mr-docs am: d4c3473906
am: 4521e4e605

* commit '4521e4e60518b9c32948b2027822af8c30de8576':
  Add vanity url for bg optimations.

Change-Id: Ic05784b96e79b0222f59f1f9ee07edf9bce447f6

Merge "Add vanity url for bg optimations." into mnc-mr-docs
am: d4c3473906

* commit 'd4c3473906ea86f9b18ffaa3856cb9d97fca139b':
  Add vanity url for bg optimations.

Change-Id: I765f0c558ff2d74c536cbdf4c0fbdf29dd690fdb

Update MIDI package summary

Bug: 28625060
Change-Id: If552ca8e1a0666d402b5f536699bf3fb09c1e324
(cherry picked from commit 9a279ef267f194dd3f0b777da5a219b6ef04abac)

Merge "Add vanity url for bg optimations." into mnc-mr-docs

Merge "Fix status bar background flicker" into nyc-dev
am: d0fa4d3aaf

* commit 'd0fa4d3aafa42da696c006ab96a11ed54deec14c':
  Fix status bar background flicker

Change-Id: I8cb0909f952aa0e8bf7705d753fd1bc0953187ba

Merge "Fixed deadlock between AM and WM when animating pinned stack." into nyc-dev
am: 3a94ec5617

* commit '3a94ec56173170306b2304c858dbbeb99c3c93cc':
  Fixed deadlock between AM and WM when animating pinned stack.

Change-Id: I51938073894ec18042a987645a424811213061ad

Merge "Fix status bar background flicker" into nyc-dev

Merge "Fixed deadlock between AM and WM when animating pinned stack." into nyc-dev

Properly map runtime permissions to app ops

The code assumed mapping from a permission to an app op
is one to one but this is not always the case. For example,
READ_SMS is mapped to OP_READ_SMS and OP_READ_ICC_SMS which
resulted mapping the READ_SMS permission to the OP_READ_ICC_SMS
instead of OP_READ_SMS resulting in a failure to find the op
name given the permission.

This breaks the AppOpsManager.permissionToOp() API for READ_SMS
returning null instead of OPST_READ_SMS. The consequence of this
is that the apps that proxy permission protected operations may
let the operations for READ_SMS go through as they would get a
null app op, i.e. no app op while there is one and it can be
disabled for the caller.

bug:28620132

Change-Id: I92f8ef375ae2122b7266c50653ce73f3d90f4b28

Fix edge case crash with invalid bitmap dimensions.
am: f9e79e0499

* commit 'f9e79e0499a114b184e0d0f4dbd45b3d690a4657':
  Fix edge case crash with invalid bitmap dimensions.

Change-Id: Id8464d81b228ff917226a7088ab9199b885c32c2

Fix edge case crash with invalid bitmap dimensions.

- In these cases, skip returning an animation spec for that task

Bug: 28235453
Change-Id: I611dc4390c0cbd2ca5f16112e4784848b2d6896f

Fix NPE in KeyphraseEnrollmentInfo.toString()

If there are no enrollment applications on the system, but someone still
makes a KeyphraseEnrollmentInfo and tries to print it, it would generate
a NPE on a map object. Instead of setting the map to null when we don't
find any enrollment applications, we can just set it to an empty map.

Bug:28622866
Change-Id: I023e6fd90effd3143c19817a0d6637a013bebc31