Merge "Fixed a bug where the statusbar could become empty" into nyc-dev

Merge "Fixed a bug where Notifications could overlap" into nyc-dev

Merge "Don't include IME in app screenshot while in multi-window mode" into nyc-dev

Merge "Change ForcedResizableInfoActivity to handle config. changes" into nyc-dev

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

Merge "Autobundle all notifications; don't leave the most recent one free." into nyc-dev

update the older redirects based on studio restructure (https://critique.corp.google.com/#review/121836018) am: f2f8dff6eb am: d20d4c6cf3
am: 0d1bce5e5d

* commit '0d1bce5e5d3a0521466e8d12de792024325d8d91':
  update the older redirects based on studio restructure (https://critique.corp.google.com/#review/121836018)

Change-Id: Iae2be6e4c6428e46ff1317d35eb3e12f97c6a813

update the older redirects based on studio restructure (https://critique.corp.google.com/#review/121836018) am: f2f8dff6eb
am: d20d4c6cf3

* commit 'd20d4c6cf3bc5891d2a63024b1c5ba3abf5ed77d':
  update the older redirects based on studio restructure (https://critique.corp.google.com/#review/121836018)

Change-Id: Ib717a1f84f3c32ba6ccb762d0b3ffba6a1bd37e0

update the older redirects based on studio restructure (https://critique.corp.google.com/#review/121836018)
am: f2f8dff6eb

* commit 'f2f8dff6eb50e673b32b8f6bb3c181267997aefd':
  update the older redirects based on studio restructure (https://critique.corp.google.com/#review/121836018)

Change-Id: Ib906d725e2de7012ebd305d64e12fc434c16a5ca

update the older redirects based on studio restructure
(https://critique.corp.google.com/#review/121836018)

Change-Id: I5c7cb01da2afe18885444c9bd6a3fc09526a42a8

Merge "Fix recents row blinking when PIP is focused" into nyc-dev

Merge "MediaRouter: Add more logs for debugging" into nyc-dev

Merge "Hold a ClassLoader reference in NativeAllocationRegistry." into nyc-dev

Merge "Remove inconsistent 'cmd package compile' documentation." into nyc-dev

update redirects for /studio files for the restructure from this CL: https://critique.corp.google.com/#review/121726749 am: ab5bf95626 am: 1a8f9e3ca0
am: 5e42875297

* commit '5e428752971f2e3dfc3fda73633fc346f50f2763':
  update redirects for /studio files for the restructure from this CL: https://critique.corp.google.com/#review/121726749

Change-Id: I76513a274909a296589c1354dfc0551764390e72

update redirects for /studio files for the restructure from this CL: https://critique.corp.google.com/#review/121726749 am: ab5bf95626
am: 1a8f9e3ca0

* commit '1a8f9e3ca00fb4436710e34b1f7fb2259adc066f':
  update redirects for /studio files for the restructure from this CL: https://critique.corp.google.com/#review/121726749

Change-Id: Id0d7cf68025c8f6b082ebfdec6c9727e963d6b9a

update redirects for /studio files for the restructure from this CL: https://critique.corp.google.com/#review/121726749
am: ab5bf95626

* commit 'ab5bf95626850e97124b912ca2699fb164574186':
  update redirects for /studio files for the restructure from this CL: https://critique.corp.google.com/#review/121726749

Change-Id: I7524aa066efa4241c6a14d881b23dee0da8e2632

update redirects for /studio files for the restructure
from this CL: https://critique.corp.google.com/#review/121726749

Change-Id: I2c06eb12267f11d69a1dbd6f1c96a7bf1ef4a61c

Merge "Always mutate child when added to drawable container" into nyc-dev

Merge "Cancel animation and jump to current state if not animating picker" into nyc-dev

Merge "QQS: Fix ripple clipping" into nyc-dev

Import translations. DO NOT MERGE

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

Merge "Ensure Netd Service is alive" into nyc-dev

Ensure Netd Service is alive

At least on android TV emulator system image, the Netd
service could be null when the setDnsConfigurationForNetwork
is invoked.

bug: 28553503
Change-Id: If059495c8573ede287c4e9d8f343318d519fca7f

Don't include IME in app screenshot while in multi-window mode

We don't screenshot the IME in multi-window mode because the frame
of the IME might not overlap with that of the app. E.g. IME target
app at the top in split-screen mode and the IME at the bottom
overlapping with the bottom app.

Bug: 28446505
Change-Id: Iae4f7eafa582779a4cf2aba3d83c4d9f76a4ccca

Change ForcedResizableInfoActivity to handle config. changes

This created extra churn in the system during resize due to
the activity relaunching.

Bug: 28614747
Change-Id: I148b6fca3dad7e10c90085a04bccb99587397912

MediaRouter: Add more logs for debugging

Bug: 28517884
Change-Id: I325b407983e79a78701acfbfd8f109657f006846

Merge "Fix minimized state messup" into nyc-dev

Merge "Reset transparent region when saving a surface" into nyc-dev

Merge "Fixed connectivity state in some restricted network scenarios." into nyc-dev

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

Fix minimized state messup

We didn't dispatch the change, but still set the field. This lead to
issues where SysUI disagreed on the minimized state. To fix this, we
only check for state changes if the docked stack is visible, but then
dispatch the change always.

Bug: 28445644
Change-Id: Ie460135049c78e11c97f3160994ca4069bf82e6c

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

Reset transparent region when saving a surface

Fixes: 28432088

Ensure a transparent region on a saved surface is reset for future use,
since the surface should be like-new if used again.

This prevents an issue where the region - used to signal a portion of
content doesn't need to be composited - is persisted when a saved
surface is reused. The client assumes it's new and in default
state (composite everything), but the window is clipped when composited.

Change-Id: Icf2ec94c735679d715aded58de7eab12e9c43367

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

Fixed connectivity state in some restricted network scenarios.

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.

Another scenario that is not properly handled is when a UID whitelisted
for Data Saver is brought back to foreground: although the proper
firewall rules are set, CS is not notified, and the apps state would be
DISCONNECTED / BLOCKED.

This CL introduces many changes that fix this issue:

- Fixed updateRuleForRestrictBackgroundLocked() to invoke
  onUidRulesChanged() when the Battery Saver status changed.
- Fixed updateRuleForRestrictBackgroundLocked() to invoke
  onUidRulesChanged() when an app whitelisted for Data Saver is brought
  back to the foreground.
- Added a new API (onRestrictPowerChanged() and getRestrictPower())
  to notify external services about Battery Saver mode changes.
- Fixed CS logic to properly handle the Battery Saver changes.

Externally to this change, the CTS tests were also improved to verify
the apps get the proper connection state; they can be verified running:

cts-tradefed run commandAndExit cts -m CtsHostsideNetworkTests \
    -t com.android.cts.net.HostsideRestrictBackgroundNetworkTests

BUG: 28521946

Change-Id: I8eaccd39968eb4b8c6b34f462fbc541e5daf55f1

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

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

Always mutate child when added to drawable container

Bug: 28456908
Change-Id: I3e7bea85e056882a331aecf7e225134161e685e6

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

Cancel animation and jump to current state if not animating picker

Bug: 28629631
Change-Id: I0e1b3c2f2c774e027f2df8c9ab75c7a89bd2ddb7

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

QQS: Fix ripple clipping

Change-Id: Ib29265ca582e8563277466edefb32b0a5c852b2f
Fixes: 28175282

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

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 "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

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

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

Protect TRANSFER_COMPLETE broadcast

Bug: 28621801
Change-Id: Ia90abd00694ae80000f59ee0a17826592fd2159b

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

Autobundle all notifications; don't leave the most recent one free.

Change-Id: I0c30e34d09852673a475392411d0bb8fdc2f7aa3
Fixes: 28615503

Quicksettings accessibility.

Bug: 15696340
Change-Id: I6887e2dad4822911d3a1642aaec5703174b57330

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

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

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

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

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

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

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

* 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

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.

- 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

Fixed deadlock between AM and WM when animating pinned stack.

In Ie18cc04ceeabd94093017f79676253e1189f5249 we delay ending the
pinned stack animation until app transition is done. However, the
signal that app transition is done comes in with the window manager
lock held and the we call into AM to do some additional work which
requires AM lock to be aquired. Since AM depends on WM we shouldn't
be holding the WM lock while calling into AM since AM can call into
WM with its lock held. We now post a runnable to finish the pinned
stack animation without the WM lock held.

Bug: 28564514
Change-Id: I8450123e992e569a74609883de810cdc611ced81

Merge "Fix issue where existing Activity Resources references would not update" into nyc-dev

Fix status bar background flicker

Fixes: 28533578

Change-Id: I075f49b7d20e0e95e790a9755d104a0a51575054

Fix issue where existing Activity Resources references would not update

When relaunching an Activity, the DecorView and ViewRootImpl is re-used,
along with the IBinder activity token. This means that when we create
new resources, we end up just updating the base Activity resources.

However, we would set the configuration to the new requested override
config and then proceed to update existing Resources references, which
would return immediately due to the equality check for configuration
updates.

This change pushes the setting of the new override config to the
updateResourcesForActivity method, which will properly update Resources
references that are held in the re-used DecorView.

Bug:27915587
Change-Id: I113007a40fa464b8a234b073dcf851e16fee0177

Coerce 8-bit bsic to Integer if Invalid

Bug: 28299027
Change-Id: I1ff3314844991cf49cab6099a7a4c436980d52c9

CellInfo Parceling for Invalid Values

Within CellInfo Fields, Integer.MAX_VALUE should indicate
that a field is not present in a report. This change makes the
invalid value consistent, and also ensures that it is checked
in the calculation of power levels / asu.

-Update parcel/unparcel functions to avoid -MAX_VALUE
-Update LTE and CDMA asuLevel() and level() functions to be
 consistently aware of the possibility of incomplete information

Bug: 27946114
Change-Id: Iacdc85db894e4a7809da8b5dc759488a1b6321ea