fix service binding

there were cases where we wouldn't bind when we should. also, add
some additional debug to better track potential connection issues.

Change-Id: I381da24f490e3102817a14428ef1a63982f33d19
Fixes: 37122153
Test: kill/upgrade GMS to ensure we re-bind in a timely manner

Merge "[DO NOT MERGE] Fix potential NPE in Trampoline.requestBackup()." into oc-dev

Merge "SmartSelection ConfigUpdater: Ignore version numbers" into oc-dev

[DO NOT MERGE] Fix potential NPE in Trampoline.requestBackup().

Before it was returning null in case the service was not initialized,
but as the method signature has int as the return type, this was boxed
and unboxed which would cause NPE.

Now it will return BackupManager.ERROR_BACKUP_NOT_ALLOWED instead.

This doesn't include fix for the test, because it's going to be
cherry-picked into oc-dev, which doesn't have the tests.

Test: manually ran unit tests

Bug: 37535747

Change-Id: I186cf812a4ec67be943b5c4334c82e9af5320b7a

Merge "Update Typeface callback docs about error codes" into oc-dev

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

Merge "Add local-only hotspot info into tether state change broadcast" into oc-dev

Merge "Don't trigger new partition on fields from SaveInfo" into oc-dev

Merge "Correctly compute visible views" into oc-dev

Add local-only hotspot info into tether state change broadcast

Test: as follows
    - build
    - flashed
    - booted
    - "runtest frameworks-net" passes
    - manually starting tethering shows Settings and icon updates
Bug: 31466854
Bug: 32163131

Change-Id: I938074587dfeec221c5cdb43a392802ad3fc3589

Merge "MediaPlayer/ModularDrm API review: Document/runtime exception update" into oc-dev

Merge "Implemented FillResponse.setIgnoredIds()." into oc-dev

Merge "fix KeyEvent can't correctly finish when inputmethod time out happend" into oc-dev

Merge "Stop restoring tasks added before recent is loaded" into oc-dev

Merge "media: Change setNextOutputFile to address API review comments." into oc-dev

Merge "Fix #37305009 : Beam crash in secondary users" into oc-dev

Merge "Auto-generate docs based on annotations." into oc-dev

Merge "Always create display content with system identity" into oc-dev

Merge "Update ClipDescription.getTimeStamp to use currentTimeMillis timebase." into oc-dev

Merge "OOM-adjust an app to the foreground for startForegroundService" into oc-dev

Don't trigger new partition on fields from SaveInfo

Fixes: 37551943

Test: existing CtsAutoFillServiceTestCases pass
Test: modified LoginActivityTest.testSaveOnly
Test: new LoginActivityTest.testSaveOnlyOptionalField

Change-Id: I2ae11c4f0aebe63fd02f81fbc37466435e3a2820

Merge "Fixed NPE on dump() and other minor fixes..." into oc-dev

Merge "Fix a11y text location in clipped views" into oc-dev

Implemented FillResponse.setIgnoredIds().

Fixes: 37424141

Test: existing CtsAutoFillServiceTestCases pass
Test: PartitionedActivityTest.testIgnoredFieldsDontTriggerAutofill

Change-Id: I09bd95230c45d2970080514e25cb53320de49b56

Merge "Renaming setAsyncExecutor to setExecutor" into oc-dev

Merge "browserIntent needs proper flags for PendingIntent" into oc-dev

Merge "Hide API to listen to a11y service state" into oc-dev

Import translations. DO NOT MERGE

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

Correctly compute visible views

addToSet needs the same set as parameters as it returns. Otherwise the
returned set it overrridden.

In this case only the last visible view was added as the set was
overriden with the invisible views.

Test: CtsAutoFillTestCases
Fixes: 37550229
Change-Id: I80e16cbd8a1af83373d7ea7b0f84a7c4d157119f

Merge "Kill running animations on fragment view's when removed." into oc-dev

Merge "Execute pending transactions after FM state change." into oc-dev

Merge "Added missing important-for-autofill mappings." into oc-dev

Merge "Deprecate BSSID Hostlist system API" into oc-dev

Always create display content with system identity

If an app tried to use a virtual display that it had just created
and add a window to it before it was registered in WM, WM would try
to create a DisplayContent instance. This would be executed with
app's calling identity and fail permission check.
This CL ensures that we always clear the calling identity before
creating display content.

Bug: 37422998
Test: android.display.cts.VirtualDisplayTest#testPrivateVirtualDisplay
Change-Id: I442cca65055886b384a28eeefcc35f2a36e482d0

