Merge changes I6de5ac05,I5f8c112d into oc-mr1-dev

* changes:
  Fix system restart due to race in TSMS
  Remove redundant synchonized blocks from TSMS

Merge "Do not remove uninstalled packages from journal" into oc-mr1-dev

Merge "WebView: update SafeBrowsing documentation" into oc-mr1-dev

Merge "Don't retain a bitmap unnecessarily" into oc-mr1-dev

Merge "Mark profile password key as critical." into oc-mr1-dev

Merge "Revert session-transfer change" into oc-mr1-dev

Merge "Refactor WifiTracker sStaleScanResults." into oc-mr1-dev

Merge "Shutdown: Continue shutdown with BLE only mode" into oc-mr1-dev

Merge "Update string for not charging." into oc-mr1-dev

Don't retain a bitmap unnecessarily

Due to a limitation in the GC, LazyTaskWriterThread would always retain
the last item it processed.  Extract the second half into a separate
method to clarify the scope of the variable 'item' to the GC.

Bug: 64438652
Test: Launch several apps and check with ahat
Change-Id: Ie7357927ae61a8731285d14743187a2d811fbf8c

Merge "Cherry-pick: Add null check for Wi-Fi command" into oc-mr1-dev

Revert session-transfer change

This reverts:
5d74493f64493cd506d7458b810095259287f623
94deaf7725c418ec1950d810ab86f0d157ddf518

Bug: 64467704
Test: none, 100% revert
Change-Id: I6534c19c594dadfa41457725988485a67f0ed45f

Merge "API Review update for SQLiteDatabase" into oc-mr1-dev

Refactor WifiTracker sStaleScanResults.

In ag/2580164, in order to minimize changes, we used mutable static
state in WifiTracker to control cache eviction. This breaks a certain
flow in SetupWizard. This change reverts sStaleScanResults to an
instance member and then pipes the value into the AccessPoint update
call to control cache eviction instead of relying on error prone use of
mutable static state.

Bug: 63479352
Test: runtest --path
frameworks/base/packages/SettingsLib/tests/integ/src/com/android/settingslib/wifi/AccessPointTest.java
&&
runtest --path
frameworks/base/packages/SettingsLib/tests/integ/src/com/android/settingslib/wifi/WifiTrackerTest.java
Manual: See bug

Change-Id: Ia6d79c1904294da69841cbdf6eafbf42fa70f2d0

Merge "Always populate the ApplicationInfo object" into oc-dr1-dev
am: bb6653c404

Change-Id: I4e0dbda0e13e5419abfa25e2719d8fdcea0fa047

Mark profile password key as critical.

Currently it is erased when keystore for user 0 is cleared, e.g.
when the user clears data for Settings app.

Bug: 64467610
Bug: 35929605
Test: Manually cleared Settings data.
Test: Manually cleared credentials from Settings.
Change-Id: I7e8753a1bf53f5d68d4738a4eb84faa890f026cc

Merge "Always populate the ApplicationInfo object" into oc-dr1-dev

Merge "Merge "NetworkStats: more robust subtraction for deltas" into oc-dr1-dev am: 5e24152e33" into oc-mr1-dev

Merge "NetworkStats: more robust subtraction for deltas" into oc-dr1-dev
am: 5e24152e33

Change-Id: Id6040a899653c585d298b7be7208f5cba997f665

Merge "Add native bindings for getNamesForUids" into oc-mr1-dev

Merge "NetworkStats: more robust subtraction for deltas" into oc-dr1-dev

Merge "Device Default theming updates for Wear" into oc-mr1-dev

Merge "Update colour to match toasts." into oc-dr1-dev
am: 91866c65a5  -s ours

Change-Id: I40dd9ae63195b5f00741facc5b589f66513f1015

Merge "Update colour to match toasts." into oc-dr1-dev

Merge "FRP: Enable credential-based factory reset protection by default" into oc-mr1-dev

Merge "AOD: Follow-up to I0fa53616db09be98d33ea7d06b38d8ff3037707f" into oc-dr1-dev
am: d58c6978a7

Change-Id: I077bc4252e9129960d20109920aa5783c46e0e2d

Merge "AOD: Follow-up to I0fa53616db09be98d33ea7d06b38d8ff3037707f" into oc-dr1-dev

