Merge "Allow ImsService to return network error codes"

Allow ImsService to return network error codes

Adds new API for SMS over IMS that allows the ImsService
to pass 3GPP TS 24.011 error codes from the network for
better debugging/metrics collection.

Test: manual
Bug: 138239529
Merged-In: I5d1f41b90595238011eb0d1dc0ae0415157bb469
Change-Id: I5d1f41b90595238011eb0d1dc0ae0415157bb469

Merge "Add jji@google.com to am/OWNERS file."

Merge "Exposing flickerlib classes and layer tracing to sysui"

Merge "Use filter_packages to unbunden metalava"

Merge "Mirror fs-verity ownership UID to Java"

Add jji@google.com to am/OWNERS file.

Bug: NA
Test: NA
Change-Id: I5c5f0d1e75a66e772c95304a3dbd936a471e9a17

Mirror fs-verity ownership UID to Java

This has been done in system/core, so mirroring for later use.
https://android-review.googlesource.com/c/platform/system/core/+/1115493

Test: compile
Bug: 112038744
Change-Id: I2d7f5d38caae0b6ada1f208b3d71d937d527716e

Merge "Bugreport progress notification should use the bug icon."

Merge "Expose ConnectionService Handler to remove flaky tests"

Merge "Use UnlockMethodCache#canSkipBouncer in user switcher"

Merge "Implement stats logging for Compatibility API."

Merge "Add isApplicationOnUicc API"

Merge "Add new onSmsStatusReportReceived() without message reference"

Merge "HwBinder: don't rely on kernel headers."

Expose ConnectionService Handler to remove flaky tests

Test: atest TelecomUnitTests
Change-Id: Ibb80b5739083ad9f85ee06f4d9f0017f4cb605bd

Merge "Fix descriptions for default SIM in global preferences"

Merge "Use bionic platform headers."

Fix descriptions for default SIM in global preferences

Test: Not required.

Change-Id: I7f32eb495b53d13aba50b07d86e63f0549e9e6bb
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>

Merge "VintfObject: Fix getHalNamesAndVersions doc"

Merge "Ignore physics animation tests that have suddenly become flaky."

Merge "Add carrier config KEY_CALL_FORWARDING_WHEN_UNREACHABLE_SUPPORTED_BOOL"

Merge "AudioService: fix missing CEC message on unmute after vol down"

Ignore physics animation tests that have suddenly become flaky.

Test: atest SystemUITests
Change-Id: Ia15f767fdb251338a7e7b15be7836b6928e48ba5
Merged-In: Ia15f767fdb251338a7e7b15be7836b6928e48ba5
(cherry picked from commit 2b991d91c6d6a2341a06be329962d2f5a8f3ee28)

Use bionic platform headers.

Bug: 130763340

Test: Builds.
Change-Id: Ieed72d4e15c0ce0dfc3a4f3383ea6563905bc236
Merged-In: Ieed72d4e15c0ce0dfc3a4f3383ea6563905bc236
(cherry picked from commit 8269f3a3770d83b139cc3b3571d72eb30f62191f)

Exposing flickerlib classes and layer tracing to sysui

- Allow recents component to enable/disable layer tracing
- Expose flickerlib classes for use with sysui/launcher tests
- Allow trace files to be moved to a path that the client can read
- Ensure layers are flattened with depth traversal to encode ordering

Bug: 140244969
Test: atest FlickerTests
Change-Id: I7af5699ff5b8a4bc62c1a8105c67b31bc45a2236
Merged-In: Ia0934ec7c2b9484fa6c85f5aa8d9b2e5e6f7dc0f

Implement stats logging for Compatibility API.

Add a new atom and log from both the app process API and the system server API
Bug: 136794938
Bug: 138378110
Test: statsd_testdrive 228

Change-Id: I80f07d0beb30c779c4bce70bebf2bb4ab22f6bfe
Merged-In: I80f07d0beb30c779c4bce70bebf2bb4ab22f6bfe

Add new onSmsStatusReportReceived() without message reference

IMS providers must generate unique token and don't need to parse the PDU
to get the message reference for new onSmsStatusReportReceived() API.
The platform will extract the message reference from the PDU and return
it to the IMS providers via acknowledgeSmsReport(). And @Deprecated tag
is added on existing onSmsStatusReportReceived() API.
This CL includes some fixing doc bug and description for APIs also.

Test: atest FrameworksTelephonyTests:ImsSmsDispatcherTest

