Merge "Fixes the system server crash issues caused by null pointer in NetworkPolicyManagerService."

Merge "While turning OFF do not honor ON requests."

While turning OFF do not honor ON requests.

Native stack does not handle being put from OFF -> ON state without
doing a complete cleanup. Hence instead of going from start -> ON -> OFF
-> cleanup it goes start -> ON -> OFF ->ON -> ... usually leads to race
conditions down the road in native.

This patch is a workaround so that we can throw away the requests if we
are in currently "turning off" phase. The side-effect would be that user
will need to turn it ON again. The race happens when the turn OFF time
is longer but usually it is found to be close to order of seconds hence
the wait should be bounded.

Bug: b/28318203
Change-Id: I14f6633f31311e5b561e1dcbc8a9d6d2a5dd6fdc

Fixes the system server crash issues caused by null pointer in
NetworkPolicyManagerService.

if the multi user occurs switch and remove then to getPackagesForUid
the return value will got null,because the user remove is not synced
with the getPackagesForUid,so we must check the return value for
getPackagesForUid to avoid the crash.

https://code.google.com/p/android/issues/detail?id=211972

Change-Id: If921582da86dbe862f2dad337b0a8d1f20593c99
Signed-off-by: songjinshi <songjinshi@xiaomi.com>

Merge "Track lilbcore change 3ad0f67b98b61cc3f047e3cf6fe8f94d5f2ef5e5"

Merge "Revert "Merge "Add 900 for Russia - Short code used by Sberbank."""

Revert "Merge "Add 900 for Russia - Short code used by Sberbank.""

This reverts commit 9a769a482861496686a4138a0a9b612a08fce8f8, reversing
changes made to 6fb921a61b0ca6bac8002e0fdbeb521453dce408.

Merge "Add 900 for Russia - Short code used by Sberbank."

Add 900 for Russia - Short code used by Sberbank.

Sberbank SMS banking solution uses 900 short code.
They have requested this shortcode to be listed has premium to reduce banking fraud.

bug: 28423611

Change-Id: I82dfe012a1e55b2d025c24e50149fa01551a92ed
Signed-off-by: Vasco Brito <vascob@google.com>

Merge "Revert "AlarmManagerService: remove legacy /dev/alarm support""

Revert "AlarmManagerService: remove legacy /dev/alarm support"

This reverts commit f68cdb9cb4841765d9d55d4a95259e9c3bb3ecc1.

Change-Id: I13e954656771f32c6156b4c4adb52922bcd0717a

Merge "AlarmManagerService: remove legacy /dev/alarm support"

AlarmManagerService: remove legacy /dev/alarm support

We've removed the Android alarm driver from our supported kernels.  It's
time to remove the userspace callers too: they make the native side of
AlarmManagerService more complex, and send the wrong signal about
/dev/alarm still being supported.

Bug: 28357356

Change-Id: I225d76c3f4e695a831d4419e050e8e178e1224b5
Signed-off-by: Greg Hackmann <ghackmann@google.com>

Merge "Fix misc-macro-parentheses warnings in DngCreator."

Merge "Chronometer: Update the value only if displayed on screen"

Merge "[RenderScript] Fix ScriptIntrinsicBlur documentation."

Merge "Update documentation about copyTo and copyFrom."

[RenderScript] Fix ScriptIntrinsicBlur documentation.

Bug: 28177406
Change-Id: Icccf35857cf29f827a78704e7751c8791cda0d7b
(cherry picked from commit c242fa6b5d7b8624b5ce710c1e2740c5bb9679dd)

Update documentation about copyTo and copyFrom.

Bug: 23159764
Bug: 26862970

  - Add detailed description of AutoPadding
  - Add comments to all the copy related APIs.
  - Fix typos in the comments.

Change-Id: I2d045e0d90efd94f1407f88d3e35bcd42ea93fb9
(cherry picked from commit 3231e8e0220614f7b1628493da5f276f19faab7c)

Merge "Add missing "try ... finally" safeguards"

Add missing "try ... finally" safeguards

