Revert "Call ActivityCallbacks after corresponding activity methods"

This reverts commit 660cf30e2100d8ed40198c95f93b61e5d1cdb10c.

bug: 34415265
bug: 34914977
Test: chrome launches

Change-Id: I170365fd47313bafb0d722e6d55ee119336137c4

Handle nullable drawables inside MaskableIconDrawable
Test: manual test
b/34902453

Change-Id: I36fc48f7e4c1f6503a9c26b8f239c62da729b490

Merge "Prevent division-by-zero error when calculating battery capacity."

Merge "Hardware bitmaps: Destroy texture ID generated for the copy"

Merge "No warning/info log for expected/normal situations"

Merge "Avoid NPE in NightDisplayService when VR mode is invoked"

Merge "Add plain mode for the quick viewer."

Merge "Fix AutoFill Save window layout params."

Add plain mode for the quick viewer.

When EXTRA_QUICK_VIEW_PLAIN is passed, then plain UI should be shown.
This is just a hint for third party apps, whic may ignore it.

Test: Not testable, as it's just a hint.
Bug: 32161075
Change-Id: Ie244d28d552f6c654be93a5749ac164d2a77d25f

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 4ffe72dcc8 am: ab8d3febd3 am: 2e6bdbeaab am: 588411fc7c  -s ours am: 773f85ec60 am: 029364c505 am: 2f1b2d82ca am: ac8f44e387 am: 3ee26841fd
am: a843ab955c  -s ours

Change-Id: I5de7b85e7cf6aae708512ec068edee82ad5ee239

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 4ffe72dcc8 am: ab8d3febd3 am: 2e6bdbeaab am: 588411fc7c  -s ours am: 773f85ec60 am: 029364c505 am: 2f1b2d82ca am: ac8f44e387
am: 3ee26841fd

Change-Id: I7b4d65cc7a3d1b4f575d885e1c71b40a52cc90a4

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 4ffe72dcc8 am: ab8d3febd3 am: 2e6bdbeaab am: 588411fc7c  -s ours am: 773f85ec60 am: 029364c505 am: 2f1b2d82ca
am: ac8f44e387

Change-Id: I7e4376740ddf167035a3274c19ca5ae3f113a9b0

Merge "QS: Remove circles and add lines"

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 4ffe72dcc8 am: ab8d3febd3 am: 2e6bdbeaab am: 588411fc7c  -s ours am: 773f85ec60 am: 029364c505
am: 2f1b2d82ca

Change-Id: I8094afc5c919b65d7f180ef9d3ad875f3ea4dacf

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 4ffe72dcc8 am: ab8d3febd3 am: 2e6bdbeaab am: 588411fc7c  -s ours am: 773f85ec60
am: 029364c505

Change-Id: Ibef494c57a80a7b0d871dd8a80f877816e6897d0

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 4ffe72dcc8 am: ab8d3febd3 am: 2e6bdbeaab am: 588411fc7c  -s ours
am: 773f85ec60

Change-Id: Ie83cb2dfd89fd808e906a802e16fd1943a898012

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 4ffe72dcc8 am: ab8d3febd3 am: 2e6bdbeaab
am: 588411fc7c  -s ours

Change-Id: I1511e5dff681d06161cd18b78e7edaf7eddb897a

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 4ffe72dcc8 am: ab8d3febd3
am: 2e6bdbeaab

Change-Id: I7e286445a887e467a7de42ce9b27a9dc009e47fd

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 4ffe72dcc8
am: ab8d3febd3

Change-Id: If42411391fe2010162c51adf48e720e308a650ec

DO NOT MERGE Do not call RecoverySystem with DPMS lock held
am: 4ffe72dcc8

Change-Id: Ic7b9ef6484df0b57c8f1ee7a483ef3bda05c7cc7

Merge "Improved AutoFill Save workflow."

Hardware bitmaps: Destroy texture ID generated for the copy

Test: manual
bug:34807359
Change-Id: Idb1ef2a2efcb3cf7fb96e2a1f0554a06e7cd0fc2

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 2317451acc am: 01f971dcdc am: 963cba5b5b am: f1e276384d am: 46bfcb9961  -s ours
am: 62c651d4d5  -s ours

