Merge changes I7261cf87,Icabfcf47,I09e6b032,I252cf139,If85ade73, ... into nyc-dev

* changes:
  Revert "Death to synchronous transactions (2/2)"
  Fix a few weird state issues from race-conditions
  Fix lifecycle bug in when calling positionTask
  Animation fixes when task is not resumed
  Keep stack from mReuseTask
  Final fixes for growing recents transition

Merge "Update divider controller's dim layer when the layer is changed" into nyc-dev

Update divider controller's dim layer when the layer is changed

The divider's layer assignment could change after the initial adjust
animation has finished.

bug: 28255739
Change-Id: I5899fe22e4fec680c49c881437c7f85d8ee9ca74

Revert "Death to synchronous transactions (2/2)"

This reverts commit dd05f0365aa0df9ba9a7631c8e02ed67f472132d.

There seems to be some logic that gets triggered inside surface flinger
regarding commit time of the transaction, and without it seems that
animations got jankier. No time to investigate...

Bug: 28305789
Change-Id: I7261cf8719d3528f8f5bee19e507b8ba88b8b729

Fix a few weird state issues from race-conditions

- When starting another transition while the dock transition
is already set, always override that because that transition
starts prolonged animations which will never be ended.
- Only dock if we get a good UP event.

Bug: 28257206
Change-Id: Icabfcf475f419696d2bcff8fbb53cf3b29dedd69

Fix lifecycle bug in when calling positionTask

Sometimes the resumed activity from another stack gets positioned
BELOW the top activity of the current stack. We need to carry over
the mResumedActivity to make sure onPause is called on the moved
activity.

Bug: 28235047
Change-Id: I09e6b032cd8d0b00dc5fdd06bec68e53c25aae73

Animation fixes when task is not resumed

- Make sure to add the activity to mWaitingVisibleActivities
to not destroy the activity before the app transition has
started.
- Defer layouts in startActivityUnchecked so when clearing the
current activities in the current stack, we don't execute the
app transition yet. We need to wait until the new activities
are added as well.

Bug: 28026847
Change-Id: I252cf139e06197ea34329d1466f1f6162f3e7a30

Keep stack from mReuseTask

When we launch an activity with CLEAR_TOP and NEW_TASK, we clear
all the activities in that task but then we don't reuse the stack,
which can lead to weirdness.

To fix this, we need to keep the target stack in that case.

Bug: 28000613
Change-Id: If85ade73b1036e8a7c0eda418d9f39e8596b8b16

Final fixes for growing recents transition

- Make sure to reposition windows during animations to avoid that
they lag one frame behind.
- Don't put windows that are gone for layout into resizing mode.
- Don't layout windows that are gone for layout, to avoid resizing
the surface but the client won't draw anymore.

Change-Id: I809feffef00f9a086b44504126e03f509eb7f190
Fixes: 27855229

Merge "Rename DropPermissions to DragAndDropPermissions" into nyc-dev

Merge "Fix broken build due to @RemotableViewMethod" into nyc-dev

Fix broken build due to @RemotableViewMethod

Change-Id: I63c6cb4fa68df114220efb2f263115e773e25f5d

Merge "API review feedback for ProgressBar" into nyc-dev

Merge "Don't overwrite vertical and horizontal offsets unless explicit" into nyc-dev

Merge "add @RemotableViewMethod to TextView#setEnabled b/28304068" into nyc-dev

add @RemotableViewMethod to TextView#setEnabled
b/28304068

Change-Id: Ia5464030a65c87dacdd75e5be331da017d7ef593

Merge "Make sure don't adjust IME when docked left-right" into nyc-dev

Merge "Remove pre-N db after successful migration" into nyc-dev

Merge "QS: Use current tile state when removing custom tiles" into nyc-dev

Merge "Announce when volume controls appear and disappear." into nyc-dev

QS: Use current tile state when removing custom tiles

Change-Id: If12415c051239348261386fb23e88d6725767ba0
Fixes: 27912532

Merge "Make Dismiss Icon under Cards show up only when focused." into nyc-dev

Make sure don't adjust IME when docked left-right

