Merge "Split network monitoring and portal probe events" into nyc-dev

Merge "Remove MANAGED_PROFILE_AVAILABILITY_CHANGED broadcast" into nyc-dev

Merge changes from topic 'blockOnProcessingUnsolicited' into nyc-dev

* changes:
  Tethering: interface updates can be handled again
  NativeDaemonConnector: add waitForCallbacks method

Tethering: interface updates can be handled again

When reloading wifi firmware, unsolicited responses from netd were
processed after softap had started and caused wifi tethering to be torn
down.

The NetworkManagementServer.wifiFirmwareReload call has been changed to
not only block for the command to finish, but also until all unsolicited
messages (interface updates) have been handled.  We should now be able
to handle interface updates in tethering without suffering from the
softap bringup/interface down notification race condition.

BUG: 27857665
Change-Id: Ie57cb8f760781b3227df575b577b33667070d63e

Split network monitoring and portal probe events

This patches changes how captive portal tests and network lifecycle
events are logged as connectivity events:
  - it splits NetworkMonitorEvent into two event classes:
      - ValidationProbeEvent for logging individual probe events.
      - NetworkEvent for logging network connection, validation,
        lingering, and disconnection.
  - it removes the redundant CaptivePortalCheckResultEvent class.
    The information logged in CaptivePortalCheckResultEvent was already
    logged by NetworkMonitorEvent, but missing the evaluation durations.
    It is now logged by ValidationProbeEvent.
  - it removes the CaptivePortalStateChangeEvent class, which is now
    redundant with NetworkEvent, but missing evaluation durations.

In addition, it adds event logging when ConnectivityService puts a
network into lingering or removes a network from lingering.

Bug: 28204408
Change-Id: I8f9752e4d36175ecfcbd1545a01a41bad6e06ea4

Merge "Fixed a bug where the icon of children could be colored" into nyc-dev

NativeDaemonConnector: add waitForCallbacks method

Add a method that allows callers to wait until all unsolicited
responses received from the native daemon during a command are
processed.

When commands are issued to a native daemon (such as netd) through the
NativeDaemonConnector we block until the command response is received.
Any responses or events that are a side-effect (considered
"unsolicited") of the command are placed in a Message and handled as
callbacks. The order of their processing is not guaranteed and, as we
have seen from bugreports, can be handled several seconds
later - causing the SoftAP that was just set up to be torn down
because a late interface down/removed is indistinguishable from a
new interface down/removed.

This CL adds a method that first checks to make sure callback thread
is not the same thread as used for the blocking call.  The new
waitForCallbacks method uses a CountDownLatch to force the calling
thread to wait until all unsolicited responses received from the
native daemon during the execution of the command are handled.

The wifiFirmwareReload method is also updated to use the new
waitForCallbacks method.

BUG: 27857665
Change-Id: I3e22978f720b1cbf57fbb64ad4fea73f8c2d408a

Merge "Fixed a bug where notification header could become empty" into nyc-dev

Merge "Fix signal cluster animation leak" into nyc-dev

Fix signal cluster animation leak

When subscription id's change during the animation, make sure to
clear them out as well. This happens when doing a network switch
on Fi.

Bug: 27434767
Change-Id: I9cbd65888419ca630e884c51d41a0d785237f35b

Merge "Fixed a crash when the heads up content view was null" into nyc-dev

Merge "Changed the name of chronometer countdown" into nyc-dev

Merge "IpManager,DhcpClient: catch some errors and better logging" into nyc-dev

Merge "Fix squished UI elements" into nyc-dev

Merge "Clear calling identity before calling mountService" into nyc-dev

Fix squished UI elements

Stack bounds were wrong after moving all tasks from docked stack to
fullscreen stack, if fullscreen stack was created while moving.
This lead to very wrong bounds when rotating the screen the next
time.

Bug: 27870534
Change-Id: I6d054922ffc7baa7cd1a86d0d3448050cbacd21f

Merge "Don't worry about an extra weak indirection with callback alarms" into nyc-dev

Merge "leanback: Fix link to changed API setTitle()" into nyc-dev

Merge "Add throws definition to MtpDevice#readEvent." into nyc-dev

Merge "Handle API review comments from the council." into nyc-dev

Merge "Switched the notification time to relative time" into nyc-dev

Merge "Move child FragmentManager shutdown for retained fragments" into nyc-dev

Merge "Fix flag value collision" into nyc-dev

Fix flag value collision

Bug: 28314544
Change-Id: If73c1e8fd0e7b313bccbeffa172bdc3ab9496a43

Merge changes I036fbc5f,I979f0107 into nyc-dev

* changes:
  If we can't call into an app process to deliver an intent, crash the process.
  If a crash dialog can't be shown, just kill the process.

If we can't call into an app process to deliver an intent, crash the process.

Rather than hoping the process will crash for some other reason, it is
better to just get rid of it right away. The Intent was not delivered,
so there are no guarantees that the app will continue to function
correctly.

Bug: 28196243
Change-Id: I036fbc5ffd42dc7259437cc5a733976478a2dd3a

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

Fixed a bug where the icon of children could be colored

Change-Id: I12e577de648f8b1734521ffe713f0168f714572a
Fixes: 28320187

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

Fixed a bug where notification header could become empty

Also fixed a few more issues with the header util that could
lead to wrong states.

Change-Id: I95c3479f5d9e5221ee9e91120271e7957b887607
Fixes: 28295743

Fixed a crash when the heads up content view was null

Change-Id: I02c47eb0f4bcc3620103882e34b079aa83790a44
Fixes: 28301534

Changed the name of chronometer countdown

Change-Id: Ie0339205b26f2dff40e0c19760750a0ce6aaf7da
Fixes: 28295825

Switched the notification time to relative time

The time is now displayed in a relative manner instead
of absolute.

Change-Id: Idcc4411ed07991b6e176c4890a0d804394d8598a
Fixes: 27435284

Don't worry about an extra weak indirection with callback alarms

Bug 27995384

Change-Id: Ie9cf3e05dca4af7085b479fb65d4cf45ee0239ba

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

leanback: Fix link to changed API setTitle()

To match ag/952395.

Change-Id: I96cf9caa34bd560f14f7864201f3ac228cd2537d

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

Move child FragmentManager shutdown for retained fragments

Some apps out there do some creative things in terms of calling their
own onDetach methods from elsewhere. Perform the child FragmentManager
shutdown for retained fragments in a package-level perform method
instead of in onDetach itself.

Also mark all existing fragment methods with @CallSuper.

Bug 28293847

Change-Id: I343b610f8d427fe5cabd523a78fbf0e6f3b34c57

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

IpManager,DhcpClient: catch some errors and better logging

Bug: 28196843
Change-Id: Ia91e926da569bf9c503ea00e1e0f767d196547da

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