Safeguards for code protected by ReentrantReadWriteLock.

Bug: 28734075
Bug: 28799467
Change-Id: Ib7f598a92e8df6bd855ca48cdd094c1c73a935f2
(cherry picked from commit e957a8a0b4100d001f79c866e7904d2426ac8da0)

Merge "Reduced the impact of "synchronized" statements"

Reduced the impact of "synchronized" statements

* Removed "synchronized" statements that are not needed
* Replaced "synchronized" statements with Read/Write lock as
 appropriate. The lock protects the access to and the setting of
 BluetoothAdapter.mService and BluetoothManagerService.mBluetooth and
 associated state.

Bug: 28734075
Bug: 28799467
Change-Id: I8f8281c505f0a1ae0add1e14a3caba1f5b2a98e4
(cherry picked from commit eb50a39e98acb78d16465041bb5c172aa1637e97)

Merge "Fix misc-macro-parentheses warnings in common_time."

Merge "Fix misc-macro-parentheses warnings in audio media jni."

Merge "Add free short codes for Direct Carrier Billing."

Merge "ZygoteInit: install AndroidKeyStoreProvider in the Zygote"

ZygoteInit: install AndroidKeyStoreProvider in the Zygote

Instead of in activity thread. That way, we can warm up (ie,
precompute cached values) this provider and AndroidBCWorkaroundProvider
(which are installed together) so that the computation doesn't
happen in the app. As a result, the time spent in the first call to
SSLSocketFactory.getDefault() decreases by ~5ms in angler userdebug.
Measured with an app calling SSLSocketFactory.getDefault in onCreate
and timed it with System.currentTimeMillis() .

(cherry picked from commit 69de32071c8ca7fa2277e4340a320b7df6fe963d)

Bug: 28545496
Change-Id: Ic4e11d058fb404eaa92db925a3e765fc3bef7ae2

Add free short codes for Direct Carrier Billing.

All short codes are used by DCB on Google Play. Fixed default pattern for countries added recently.
Internal bug b/27911552.

Change-Id: I9b4a2bb30f2c6de058cc2adb75b67c4f69182198
Signed-off-by: Vasco Brito <vascob@google.com>

Merge "Fix misc-macro-parentheses warnings in aapt and androidfw."

Merge "Fix misc-macro-parentheses warnings in services jni."

Merge "Fix misc-macro-parentheses warnings in hwui and graphic jni."

Fix misc-macro-parentheses warnings in services jni.

Bug: 28705665
Change-Id: Iaf965875b667e45e1557e447de7988f729adfd52

Fix misc-macro-parentheses warnings in hwui and graphic jni.

Bug: 28705665
Change-Id: I7d1eb38a713fe6cc7f41a3cc3f2f9d0aa0114b2c

Fix misc-macro-parentheses warnings in common_time.

Bug: 28705665
Change-Id: Ibdab4631f0e692b8a291faecdeb4a6062f906ea5

Fix misc-macro-parentheses warnings in aapt and androidfw.

Bug: 28705665
Change-Id: Iac353e66718aadad384183cf300330695df25909

Fix misc-macro-parentheses warnings in audio media jni.

Bug: 28705665
Change-Id: Iff476943a0a2c2ae68778d1f4bca5d3e9e04bd26

Fix misc-macro-parentheses warnings in DngCreator.

Bug: 28705665
Change-Id: Ia9b513411456b34b3e508793166ddb17e34dec13

Merge "Fix GATT autoConnect race condition"

Fix GATT autoConnect race condition

As pointed out here:
https://code.google.com/p/android/issues/detail?id=69834

registerApp() causes onClientRegistered() to happen before autoConnect
is set. This patch fixes that.

Bug: 28861330
Change-Id: Ie1174c0f224f5084178439420b383164d22d542c

Track lilbcore change 3ad0f67b98b61cc3f047e3cf6fe8f94d5f2ef5e5

Bug: 27331465
Change-Id: I1a3dd1a550b081535eb8bf79b43edff46da58d6b

