Merge "Remove newly added remove methods in Location and deprecate the old ones, this is the request from the android API council Test: Existing unit tests still pass. BUG: 36552859" into oc-dev

Merge "update docs per API council" into oc-dev

Merge changes I062b6c37,Ie5165364,Ib3cce253 into oc-dev

* changes:
  Add remaining default media icons.
  Disable PiP when a task is locked.
  Fix issue with PiP actions flashing.

Merge "Hide Socket+DatagramSocket IpSecTransform methods" into oc-dev

Merge "better defualt behaviour for msim" into oc-dev

Merge "Consistenly Throw IOExceptions from IpSecManager" into oc-dev

Merge "Update Overview content descriptions." into oc-dev

Merge "Clone autofill service settings on managed profile." into oc-dev

Merge "Add logging to WebView zygote package preload." into oc-dev

Merge "guard against nulls in LogMaker" into oc-dev

update docs per API council

Change-Id: Id32be4db7671d3ef99e155f5fc2f3946eac707dd
Fixes: 37322490
Test: treehugger

Merge "Change nullability of colorspace parameter" into oc-dev

Add remaining default media icons.

- Fix issue where strings were only in overlay and not in base resources

Bug: 36778575
Test: Launch PiP activity with media session, ensure actions show
Change-Id: I062b6c37ff4789bf375670e8ce872f28c578ffca

Merge "Use wifi icon for wifi devices in companion chooser" into oc-dev

Remove newly added remove methods in Location and deprecate the old ones, this is the request from the android API council
Test: Existing unit tests still pass.
BUG: 36552859

Change-Id: Ib4cdcef90be3d5b3be329dd780bcb40275b89ef4

Merge "Part of global strings change from cellular —> mobile." into oc-dev

Merge "Ignore nested keyboard navigation clusters" into oc-dev

Merge "Change saveOnAllViewsInsible from boolean to flags" into oc-dev

Merge "Fixed WTF that happens on race condition." into oc-dev

Clone autofill service settings on managed profile.

Test: manual verification
Change-Id: I9d13e4230265b18444bfd30fd403c5fb98b88159
Fixes: 37351722

Merge "update docs per API council" into oc-dev

Merge "Re-hide shader mutation, remove colorfilter mutation" into oc-dev

Disable PiP when a task is locked.

Bug: 37135610
Test: Start PiP, enable screen pinning, ensure that PiP is dismissed.
Change-Id: Ie5165364db891b13dd4ef282089f915fe9f99615

guard against nulls in LogMaker

Bug: 36909905
Test: runtest --path frameworks/base/core/tests/coretests/src/android/metrics
Change-Id: Ibd12bf12f9b1ed5fbc926ec04bbfa429ff98f7d2

Merge "Adjust some of the timings around animating / showing the dismiss gradient" into oc-dev

Merge "Allow at most one pinned stack task." into oc-dev

Add logging to WebView zygote package preload.

This is a potentially slow event and having a timestamp for when it
started/completed may help to track down a currently-open bug (as well
as enable us to see how long preloading took). This only happens once
per boot and once per WebView update so won't cause a significant amount
of logging.

Bug: 37347817
Test: check for messages in logcat on first use of WebView
Change-Id: Ib6c50eea71324f81cda5b5b1ac91f9dc806e0ab4

Use wifi icon for wifi devices in companion chooser

Test: Ensure wifi devices have wifi icon
Change-Id: I08c9fba1acaefb097a010b1150cc052d5a397adc

Part of global strings change from cellular —> mobile.

Bug: 33049102
Fixes: 36389122
Test: no
Change-Id: I0438f63f643d144e065b8394bec36fb6e1ec7aa9

Merge "Minor UI adjustments to a11y button chooser" into oc-dev

Merge "Fixing javadoc errors for pm#canRequestPackageInstalls" into oc-dev

Change nullability of colorspace parameter

