Merge "Deprecate NetworkInfo"

Merge "Fix/suppress startop google-explicit-constructor warnings"

Merge "Fix/suppress renderthread google-explicit-constructor warnings"

Merge "whitelist pri permission for carrierconfig app"

Merge "new SystemAPI for preciseCallState and DisconnectCause"

Merge "Moved data settings to phone"

Merge "Public SIM_APN_URI uri and deprecated mcc, mnc, mvno_type, mvno_data"

Merge "Fix/suppress core/jni google-explicit-constructor warnings"

whitelist pri permission for carrierconfig app

Bug: 121350842
Test: Build
Change-Id: If804e18265a917e841d6afc1b7a450321cc1cc77

Merge "Add APIs to indicate which subscriptions should be hidden from user"

Merge changes from topic "118349116"

* changes:
  Fix broken building offline-sdk-docs.
  Add APIs to remove sub from a group and get subs in the same gorup.

Merge "Changed uid output parameter from an int array to a list of strings."

Moved data settings to phone

Since there are multiple DcTrackers, but we only have
one set of data settings. So we need to move them from
DcTracker to Phone.

Test: Telephony sanity tests + unit tests
Bug: 73659459

Merged-In: Ica24b708beb63156937e88198a4c915485771b0a
Change-Id: Ica24b708beb63156937e88198a4c915485771b0a
(cherry picked from commit 335860e44f6c7d040646be9d50c862b9c6488dec)

Fix/suppress renderthread google-explicit-constructor warnings

* Add explicit to conversion constructors/operators
* Use NOLINT or NOLINTNEXTLINE to suppress warnings on intended converters

Bug: 28341362
Test: make with WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=-*,google-explicit-constructor
Change-Id: Ib7d634bf1e0e9a1dda35b0a1d92de4d460193296

Merge changes from topic "fw_container"

* changes:
  DO NOT MERGE Set ContainerLayer for buffer-less surface
  DO NOT MERGE: WM: Restrict SC Builder to set a single surface type
  Implement construction of container layers

Merge "Statsd: Enable host_supported for libstatslog"

Fix/suppress startop google-explicit-constructor warnings

* Add explicit to conversion constructors/operators

Bug: 28341362
Test: make with WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=-*,google-explicit-constructor
Change-Id: I9e7b929cfa65c68f4cdb4710c1c5c6760d56e4d5

Fix/suppress core/jni google-explicit-constructor warnings

* Add explicit to conversion constructors/operators

Bug: 28341362
Test: make with WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=-*,google-explicit-constructor
Change-Id: I15ac3511154b652402c5a21a22ff0c306d806d84

new SystemAPI for preciseCallState and DisconnectCause

1. decouple disconnectCause and preciseDisconnectCuase from
 preciseCallState.
 2. expose preciseCallState to SystemAPI and expose
onPreciseCallStateChanged to System API
3. remove/hide ImsReasonInfo from PreciseDisconnectCuase
4. expose DiconnectCuase and PreciseDisconnectCause to System API
5. new onDisconnectCauseChanged API on PhoneStateListener for CS call
 and use ImsCallSessionListener for IMS call(imsreasonInfo)
6. add read_precise_phone_state to shell apk to enable GTS test

Bug: 119890559
Test: atest GtsGmscoreHostTestCases:TelephonyHostTest#testPhoneStateListener
Change-Id: If9256feff16b6bcaef1dfe194c2b0df2a48436b0
(cherry picked from commit 6a942d488af3356fe84b0cef396e613d8d74101e)
Merged-in: If9256feff16b6bcaef1dfe194c2b0df2a48436b0

Merge "Adds vishnun to owners files for wm/am"

Merge "Annotate two non-SDK methods known to be used by apps"

Annotate two non-SDK methods known to be used by apps

Exempted-From-Owner-Approval: Mechanical changes to the codebase
which have been approved by Android API council and announced on
android-eng@

Bug: 121237128
Bug: 120783643
Test: m appcompat
Change-Id: Ib7a8bdf3151290aa8a5ca85dc8650612432f0d59

Merge "Use ICU in libandroidicu"

Merge "Show version for APEXes"

Statsd: Enable host_supported for libstatslog

* libutils must be used as a static library when compiled on host
* Host does not have Android system properties and hence we cannot
  use <cutils/properties.h>. In fact, properties.cpp is not even
  compiled on host for libcutils. Therefore, this CL adds a check
  for __ANDROID__ macro before including <sys/propoerties.h> and
  before calling properties_get_bool()
* On host, statsd logging will be disabled since host does not
  use statsd for anything