Change-Id: Ib788e291f2e70c3e062319e17e2c9efe33f31164
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>

Use filter_packages to unbunden metalava

This change fixes a problem that entire framework source files are given
to metalava even when many of the source files are not for public. Using
the new property filter_packages, only source files that belongs to the
public packages are given to metalava.

This CL also changes the name of the filegroup 'framework-srcs' to
'framework-non-updatable-sources' to better describe its contents. In
addition, a build-time only library 'framework-all' is introduced. This
library produces a header jar for all framework sources for both
non-updatable and updatable parts. It is put to the classpath when
invoking metalava to give information on the hidden classes that were
filtered-out.

Bug: 140764681
Test: m
Change-Id: I6e16679109ac07820e35037754d4327442c014f2

Merge "SoundPool: Do not sleep on stop for USAGE_GAME or on BUFFER_END"

Merge "Listeners of active emergency numbers"

Merge "WindowOrientationListener: Check if proposed rotation is in range"

AudioService: fix missing CEC message on unmute after vol down

  Group in one method sending volume / mute updates to HDMI CEC
when applicable. Use this method, maybeSendSystemAudioStatusCommand,
to avoid code duplication.
  Add missing synchronization in onUnmuteStream for stream mute check.
  When STREAM_MUSIC is unmuted, also call
maybeSendSystemAudioStatusCommand for a mute adjust, to ensure
other CEC devices' UI is in sync with the internal state.

Bug: 136977965
Test: with ATV device in system audio mode, mute with remote, then \
  press vol down on TV remote, verify TV display matches ATV UI    \
  with current volume.

Change-Id: I279a97871ba98e67f282bf148969ba41ed3f6062

SoundPool: Do not sleep on stop for USAGE_GAME or on BUFFER_END

Test: SoundPoolApplication, SoundPoolOggTest, Games
Bug: 139787914
Bug: 139788416
Bug: 140579743
Bug: 140779732
Merged-In: I83b9b1d61b65dcc4a73d676e1f8be1e022c70302
Change-Id: I83b9b1d61b65dcc4a73d676e1f8be1e022c70302
(cherry picked from commit a527524356a9a6bc9597e4d06cee5c02cf936a68)

Merge "Fix QuotaController constants."

Merge "Add nullability anotations"

Merge "Honor HDMI_CONTROL_ENABLED setting in AudioService"

Merge "Check FIPS_mode() rather than explicitly calling BORINGSSL_self_test."

Listeners of active emergency numbers

1) Introduce a new signature-level system permission allowing system apps to listen the
 active emergency number, placed by calls or texts.

2) Add PhoneStateListener API surface to provide a mechanism to listen the
outgoing call emergency number or outgoing sms emergency number.

Bug: 135204580
Test: atest
Change-Id: I6e6bce6d4b97d492ca20302002911f9daa20767e
Merged-In: I6e6bce6d4b97d492ca20302002911f9daa20767e
(cherry picked from commit 7a1eb6d56b3fb37c746664887b6f754e8d09d715)

Bugreport progress notification should use the bug icon.

(Don't worry, stat_sys_adb isn't going away, but it will
only be used by the ADB notification now.)

Bug: 138661761
Test: initiate bugreport from globalactions
Change-Id: I238659d50f70bfbc047133d89afb76f78ec841f9

Fix QuotaController constants.

1. QuotaController wasn't using the updated constants on system start.
It would only use the updated constants if they were changed after
system start.
2. Updating ACTIVE limits to more workable numbers.

Bug: 137166468
Test: atest com.android.server.job.controllers.QuotaControllerTest
Change-Id: I3e2848446b20f7258fac2d96566c96aea5715949

Honor HDMI_CONTROL_ENABLED setting in AudioService

This change updates how AudioService is updated of changes to Hdmi Cec
status (enable/disable) and availablity.

Bug: 138956107
Bug: 138373017
Test: Android TV STB: make; flashall;Volume -/+; HDMI control enable/disable.
Change-Id: Iaf83a45206be076858e13d113ba772ed5c59ef60
Merged-In: Iaf83a45206be076858e13d113ba772ed5c59ef60

Merge "Remove android.permission.PermissionManager from blacklist"

Remove android.permission.PermissionManager from blacklist

This is getting pulled in from somewhere, removing to fix
PreloadCheck test.

Bug: 140912480
Test: atest PreloadCheck
Change-Id: I8eea07930732865cc72fea0e352a1704d1c26118