Merge "ScrimController: Wait with clearing wakup animation until done" into oc-dr1-dev
am: 6e4de6ccdf

Change-Id: I55938f50f29289e1dcc51be18561d0819caa2b9b

Merge "ScrimController: Wait with clearing wakup animation until done" into oc-dr1-dev

Merge "Recents scroll backwards (accessibility) if clear all button invisible" into oc-mr1-dev

Merge "Scrolling recents to next/prev page focuses next/prev node" into oc-mr1-dev

Merge "system_server : trigger runtime restart when we're close to the soft FD limit." into oc-mr1-dev

Merge "app link handling under restricted mobile data" into oc-dr1-dev
am: 93bf6200eb

Change-Id: I1a506e9f8dae6fd76c74c3f9d7f93c40ce64a387

Merge "app link handling under restricted mobile data" into oc-dr1-dev

Shutdown: Continue shutdown with BLE only mode

Bug: 64213593
Test: shutdown no timeout
Change-Id: I5007e7cfc00ab22fce3b6ef13bacbcc951197b9e

Merge "Start ARC action when setSystemAudioMode" into oc-mr1-dev

Merge "Bump parcel version to reset batterystats" into oc-mr1-dev

Merge "thermalservice: Add ThermalService java interfaces" into oc-mr1-dev

Merge changes from topic 'TransferInstallSession-oc-mr1-dev' into oc-mr1-dev

* changes:
  Allow to read all params used to create session.
  Allow to transfer+seal a install session

Merge "Reset wait for transition state to false when recents goes invisible" into oc-dr1-dev
am: 673e3c41ec

Change-Id: Iedcf8f27eacdba8ad96a4753fb108f4539930fb9

Merge "Revert "Revert "Fixing occasional jump when launching certain apps from Recents.""" into oc-dr1-dev
am: c7d24aed82

Change-Id: I96ed9a5f7f295b1a3f3a9199547945e4958a34a3

Merge "Change VENDOR program types to range." into oc-mr1-dev

Merge "Reset wait for transition state to false when recents goes invisible" into oc-dr1-dev

Merge "Revert "Revert "Fixing occasional jump when launching certain apps from Recents.""" into oc-dr1-dev

NetworkStats: more robust subtraction for deltas

This patch ensures that subtract() between two NetworkStats object will
return a delta with no negative entries in all cases.

When the stats delta contains some negative values, there are clamped to
0. Some logging is added when this happens.

This is what's expected by NetworkStatsHistory#recordData().

Bug: 64365917
Test: runtest frameworks-net
Change-Id: I16e97e73f600225f80e0ce517e80c07c6f399196

Merge "Change the location of current profiles for secondary dex files" into oc-mr1-dev

Merge "Minor Autofill changes after API council review." into oc-mr1-dev

Merge "Track per-cluster times of each uid in microsec." into oc-mr1-dev

Merge "Catch (and ignore) any timeout exceptions when releasing WakeLock" into oc-mr1-dev

Merge "Package wakeup alarms are now on screen-off timebase" into oc-mr1-dev

thermalservice: Add ThermalService java interfaces

Binder service ThermalService broadcasts thermal events to registered
listeners.

Test: manual: marlin with modified thermal-engine.conf and temporary
      java thermal event listener
Bug: 30982366
Change-Id: I11f7fd18feff3b9af0eecc3fd3a13d54c0b97ff0
(cherry picked from commit dd05582eabb6809899e3187d3afc1e97e148fdb1)

Merge "Bind SpellCheckerService as IMPORTANT_BACKGROUND" into oc-mr1-dev

Merge "Add black frame behind app window" into oc-dr1-dev
am: cbdaf04101

Change-Id: I9ce5cf5aea7f0fd93fbe24664bbd583d5057d3e6

Merge "AAPT2: Add targetConfig to AndroidManifest.xml" into oc-mr1-dev

Merge "AAPT2: Ensure style strings are always first in StringPool" into oc-mr1-dev

Merge "Add black frame behind app window" into oc-dr1-dev

Scrolling recents to next/prev page focuses next/prev node

Recents did not report that it finished scrolling to Talkback so it did
not focus the next item. Now sending the accessibility event when
scrolling will focus the next node.

Test: talkback on, open recents with tons of apps, scroll to bottom app,
keep swiping left until multiple tasks move down
Bug: 37005488
Change-Id: I07e8d686e7cc0f1a43f13769181d4c7cb87853e8