Fixes: 121294178
Test: test drive statsd
Change-Id: I838ff02468c650c5f7d85e68fa5008b98f08ce8c

Deprecate NetworkInfo

Currently NetworkInfo is used by Apps to get information of
network. However, to get such information, Apps need to poll
NetworkInfo frequently from ConnectivityService.

In order to increase the stability and reduce the maintain
effort, all functionalities provided by NetworkInfo are targeted
to be replaced or removed entirely.

Apps should use ConnectivityManager.NetworkCallback instead, to
get faster and more detailed updates from connectivity changes.

Or, apps could use getNetworkCapabilities or getLinkProperties
to get information synchronously, but should not mix the
callbacks and synchronous methods together.

Bug: 113629330
Test: atest FrameworksNetTests
Change-Id: Ie8faf620958c3fa0a4a2f233b35b825de0e99ffc

Merge "Revert "[framework] Compile secondary dex files in isolation""

Revert "[framework] Compile secondary dex files in isolation"

This reverts commit 8a5a708df18c2377c7ec3cb03d406bdc9cb99bd6.
ART does not compile secondary dex files in the user process
any more. The original issue of a large overhead of providing
a class loader context is therefore not relevant any more and
we can start passing the context to installd.

Note that this also partially reverts commit
6dba50d63375b6738c6e1ad8a8758f181d295d8b which restricted
visibility of methods in PackageDexUsage. The commit also
removed data collection for unsupported class loaders. The
logic in PackageDexOptimizer was adjusted accordingly.

Bug: 64530081
Bug: 111174995
Test: manual
Change-Id: Id78b5a6d8841b199c12a63a8d45d12efbcc32275

Merge "Include annotation property values in output."

Include annotation property values in output.

When editing annotations, we want the ability *not* to overwrite any
existing annotation properties already in place. Include any properties
set on the annotation in the output, so that the edit_annotations script
can know that they're there.

The annotation properties are encoded like URL query parameters for
convenience; it makes them easy to encode here & subsequently decode on
the other side (in Python).

Test: m framework-annotation-proc & inspect output.

(cherry picked from commit bd7077065c211f49b090aa4250b53b58461adfd9)

Merged-In: I71fb1215ad2790751be336b4955c163bb323a4a6
Change-Id: I0b33e2b379076346ce258d93a9225a9143b7d91a

Public SIM_APN_URI uri and deprecated mcc, mnc, mvno_type, mvno_data

Deprecated legacy mno and mvno columns to support carrier id and public
the SIM_APN_URI to provide app more simplier query current SIM's APN.

Bug: 120031619
Test: Build pass.
Change-Id: Ie06dfc55248b871c45ff3274de17d6eafd82d846

Merge "refactor dataFailCause"

refactor dataFailCause

1. add comments for each data fail cause.
2. refactor dataFailCause from enum to int

Bug: 119891533
Test: Build & unit test
Change-Id: Ic453f80a3d0ccd22c6c4c1f4b2c90e2bf40cfbe1
(cherry picked from commit 3016167192d301380120b4eb47cd051200c8b5b8)
Merged-in: Ic453f80a3d0ccd22c6c4c1f4b2c90e2bf40cfbe1

Add APIs to indicate which subscriptions should be hidden from user

Bug: 120573892
Test: manual
Change-Id: I46b20241b71fef2938d7340fcf0b02173d83ed87
Merged-In: I46b20241b71fef2938d7340fcf0b02173d83ed87

Fix broken building offline-sdk-docs.

TelephonyManager#hasCarrierPrivileges(int) is hidden API so it
shouldn't be referenced in public API descriptions.

Bug: 120622105
Test: none
Change-Id: Ia6b6ec2c16b824b8eb092920089cb77cdc54bfb7
Merged-In: Ia6b6ec2c16b824b8eb092920089cb77cdc54bfb7

Add APIs to remove sub from a group and get subs in the same gorup.

Add APIs in SubscriptionController so that caller with permissions
can remove subscriptions from a group or get all subscriptions with
the same group.

Bug: 118349116
Test: unittest
Change-Id: Iba4d31b437b372b3f41a6ed23f03b96a685a324c
Merged-In: Iba4d31b437b372b3f41a6ed23f03b96a685a324c

Adds vishnun to owners files for wm/am

Test: n/a
Bug: n/a
Change-Id: Ie17a280d8f4435490b039a8c95f65cd78a8d6cd1

Merge changes from topics "dialer-account-suggestion-3", "dialer-account-suggestion-2"

* changes:
  Enable CTS testing for phone account suggestions
  Add the PhoneAccountSuggestionService

Merge "Annotate getMobileNetworkOperator() as Nullable"