Add nullability anotations

Add missing nullability for applyPayload.

Test: manual
Change-Id: Iafce9091767bd7a31937b0be329ede6639239d73

Merge "Revert "Revert "Remove a misleading "flush" function."""

VintfObject: Fix getHalNamesAndVersions doc

Test: CTS device info collect
Bug: 140832836
Change-Id: Iddc3c36d15370baa661e485b76c490a5d2b7815c

Merge changes from topic "protobuf-3.9.1"

* changes:
  Convert statsd_test and statsd_benchmark to proto lite
  Adapt to google::protobuf::uint64 type change

Merge "Track changes to CountryTimeZones API"

Merge "Add uhid group to system_server"

Merge "Swisscom VVM free SMS shortcodes CH"

Merge "Remove libhwbinder/libhidltransport deps"

Merge "FocusRequester: refactor handleFocusLoss"

FocusRequester: refactor handleFocusLoss

Break out the checks to see if a focus loss is handled by the
framework in a separate method, and reorder the if statements
for code clarity.

Annotate PlayerFocusEnforcer methods for nullability, add
javadoc

Test: atest AudioFocusTest
Change-Id: I9aaab804c78a2e2627ddbd7b9bd41fdec47d2b57
Merged-In: I9aaab804c78a2e2627ddbd7b9bd41fdec47d2b57

Merge "Re-enable testAppZygoteServices in pre-submit."

Merge "Reconcile Zygote process name code with Q release."

Merge changes If2b3b414,I1039748d,Ib3cd38f5

* changes:
  AudioService: fix disconnect/connect of A2DP device
  audioservice: re send BT_SCO parameter to HAL when audioserver restarts
  audioservice: fix hasMediaDynamicPolicy() for loopback and render policies

Merge "Refactor SoundEffectsHelper for asynchronous loading"

Merge "AudioService: factor out sound effects handling"

Merge "Use sendVolumeKeyEvent API instead of sendKeyEvent API"

Merge "Don't clear the output array if it failed to open the proc status file"

AudioService: fix disconnect/connect of A2DP device

 Due to the expected BECOMING_NOISY behavior associated
with a device disconnection, the disconnection is handled
asynchronously after a fixed delay. This delay caused an
inversion of commands in the processing order of the
disconnection of a device closely followed by connection
of the same device.
 The fix consists in:
- overriding the equals() operator for BtDeviceConnectionInfo
 so messages for a given device in the message queue
 can be checked / removed.
- when AudioDeviceBroker receives a command for A2DP
 connection or disconnection, remove all upcoming connection
 and disconnection commands in the queue for this device
 (see postBluetoothA2dpDeviceConnectionStateSuppressNoisyIntent)
- remove AudioDeviceBroker.handleSetA2dpSinkConnectionState, which
 was only used in BtHelper.onA2dpProfileConnected() with
 a CONNECTED state, and have this method perform a regular device
 connection (just like when coming from AM->AS).
- in AudioDeviceInventory.onSetA2dpSinkConnectionState(), support
 receiving a connection event for an already connected device,
 to support codec changes.

This change also includes modifications to the classes involved
in the device connection to make them support mocking/spying
to reproduce the bug conditions (see AudioDeviceBrokerTest).

Bug: 134932649
Test: atest AudioDeviceBrokerTest
Change-Id: If2b3b41409c77467a181a2f9b42310db9b9de8c5
Merged-In: If2b3b41409c77467a181a2f9b42310db9b9de8c5

audioservice: re send BT_SCO parameter to HAL when audioserver restarts

Fix omission in AudioDeviceBroker.onAudioServerDied() where parameter
"BT_SCO=on|off" must be send to HAL according to current forced usage
for communication.

Bug: 135512789
Test: killaudio server during call over BT SCO
Change-Id: I1039748db5591321869f70c3dc589fbc62d41c31
Merged-In: I1039748db5591321869f70c3dc589fbc62d41c31

audioservice: fix hasMediaDynamicPolicy() for loopback and render policies

Exclude mixes with LOOPBACK + RENDER flags when looking for dynamic
policies matching media usage. As this is only used in the context
of sending becoming noisy intent we want to send the intent when playback
capture is active.

Bug: 137055231
Test: enable live caption and disconnect headset while music is playing
Change-Id: Ib3cd38f58c2ff78a2f2f13c5c22b637f9701e345
Merged-In: Ib3cd38f58c2ff78a2f2f13c5c22b637f9701e345

