Merge "Fix fingerprint onEnumerationResult to use remaining instead of fid" into oc-mr1-dev

Merge "WebView: fix SafeBrowsingResponse javadocs" into oc-mr1-dev

Merge "Add package use info to dexopt dump" into oc-mr1-dev

Merge "Intern process names." into oc-mr1-dev

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

* changes:
  Allow to close session from wrong context.
  Prevent closing by prev owner after transfer.
  Revert "Revert session-transfer change"

Merge "Update FINGERPRINT_ERROR_LOCKOUT documentation" into oc-mr1-dev

Merge "Add virtual preload bit to ApplicationInfo" into oc-mr1-dev

WebView: fix SafeBrowsingResponse javadocs

Docs change only.

This fixes javadoc errors in SafeBrowsingResponse.java:

 * Typo: capital "B" in "onSafeBrowsingHit"
 * Add full package names
 * Add parameter lists to all linked methods

Bug: 64077668
Test: make update-api and check javadocs manually
Change-Id: I5e7b0e59ba865619b252f7e9e431ffac6e1bab1f

Add package use info to dexopt dump

Bug: 64153885
Test: adb shell dumpsys package dexopt
Change-Id: I57768076de821980b1de84d5fe44ab642ceebeba

Merge changes from topic 'info-per-split' into oc-mr1-dev

* changes:
  Record usage information per split
  Use PackageUseInfo in DexOptimizer

Merge "Remove logic for parsing dismiss rule" into oc-mr1-dev

Intern process names.

Bug: 62144301
Test: Boot and start random apps on a secondary user
Change-Id: I8a9b475410c52e1063cff5519b0297ad69dd7925

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

Add virtual preload bit to ApplicationInfo

Change-Id: I2735b3823a8709b2ffb65cc8085ffcd952d3e1f2
Fixes: 64205417
Test: Manual
Test: Create a sample app and install it as a normal app
Test: See that it returns 'false' for "isVirtualPreload"
Test: Create a sample app and install it as a virtual preload ["--preload"]
Test: See that it returns 'true' for "isVirtualPreload"
Test: Run sample apps after reboot and see they return the correct value

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

Allow to close session from wrong context.

commitLocked runs on a handler, thereby as system. It will call close()
if the package installer is popped-up to confirm the installation. Hence
we have to allow this use case without checking the caller.

Bug: 64467704
Test: gts-tradefed run singleCommand gts -m GtsPackageInstallSessionTestCases
Change-Id: I0bd0329c0485c50afa459d90c3372d7bf7408287

Prevent closing by prev owner after transfer.

Bug: 64467704
Test: gts-tradefed run singleCommand gts -m GtsPackageInstallSessionTestCases
Change-Id: I81fb1b641295cbb9e36504eb3e40f51ce6b2de94

Revert "Revert session-transfer change"

This reverts commit 9890f8b426550485aaab164a7bedbcd545862b85.

Bug: 64467704
Test: cts-tradefed run singleCommand cts-dev -m CtsContentTestCases --test=android.content.pm.cts.InstallSessionParamsUnitTest
      cts-tradefed run singleCommand cts-dev -m CtsContentTestCases --test=android.content.pm.cts.InstallSessionTransferTest
Change-Id: I0cc7e1129d85e4d0de16ade44232a9bd381d6c04

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

Fix fingerprint onEnumerationResult to use remaining instead of fid

Fixes: 64087535

Test: make -j56, this code isn't used until we figure when to do
b/63606218

Change-Id: I4c8804e71695ffc46e9b70c7f623e14e420f6242

Record usage information per split

Increase the granularity of usage information to store data on each split
separately.

Now, splits get their own useByOtherApps flag and can be compiled
speed-profile when only the primary apk is loaded by other apps.

Bug: 64124380
Test: runtest -x
services/tests/servicestests/src/com/android/server/pm/dex/*

Change-Id: Ibf9e7b9e67db9c6f0f45dc695bce8fbeb7be20ae

Use PackageUseInfo in DexOptimizer

Pass the PackageUseInfo directly to DexOptimizer and use it to detect if a
package is used by other apps. Move the usage checks closer to dexopt so
that they can be easily adapted when we add usage info for each of the
app's code paths separately.

This is a refactoring CLs to reduce the size and complexity of the
upcoming CLs which record the usage info for each of the application
splits.

Bug: 64124380
Test: runtest -x
services/tests/servicestests/src/com/android/server/pm/dex/*

Change-Id: I8031590cdaff81ab1792ca19baddb6cb36dc021d

Update FINGERPRINT_ERROR_LOCKOUT documentation

Fixes: 64330785

Test: make -j56
Change-Id: I40f0e71b9fd7dcdd12340bc03bb077e56813f5b7

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>