Merge "Annotate CallbackExecutor in requestCellInfoUpdate"

Merge "add carrierid in subscriptionInfo"

Merge "Fixed toString() printing error"

Merge "Convert to ISO 639-2/B for menu language"

Merge "Following commit for asynchronous DNS query API"

Merge "Use the boot class loader as parent of system server loader."

add carrierid in subscriptionInfo

add carrierid in subscritionInfo so that apps can get
subId, carrierid and other sim related info from a single call

Bug: 119676543
Test: gts SubscriptionInfoTest
Change-Id: Ia2d01ad6bd59565adbf989b4d95e7f64f3ee6d45
(cherry picked from commit 0034bb393ca7054bfebb606cedb823e8a49a1920)
Merged-in: Ia2d01ad6bd59565adbf989b4d95e7f64f3ee6d45

Merge "BluetoothManagerService: catch NoSuchElementException for unlinkToDeath"

Merge "Print carrier Id information in toString()"

Merge "[view-compiler] DexBuilder: Add check-cast instruction"

Annotate CallbackExecutor in requestCellInfoUpdate

Add a missing annotation for CallbackExecutor in
requestCellInfoUpdate().

Bug: 120489428
Test: compilation (no functional change)
Change-Id: I33bba9eb59f81aaefa0194b58936d4f809a8d51f

Annotate getMobileNetworkOperator() as Nullable

Add Nullable annotations for the classes that support
getMobileNetworkOperator().

Bug: 118675404
Test: compilation (no functional change)
Change-Id: I914ec8242aff2f578dfe1f97f2e7cfe2d109c7dc

Use ICU in libandroidicu

libicuuc and libicui18n are moved into APEX, but
they have no stable ABI due to the version suffix.

Use libandroidicu which provides stable symbol.
See http://go/apex-stable-icu4c-interface for the design.
See http://aosp/801855 for libandroidicu implementation.

Bug: 117094880
Test: m checkbuild
Change-Id: I1a8e5d7ddb0beed61257706003d93ac4d5f9566f

Merge "NativeHandle.java: dup() with O_CLOEXEC"

[view-compiler] DexBuilder: Add check-cast instruction

Bug: 111895153
Test: atest

Change-Id: I767e56713fab6beaa6970e58c4fc4d3560cc1304

Merge "Add InetAddresses class to API"

Merge "Remove 'hasDns' parameter"

Merge "Replace InterfaceParams with ifName in DhcpServer"

Merge "Add new netd event callback for changes to NAT64 prefix"

BluetoothManagerService: catch NoSuchElementException for unlinkToDeath

linkToDeath fail will cause NoSuchElementException when following
unlinkToDeath, add try catch to handle it.

Bug: 121104808
Test: build pass
Change-Id: Idc4959331841946e2fc099bdc0fab7c753caf369

Enable CTS testing for phone account suggestions

Add TestApi annotation on PhoneAccountSuggestionService and add sensible
equals and hashCode methods to PhoneAccountSuggestions

Bug: 111455117
Test: CTS
Change-Id: I112e350cc637cb086252c53be5e08596e743537d

Add the PhoneAccountSuggestionService

Add a service base class for phone acct suggestion. Also add associated
aidl plumbing and a new permission for the service to require.

Test: CTS to come
Bug: 111455117
Change-Id: I30d7d004bdddbab3dfbb5408c07775ea56d490fe

NativeHandle.java: dup() with O_CLOEXEC

Don't use Os.dup(), as it creates file handles which leak across exec()
boundaries. Instead, use fcntl(F_DUPFD_CLOEXEC);

O_CLOEXEC is essential for ensuring that file descriptors do not leak
across an exec() boundary. Setting O_CLOEXEC ensures that file
descriptors can't linger around unnecessarily in an exec()ed process
which doesn't use them, making more efficient use of resources.
Additionally, O_CLOEXEC is important in ensuring that untrusted
exec()ed code cannot take advantage of leaked file descriptors.

Test: Android compiles and boots
Bug: 120983106
Change-Id: I99a66834cc6b9bb25e1b4daf75384ec6a91ae9e2

Merge "Fix performance-for-range-copy warnings"

Merge "[view-compiler] Add layout validation"

Merge "expose Carrier Id related APIs"

Fix performance-for-range-copy warnings

Bug: 30413223
Test: make with WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=-*,performance*
Change-Id: Ie481e88025a7a1f3abde8ff63420d5ccd8577e52

Merge "Add FrequencyRange comparator to ServiceState"

Merge "public API for VideoProfile.CameraCapability with zoom"

Merge "[view-compiler] DexBuilder: Add more instructions"