Refactor SoundEffectsHelper for asynchronous loading

Use a dedicated thread for all SoundPool and MediaPlayer
operations. This ensures that the client (AudioService) doesn't
get blocked or delayed. This also removes the need to use
locking as all access to data happens on the same thread.

Simplify the implementation of the mapping between UI effect IDs
and SoundPool sample IDs.

Add event logging for sound effects loading.

Bug: 135763139
Test: normal scenario
      media volume responsiveness with stuck SoundPool
      behavior when declared sound assets are missing
Change-Id: Ie5a1684a2cabe05ff53270ad97cab161a87a59f9
Merged-In: Ie5a1684a2cabe05ff53270ad97cab161a87a59f9

AudioService: factor out sound effects handling

Move the code related to sound effects loading and playback
into a separate class SoundEffectsHelper.

Bug: 135763139
Test: make and run phone
Change-Id: I30a7d897d740b02943c68924074f14fae3c3eff8
Merged-In: I30a7d897d740b02943c68924074f14fae3c3eff8

Use sendVolumeKeyEvent API instead of sendKeyEvent API

sendKeyEvent API always forwards the key control to the TV.
But for volume keys, we want it to be forwarded to the audio system if
system audio mode is on and if we are not a audio system itself.
sendVolumeKeyEvent can handle this logic.

Test: manual
Bug: 129116245
Change-Id: I6b81072ab49c092fac65cd542c19e611fc939cea
Merged-In: I6b81072ab49c092fac65cd542c19e611fc939cea
(cherry picked from commit ada40f59b63fe613448278a7cfbb7055146f8ccd)

Merge "Rename profileSystemServer to shouldProfileSystemServer"

Merge "AudioService: Add the AudioDeviceBroker's message queue to the dump"

WindowOrientationListener: Check if proposed rotation is in range

* Some OEMs *coughs OnePlus* decided it was a great idea to add a fourth kind of rotation ( flat )

Change-Id: I54c6e6360e897fbc8445edabbf411807521c624b

Track changes to CountryTimeZones API

Track changes to CountryTimeZones.lookupByOffsetWithBias()
API: the method now takes an extra argument: dstOffsetMillis,
which is unused in this case.

This commit also improves the parameter names to be more descriptive.

Bug: 139091367
Test: treehugger
Change-Id: I2090965640bb389ea41bde460f380f9ea8a6b176

Merge "Simplify APIs exposed for time zone lookups"

Simplify APIs exposed for time zone lookups

The libcore.timezone APIs may form the basis for some
new SystemApis. Before starting that process the API
surface is being rationalized to establish the core
use cases.

Test: Treehugger
Bug: 139091367
Change-Id: I6c8b791524d16223c414612913b22f4d70c27d61
Merged-In: I6c8b791524d16223c414612913b22f4d70c27d61
(cherry picked from commit 400efa36ef8666ad8341c36679a4f2cb9fd00f9f)

Merge "Revert "Fix: vendor public libraries are accessible via System.loadLibrary""

Swisscom VVM free SMS shortcodes CH

Visual Voicemail uses SMS for provision, these SMSs are free to the
user. To add the shortcode 30047 into whitelist to avoid the premium
SMS notification appear.

Bug: 133877441
Test: manual test
Change-Id: I2f7da544c2a948d1528781ebd11b75f027779e0d

Revert "Fix: vendor public libraries are accessible via System.loadLibrary"

This reverts commit cfe38cdb1cc45c8b7dcbe4f39329551f6602b9ce.

The CL 8f712189dfc02285573337e2b4ab17678011db14 in libcore project avoids
the need for adding the new paths in LoadedApk. With the CL, a classloader
does not give up even when loader.findLibrary() has failed due to some
paths are not added to it. Instead, the classloader converts the given
libname into a filename (e.g. foo -> libfoo.so) then calls dlopen()
with the filename. The classloader reports failure only when the dlopen()
call has failed. Therefore, manually adding these paths to the classloader
is no longer needed and thus removed.