Change-Id: I459ff4ec03824e661d286aa718e029c7a5a60c7d

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 2317451acc am: 01f971dcdc am: 963cba5b5b am: f1e276384d
am: 46bfcb9961  -s ours

Change-Id: Ie1cd5114414daeb1216827563333ed91df873604

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 2317451acc am: 01f971dcdc am: 963cba5b5b
am: f1e276384d

Change-Id: I856ad657b894531f03bc145e033b0cebf4e722cd

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 2317451acc am: 01f971dcdc
am: 963cba5b5b

Change-Id: I3a1421f656d1a90970eac6156e0840625662fb17

DO NOT MERGE Do not call RecoverySystem with DPMS lock held am: 2317451acc
am: 01f971dcdc

Change-Id: If5f5e5d47b97e2b8480adc5ca2cb08702056ca87

DO NOT MERGE Do not call RecoverySystem with DPMS lock held
am: 2317451acc

Change-Id: Id3441141d4feea4c7291350c9670bd59ec017240

Merge "Add dumpsys options to procstats to inject screen state"

Merge changes Iad270dfd,Ice608e6d,I375369a8

* changes:
  Adding initial TRON logging for picture-in-picture.
  Disallow entering PiP when activity is locked.
  Preventing cases where an app can be stopped while entering PiP.

Merge "Start MtpDocumentsService as foreground service."

Start MtpDocumentsService as foreground service.

Prevously MtpDocumentsService was started as background service, then it
turns into a foreground service by calling Service#startForeground.

The workflow did not work until this, because now background activity
cannot launch a background service.

The CL starts using NotificationManager#startForegroundService to launch
MtpDocumentsService so that the service can be started as foreground
service directly.

Bug: 34468813
Test: MtpDocumentsProviderTests
Test: manual testing with connecting MTP device to Android
Change-Id: Ic35d3e92f234881846e5d82ed04d6681a83035f7

Avoid NPE in NightDisplayService when VR mode is invoked

Bug: b/32086748

Test: I was seeing the crash without the CL. With the CL no more crash.
Change-Id: I286af1066ce8928ed72a83a8da645fe99953fd85

Merge "Removing extraneous translations"

Merge "Fix a bug in View#hasFocusable with new auto-focusable behavior"

Fix AutoFill Save window layout params.

Change-Id: I54ae23e9ab1b12ab868b37fd8e87e89dad7ae8fc
Test: CTS

Merge "Scale PiP to stack instead of crop."

Add dumpsys options to procstats to inject screen state

Test: CTS CtsDumpsysHostTestCases, not submitted yet though.
$ cts-tradefed run cts --skip-device-info --skip-preconditions --skip-system-status-check com.android.compatibility.common.tradefed.targetprep.NetworkConnectivityChecker -a armeabi-v7a -l INFO -m CtsDumpsysHostTestCases -t 'android.dumpsys.cts.ProcessStatsDumpsysTest'
Bug 34228624

Change-Id: I9cf7f737624fe9f188956be7ba7485151555bc4b

Merge "Allow opening activity to specify orientation."

Merge "Explicitly destroy Layer in DeferredLayerUpdater on destroyHardwareResources()"

Merge "Properly clear textures for Hardware Bitmaps"

QS: Remove circles and add lines

 - Remove circles from icons.
 - Add lines to indicate tiles with dual tap targets.
 - Change default order

Test: visual
Change-Id: I1eb02641757a76ddf3e3876182f7b2503161bc9e

No warning/info log for expected/normal situations

This CL hides warning and info log messages from
InputMethodManagerService (IMMS) and InputMethodService (IMS) behind
DEBUG flag like other logs unless the state is certainly unusual.

Of course the definition of "unusual" is still an open question, but
basically that we should not see any suspicious message from IMMS/IMS
just by turning on a new phone, launching some applications, typing
something, and turning off the device. IMMS and IMS should expect all
events that can (easily) occur in that scenario, and no log is
necessary for such things.

Note that warnings suppressed with TODO comments will be tracked
under Bug 34851776 (and Bug 34886274).