bug: 28295645
Change-Id: Ia8f7f9a92fcc562bd672abae4d3ddfd6e6201a79

Merge "Framework: change the TTS text to use the current locale" into nyc-dev

Merge "Respect manually set HARDWARE_ACCELERATED flag." into nyc-dev

Merge "Fix order of operation issue with ResStringPool" into nyc-dev

Announce when volume controls appear and disappear.

Bug: 28120156
Change-Id: I4e17b9cf1cd4a442adb7513168c12e3af24c424f

Remove pre-N db after successful migration

Bug: 26279712
Change-Id: I53e4bd1b04b4ae6b94e8b5642d7f11a4edd6f1e4

Merge "AudioManager: fix isBluetoothA2dpOn()" into nyc-dev

Make Dismiss Icon under Cards show up only when focused.

BUG: 28299529
Change-Id: I37cd029734da5de8416705746457151c22f64280

Merge "Fix dead surface/window left on screen" into nyc-dev

Merge "Accept null subtype in InputMethodSubtypeHandle." into nyc-dev

Merge "Fix shadow radius too large for RS" into nyc-dev

Fix order of operation issue with ResStringPool

Due to ! taking precedence over bitwise &, the condition for
checking non-null terminated string blocks is incorrect.

Adds parentheses for the correct behavior.

Bug:28288210
Change-Id: Ie31fa239e5f869e6bb28deb6ae190f41f1aa4d92

Merge "PIP: Fix system UI crash" into nyc-dev

Merge "Fix wrong app is launched after solving challenge from tapping notification" into nyc-dev

Fix wrong app is launched after solving challenge from tapping notification

PendingIntent.FLAG_ONE_SHOT is needed to prevent the PendingIntent being
reused.

Also, flag and activity option of credential intent in BaseStatusBar
is not same as the one in ActivityStarter. Added a new
function startConfirmDeviceCredentialIntent in AM to centralize it.

Change-Id: I2d9e5923ad5d4d012f10057c409c666c8ca299a3
Fix: 28309964

Merge "Accept the calendar name in addition to the owner as a calendar filter." into nyc-dev

Merge "Always reevaluate rules when a new service is added." into nyc-dev

Merge "Include work app in battery optimization settings" into nyc-dev

Merge "Don't call into audiomanager holding a zen lock." into nyc-dev

Merge "Reinflate QS view on Locale configuration change" into nyc-dev

Always reevaluate rules when a new service is added.

Otherwise the condition providers may never get an
onSubscribe call and may not turn on at the right time.

Bug: 26981870
Change-Id: Ia622597f10cc26f7c34ece1748013655543f488f

Don't call into audiomanager holding a zen lock.

Bug: 27642196
Change-Id: Ib510f2ddf966f36b3670290a4f3f695bf6b7810e

Merge "Volume updates should appear continuous when changed by touch." into nyc-dev

Merge "Fix silent errors when launching Keyboard Shortcuts from Settings" into nyc-dev

Merge "Cap number of NetworkRequests a UID can make to 100" into nyc-dev

Reinflate QS view on Locale configuration change

The existing DensityContainer already did this for density changes,
repurposed it to handle other Configuration changes. Also renamed
to AutoReinflateContainer.

Bug: 27952059
Change-Id: Ieb572abcbafe49aec73b16e1bcb8c294011afdd3

Fix silent errors when launching Keyboard Shortcuts from Settings

The following error was thrown, given that Activity is generating a
protected broadcast:
Sending non-protected broadcast
android.intent.action.SHOW_KEYBOARD_SHORTCUTS from system

Bug: 28138503
Change-Id: I50d5212d7776d33ffe6e9df1c5f080205ac4a29d

PIP: Fix system UI crash

Bug: 28306480
Change-Id: Ie36f73c46f3ee8d6a2cb8264458cb0e0410aaecd

Merge "Expand NetworkMonitor metrics" into nyc-dev

Merge "Some fixes in ConnectivityService event logging" into nyc-dev

Expand NetworkMonitor metrics

    - add netid to CaptivePortalStateChangeEvent
    - add NetworkMonitorEvent for simple duration measurements