Merge "media: Address api council comments." into oc-mr1-dev

Catch (and ignore) any timeout exceptions when releasing WakeLock

Bug: 64424030
Change-Id: I4f1f71bc4f16d1485b360aec70c82818aa7510fb

Merge "Only replace new line characters if the wakelock name contains one The original operation creates new string everytime which is expensive" into oc-mr1-dev

Merge "Reduce duplicate strings due to the package cache." into oc-mr1-dev

Cherry-pick: Add null check for Wi-Fi command

When WifiService is not ready during boot phase and Wi-Fi command
is executed on console, NullPointerException happens.
To prevent this, null check is needed.

Test: Manual test
Bug: 64040515
Change-Id: I5684cdc7661b7d93fd6f2ac61863919077d12935
Merged-In: I5684cdc7661b7d93fd6f2ac61863919077d12935

Merge "Revert "PackageManager: update documentation for FEATURE_WIFI_PASSPOINT"" into oc-mr1-dev

Revert "PackageManager: update documentation for FEATURE_WIFI_PASSPOINT"

This reverts commit 84933321668c79202de39ea491329f7234ce6ee5.

Change-Id: Ibecf24689952d4888269863ff02d8c97e14a0c31

Device Default theming updates for Wear

- aligning styled attributes with those defined in O RC
- reduce coupling to/mutation of the material theme
- includes all previously customized values (i.e. WTS neutral)

Test: should be no visible difference (changes are focused
      on simplifying resources and their inheritance via theme
      definitions, as well as explicitly defining additional
      resources not currently used in the application layer).