Test: adb logcat -s InputMethodManagerService:* InputMethodService:*
      to monitor log in the following scenario:
       1. Boot the device.
       2. Complete the setup wizard.
       3. Launch Dialer and type something on it.
       4. Launch Contacts app and type something on it.
       5. Try some special modes:
           - Turn on/off display
           - Recents screen
           - Split-window mode
           - Guest user
           - Multi user
           - Direct-boot (setup a device password and reboot)
      except for logs about actual IPC calls from a background user.
Bug: 34838583
Bug: 34851776
Bug: 34886274
Change-Id: I3fcdeb919bb2f2940da9ff45e17ac00baa1253f4

Merge "Backbone of the new aapt2 strip command."

Merge "Cleanup hwbinder getService code." am: 9be24a3040 am: 4ede371d80 am: ec840d1464
am: 8bb391016b

Change-Id: I56d2b0c8d466e78316497264b7ca96f6526b3d9d

Merge "Cleanup hwbinder getService code." am: 9be24a3040 am: 4ede371d80
am: ec840d1464

Change-Id: I83a7077f07689dc57ac4d8828508443cc4385292

Merge "Add a filter for audio apps."

Merge "Cleanup hwbinder getService code." am: 9be24a3040
am: 4ede371d80

Change-Id: Ia0f3f88d8881f89cec6939dbb44d7dbbb8da9a7f

Merge "Fix setting null target fragments"

Merge "Cleanup hwbinder getService code."
am: 9be24a3040

Change-Id: Idb75f2c41e6e0fd9c2eb03ef4960e40cb6556308

Merge "App Widget Service with dump in protobuf format"

Merge "Cleanup hwbinder getService code."

Properly clear textures for Hardware Bitmaps

Test: manual  (Description in the bug)
bug:34858530
Change-Id: I13eb89077c43ca28436509a7af5b7c11374446c4

Allow opening activity to specify orientation.

An activity's window is initially not visible. This means it is not
accounted for until it becomes visible, which means requested
orientations will not be honored until after onCreate.

This changelist removes the visibility check previously used determining
the orientation.

This changelist also addresses an issue where an ActivityRecord member
variable tracking the last configuration sent was not being initially
updated.

Fixes: 33844887
Fixes: 33814250
Fixes: 34165818
Fixes: 34177026
Fixes: 33809086
Fixes: 34111451
Fixes: 33844423

Test: Verified reported issues fixed in apps mentioned in bugs
Test: bit FrameworksServicesTests:com.android.server.wm.WindowContainerTests
Test: cts/hostsidetests/services/activityandwindowmanager/util/run-test android.server.cts.ActivityManagerAppConfigurationTests
Change-Id: If9243792b9f2d137dc5addbf6fb735a0048fa192

Merge "Check disqualifying notifications synchronously."

Merge "Deprecate notification fields"

Merge "Correct accessibility check for current user"

Merge "Fix incorrect comments"

Merge "Remove unneeded callbacks to systemui."

Merge "AOD: Fix in-place ambient layout updates"

Merge "Turn off Dynamic IMS switching for now" am: 2bb0d95e91 am: 9f1ccbf5f9 am: cb8180033a
am: aca8a2a707

Change-Id: I2dc6102e9bd875603f26fcfb87ce34b1a72e1a23

Merge "Turn off Dynamic IMS switching for now" am: 2bb0d95e91 am: 9f1ccbf5f9
am: cb8180033a

Change-Id: If5baf2ffc7b4010c6598ec730d48602b48738cc2

Merge "Import translations. DO NOT MERGE"

Merge "Turn off Dynamic IMS switching for now" am: 2bb0d95e91
am: 9f1ccbf5f9

Change-Id: If9be849f743e9722ca49543a92d33d09978dc6ce

Merge "Adds @hide permission granting API for ImsServices" am: 390ec27f9d am: 6734275c2f am: fa1dca0ad6
am: 3aaf673c60

Change-Id: I6a6a2d136f0f31bbac984e4ba18f9f6b90808705

Merge "Turn off Dynamic IMS switching for now"
am: 2bb0d95e91

Change-Id: Ie74b53ca5f0b6463b371368608c793d402878d50