Bug: 28204408
Change-Id: I62035e07d64013878a0bce30c67c929c2c7e7a85

Merge "Various fixes in IpReachability event logging" into nyc-dev

Merge "Removed obsolete screenshot mention from full bugreport." into nyc-dev

Merge "Remove obsolete TODO" into nyc-dev

Merge "Do not make files in zip disabled in ACTION_VIEW." into nyc-dev

Merge "fix build: remove SUSTAINED_PERFORMANCE_WAKE_LOCK link" into nyc-dev

Merge "Additional checks for ModemActivityInfo.isValid()" into nyc-dev

Some fixes in ConnectivityService event logging

Bug: 28204408
Change-Id: I44ed9539bb213a845c2bcc2861c947af7c3c9ef4

Merge "Fix a11y key event handling from multiple devices." into nyc-dev

Remove obsolete TODO

Change-Id: Ia76b7f1cea0799c46ae5c933741c865869e39f5f
Fixes: 27550895

Merge "Fix IME adjust in SysUI" into nyc-dev

Merge "Only invoke k/v helpers during k/v operations" into nyc-dev

Merge "PIP: Support accessibility" into nyc-dev

Only invoke k/v helpers during k/v operations

In particular, don't try to call the k/v wallpaper restore helper's
onRestoreFinished() if we are actually doing a full-data restore.

Bug 28132784

Change-Id: I2eaf748f6d54601f0ca195ae2781aa177daf0b93

Fix dead surface/window left on screen

If the surface is explicitly hidden while we're changing resizing mode,
we need to discard the preserved surface immediately, otherwise it could
linger around on screen.

Do not mark mAnimatingExit in setTokenVisibilityLocked. This flag can only
be set after client relayouts to invisible. There we set mAnimatingExit and
mAnimating, so that when animation finishes the surface is marked to be
destroyed. If we marke it too early (and not mark mAnimating at the same
time), the surface may get stuck in exiting state.

bug: 28236574
Change-Id: I5cc916dfa698113e1d783a66547e1878469a9879

Merge "Exclude IME dialogs from tap outside task bounds funcationality" into nyc-dev

Merge "Throw better exception when startVoiceActivity fails" into nyc-dev

Merge "BatteryStatsHelper: Include screen on baseline power draw" into nyc-dev

Merge "Fix logspam by JobInfo" into nyc-dev

Merge "Increase the max binder thread pool size for system_server." into nyc-dev

Merge "Prevent printing from finishing activity." into nyc-dev

Merge "A11y: Fix battery icon and clock descriptions" into nyc-dev

fix build: remove SUSTAINED_PERFORMANCE_WAKE_LOCK link

Change-Id: Ic12b98cbafcc8c81eadba119ab416a9eddb7214a
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>

Various fixes in IpReachability event logging

- fold IpReachabilityMonitor* classes into a single IpReachabilityEvent.
- only log an event for NUD_FAILED Netlink answers.
- distinguish between NUD_FAILED with or without lost of provisioning.
- do not record host ip addresses.
- record interface name instead of interface index when losing
  provisioning.

- also return an error code when probeNeighbor fails, and log this error
  code in IpReachability events.

Bug: 28204408
Change-Id: I5f0def0ab1ace7e467a0c69b3b82d07ef2252307

Merge "Print API for N: Fix docs and change interfaces as requested." into nyc-dev

Merge "Disallow changing stacks if top activity" into nyc-dev

Merge "Add JavaDoc for View.updateDragShadow" into nyc-dev

Merge "Prevent recents from closing while moving PiP to fullscreen." into nyc-dev

Accept null subtype in InputMethodSubtypeHandle.

There are two types of IMEs:
 A. IMEs that have one or more subtypes
 B. IMEs that have no subtype

The initial implementation to update hardware keyboard layout per
subtype change of layout (See Bug 25752812) has supported IMEs in the
category A only, and IMEs in the category B are just ignored in both
system and Settings app.

In order to support IMEs in the category B, InputMethodSubtypeHandle and
related methods need to accept null InputMethodSubtype. Technically
this is a straightforward change, because in InputMethodManagerService
we have already used InputMethodUtils.NOT_A_SUBTYPE_ID for those IMEs in
the category B.  We also need to update Setting App, which will be done
by a different CL [1].

  [1]: I46b9c5b018f08e3eaa4614a0893db0be91652f3c

