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

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

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

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

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

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

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

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

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

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

Account for content insets in source rect hint for transition into PiP

- As a part of transitioning into PiP, we remove the content insets, and
  as a result, we also need to adjust the source hint rect for the change
  in insets as well.

Bug: 37418994
Test: Enter PiP from YT, ensure no jump at the end

Change-Id: I74219b05c91d3c8b8466dd8fff4d6dd692f08725

Merge "Disable crashing drop shadow tests" into oc-dev

Merge "Accessibility for Notification Inline Controls" into oc-dev

DecorView: Avoid changing shadow size in Pinned Stack.

It seems at the end of the animation, sometimes we will
come to think we have focus, before we have properly
updated mStackId. This in-and-of itself seems like
somewhat of an issue...but I'm not inclined to dig too
deeply at the moment. For now just ensure a static shadow
size in the pinned stack.

Bug: 37425446
Test: Manual
Change-Id: I379f7a6de51670c90eacb52a5be5e17784be8ac5

Merge "Warning if app targeting O posts a notification with no channel" into oc-dev

Deprecate ActivityManager#getRunningServices

This change deprecates the method akin to the previous deprecation of
ActivityManager#getRunningTasks. The documentation has been updated
to reflect the current limitations of the method.

Change-Id: I6f35309c1224fdf1f890bce3cc614be8aa343368
Fixes: 36937370
Test: documentation

AOD: Implement redacted notificatinos

Test: Turn on "show sensitive notifications on lockscreen", receive notification, observe AOD
Bug: 36610748
Change-Id: I981ef4cea7cefc76dcd55ebad9d05c758927dabb

Merge "Fix up color issues as a result of theme change" into oc-dev

Merge "Status bar updates" into oc-dev

More small fixes/adjustments to job scheduler.

- JobServiceEngine now takes a concrete Service instead of
generic Context in its constructor, since it really must be
associated with a real Service.

- Expand documentation of how dequeueWork() operates.

- Fix some job scheduler implementation to hopefully actually
match the docs: transfer remaining executing work to the new
job, and actually correctly transfer state from old and new
jobs if we are rescheduling due to a true return from onStopJob().

Test: bit CtsJobSchedulerTestCases:*
Change-Id: Ia66797049883eefb566264f930070afb69d469b1

Merge "Storage API polishing." into oc-dev

update docs per API council

Change-Id: I182e29912276bc09734bd604e9e2fd866094a95f
Fixes: 37328564
Test: treehugger

Merge "AOD: Track missed AOD time ticks" into oc-dev

Merge "Fix date going missing in QS" into oc-dev

AppErrors: Add test for AppErrorDialog.onCreate

Also refactor to make sure we always remove the TIMEOUT
message and clear the mProc.mCrashDialog field, even when
dismissing without the Handler.

Bug: 37351370
Test: runtest -c com.android.server.am.AppErrorDialogTest frameworks-services
Change-Id: If9fb348e8ed83c6c1c0f48fa7fd27ffd33b530f2

only log calling package for instant apps

Change-Id: I71b09fd130f27d5814638fdaaec33f759b0b2346
Fixes: 37425492
Test: manual

Merge "Include vndk/hardware_buffer.h for AHardwareBuffer_getNativeHandle" into oc-dev

Tightening up rotation behavior for PIP (3/3)

- Fixing up edge case when SysUI resizing conflicts with updating the
  display rotation bounds.  When an interaction causes both a display
  rotation and a resize from SystemUI, we should defer the resize animation
  until the rotation has been propagated to SystemUI, otherwise the bounds
  used will be incorrect.

Bug: 36879891
Test: android.server.cts.ActivityManagerPinnedStackTests
Change-Id: Ife1b7ab0c2f1f11f33cbc9614778ff49a28c79f6

Cleanup that wasn't uploaded with ag/2094132

I saw that Tree Hugger had verified, and hit submit without
remembering to repo upload the last patch set containing toddke's
most recent set of suggestions.

Bug: 37192695
Test: runtest -c com.android.server.pm.PackageManagerServiceTest frameworks-services
Change-Id: I4d6376165aaa615007891ba045c9b5c34e4b0394
(cherry picked from commit 338d332873c431c46057eae3c93ed79351ddd2e4)

Merge "Add tests for ag/2086796" into oc-dev

Notification settings can contain dashboard settings

Bug: 36562211
Test: NotificationSettingsTests, RunSettingsRoboTests
Change-Id: I4da338a90e0f094202bae73f9bd8622b37a3acf6

Merge "Add user to channel/group modification callbacks." into oc-dev

Turn down the preliminary foreground service API

The NotificationManager.startServiceInForeground() experiment is over,
and will not ship as API, so it's time to tidy up and get rid of it.

Bug 36130212
Test: manual

Change-Id: I834d1ce059aa464ff27f69f5e5d3625cc5e61d8a

AOD: Do not show guts nor snooze

Fixes: 37179751
Test: receive notification while phone is off, try longpress or accessing the gear icon through swiping, ensure neither works
Change-Id: If18ac166eac28cd7ad05a12f670b2ddbad9e29b8

Merge "Schedule PIP mode changes at the beginning/end of the transitions." into oc-dev

Accessibility for Notification Inline Controls

Send WINDOW_STATE_CHANGED event in status bar
Disable falsing when in touch navigation
Accessibility string for enable/disable switch
Bug: 34840327
Test: runtest systemui
Change-Id: I60cc7e49d9e6323e23a17183b8b10c9d857501bb

Merge "OEM single-build/multi-SKU via dynamic RRO support" into oc-dev