Merge "Adds @hide permission granting API for ImsServices" am: 390ec27f9d am: 6734275c2f
am: fa1dca0ad6

Change-Id: Id2875b5b60401405e31e76eb07968798144df700

Merge "Turn off Dynamic IMS switching for now"

Merge "Adds @hide permission granting API for ImsServices" am: 390ec27f9d
am: 6734275c2f

Change-Id: I6d502db3362aafcfed426b6598be0bd39dd38890

Scale PiP to stack instead of crop.

When changing stacks the finalCrop or crop may not be set
so relying on either one for the scaling factor is error prone.
Just scale to the stack size. I initially avoided this because I
thought some of the crop code may be relevant in computing the size
we wan't...but I'm now pretty sure it won't.

Test: Expand PiP back to fullscreen on phone, verify no crash.
Change-Id: I172daade74bec7374b5fd3310c1bf554e46d8832

Merge "Adds @hide permission granting API for ImsServices"
am: 390ec27f9d

Change-Id: I0f3d38811925884681d11cf18e12f1b0574d64c6

Merge "Adds @hide permission granting API for ImsServices"

Merge "Fixed NPE during monkey testing."

Adding initial TRON logging for picture-in-picture.

Bug: 33756317
Test: Run PIP activity
Change-Id: Iad270dfdf6521d4659653a0697effaed4a0d4137

Disallow entering PiP when activity is locked.

- Also ensuring that when we request for a task to be locked
  via the shell, we directly set the locked state instead of
  relying on the SystemUI to show a prompt to lock (necessary
  for the CTS test to run).

Test: android.server.cts.ActivityManagerPinnedStackTests
Test: #testDisallowEnterPipActivityLocked

Change-Id: Ice608e6d8e7f018dbd5e20ae457343bfdd0a8630

Preventing cases where an app can be stopped while entering PiP.

- Ensure that we clear the timeouts on the old stack and reschedule
  them on the new stack when moving an activity to a new stack,
  otherwise the pause timeout from the old stack will cause onStop()
  to be called.
- When adding an activity to the stopping list for processing, prevent
  scheduling an idle immediately in case an activity tries to enter
  picture-in-picture when handling onUserLeaveHint().  In that case,
  schedule an idle after the default idle delay instead.
- In addition, when resuming a resumeWhilePausing activity, prevent the
  activity idle to trigger pausing activities to be immediately put into
  a stopped state.  This was a race between the handling of pause/resume
  that would cause a pip activity to get onStop() even if it called enter
  pip on pause. Instead, when processing an idle from the activity or
  from an immediate idle scheduled by the system, we defer processing
  pausing activities until a later idle (that it reschedules).

Test: android.server.cts.ActivityManagerPinnedStackTests
Test: #testEnterPipWithResumeWhilePausingActivityNoStop

Change-Id: I375369a800b7fadaa57d6e00e0564bc3ee338979
Signed-off-by: Winson Chung <winsonc@google.com>

Improved AutoFill Save workflow.

Currently, the onProvideAutoFillStructure() methods can be called
twice: to auto-fill an activity and to save the activity's data
in the service.

The problem with this approach is that when the save workflow is
called, the activity might have been gone. Hence, a proper approach
is to keep the initial AssistStructure data in the system_service
memory, watch for view changes, and then passed the new structure
back to the AutoFillService.

A side effect of this change is that we need another way to determine
if the view is sanitized or not. For "standard" views, that will be
defined based on whether the view content come from a resource or not,
but that logic is not implemented yet (for now, all views will be
considered sanitized, except for TextView passwords). For "custom"
views (such as WebView), this logic is responsibility of the view
implementation, through the newChild() method, which now takes a
flag (whose value could be AUTO_FILL_FLAG_SANITIZED for sanitized
views).

The SaveCallback.onSuccess() method was simplified: it does
not need a list of saved ids anymore the auto-fill UI will not use it
anymore.

Another side effect is that the Save notification is gone - until
it's attached again, it can be test by using:

    adb shell cmd autofill save

Finally, hook AutoFillUI on ACTION_CLOSE_SYSTEM_DIALOGS events.