Bug: 37323693
Test: android.graphics.cts.BitmapColorSpaceTest
Change-Id: I705ac18e5c64c028c2e8366a18ec4ef20f44a630

Merge "Check if singleLineTitle attribute is set for the preference." into oc-dev

Fix issue with PiP actions flashing.

- Instead of recreating the whole layout, only add/hide action views as
  necessary

Bug: 35467434
Test: Launch PiP, tap some actions
Change-Id: Ib3cce253228d737bb699f80298001f08121640e4

Merge "Move TextClassificationManagerTest to coretests" into oc-dev

DO NOT MERGE Mask case of base or tree permission being null am: cb8a640cef
am: 28fc6a28aa  -s ours

Change-Id: I3e83332cd4bd7e51a3394ee5de61afc613fe9552

Merge "[AWARE] Implement custom Wi-Fi Aware network specifier" into oc-dev

DO NOT MERGE Mask case of base or tree permission being null
am: cb8a640cef

Change-Id: I43289a33b8e1e6df9307cd11891feff4d739da14

Re-hide shader mutation, remove colorfilter mutation

Bug: 36025103
Bug: 37327129
Test: boots, CtsGraphicsTestCases passes

Change-Id: I937f8ae26ed47b4811dc033ed91c257670fc3a39

Merge "Decrease number of network logging notifications shown" into oc-dev

Adjust some of the timings around animating / showing the dismiss gradient

Dan wanted longer + linear fade in of the dismiss gradient. Also wanted
fade out to match the timing of 'snap to edge'.

Test: Manual -- move PIP around, observe how gradient fades in / out
Bug: 35358628
Change-Id: I847d96e70184c0093e64a7e2e895a9c5f30fea8b

Merge "Fixed some pip state acitvity accounting." into oc-dev

Merge "Lock all the things" into oc-dev

Update Overview content descriptions.