Merge "Remove uses of libcore.io DropBox and EventLogger"

Remove uses of libcore.io DropBox and EventLogger

DropBox and EventLogger provide hooks for use by framework to intercept
messages sent by libcore. However, neither of these mechanisms are
actually used by libcore so there is no point in framework code using
it.

(cherry picked from commit ecb6124d540975d8f2466bc9d9bf4b79bc6ff972)

Bug: 119748341
Test: make checkbuild, flash
Merged-In: I9dbd1b6dea92f7441efb6d0a579efd13bc0e5139
Change-Id: I65425e49ab9ad9752b65772d27f788d1a674eeff

Add InetAddresses class to API

Adds methods needed by app developers to avoid them having to call
non-SDK APIs.

Test: tests added to cts
Bug: 78686891
Change-Id: Ic0eadfdab2b111d5ca4becb753f4fe3557806433

Following commit for asynchronous DNS query API

Test: built, flashed, booted
      atest CtsNativeNetDnsTestCases

Change-Id: Ia0f54ff684d2ad37ca82d94bad4e4dd582783d85

Merge "fd_utils: carry over O_CLOEXEC on duplicated FDs"

Print carrier Id information in toString()

Bug: 115709816
Test: Build pass
Change-Id: Idd3336c472860ad21121029abecb91aa9b8325ec
Merged-In: Idd3336c472860ad21121029abecb91aa9b8325ec

Replace InterfaceParams with ifName in DhcpServer

The current code only uses InterfaceParams#name, and InterfaceParams is
defined in services/net which DhcpServer cannot depend on once moved to
a separate app.

Test: atest FrameworksNetTests
Bug: b/112869080
Change-Id: I94c7dce33200c111666a9dddde82ac2e66a6794f

Show version for APEXes

--show-versioncode now works for APEXes as well.

Bug: 117589375
Test: $ cmd package list packages --apex-only --show-versioncode
package:com.android.conscrypt versionCode:1
package:com.android.resolv versionCode:1
package:com.android.runtime versionCode:1
package:com.android.tzdata versionCode:31000010

Change-Id: I3f29bb560cdbd54d7e8377d09ec261a569f35901

Merge changes from topic "statsd-allow-bytes-type"

* changes:
  Add byte size of the bytes fields to statslog c++ APIs.
  Skip writing message field in an atom if it's empty
  Adding tests and move launcher.proto
  Add launcher.proto to atoms.proto
  Allow atoms to log fields in bytes format.

Add byte size of the bytes fields to statslog c++ APIs.

The proto binary data can contain '\0's and in the native layer,
the current liblog api would convert that into string and thus
the data is truncated.

This CL adds a "size_t bytes_field_len" after the bytes fields so that
we can correctly pass the data from JAVA to native.

Java StatsLog.write() APIs remain the same

Bug: 120635548
Test: test_drive with atom 103
Change-Id: I34f1c4ddd6a4ec5f3604b0c67a47a5399e3c6ddd
Merged-In: I34f1c4ddd6a4ec5f3604b0c67a47a5399e3c6ddd
(cherry picked from commit 1fe9f594984b47144b958f0ac423eeffc56b62ea)

Skip writing message field in an atom if it's empty

Test: unit test added
Bug: 120635548
Change-Id: I825b1ce526944a20fe65705508ad180ece37492c
Merged-In: I825b1ce526944a20fe65705508ad180ece37492c
(cherry picked from commit 8e6f9983009eadf32d05ad65e38906d3985bc9ba)

Adding tests and move launcher.proto

Test: statsd_test
Bug: 120635548
Change-Id: I0655f70098d78f8f6dc9381921c4a687d2195bd4
Merged-In: I0655f70098d78f8f6dc9381921c4a687d2195bd4
(cherry picked from commit 8c4338614513f59e53c2d7a164a05d309a470e81)

Add launcher.proto to atoms.proto

Test: blank
Bug: 113043444
Bug: 120635548
Change-Id: I7e0e72cdf68dcf195188dbab8fe2567fcbd5964e
Merged-In: I7e0e72cdf68dcf195188dbab8fe2567fcbd5964e
(cherry picked from commit c6d6b77ddc805d6e8f70ed16f0c73177840791e7)

Allow atoms to log fields in bytes format.

There are an increasing number of requests to log data in complex format to statsd, while the data
is not expected to be parsed or aggregated by statsd and only to be uploaded as events.

Instead of making an exception for each of these cases in a hard coded way, this CL add a feature to
annotate these field in atoms.proto and the stats-log-api-gen tool will produce byte array
interfaces for them.