WTS OemColorsTest should pass:
run wts --module WtsOemConfigDeviceTestCases
(no failure on colors... doesn't mean no failures)

Bug:b/63525308

Change-Id: I5c3c3b848e0090169b884ec005113aeac2560993

Merge "AOD: Fix shelf animating when it should not" into oc-dr1-dev
am: 2b7fbd010e

Change-Id: I2956ecaff7b471af05e9baa275d7ea0742bfb4eb

Merge "AOD: Fix shelf animating when it should not" into oc-dr1-dev

Reset wait for transition state to false when recents goes invisible

From ag/2572257, this change will ensure the transition wait flag to be
false when recents runs "onStop" or when it becomes invisible. This
should fix edge cases when a user goes from Recents to somewhere that
is not a fullscreen app (like home). Resetting the state should allow
users to use the recents button without getting blocked.

There is only two locations where the flag is set to true, when user
starts recents and when an app is started from recents. When a user goes
from recents to home with the recents button (when the phone gets into a
wierd state), then the flag is true but home does not reset the flag
because the old change expects a non-home app to start. Now whenever
recents is dismissed, the flag will be reset.

Fixes: 37277853
Bug: 64098508
Test: start recents with apps in it, go to guest account, start apps in
recents, go back to main account, unlock keyguard, press recents button
to dismiss recents (do not press home or back), it will go to home,
press recents again
Change-Id: I07c292d701a621685c836a4f6040864377f0a3dd

Merge "PackageManager: update documentation for FEATURE_WIFI_PASSPOINT" into oc-mr1-dev

API Review update for SQLiteDatabase

SQLiteDatabase openDatabase -- should take File for first parameter
  instead of a String path

SQLiteDatabase.OpenParams.Builder -- make sure the javadocs says what
  the default openFlags is and default idle connection timeout

SQLiteDatabase createInMemory -- throw if it has trouble instead of
  returning null.

Test: cts/SQLiteDatabaseTest
Bug: 64331777
Bug: 64331778
Bug: 64330914

Change-Id: Ibecf4f4a6498795f9a5d12b94b77481e5745b523

Allow to read all params used to create session.

So that they can be verified by the app calling commit().

This really only makes sense if the app calling commit is different from
the app that created the session.

Bug: 37281396
Test: cts-tradefed run cts-dev -m CtsContentTestCases --test=android.content.pm.cts.InstallSessionTransferTest
      Installed and uninstalled packages via the PackageInstaller app

Change-Id: I5c954ca59b7582555bea847f3ddbba0aeefba301

Allow to transfer+seal a install session

... so that one package can supply the data and another one can issue
the commit.

Also allow reading of sealed sessions.

Also lock more in PackageInstallerSession so that we can be sure the
session is not used by the old package anymore once transferred and that
all calls into the session work on consistent data.

Bug: 37281396
Test: cts-tradefed run cts-dev -m CtsContentTestCases --test=android.content.pm.cts.InstallSessionTransferTest
      Installed and uninstalled packages via the PackageInstaller app
      Installed and uninstalled packages via the Google Play Store

Change-Id: Id4b7a0071d703b7d18c9f5bf2bd15ebf67086d07

Merge "AAPT2: Disable locale domination for deduping" into oc-dr1-dev
am: 242c22c900

Change-Id: I8be36dcf67b6d32f69fbefdf2bd78e04034f6bb8

Merge "AAPT2: Disable locale domination for deduping" into oc-dr1-dev

AOD: Follow-up to I0fa53616db09be98d33ea7d06b38d8ff3037707f

Address reviewer's requests on I0fa53616db09be98d33ea7d06b38d8ff3037707f.

Bug: 64287908
Test: make SystemUI
Change-Id: I804c9408742cc5bd1d69fd5fd8ad42074953b0f6

Change VENDOR program types to range.

Bug: b/64331420
Test: instrumentation
Change-Id: If5c56f09e2b9e272a40f0ca074a590ca6ec3a703

Location: adding NULL check for the LPP_PROFILE property
am: 8eec744ccb

Change-Id: Ia72f80b5d809190c7e2f733c8e59bf989f826973

Fix system restart due to race in TSMS

This is a follow up CL to my previous CL [1], which aimed to get rid of
sync IPCs from TextServicesManagerService (TSMS) to
SpellCheckerService (SCS).

What was overlooked is that mSomeArrayList.forEach(action) throws
ConcurrentModificationException when the specified action adds or
removes items to/from mSomeArrayList.  This can happen in my previous CL
when

 1. An app requests a new spell checker service.
 2. TSMS puts the request to mPendingSessionRequests and launchs the
    current SCS by binding to it.
 3. SCS#onCreate() succeeds.
 4. TSMS receives onServiceConnected() callback.
 4. SCS crashes.
 5. TSMS tries to dispatch pending requests to the SCS as follows
    mPendingSessionRequests.forEach(this::getISpellCheckerSessionLocked)
 6. In getISpellCheckerSessionLocked() the IPC to the SCS fails because
    the target SCS process is already dead.  This triggeres removeAll(),
    which clears mPendingSessionRequests.
 7. ConcurrentModificationException is thrown, which will never be
    caught and results in system restart.

Essentially this is a timing issue, but you can most likely reproduce
the issue by manually adding a certain delay in
SpellCheckerBindGroup#onServiceConnected() as follows.

  public void onServiceConnectedLocked(ISpellCheckerService spellChecker) {
      if (DBG) {
          Slog.d(TAG, "onServiceConnected");
      }

 +    try {
 +        Thread.sleep(100 /* msec */);
 +    } catch (InterruptedException e) {
 +    }
      mSpellChecker = spellChecker;
      mConnected = true;
      // Dispatch pending getISpellCheckerSession requests.
      mPendingSessionRequests.forEach(this::getISpellCheckerSessionLocked);
      mPendingSessionRequests.clear();
  }

Then you can also emulate the SCS crash as follows.

  public class CrashingSpellCheckerService extends SpellCheckerService {
      @Override
      public void onCreate() {
          super.onCreate();
          new Thread(new Runnable() {
              @Override
              public void run() {
                  try {
                      Thread.sleep(10 /* msec */);
                  } catch (InterruptedException e) {
                  }
                  Process.killProcess(Process.myPid());
              }
          }).start();
      }

      ...
  }

This CL addresses the above issue by simply inlining
getISpellCheckerSessionLocked() to each call site to make it clear what
will happen in RemoteException is thrown.

 [1]: I92e7aa40dc9ea14f67d355f0bfa15325b775d27b
      7fa65eef203c4ed3ce00ddef96ccf311d3bfb58c

Test: Manually made sure that without this CL the system can restart
      with the above technique.
Test: Manually made sure that AOSP spell checker service still works
      as expected.
Fixes: 63452807
Change-Id: I6de5ac0507574f28a9859b50d081378112f1f7df

Remove redundant synchonized blocks from TSMS

As a preparation to fix Bug 63452807, this CL cleans up some redundant
synchonized blocks in TextServicesManagerService (TSMS).

This is purely a code clean-up that has no behavior change.

Bug: 63452807
Test: compile
Change-Id: I5f8c112d72dad46297b59da623f066fa944c1a27

Location: adding NULL check for the LPP_PROFILE property

Bug: 35485906

Change-Id: I5e40be8ce0d3832f0e913bfe8fd9a38d87d0d28c
Signed-off-by: Ecco Park <eccopark@google.com>

Merge "Move getNetworkCountryIso impl from TM to PIM." into oc-dr1-dev
am: 35a563ef62

Change-Id: Ib898af17ce2985a1e19759dd9296f3cea8077018

Merge "Move getNetworkCountryIso impl from TM to PIM." into oc-dr1-dev

Merge "Allow marking fields with @TargetApi(XX)" into oc-mr1-dev

FRP: Enable credential-based factory reset protection by default

Bug: 64436700
Change-Id: I3aed6313e3b6b7e6d6e65c9e822ff123b8cd52cd
Test: Set up PIN, add Google Account, adb shell reboot-bootloader; fastboot -w; verify credential-based unlocking is possible using the PIN from before factory reset

system_server : trigger runtime restart when we're close to the soft FD limit.

We arbitrarily define a high water mark as 12 below the max limit (1024)
and dump the list of open descriptors and restart the system_server when we
reach that mark.

The list of file descriptors is dumped to a file with prefix /data/anr/anr_fd_xxxxx.
This might be construed as a hack, but it allows us to take advantage of
existing code in dumpstate that will capture this file and add it to any
bugreport that's collected after.

Test: Manual
Bug: 63004717
Change-Id: I4052625574a3ab2df9ddf591f281a412e7b4b511

ScrimController: Wait with clearing wakup animation until done

Fixes an issue where if another animation was started during
the wakeupFromDoze animation, the scrim would flash white
because the original animation got canceled. Instead now wait
until the animation is actually completed, not just altered.

Change-Id: I7738ec838d77f78dde2c5810369c50b4fafaeb67
Fixes: 64412754
Test: Trigger assistant from AOD; verify there is no white flash.

AOD: Fix shelf animating when it should not

Fixes an issue where the shelf would animate downwards when going to AOD.

The issue is that setDark gets called in updateIsKeyguard even though nothing changed,
yet it still re-requests the dark transition. Consequently, when the keyguard goes dark
later, the requested transition executes and moves the shelf downwards.

Change-Id: I8c590aeb4f330d8c3c1b51285d899d5f6db36323
Fixes: 64412244
Test: Double tap on notification on AOD2; press power; verify shelf does not move downwards

Merge "AOD: Fix launching notifications from occluded AOD" into oc-dr1-dev
am: 9f3bac52ee

Change-Id: I8c17429ce2b79a3e14e7b6cbabf7b377ace44718

Merge "AOD: Fix launching notifications from occluded AOD" into oc-dr1-dev

Merge "AOD: Never show bouncer when dozing" into oc-dr1-dev
am: 81e3b48f9c

Change-Id: I6f0ae86412eb174474d640c437e0b0b006507c00

Merge "AOD: Never show bouncer when dozing" into oc-dr1-dev

Merge "Defer updating dim layers if layout is deferred" into oc-dr1-dev
am: 9aa43e6320

Change-Id: I0889a00cbdc54d401484391d66e0dd06ac92fd71

Merge "Fix light bar flickers" into oc-dr1-dev
am: 997f2f71fa

Change-Id: I8b5fb57655afe1b9cab6afe3f5f34d48413af062

Merge "Defer updating dim layers if layout is deferred" into oc-dr1-dev

Merge "Fix light bar flickers" into oc-dr1-dev

Merge "screencap: always exit with _exit()." into oc-dr1-dev
am: 4499576db0  -s ours

Change-Id: I6119b3e45a4b104aa9a7cbad576a6b3d8a273ca4

Merge "screencap: always exit with _exit()." into oc-dr1-dev

Merge "Avoid entering orientation changing state when seamlessly rotating." into oc-dr1-dev
am: 17844f2257

Change-Id: Ic1d7779f66958961da22230187f71310d9fedec8

Merge "Avoid entering orientation changing state when seamlessly rotating." into oc-dr1-dev

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