- If there is a task label with a provided task description, ensure that
  it is used instead of the activity label when generating the content
  description (the concatenation of the badged application name and
  activity label.
- Also fix a case where content descriptions can go stale if they are
  updated between first preloaded. Just don't cache the content
  description if there is a task description.

Bug: 37417229
Test: Launch Play store, go to Overview
Change-Id: Ieeef9ecd93759c130dc66902f249783e7d218be7

DO NOT MERGE Mask case of base or tree permission being null

Bug: 34850544
Change-Id: I4bf9e85d7e615ccdb75752503e58d7e7f47b04c2

Merge "MediaPlayer/ModularDrm API review" into oc-dev

Lock all the things

Fix some instances of .*Locked() methods being called without
holding mNotificationLock.

Also, if this happens again, don't crash system server

Change-Id: I0a51310a6f6c8b07266092176e189e7cd08778cc
Fixes: 37444864
Test: runtest systemui-notification

Allow at most one pinned stack task.

This changelist enforces only one task may be present in the pinned
stack at a time. If a task is already persent, the existing task is
moved to the fullscreen stack.

Fixes: 36844394
Test: cts/hostsidetests/services/activityandwindowmanager/util/run-test CtsServicesHostTestCases android.server.cts.ActivityManagerPinnedStackTests#testPipMovedToFullscreenStack
Test: bit FrameworksServicesTests:com.android.server.am.ActivityStackSupervisorTests#testDisallowMultipleTasksInPinnedStack

Change-Id: Iaf0fbda6df835d93738fdf6f7f3a8c5956c2b262

Fixed some pip state acitvity accounting.

- Corrected ActivityRecord.checkEnterPictureInPictureState() to return
false if the activity doesn't support Pip.
- Wait for previous activity that supports pip to pause before resuming
the next activity with FLAG_RESUME_WHILE_PAUSING so that the activity
has a chance to enter Pip before we resume the next activity. Currently
disabled until we figure-out if callers are passing in the right value
for prev.
- Changed stopActivityLocked() to call setVisible() instead of
setVisibility()...yeah this isn't confusing...

Change-Id: If2336ece4fe5b7104c9b301a4ded5cadc2c2f7cb
Bug: 37370508
Test: manual

Merge "Don't let toasting throw a runtime exception." into oc-dev

Don't let toasting throw a runtime exception.

Not sure why is complaining about the Looper.
(The toast is debug code that will be removed for ship.)

Change-Id: I5e2ecbe40687cd626b235e248fe6d12fea14204a
Fixes: 37455183
Test: runtest systemui-notification

Change saveOnAllViewsInsible from boolean to flags

Change-Id: Ibd6f283479bad841eb7b20bb8d006562c4398e9f
Fixes: 35708237
Test: CtsAutoFillServiceTestCases

Merge "Remove round rect clipping optimization." into oc-dev

Merge "Log screen-on latency with tron" into oc-dev

Merge "Vol key presses: active stream has priority of set control stream" into oc-dev

Merge "AutoSize TextView - take horizontalScrolling into account" into oc-dev

Move TextClassificationManagerTest to coretests

Test: bit FrameworksCoreTests:android.view.textclassifier.TextClassificationManagerTest
Bug: 37296604
Change-Id: I8f53032dd63db18d98c6e26ee5689a5dafb3d98b

Merge "Deprecate ActivityManager#getRunningServices" into oc-dev

[AWARE] Implement custom Wi-Fi Aware network specifier

Convert Wi-Fi Aware connectivity requests to use a custom
network specifier (as opposed to a string). Aside from
simplifying code, allows for more control of information
printed out, and of additional information to be packaged
and used in network decisions (uid - TBD).

(cherry-pick of commit a5028455a4b7592091f12c80d4b4d42c9c866d66)

Bug: 36275276
Bug: 36053921
Test: unit tests and integration (sl4a) tests passing.
Merged-In: I776fb348b15ec4830f890a8ac5c41f24594e12a7
Change-Id: I776fb348b15ec4830f890a8ac5c41f24594e12a7

Decrease number of network logging notifications shown

Now, network logging will show one notification when it is enabled
and one after the next reboot.

Bug: 36254499
Test: CTS Verifier > Managed Provisioning > Device Owner Tests
      > Network Logging UI

Change-Id: I60fc64e96ceb0ec0ae7ca832b74ac8b47e581be4
(cherry picked from commit 55dba53ed433d713a075ba0de15504a1ed42852b)

Merge "FloatingToolbar: Rules for ordering menu items." into oc-dev

Merge "Flatten the transitions to tethering and hotspot." into oc-dev

Merge "Unhide MethodHandles.explicitCastArguments()" into oc-dev

AutoSize TextView - take horizontalScrolling into account

When passing the characteristics of the current TextView
to the StaticLayout for auto-size measurements also take
the horizontallyScrolling into account similarly with how
the actual measurements are done (@ TextView#onMeasure).

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

Change-Id: I44b85e77d9b1770f188a0f59f103e3fda284e97a

Merge "Renamed ViewNode's getAutoFillHints() to getAutofillHints()" into oc-dev

Hide Socket+DatagramSocket IpSecTransform methods

These methods do not currently work for IPv4 sockets,
so they need to be hidden until the underlying kernel
code can be made to work properly in a dual-stack
environment.

Bug: 36073210
Test: compilation
Merged-In: Idce367f638a42da374a7f1dc5ebb8931c9555e06
Change-Id: Idce367f638a42da374a7f1dc5ebb8931c9555e06
(cherry picked from commit da18b028f85e9a2c969c636aea6abf7f4bac3922)

Consistenly Throw IOExceptions from IpSecManager

Bug: 36073210
Test: api-update
Change-Id: Ia3825e85b09b6330f1326a26cf9e1cadf9dacf71
Merged-In: Ia3825e85b09b6330f1326a26cf9e1cadf9dacf71
(cherry picked from commit c2fd6cfdc9dce33d57a3c2780e0527dd4b1bfeb9)

Merge "OtaDexopt: Add version support" into oc-dev

Merge "Update doc string for getForbiddenPlmns() to READ_PHONE_STATE" into oc-dev

Fixed WTF that happens on race condition.

Test: it's a log change, and it compiles :-)
Change-Id: Ie6409b92a5b7ad21ade5aecc0b7e78602aa3f605
Fixes: 37216544

Vol key presses: active stream has priority of set control stream

When deciding which volume stream is to be controlled by key
  presses, give priority to the stream that is actually playing,
  unless the user has selected a another volume to be controlled
  by the key presses.

Test: play some music over headset, pause, go to home screen.
  Press vol up, it changes ringer volume, immediately press play
  and press vol up again, it should now change music volume
  (and UI updates to indicate that).
Bug: 33355229

Change-Id: Ibad9b59a692e1c6375487b739c55fdbf07f107dd

Renamed ViewNode's getAutoFillHints() to getAutofillHints()

Test: CtsAutoFillServiceTestCases pass
Bug: 35956626
Bug: 35364993
Change-Id: I2ad5a625da6ed4ff889315923c81cfd629e34074

Merge "Turn down the preliminary foreground service API" into oc-dev

Merge "AudioService: fix updating of stream alias for A11y" into oc-dev

Merge "Shows navbar in SUW if the a11y button is requested" into oc-dev

Merge "Defer showing pip menu activity until pip animation ends" into oc-dev

Merge "Account for content insets in source rect hint for transition into PiP" into oc-dev

Merge "DecorView: Avoid changing shadow size in Pinned Stack." into oc-dev

Merge "If gnss service dies, then, upon receiving the death notification on system-server side: Proactively take down the whole system_server." into oc-dev

Merge "A new power button mode to hide the IME when shown" into oc-dev

Merge "Finish impl of job queue: handle URI permissions." into oc-dev

Merge "More small fixes/adjustments to job scheduler." into oc-dev

Merge "Revert "MediaSession: Add methods for setting shuffle and repeat modes"" into oc-dev

Merge "AOD: Implement redacted notificatinos" into oc-dev

Merge "Revert "MediaSession: Add methods for queue managements"" into oc-dev

Merge "MediaBrowser: Revert Search API" into oc-dev

Merge "AOD: Do not show guts nor snooze" into oc-dev

Merge "Fix issue #36858643: Runtime restart on OPR1.170323.002" into oc-dev

Merge "only log calling package for instant apps" into oc-dev

Merge "Cleanup that wasn't uploaded with ag/2094132" into oc-dev

Merge "Tightening up rotation behavior for PIP (3/3)" into oc-dev

Merge "AppErrors: Add test for AppErrorDialog.onCreate" into oc-dev

Minor UI adjustments to a11y button chooser

- Allow scrim-touch or downward-drag to dismiss the chooser
- Ensure the bottom drawer takes up only as much space as it needs
  to prevent it from being draggable

Bug: 36871827
Test: Manual - verified appearance and behavior changes
Change-Id: I1779814a7ee386f4a32faed5c0eb799ebe4706e5

AudioService: fix updating of stream alias for A11y

VolumeStreamState.setAllIndexes():
  - don't do anything if the target and source VSS have the same
    stream type,
  - javadoc of (old+new) behavior.
When STREAM_ACCESSIBILITY becomes independent of any other stream,
  read the settings to restore the volumes previously set.

Test: see bug
Bug: 36286803
Change-Id: I2afcff3a93cc81f3958af252986ef1ac9c38899e

Merge "TIF: Add preview parameter to filter only preview or non-preview channels." into oc-dev

A new power button mode to hide the IME when shown

As discussed in Bug 33038203 on certain platforms there is a demand
that the power button can change the behavior depending on whether an
IME window is shown on the screen or not.  The behavior requested here
can be summarized into two parts:

  * Hide the IME window if it is shown [1]
  * Go to the home screen if no IME window is shown

This CL implements the above request by introducing a new config mode
for config_shortPressOnPowerBehavior.  Note the definition of when an
IME is shown is often tricky than people would expect.  The way this
CL is implemented is to propagate IME window state from
InputMethodManagerService (IMMS) to PhoneWindowManager via
WindowManagerService regarding when the back button on the NavBar for
phones/tablets should be shown as an IME dismiss key [2].

 [1]: Even with this CL the IME still is allowed to ignore the request
      to hide the software keyboard. Currently there is no official
      protocol to forcefully hide the software keyboard.  How to deal
      with such a situation is a long standing TODO task.
 [2]: Internally this is controlled by the following IMMS fields:
       - InputMethodManagerService#mImeWindowVis
       - InputMethodManagerService#mBackDisposition
      Note that those fields rely on self-report from the IME.  To be
      precise, the base implementation of InputMethodService is
      responsible for report back its internal state to IMMS when
      necessary.  The important point is that, although this could
      allow a malicious IME to confuse the system UI to some extent,
      supporting malicious IMEs is not clearly a goal of Android.
      Anyway, the definition of when an IME is shown is a kind of
      hot topic in several system services recently.  Hopefully we
      can come up with better definition and reliable mechanism in
      a future release.

Fixes: 33824860
Test: Manually verified as follows
       1. Change config_shortPressOnPowerBehavior to "5"
       2. Rebuilt the OS image and flash it to the device
       3. Make sure that the power button works like a home button
          if software keyboard is not shown.
       4. Open dialer and focus in to the text field shown on top
       5. Make sure that the AOSP keyboard is shown.
       6. Run 'adb shell dumpsys input_method' to observe the
          following line:
            mImeWindowVis=Active|Visible
       7. Tap the power button to make sure that the AOSP keyboard
          gets dismissed.
       8. Tap the power button again to make sure that it works
          as if a home button.
Test: Manually tested as follows
       1. Open dialer and focus in to the text field to show an IME
       2. Run 'adb shell dumpsys window policy' to make sure
            mDismissImeOnBackKeyPressed=true
       3. Tap the back button to dismiss the IME
       4. Run 'adb shell dumpsys window policy' to make sure
            mDismissImeOnBackKeyPressed=false
Change-Id: I20721547c73360a70b5fc5cbe06824d577d1768a

OtaDexopt: Add version support

Add version number to dexopt command. Current version is "2."

37256688
Test: m
Test: Manual OTA O (v2) -> O (v2)
Change-Id: I9fff673f3ba99833e88cf1c5c9625f42436ec150

Defer showing pip menu activity until pip animation ends

This was causes additional churn in the system delaying the visibility
of the activity behind the one we are pipping. We now delay starting the
pip menu activity until the enter pip animation is done.
This doesn't fix the real issue where we don't wait for app transition
to be finished before starting pip, but it does make it much harder to
the user to trigger the case. Will do that in a follow-up CL.

Test: manual
Bug: 37420370
Change-Id: Ic0271d693d45c3e414b3abdf0ad24be44ff5ae5c

Merge "Notification settings can contain dashboard settings" into oc-dev

Finish impl of job queue: handle URI permissions.

The job queue now handles URI permissions associated with
the Intent of each job.  Just (kind-of) like Service!

Also do the second pass of locking in job scheduler, getting
rid of all the async dispatching on a handler, and just executing
calls right in line with simple locking.  This probably fixes
a few other race issues, and allows us to make sure that we
always finish a job correctly when dequeuing the last work (we
will always atomically dequeue and finish, so no new work can
slip in between).

And fix a little debug output in IntentFilter.

Test: ran CtsJobSchedulerTestCases, added new test for URI perms.

Change-Id: I52f700ef0cd5be3ff70050f9c0f5fe3e8a5ccac1