Merge "Make fake libart for misbehaving apps be loaded."

Make fake libart for misbehaving apps be loaded.

Bug: 27775991
Change-Id: Ie4eba211e876ec0643ce6ecf6a110f3cb5e0f57e
(cherry picked from commit 20ed24fc0cbbada36e84dc1f1e86946db6e6c5a4)

Merge "Add support for ICU data pinning in the Zygote"

Add support for ICU data pinning in the Zygote

Upstream ICU caches use SoftReferences. On Android this means
that useful cached data initialized in the Zygote are "lost" when
the Zygote GCs and cannot be shared with apps. This change makes use
of an Android patch to ICU to ensure References created during
Zygote initialization are "strong". i.e. they are never collected.
This prevents them being GCd and ensures they can be shared between
applications.

After switching ICU to use strong references, this change
also creates DecimalFormatSymbols objects for common ULocales
(ROOT, US and the user's default, if different). DecimalFormatSymbols
makes use of an ICU Reference cache and this alone has been shown to
improve the construction time of java.text.DecimalFormat by 1-1.5
milliseconds on a Seed device. This saving applies the first time one
is created in each app for each locale, and again if SoftReferences
have been cleared.

The cost to the heap size of the Zygote has been measured at ~107k.
This value will change as more caches are switched to use the new
CacheValue class.

Formatting is typically performed on the UI thread and the intention
of this change is to reduce app start up time and jank in apps like
the Dialer which do a lot of formatting when scrolling lists. The
change may also enable more virtual memory page-sharing between
apps, though this is not the specific goal.

Bug: 28326526
Change-Id: Ia2c73f6525f05b1aa81e57a31eed1616decf6bb5

Merge "ZygoteInit: warm up JCA providers during preload"

Merge "Synchronize access of mSubscriptions"

ZygoteInit: warm up JCA providers during preload

This makes the time spent in the first call of an app to
SSLSocketFactory.getDefault() drop from ~240 ms to ~50 ms. In M
it was around ~6ms. This is due to the fact that, while instantiating
the default factory, all providers are initialized.

In order to obtain the timings above, I created an app calling
SSLSocketFactory.getDefault in onCreate and timed it
with System.currentTimeMillis() .

(cherry picked from commit 6cb7b1c4765e9bc5175056826523dbd88426e9aa)

Bug: 28545496

Change-Id: Ic5aab3ece609d9fef06fee4ccb83d8371af075b2

Merge "Fix GATT Characteristic write type serialization"

Merge "Fix "TelephonyRegistry notifies wrong data status""

Merge "Make android_net_NetUtils include what it actually uses."

Synchronize access of mSubscriptions

Introduction of setConfigAsync to set Zen mode config
asynchronously leads to synchronization issues like
ArrayIndexOutOfBoundsException while accessing mSubscriptions.

Change-Id: I6f72d894605498e0a4b48eb58af19851be53b9dc

Fix "TelephonyRegistry notifies wrong data status"

TelephonyRegistry should notify data status by correct subId,
otherwise in some case, SIM1 data status will be overried
by SIM2 data status.
Such as screen on/off action.If in fact SIM1 is data connected
to download something, but SIM2 is not,then sreen off,
both SIM1 and SIM2 will be notified to update data status.
Then SIM1 data status will be overried by SIM2 to be "no data
connected", before SIM1 data downloading is finished,SIM1
will not be able to show "data conneced" status.

Change-Id: Ia7d0767b3b4520294ee0d024903f7522cb0a0110

Merge "Fix documentation bug in SQLiteDatabase"

Merge "ContentValues: improve documentation"

ContentValues: improve documentation

Correct and clarify the documentation for ContentValues.

Change-Id: Iddf54da093e97c32da2568eab8802b1b2715303b
Signed-off-by: Steve Pomeroy <steve@staticfree.info>

Fix documentation bug in SQLiteDatabase

The documentation incorrectly states that, "[insertWithOnConflict
returns] the primary key of the existing row if the input param
'conflictAlgorithm' = CONFLICT_IGNORE". Unfortunately, SQLite does
not provide such functionality.