Note that log_msg does not have byte array type, and only has string type, when statsd receives the
log, these fields are in string type. Only when the atom is written to proto, we will check if this
field should be bytes field and write it to protobuf in message format.

Change-Id: If53dd95c5826710c76d7fe982bf951a435dfc738
Merged-In: If53dd95c5826710c76d7fe982bf951a435dfc738
Fix: 118386797
Bug: 120635548
Test: unit test & manual test
(cherry picked from commit bbdd67d19f4912fbec00220b22e44c68eff5ab3f)

fd_utils: carry over O_CLOEXEC on duplicated FDs

Due to b/30963384, every time zygote creates a new child, zygote reopens
all existing file descriptors, taking careful measures to preserve the
file status flags, file descriptor flags, and seek offset. However, dup2
resets the sole file descriptor flag (FD_CLOEXEC) on duplication,
defeating the hard work done to preserve the flag.

From "man dup"

  NAME
    dup, dup2, dup3 - duplicate a file descriptor

  DESCRIPTION
    The dup() system call creates a copy ...
    ...
    The two file descriptors do not share file descriptor flags
    (the  close-on-exec flag).  The close-on-exec flag
    (FD_CLOEXEC; see fcntl(2)) for the duplicate descriptor is off.
    ...
    The  dup2() system call performs the same task as dup() ...

Use dup3 instead to allow us to preserve the FD_CLOEXEC status.

Bug: 120983106
Test: Android compiles and boots
Change-Id: Idbb27c83092f30d8394c254cfbdf33406f74eb94

Merge "api/current.txt: Add SOCK_CLOEXEC and SOCK_NONBLOCK"

public API for VideoProfile.CameraCapability with zoom

Bug: 120796998
Test: Build & CTS videoprofiletest
Change-Id: I12b3d1c510065666e2fdf5186d924e657f66d039

Merge "Renamed data/net service provider onDestroy to close"

Add FrequencyRange comparator to ServiceState

The FrequencyRange is not comparable beucase it's an enum. This change
add a comparator to ServiceState to make the enum comparable.

Bug: 111453000
Test: atest FrameworksTelephonyTests
Merged-In: I59a595bdaf5451d2c4d95b1b5bdfefcba123fea7
Change-Id: I59a595bdaf5451d2c4d95b1b5bdfefcba123fea7

Merge "API: Enable UnhiddenSystemApi check for frameworks/base"

[view-compiler] Add layout validation

Layout validation walks over a layout resource and verifies that the layout only
uses features the layout compiler supports.

Currently this means we reject layouts that use any of the following tags:
  * include
  * fragment
  * merge
  * view

In the future, we will extend the layout compiler to handle these, but for now
we want to be liberal in what we reject.

Bug: 111895153
Test: atest view-compiler-tests
Change-Id: I8dca30c035a83b6763497a36fc60c68438fa1b0c

[view-compiler] DexBuilder: Add more instructions

This CL adds the ability to generate code that calls static and virtual methods
which return objects, as well as the not-equal-to-zero comparison operator.

Bug: 111895153
Change-Id: I4ae9b3cb2edc6540671112b73c02bf6380d23051

Merge "Update CallScreeningService API docs."

API: Enable UnhiddenSystemApi check for frameworks/base

Test: m checkapi
Bug: 115333477
Change-Id: I84d454f3495d7ab8e5f0bc2e4c24725f1cd2324f

Renamed data/net service provider onDestroy to close

Renamed them to avoid confusion with Android services'
onDestroy().

Test: Telephony sanity tests
Bug: 73659459
Change-Id: I05f68ced65498d6bfeb9e3c9ed3fb6164643bbaa

Merge "Covert list<String> into String[] in INetd"

Merge "apf: Drop ARP reply if SPA is 0.0.0.0"

Merge "move and rename DcFailCause to DataFailCause"

Merge changes from topic "schematize_car_props"

* changes:
  Fix missing schematized Car system properties
  Schematize Car system properties

api/current.txt: Add SOCK_CLOEXEC and SOCK_NONBLOCK

Android makes the function socketpair() publicly visible, but never made
SOCK_CLOEXEC nor SOCK_NONBLOCK visible. Java socketpair() users have no
approved flag to ensure sockets are immediately created with O_CLOEXEC,
nor set the non-blocking status on sockets.

socketpair() documentation:
https://developer.android.com/reference/android/system/Os.html#socketpair(int,%20int,%20int,%20java.io.FileDescriptor,%20java.io.FileDescriptor)

Test: compiles
Bug: 120983106
Change-Id: Id7d1c853f3b00d6766ecf6dd88141da136011d6b

Merge "Propagation of additional Ims Call Fail Causes"