Bug: 109720125
Test: System.loadLibrary("adsprpc") is successful in Pixel (because
libadsprpc.so is in Pixel's vendor public lib list)
Test: atest cts/tests/tests/jni

Merged-In: I1970eba7e732728699042a36b89571915ec81451
(cherry picked from commit 37131e1ee61dd83121ef02744855ee514f0aa6c5)
Change-Id: I1970eba7e732728699042a36b89571915ec81451

AudioService: Add the AudioDeviceBroker's message queue to the dump

Clogged queue means device-related requests will not be served.

Example dump snippet:

AudioDeviceBroker:
  Message handler (watch for unhandled messages):
    Handler (com.android.server.audio.AudioDeviceBroker$BrokerHandler) {753fed2} @ 359741
      Looper (AudioDeviceBroker, tid 115) {ed3aba3}
        (Total messages: 0, polling=true, quitting=false)

Bug: 135763139
Test: adb shell dumpsys audio
Change-Id: If2130a4bd8bd9536df01656fe1a8f94548912ab9
Merged-In: If2130a4bd8bd9536df01656fe1a8f94548912ab9

Merge "AudioService: Add the service's message queue to the dump"

Merge "audioservice: do not show volume panel when system audio is on"

AudioService: Add the service's message queue to the dump

Clogged queue means UI and other commands can not reach
the service and will cause volume controls to malfunction.

Example dump snippet:

Message handler (watch for unhandled messages):
  Handler (com.android.server.audio.AudioService$AudioHandler) {a904698} @ 61825
    Looper (AudioService, tid 118) {6808ef1}
      Message 0: { when=-52s689ms what=25 ... }
      Message 1: { when=-52s8ms what=8 ... }
      Message 2: { when=-52s4ms what=8 ... }
      ...
      Message 35: { when=-2s436ms what=0 ... }
      (Total messages: 36, polling=false, quitting=false)

This indicates that the command queue is not being processed
because the handler got stuck. Checking stacks of the system_server
can help in figuring out the reason.

Bug: 135763139
Test: adb shell dumpsys audio
Change-Id: I02009720668af218edb38d5a5948a5964b72c685
Merged-In: I02009720668af218edb38d5a5948a5964b72c685

Rename profileSystemServer to shouldProfileSystemServer

Makes it clearer and addresses previous CR comment.

Bug: 139883463
Test: make
Change-Id: I7c1a1d99cf307b89ee8f2c5c906168d196b35d0e

Merge "Add phenotype property for profiling system server"

audioservice: do not show volume panel when system audio is on

Do not show volume panel on TVs when system audio mode is ON.

Bug: 130524216
Test: make
Change-Id: Id8730455ee79f0ea3fa4c23e0a8b6bd2791eb2d3
Merged-In: Id8730455ee79f0ea3fa4c23e0a8b6bd2791eb2d3
(cherry picked from commit db961d82edee13b7f2a1e83f451c4f20430f8fce)

Merge "Fix regression in PFD#fromData"

Merge "Add TelephonyManager APIs to take physical slotIndex - getSimCardState - getSimApplicationState"

Remove libhwbinder/libhidltransport deps

Since these were combined into libhidlbase.

Bug: 135686713
Test: build only (libhwbinder/libhidltransport are empty)
Change-Id: I136e4d4459bb8c3c5abf3c0758dd5ae14d0ad2a2

Add phenotype property for profiling system server

Added the property profilesystemserver in the RUNTIME_NATIVE_BOOT
namespace. This property is overrides the system one if it is
present.

Bug: 139883463
Test: set the property manually and verify that system server is started
Test: with profiling

(cherry picked from commit 7b31c74ddb844e2f0af79be6c87b9fd02a627987)

Merged-In: Ifd69530e52a717a381b3f91b15a74329614906f2
Change-Id: I00594949b845a75152c7ab3c94aa84a55b072776

Add carrier config KEY_CALL_FORWARDING_WHEN_UNREACHABLE_SUPPORTED_BOOL

Adds carrier config KEY_CALL_FORWARDING_UNREACHABLE_SUPPORTED_BOOL.
Call forwarding option "When Unreachable" in Call Settings will be
disabled and greyed if this value is set to false.

Default value is true, meaning that the option will be available for
most carriers.

Bug: 118861990
Test: Manual
Change-Id: Ib1e525d6c3983f0675c1730defaa18832c504a43

Reconcile Zygote process name code with Q release.

Some Zygote code that was pushed to AOSP master is different from the
pushed Q release. In particular, a call to set the process name for a
Zygote child was missing, causing an app zygote test to fail.

Bug: 139535125
Test: atest android.app.cts.ServiceTest#testAppZygoteServices
Change-Id: I53b0bc0116f1573cb1e52a998e10346bd5601f67
Merged-In: I2bce277ff8f2de4614e19d5385fe6712b076f9c9

Re-enable testAppZygoteServices in pre-submit.

Code that caused it to fail was fixed.

Bug: 139535125
Test: atest android.app.cts.ServiceTest#testAppZygoteServices

Change-Id: I5f8eed38d4a8a56e750a6bb7fa7d397d9cd4c13d
Merged-In: I2bce277ff8f2de4614e19d5385fe6712b076f9c9
Exempt-From-Owner-Approval: Re-enabling fixed TEST_MAPPING tests
for AOSP after Q push.

Merge "Crash app on foreground service notification error."

Add TelephonyManager APIs to take physical slotIndex
- getSimCardState
- getSimApplicationState

Bug: 134627366
Test: Manual test on device
Change-Id: I72612afb9f8ff3d898db99afaaee0c0d3654f754
Merged-In: I72612afb9f8ff3d898db99afaaee0c0d3654f754

Don't clear the output array if it failed to open the proc status file

According to the original documentation, the functions in Process.java
should return -1 when the process doesn't exist,this should be detected
by Process.readProcLine() when it fails to open the /proc/<pid>/status
file. Unfortunately readProcLine() zeroes-out the output array before
attempting to open the file, so the expected return value -1 is changed
to be 0  when the process doesn't exist. It's inconsistent with the
interface description definition, so correct it.

Test: try to use Process.getUidForPid(int pid) api to read a non-existent
process pid's uid, it will return -1 instead of 0.

Merged-In: Ic2ac7defea8b15dd65f907d04e168a1283a4f621
Change-Id: Ic2ac7defea8b15dd65f907d04e168a1283a4f621
Signed-off-by: qiwang <wangqi_a@xiaomi.com>

Convert statsd_test and statsd_benchmark to proto lite

libplatformprotos for the device is compiled using proto lite, and
statsd_test and statsd_benchmark have protos that reference the
libplatformprotos.  With protobuf 3.9.1 compiling a full proto
against a lite proto doesn't work.  Convert them to lite.  Requires
adding the protobuf internal protos to the sources,
libprotobuf-cpp-lite does not contain the compiled version of
field_options.proto.

Bug: 117607748
Test: m checkbuild
Change-Id: I0b101555ff604df17b2e3e06f3b3ce25adb1c9fb
Merged-In: I0b101555ff604df17b2e3e06f3b3ce25adb1c9fb

Adapt to google::protobuf::uint64 type change

Protobuf 3.9.1 redefines google::protobuf::uint64 from unsigned long
long to uint64_t, which is sometimes unsigned long and sometimes
unsigned long long.  Use PRIu64 to print it, and add an implementation
of ProtoOutputStream::write for long.

Bug: 117607748
Test: m checkbuild
Change-Id: Ib2d3f4e17857f8ccbbe342ce6678e76b591df510
Merged-In: Ib2d3f4e17857f8ccbbe342ce6678e76b591df510

Merge "[view compiler] Add test coverage for layouts compiled from APK"

Merge "Re-deprecating Capabilities in ImsFeature"

HwBinder: don't rely on kernel headers.

Kernel headers are no longer exported from libhwbinder, rely on the API
it exports instead (IBinder::FLAG_ONEWAY instead of TF_ONEWAY).

Bug: 124524556
Test: automated boot test is sufficient
Change-Id: I4d040dbde32d25c21437e06fa805eb0f10c43532

Merge "Support maximum wait time for Cell broadcast"

Crash app on foreground service notification error.

Resolved issue 118612296.

On any notification error, the NMS silently cancels the notification,
including foreground service notifications. Thus, an app could pass in a
garbage notification deliberately and start a foreground service silently.

This patch resolved this issue by judging the notification's flag, and if
it is a foreground notification, still crash the app as previous platforms,
and if it is a normal notification, don't crash the app.

Background: In 3ad4cdd1, which was merged into Android 9 release, the crash
behaviour is removed. But it is an important rule that foreground services
guaranteed to show an ongoing notification.

Test: Run the sample apk provided in the issue, it's main thread received
 a RemoteServiceException: Bad notification posted from package...
as intended behaviour.

Bug: 118612296
Merged-In: I94d9de50bb03c33666471e3dbd9c721e9278f7cb
Change-Id: Ice6faab055e2dacd3d2d12803fcf51e13d3b0f21