Fix #37305009 : Beam crash in secondary users

The background-policy check for a-priori app uids needs to be an appId
check rather than nominal uid.

Along the way, even though the code is not invoked after the boot
sequence currently, make the "is this uid on the a-priori list?"
check thread safe.

Bug 37305009
Test: manual

Change-Id: I7c2525f86e73b213057cd4b7f327191ec20c4a6d

Auto-generate docs based on annotations.

We have a handful of annotations that we've been sprinkling across
the platform APIs, such as @Nullable, @NonNull, @IntDef, etc.  It
would be really helpful to surface these contracts to developers
through the SDK docs.

This change allows annotations like those mentioned above to declare
the following new javadoc fields:

@memberDoc: docs to append to a field or method definition.
@paramDoc: docs to append to a @param definition.
@returnDoc: docs to append to a @return definition.

This change also builds a docstring to describe the list of all
constants listed in an @IntDef annotation.  Sadly AnnotationDesc
only passes along raw constant values, so we need the help of the
new "prefix" annotation argument to help find the field names.

Test: builds
Bug: 37526420
Change-Id: I4cfc00dd904e5dfa945b406d546e36846b7c0c28

Fixed NPE on dump() and other minor fixes...

Bug: 36871500
Test: manual verification
Change-Id: I1c3a8406344280da37b728eae56f26447c48e0b2

Renaming setAsyncExecutor to setExecutor

Bug: 37534990
Test: Tests updated
Change-Id: I9ea467d05b4ac4d96c23d53750727096a511686d

Merge "Fix status bar not hiding quickly" into oc-dev

OOM-adjust an app to the foreground for startForegroundService

That is, OOM adjust it appropriately when the service is kicked off.
Previously it was mistakenly not recognizing the foreground intention
at the OOM level until the service called startForeground(), so there
was a period of inappropriate OOM vulnerability.

Bug 36990485
Test: manual

Change-Id: Ic9b56ef29d5120bf7912681900e9ebaeb0abc5d6

browserIntent needs proper flags for PendingIntent

Previously the browserIntent was updated after the call to
PendingIntent.getActivity. It should be updated correctly
before the PendingIntent.getActivity call so that the
PendingIntent is correctly created for the browser instead
of the Instant App.

Test: Existing tests, manually tested fixed behavior.

Bug: 35873804
Change-Id: Ie77344a4190fbc6a3896b02e3adfcdd29985006d

Merge "docs: Restoring file to allow staging reference docs from Gerrit" into oc-dev

media: Change setNextOutputFile to address API review comments.

Bug: 37514763
Test: CTS test
Change-Id: If0fb59f26ecf3882600ca7a8b26b6c248615bbd0

Merge "fix NPE when fragment container is null" into oc-dev

fix NPE when fragment container is null

Bug 37529822

Test: ran against breaking application

Change-Id: Ib829b817eddf2e91898989a2f9e7885a6f5847e5

Merge "Enhance the shortcut manager javadoc." into oc-dev

Merge "Address API review comments for registerReceiver" into oc-dev

Fix a11y text location in clipped views

Bug: 37482849
Test: Run Select to Speak, open apps, scroll so top row is
partially visible, have it read the screen.

Change-Id: Ibbd51221f8e93372ccff6d5fb57cdc2a09f2c19f

Merge "Stop fetching font data if the context is restricted." into oc-dev

Update ClipDescription.getTimeStamp to use currentTimeMillis timebase.

Bug: 37419581
Test: cts-tradefed run singleCommand cts-dev -m CtsContentTestCases -t \
      android.content.cts.ClipDescriptionTest

Change-Id: I699bfc073c471534eee0b3aa0300a6b6682ab8dd

Enhance the shortcut manager javadoc.

Test: make -j offline-sdk-docs
Bug 37418396

Change-Id: Ic82eb45ea232c2f032abaade12ac9ca967c35cae

Merge "Fix overlay view not updated during scaling TvView" into oc-dev

Merge "StopScan before each new Companion device scan" into oc-dev

Merge "Moved getAutofillOptions() logic from SpinnerAbs to AdapterView and Adapter." into oc-dev

Merge changes I8953967a,If17eddad into oc-dev

* changes:
  Add go to web action for instant apps
  Fix when instant app notif is showing

Merge "Use SubscriptionId instead of SubId" into oc-dev

Merge "Remove swipe to exit persistent Vr mode behavior" into oc-dev

Merge "Revert "Add new api Activity.getStartInitiatedTime."" into oc-dev