Close: https://code.google.com/p/android/issues/detail?id=13045

Change-Id: Ife919c4dcf7bf703b692d338d1fb5844a532dd39
Signed-off-by: Steve Pomeroy <steve@staticfree.info>

Merge "Support TrustedCertificateStore.findAllIssuers"

Support TrustedCertificateStore.findAllIssuers

(cherry-picked from commit aa6c3c3e252252b80c3900bd4c1ff27d37265c6d)
Change-Id: I176ec42c9907e50ee218e4fb352b530ca797be46

Make android_net_NetUtils include what it actually uses.

Bug: http://b/28519060
Change-Id: I49dcd358d3fc8e0ba5c5e0b50a0f5dcc63281d22

Merge "[RenderScript] Interrupt mMessageThread when tearing down context."

Merge "Fix AbstractStringBuilder gaining extra APIs."

Fix AbstractStringBuilder gaining extra APIs.

AbstractStringBuilder showed up in the API as gaining methods originally
belonging to StringBuffer/StringBuilder (mostly append(*) and
insert(*)).

Ideally it should not be in the API at all, but it's traditionally been
there and we'll keep it for now.

@hide the affected methods and move the "implements Appendable"
definition to StringBuffer/StringBuilder (as methods implementing an
interface can't be @hidden).

(cherry picked from commit 91f4aec3ccb4b2a2b66fbde095baa584b0862c11)

Bug: 27582395
Change-Id: I0b85c2c84f3e52ffe7864728d083f9c6ea7304ef

Fix GATT Characteristic write type serialization

Default writeType for GATT characteristic was not being
serialized into Parcel. This cause errors when trying to
write into Characteristic.

Bug: 27910548
Change-Id: Ib2f88cf991123eaea244f16fa36deb0d773c5a33

[RenderScript] Interrupt mMessageThread when tearing down context.

  - Interrupt mMessageThread, so it gets to see immediately
    that mRun is false and exit rightaway.

Change-Id: Ic586ef225a5c6f3a77f89de00b7fb3f3547c9838

Merge "Clear more native pointers in finalizers"

Clear more native pointers in finalizers

This hopefully handles the remaining finalizers in frameworks/base
that didn't guard against calls from other finalizers after being
finalized.

The goal here is to reduce the possible corruption causes we have
to think about by a few.

If any of theses are either very frequently used, or use large
Java (as opposed to native) objects, we should probably convert them
to NativeAllocationRegistry instead. But that's more work.

Bug: 18178237
Change-Id: I79bc0e8fa20af8f340418a6226022e0294d77f02

Merge "Track libcore change e2f147b9b14c7645f29e92758f811a18258feef4"

Track libcore change e2f147b9b14c7645f29e92758f811a18258feef4

Bug: 28582540
Change-Id: Ic944995957aa1a5ec65ecab2a8d4b8587094ccc8

Merge "Read -Xjittransitionweight from system properties."

Merge "Fix divergent equals and hashCode behavior"

Merge "List the CONNECTION_STATE_CHANGED broadcast for MAP as protected"

Merge "Protect TRANSFER_COMPLETE broadcast"

Fix divergent equals and hashCode behavior

Calling Objects.hash with a byte[] will call the identity hashCode on
the byte[] and this doesn't agree with the use of Objects.deepEquals
in equals.
Bug caught by error prone.

Also, replaced usage of Objects.deepEquals(mServiceDataUuid, ...) with
Objects.equals(mServiceDataUuid, ...), because mServiceDataUuid
is an Object of type ParcelUuid.

Bug: 28585195
Change-Id: Id92734874339985fedafe1a28286a6a4dcd88d3b

Merge "Improve grammar in startService() documentation."

Protect TRANSFER_COMPLETE broadcast

Bug: 28621801
Change-Id: Ia90abd00694ae80000f59ee0a17826592fd2159b

Improve grammar in startService() documentation.

Change-Id: If72f710caa2e85512636d6f7ddacff2426cfb835

Merge "Fix the issue that App crash on startForeground."

Merge changes from topic 'icu_cp_catchup'

* changes:
  Track external/icu API changes for factory registration
  ICU4J: Hide constants that can change between releases
  Remove RawCollationKey/ByteArrayWrapper from the public API

Fix the issue that App crash on startForeground.

Install single APP in any user space except Owner user, the APP
got crash when call startForeground in Service Component with
Notification which without SmallIcon.

bug:https://code.google.com/p/android/issues/detail?id=209043

Change-Id: I88a3b8afc213cafe0f280f9b44d6acd2ea0df207
Signed-off-by: Wang Le <wangle@xiaomi.com>

Track external/icu API changes for factory registration

Bug: 28583844
(cherry picked from commit e7fda97fff236afa5597da7df69b1462390c5a14)

Change-Id: I4a411184fe002ea812349e49d07d17c62c66d842

ICU4J: Hide constants that can change between releases

Bug: 27830659
(cherry picked from commit 71498f107ce41a1c3f2bc612fa214579aab19db1)

Change-Id: I007ed563d78cdf36ddd0e36ae72f118189772e5c

Remove RawCollationKey/ByteArrayWrapper from the public API

Bug: 22023363
(cherry picked from commit baa72515f60af798ea9a03e53a78a7d95834ceff)

Change-Id: If81d9a67ba9edf15688f4369390a6734d92a8f3c

Merge "Fix issue #25727069: Top sleeping activity overrides foreground service"

List the CONNECTION_STATE_CHANGED broadcast for MAP as protected

Bug: 28527879
Change-Id: Ifb2db87677e92022814bb9704c26009d249ac6ce

Merge "List the SDP_RECORD broadcast as protected"

List the SDP_RECORD broadcast as protected

Bug: 28257589
Change-Id: I97ef91fa3a7f7ef62fd96ca079e0a6d5a957bc71

Fix issue #25727069: Top sleeping activity overrides foreground service

Change-Id: I9a085cd650604f9d6277ad343bf3c3852324cafc

Merge "FloatingToolbar: Fix javadoc for closeOverflow()."

Merge "AAPT: ProGuard config for components in main dex."

Merge "Fix google-explicit-constructor warnings."

FloatingToolbar: Fix javadoc for closeOverflow().

Change-Id: I450fc603ae5fe49f08f580594352cc1120cc104d

Read -Xjittransitionweight from system properties.

Bug: 27865109

(cherry picked from commit 8d74de55464c885b602b34252ee8721a9e810dcf)

Change-Id: Id4f0668ea212f0fd82d32437ea251118bfce1ae2

Merge "ZipUtils: Fix wrong timestamps when getEntryInfo"

ZipUtils: Fix wrong timestamps when getEntryInfo

"tm_mon" format should align with "ZipEntry::setModWhen" in aapt.
"tm_isdst" should be initialized, or it will because random value
and cause error in function mktime().

BUG:28021145
Change-Id: I1e8d5c14e5d7b875bf9cd940cb7f4c5b93a1bcd6

Fix google-explicit-constructor warnings.

Bug: 28341362
Change-Id: Ibdd6a210bb7ff228e3624cc319169f77aca3b51e

AAPT: ProGuard config for components in main dex.

Create an analogue of "aapt -G" which outputs a proguard configuration
that keeps only components which need to be in the main dex.

BUG: 27383099

Change-Id: Ic18c8c563794ff27a5598a214111d1b446a005f1
(cherry picked from commit 86229cb622fccde8ab8cbe85eead91a34313a708)

Merge "zygote: Remove NativeLoader initialization step"

Merge changes from topic 'target-sdk-0-fix'

* changes:
  Create linker namespace for the system server classloader
  Extract pathclassloader initialization to a separate class

zygote: Remove NativeLoader initialization step

This now done by art Runtime::Start()

Bug: http://b/28320913
Change-Id: Icf9b11954b732c9f953e02bba9ae52eb91117457