BUG: 33269702
BUG: 31001899
Test: manual verification
Test: CtsAutoFillServiceTestCases passes
Change-Id: I907a7e21d1b3cd1ab6dec3a08d144a52655da46f

App Widget Service with dump in protobuf format

Test: This protobuf format is validated as a CTS test
Bug: 33088921
Change-Id: I9d2a983052ade06cd3a13857b7e7f9e3d9bc275e

Backbone of the new aapt2 strip command.

The strip command takes an APK as argument and a --target-densities flag which
contains the list of screen densities for which to optimize the APK. All
the resources that would be unused for a device having the given
densities are stripped from the APK leaving only the ones actually being
used.

Test: Manual.

Change-Id: I68368b00876a18ccdbbfc0a8dd265bd361117933

Fix setting null target fragments

Bug 34869013
Test: CTS

Change-Id: Ibeed66f9a889a8e395bfabedcf22aebcbb848908

Merge "Notification playback synchronized with audio focus"

Cleanup hwbinder getService code.

- Remove memory leaks.
- Make sure proper errors are getting thrown.
- Avoid SIGABRT

Test: hidl_test_java
Change-Id: Ia5680fd0ebf83d0f174bf0d8109bf60327be5f9d

Merge "Add Wear keycodes to emulator key layout file. am: 119a06ce55 am: c85e503d52" into nyc-mr2-dev-plus-aosp
am: da5289b6ed

Change-Id: I34d2a4a3945579820af1c3fdb04da53b956bf665

Add Wear keycodes to emulator key layout file. am: 119a06ce55 am: c85e503d52
am: 0dea9e896f

Change-Id: I69418d80868e6cb27fcc82635150c47da03c3654

Add Wear keycodes to emulator key layout file. am: 119a06ce55 am: 9576ce6696
am: 00620e70d5

Change-Id: Ie9e186901a6d846ee1417b1b1db9ed091e854b89

Merge "Add Wear keycodes to emulator key layout file. am: 119a06ce55 am: c85e503d52" into nyc-mr2-dev-plus-aosp

Merge "Call ActivityCallbacks after corresponding activity methods"

Add Wear keycodes to emulator key layout file. am: 119a06ce55
am: c85e503d52

Change-Id: I6840265835199d454d735e443fb53228c0ae2fd2

Add Wear keycodes to emulator key layout file. am: 119a06ce55
am: 9576ce6696

Change-Id: Ic10c9e353a264c60cef3dac79a2fc4b57eebb244

Fix incorrect comments

Bug: 33010587
Test: comment only
Change-Id: Icdfa79bac50545bb73f4c40727fe51780ef4a56d

Add Wear keycodes to emulator key layout file.
am: 119a06ce55

Change-Id: I33d2b0f9b8281c22422677f03d6046a1be0bd94f

Add Wear keycodes to emulator key layout file.
am: 119a06ce55

Change-Id: Ieb06380c3bd43af7087463543960cc4ee1889050

Merge "Revert "Allow opening activity to specify orientation.""

Revert "Allow opening activity to specify orientation."

This reverts commit f0a6fdbf53882949db0a141693dd33302b42d51b.

Reason for revert: Submitted before complete.

Change-Id: I545ff45b5e870c79d66884b7a2a80981293b1b24

Merge "Allow opening activity to specify orientation."

Check disqualifying notifications synchronously.

Don't wait to check on the handler.

Test: runtest systemui-notification and
platform_testing/tests/functional/notificationtests

Change-Id: Ife1ad8bc5c40420cd6682329b8135744cfe38e4a

Merge "Remove setDockedStackMinimized credentials prompt"

Fixed NPE during monkey testing.

We tried to resume key dispatching for an activity after we destroyed
it and removed its container which led to an NPE.
We now resume key dispatching for an activity if it is currently paused
just before we remove its container to not leave WM in a bad state and
also avoid NPE later on.

Change-Id: I89d366c8667e621f4515df802c6b8278fa4bd322
Fixes: 34808323
Test: adb shell monkey -p ...

Merge "Fix a wrong UID can be logged"

Merge "Initialize bootanimation sound on separate thread"