Merge "Do not pause resumed activity in resumeTopActivityUncheckedLocked." into oc-dev

Merge "Remove QS wifi alpha state animations, they need more work" into oc-dev

Merge "Fix touch target on QS date" into oc-dev

Merge "Fix QS label layout" into oc-dev

Merge "Fix alpha on wifi disconnected icon" into oc-dev

Merge "Fix alpha on wifi connected 4 bars no inet icon" into oc-dev

Merge "Revert "Revert "AudioAttributes: remove static getVolumeControlStream method""" into oc-dev

Merge "Stop hwbinder threads from app_process." into oc-dev

Merge "Redesign QSSecurityFooter dialog" into oc-dev

Merge "String fixes in QSFooter" into oc-dev

Fix status bar not hiding quickly

Appears to be from the delay of posting in the CommandQueue, skip
post when possible.

Test: Turn screen off the right back on.
Change-Id: I285b45544f32891ab86fd1f4a4aba3204bcbfdb4
Fixes: 36463611

Added missing important-for-autofill mappings.

Bug: 36171235
Test: m update-api
Change-Id: I7bbbe7a323110c89976cab3e4f5c8f4acba407d7

Remove swipe to exit persistent Vr mode behavior

This behavior is somewhat redundant now
* It is currently already possible to exit Vr mode in normal
circumstances by pressing the X in the top left of the screen.
* If a 'safety hatch' is needed, one can just press the power button,
which will require going through the Don flow again, which can then be
cancelled by pressing the X button.

As it is, the swipe behavior is likely just to cause confusion (and
currently, it puts the device in a bad state where the vr app is still
running, but the screen is blank except for the overlay). My proposition
is just to remove it and use the power button as the safe/guaranteed way
to exit vr mode.

Bug: 36965697
Test: Swiping no longer drops the screen to the overlay when in a
vrflinger application.

Change-Id: I461828f5dce67361a5f4d60c47ac7d6371a4efb3

Add go to web action for instant apps

Test: manual
Change-Id: I8953967ac849f5a5d4129c514172238e17d54df7
Fixes: 35873804

Fix when instant app notif is showing

Test: manual
Change-Id: If17eddad60f8207fecf9518430a280261bde5b30
Fixes: 37107497

Merge "Do not check user escrow state if synthetic password is not enabled yet" into oc-dev

Merge "Make a11y clickable span work after node recycle" into oc-dev

Merge "Vulkan compute 0 only requires variable pointers" into oc-dev

fix KeyEvent can't correctly finish when inputmethod time out happend

Symptom: If inputmethod handle an KeyEvent time out ,
Did not finish this event correctly , eventually lead to app ANR
Root Cause: without seq number when obtainMessage MSG_TIMEOUT_INPUT_EVENT
Solution: add seq parameter

Test: Input
Test: InputMethod
Bug: 28635475

Change-Id: I3a2b964b34aa57267bd777305200a2bfdc66f65c
Signed-off-by: gaoshang <gaoshang@xiaomi.com>

Merge "Deprecate Activity#requestVisibleBehind" into oc-dev

Do not pause resumed activity in resumeTopActivityUncheckedLocked.

A number of operations occur between setting the resumed activity and
notifying the activity to resume. One of these steps is updating the
configuration, which can cause the device to pause the resumed
activity. Since the activity has not been told to resume, this leads
to a pause before resume.

This changelist addresses the issue by checking whether we are in the
middle of resuming the top activity before attempting to stop the
currently set resumed activity.

Change-Id: I3b5b61de6b1b5a35b7773912b4e63d6c69096c04
Fixes: 35637717
Test: bit FrameworksServicesTests:com.android.server.am.ActivityStackTests#testNoPauseDuringResumeTopActivity

Merge "Listen to USER_STARTED instead of USER_ADDED from WebViewUpdateService." into oc-dev

SmartSelection ConfigUpdater: Ignore version numbers

These model files are updated when the user changes the device
language. When this happens, there is no guarantee that the new model
file (e.g. in English) has a higher version number than the old model
file (e.g. in Italian).

Test: none
Bug: 34780396
Change-Id: Ida70c5cd85fe246d4085a299711ad8838dafa1e9

Merge changes from topics 'smartselectconfig', 'langidconfig' into oc-dev

* changes:
  SmartSelection Config installer.
  LangId Config installer.

Remove QS wifi alpha state animations, they need more work

Test: visual
Change-Id: Icd0e5ef30f04ffc86ba28eee67ea2508ffc95c40
Fixes: 35041035