Bug: 28182650
Change-Id: Ia013784a594ad3beaf30976d047f5ac0fa8185be

Merge "Removing disk access from startActivity thread" into nyc-dev

Merge "Camera2: Ignore exception on double-completed capture sequences." into nyc-dev

Fix IME adjust in SysUI

- Make sure scale is correct.
- Adjust interpolator to match WindowManager.

Bug: 28175599
Change-Id: Ica94b3dbac2fcc6e4b8a75faa311fb7861071ccc

Fix logspam by JobInfo

Print interval clamp message
only for periodic jobs having
period/flex less than the minimum value.

Bug: 28296128
Change-Id: Id0d7c3b56048582a490bb8214eca491ec2bbdc93

AudioManager: fix isBluetoothA2dpOn()

Test all A2DP type variants when checking if an A2DP device is connected
in isBluetoothA2dpOn().

Bug: 28286051
Change-Id: I756d632b12d584d8a27cc1890e758d8accff7120

Merge "Fix inline reply clobbering when narrowly missing send button" into nyc-dev

Exclude IME dialogs from tap outside task bounds funcationality

We don't want tapping on these windows to cause a change in app
focus.

Bug: 28184155
Change-Id: Ibd4ca15b2154c2e83a11f4a2a3edb4356cbfbfed

Merge "Set BugreportProgressService to run on foreground." into nyc-dev

A11y: Fix battery icon and clock descriptions

Now announces charging state on the battery icon
and AM/PM on the clock.

Change-Id: Ib8d249e60b555834e5e224287ea332178ce7c165
Fixes: 27552350

Merge "ResourcesManager: Throw exception if failed to load asset path" into nyc-dev

Disallow changing stacks if top activity

When we attach an activity to an existing
task, and that task is already on top,
disallow changing stacks altogether even
if launch stack id is set.

Bug: 28026847
Change-Id: Ie70f0585a29dc1b85a5093624fede32110be3c76

Removing disk access from startActivity thread

Code was checking whether the directory for recent_images existed in
many places unnecessarily causing StrictMode violations. Moved the
code to only attempt to check and create the directory before
actually writing to disk.

Bug: b/28195831
Change-Id: I05f77a10f1dafc8cc0b1836b62352d56549ac1ee

Merge "Added defensive code to prevent system crash" into nyc-dev

Merge "Fixed bug in upgrading GpsMeasurement" into nyc-dev

Merge "Don't back up wallpapers that we've been told not to" into nyc-dev

resolve merge conflicts of 44e07e0 to mnc-dev am: 825b713 am: 8c635e7 am: 343600c
am: b965ced  -s ours

* commit 'b965ced011bdd7a3ec5450f93d1c58b889514f11':
  Kill the real/isolated uid group, not the ApplicationInfo uid

Change-Id: Idf9a9824d411ca2b178fadda040f46864bc96a58

Camera2: Ignore exception on double-completed capture sequences.

As a workaround for b/27870771, convert fatal exception into a
warning.

Duplicate calls to finishPendingSequence with the same sequenceId
are seen in user crash reports, but with no further context and
inability to reproduce after testing various hypothesis, just
warn about it happening.

Bug: 27870771
Change-Id: Icd7d408887e04b94092689ce61809d6c664d8e3a

resolve merge conflicts of 44e07e0 to mnc-dev am: 825b713 am: 8c635e7
am: 343600c

* commit '343600c3ecff31826e50d40821927e446adb9b0a':
  Kill the real/isolated uid group, not the ApplicationInfo uid

Change-Id: I68dbf8f280672ec9770b6c57b9a17313f448e14a

Set BugreportProgressService to run on foreground.

BugreportProgressService do not persist the user-provided
information (like details and screenshot paths), so if it's killed by
the framework, that info is lost.

Running it as foreground mitigates the changes of it being killed.

BUG: 27431998
BUG: 28291423
Change-Id: I2f58507beb38309628f2f19d3f7f950d07eca16f