Merge "AutoSize TextView - API polish - no RuntimeException in signature" into oc-dev

Merge "TRON: Log "textAssist" actions." into oc-dev

Fix QS label layout

Test: visual
Change-Id: Ie334ffbef07549c9ab68fc663c25c0b4b63465ca
Fixes: 37439937

String fixes in QSFooter

Only one of the strings (monitoring_title_device_owned) needs to be
translated again. For the others, only apostrophes are fixed.

Bug: 34369166
Test: string-changes only

Change-Id: I58271393f4d8a83e55fa32885e3113845f5e4f5f
(cherry picked from commit 3e559e615359be06229a3ba9048e3b9bac5e4446)

Redesign QSSecurityFooter dialog

The dialog consists now of up to four sections: Device management,
CA certificates, Network Logging and VPN.

Also, small tweaks to the footer's coloring after the theme change of QS.

Bug: 34369166
Bug: 37492743
Test: in class QSSecurityFooterTest, which is part of "runtest systemui"

Change-Id: Ibf07a26fb6c5485a152f9d447f10c0f73acb94ba
(cherry picked from commit 0330f881447b053f88de6d30c99bfe4feed3c91d)

Merge "Add shell command for turning on/off WebView multiprocess." into oc-dev

Merge "Api Council: hide FontsContract.Columns constructor" into oc-dev

AutoSize TextView - API polish - no RuntimeException in signature

Do not mention RuntimeException subclasses in throws clauses in
method signatures.

Bug: 37514926
Test: cts-tradefed run cts-dev -m CtsWidgetTestCases -t \
      android.widget.cts.TextViewTest

Change-Id: Ia5bf4cd64e12c53dd7c409b6e44efec4983ae62e

Do not check user escrow state if synthetic password is not enabled yet

1. when adding escrow tokens, do not throw SecurityException if the user hasn't
   migrated to synthetic password yet.
2. during migration, activate any pending escrow tokens.
3. Disable escrow before attempting to activate escrow tokens, this currently
   includes whenever the user authenticates, and when synthetic password
   migration happens.

Also, add some comments in SyntheticPasswordManager and remove an invalid call
to destroy escrow data.

Test: runtest frameworks-services -c com.android.server.SyntheticPasswordTests
Bug: 36776133
Change-Id: If4b9d385fb43e09d90b6dde6d25b4fcc0c6b7ddc
Merged-In: If4b9d385fb43e09d90b6dde6d25b4fcc0c6b7ddc

Update Typeface callback docs about error codes

As requested by API council, mention that the provider may
return its own error codes within the positive int range.

Bug: 37084803
Test: none
Change-Id: Ia8ee99432cc5025a12d43049e327605095dd2b57

Api Council: hide FontsContract.Columns constructor

Bug: 37515256
Test: none
Change-Id: I2a9de7ea30ffa39af5104977f6604ce3bda220fb

Listen to USER_STARTED instead of USER_ADDED from WebViewUpdateService.

When adding a restricted user, 'hidden' applications are uninstalled for
the user after creation of the user, but before the user is 'started'.

Because WebViewUpdateService only listens to USER_ADDED and USER_REMOVED
intents it wouldn't realize that Chrome had been uninstalled for a new
restricted user (at the time of user-creation Chrome would be
installed). Instead listen to user-started/stopped intents to ensure the
restricted-user case is handled correctly.

Bug: 35813523
Test: Add a restricted user (on a Pixel C), with Chrome disabled. Start
an app using WebView (and ensure WebView doesn't crash).

Change-Id: I0c5d0a543b4bc80d269e586444b80cd3baffa11c

Merge "AOD: Fix broken triggers after failed prox check" into oc-dev

AOD: Fix broken triggers after failed prox check

Fixes an issue where after a failed prox check
we would not correctly reset mPulsePending and
thus never trigger a pulse again until the
DozeService restarts.

Also adds logging for dropped and canceled pulses
to aid diagnosing similar or different bugs
in the future.

Also adds a regression test.

Fixes: 37477968
Test: runtest -x /frameworks/base/packages/SystemUI/tests/src/com/android/systemui/doze/DozeTriggersTest.java
Change-Id: Ic465b7686ae1937a29d07a66fe44cf7ad18419ef

Merge "A11y: Better label status bar icons" into oc-dev

Merge "Add min and max learned battery capacity to batterystats." into oc-dev

Merge changes Ic029cc9f,Id917f3a2 into oc-dev

* changes:
  NsdService: minor cleanups
  Nsdmanager/NsdService: add logging