Merge "Framework's NetworkSecurityPolicy delegates to libcore's one."

Merge "Enforce tag types in KeymasterArgument classes"

Merge "Fix to crash when clicking text link without view activity"

Merge "hwui : fix memory leak due to duplicate in shadow cache"

Framework's NetworkSecurityPolicy delegates to libcore's one.

Now that the libcore's NetworkSecurityPolicy abstraction is in place,
the framework version of the abstraction should delegate to the
libcore one, to avoid them getting out of sync.

Bug: 19215516
Change-Id: Ic57341d703a13e0fb100dc414958e8fd54e03816

Merge "Address pending comments for NetworkSecurityPolicy."

Merge "Sort Bluetooth devices in quick settings by name"

Merge "systemui: Handle case when network has been lost"

Address pending comments for NetworkSecurityPolicy.

This is a follow-up to
https://android-review.googlesource.com/#/c/131920/, addressing some
outstanding comments.

This CL also switches the source file to the correct indentation.

Bug: 19215516
Change-Id: Ia48455ab351081e11619afcccadb8fa90340391d

Merge "[ActivityManager] Avoid killing unrelated processes."

Merge "Layout the window to be displayed if it would be resized"

Merge "DimLayer : remove unnecessary surface transaction calls"

Merge "Frameworks/base: Don't allocate another identity matrix"

Sort Bluetooth devices in quick settings by name

To avoid Bluetooth devices moving randomly in list, sorting by name.

Change-Id: I4f8e9f98fa29f9670678a3bb6051a6fcf7ae0b9d

Merge "Don't create unnecessary RenderThread's instance when executing 'dumpsys gfxinfo'"

DimLayer : remove unnecessary surface transaction calls

There is the case that adjustSurface()  get called even if
the size of the surface of DimLayer is not changed actually.
Since changing the size of a surface is processed synchronously
in the SurfacFlinger, there is usually a few milliseconds delay
(up to 1 vsync interval) when we launch an application.
This patch avoids such cases.

Change-Id: Ib1f76d54f9f2364ac54b70120e4b781e8534e750
Signed-off-by: Dohyun Lee <dohyun.lee@lge.com>

Merge "Prevent unexpected rotation while going back to keyguard"

Merge "[RenderScript] Add create(Context, int) to be compatible with the thunker layer & minor tweaks."

[RenderScript] Add create(Context, int) to be compatible with
the thunker layer & minor tweaks.

bug:19888167
bug:19888339
Change-Id: Ib264c4ca48c990b3476456838047f73d95f6752f

Merge "SystemUI needs the SET_WALLPAPER permission"

SystemUI needs the SET_WALLPAPER permission

If changing wallpaper when low on memory, retrieving the default
wallpaper may return null, which triggers error handling in
ImageWallpaper. This error handling tries to perform
WallpaperManager.clear(), but for that it needs SET_WALLPAPER
permission, which it does not have.

For users with apps that auto change wallpaper, this issue can
be frequent in low-memory conditions.

The solution is to add the permisson.

Change-Id: I81503c1667e3952c2dd15599969f7dcc51623e5b

Prevent unexpected rotation while going back to keyguard

The original logic would let the app hidden by keyguard be able to
decide the orientation. While going back from a show-when-locked app
to keyguard, there would be a short time that keyguard is unable to
decide the orientation, which causes WMS uses the wrong orientation
from the wrong app.

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

Change-Id: Ibc17bfe4603f68b241dc7380459ec9de42a3e259

Enforce tag types in KeymasterArgument classes

Change-Id: Ib1aca884a0b42c3e245db65d48c945b34935072d

Merge "Move arrow_pointer hotspot to better match actual tip"

Merge "Specify public exponent for RSA"

Specify public exponent for RSA

Keymaster no longer adds a default value, so these tests were failing.

Change-Id: I9c5a8d2552534db1d7fa77965b0d675acc3ccc0a

Merge "Add android.security.NetworkSecurityPolicy."

Don't create unnecessary RenderThread's instance when executing 'dumpsys gfxinfo'

To obtain the gfxinfo for each process, the static method of RenderProxy is used, which is named outputLogBuffer().
In there,
1. RenderTask is created for getting DisplayList Commands in RenderNode.
2. staticPostAndWait() is called
3. RenderThread's instance is created by 'RenderThread::getInstance()' in staticPostAndWait()

In case of the service, they don't use HW Acceleration, so don't need RenderThread.
But, by the process of No.3, RenderThread is created for all process.
As we know, RenderThread never be destroyed while the process is alive.
This patch checks RenderThread instance before the creation of RenderTask.
And, there is no one, just return to prevent the unnecessay creation of it.

Change-Id: I4fe29d83c9ced3e8b67177c0874c5d8ee62e1870

Merge "New API methods for Script Group"

New API methods for Script Group

Use a single vararg Object array for both kernel/invoke arguments
and global bindings. The assumption is that global bindings are in
the end after the kernel/invoke arguments.
This simplifies the api to add a kernel or invoke to a script group.

Change-Id: I7efb1035c5967e2cadd15e6fe27f20c80ba197a2

Merge "Added copyright notice and formatted to convention"

Merge "[RenderScript] Remove duplicate include in jni"

Merge "Make it harder to leak contexts"

Merge "Remove unused printapk.cpp."

Add android.security.NetworkSecurityPolicy.

The initial purpose of the NetworkSecurityPolicy class is to provide a
way for network libraries to check whether cleartext network traffic
(e.g., HTTP, WebSockets, XMPP, IMAP, SMTP) should be blocked from this
process.

The policy is set declaratively by the app developer in the app's
manifest and can be queried from ApplicationInfo.flags. Unfortunately,
several network stacks (bundled and unbundled) do not have a reference
to ApplicationInfo or Context.

Alternatives:
* Keep this API hidden (and thus potentially move it from framework to
  libcore), thus precluding unbundled HTTP stacks from using the API.
* Introduce a new java.lang.System property instead of this API.
  However, such properties are a mess and not as powerful/extensible
  as a public class.

Bug: 19215516
Change-Id: If22056a74d257bf1d805ebb4fc284240b3d338f1

Merge "Revert "Change jit properties to be debug properties""

Merge "Add app-level usesCleartextTraffic manifest attribute."

Remove unused printapk.cpp.

This is one of the few remaining users of libzipfile, but doesn't get built.
Not worth porting to libziparchive.

Change-Id: Ica466b22e3a7ebaf123727c0d703af39cc89cf9c

[RenderScript] Remove duplicate include in jni

Change-Id: Ib6019ab22a83d952dce3fd6f5351120d94b49d58

Revert "Change jit properties to be debug properties"

Bug: 19735273

This reverts commit 1ad10ef1e4b83ef9e845761a82e202bb57e2379d.

Change-Id: Iac930925ab6d5cd5947fa4077c8d93e2984927d5

Merge "Rename KM_TAG_USER_AUTH_ID to KM_TAG_USER_AUTH_TYPE."

Rename KM_TAG_USER_AUTH_ID to KM_TAG_USER_AUTH_TYPE.

Also change to an ENUM, to match changes in keymaster_defs.h

Change-Id: Ieda3402563b0b9e4e8b100eb69a94651e51c6347

Merge changes Id9a22ad3,I629ab2c4

* changes:
  Add KM_TAG_AUTH_TOKEN.
  Add support for KM_TAG_USER_SECURE_ID.

Make it harder to leak contexts

Change-Id: Ied60b6428bedb200d7b5b627e8196fbf2b4151e1

Merge "Add JIT late override property"

Merge "Frameworks/base: Fix potential NPE in InputMethod"

Merge "Frameworks/base: Remove duplicate check in Mesh"

Merge "Frameworks/base: Fix visibility flag in Editor"

Add JIT late override property

Required for JIT debug property since user builds can't restart
the shell.

Bug: 19735273

Change-Id: I1983852e80010c344b45e0bb459de47c56adfce8

Merge "Frameworks/base: Fix potential NPE in Script"

Merge "logd: optimize statistics"

Merge "Avoid duplicate surface creation."

Frameworks/base: Fix potential NPE in Script

Bug: 19797138
Change-Id: I5cd9394bd2c0f7c044c1f2f24665384fa797cd97

Add app-level usesCleartextTraffic manifest attribute.

The attribute declares whether the app intends to use cleartext
network traffic (e.g., HTTP, WebSockets, XMPP, SMTP, IMAP -- without
TLS or STARTTLS). The default value is true. If set to false, the app
declares that it does not intend to use cleartext network traffic. In
this case the app requests the platform, tooling, and third-party
libraries to prevent it from using cleartext traffic. The danger of
cleartext network traffic is that its confidentiality, authenticity,
and integrity are not guaranteed.

This feature is designed to help apps which care about security of
data exchanged over the network. These apps can accidentally
regress/downgrade to using cleartext network communications. This
typically happens when the server the app communicates with all of a
sudden tells it to use cleartext communications (e.g, HTTP URL
instead of an HTTPS URL) or when one of the components of the app gets
updated and regresses to cleartext communications without the
developer noticing.

In general, the prevention measures are on best effort basis. It's
impossible to automatically prevent all instances of cleartext
traffic. For example, an app bent on bypassing restrictions could
perform low-level network I/O with unusual TCP packet fragmentation,
or could use a custom application-level protocol.

The expectation is that most apps use libraries for network
communications and these libraries over time will start to honor this
flag, thus increasing the protections offered by it.

Bug: 19215516
Change-Id: I8700d51ddbc5d528faa4b6a5fa5bc9551ad02d13

Frameworks/base: Fix potential NPE in InputMethod

Don't read the size of an unchecked list.

Bug: 19797138
Change-Id: I9d8c087aff7bc9cc1e8aae9a0b489e23b5442765

Frameworks/base: Remove duplicate check in Mesh

Bug: 19797138
Change-Id: I0b11c4ff63a8031d5e58a06ac13f91ae0bbac5dc

Avoid duplicate surface creation.

Change-Id: I43104c8b48dd26681735940e6b2e1ba902af2020

Frameworks/base: Fix visibility flag in Editor

Fix double check.

Bug: 19797138
Change-Id: I95e694f384f1f25d6cf3b6a1669052940385e41d

Merge "Frameworks/base: Make IDENTITY_MATRIX final"

Frameworks/base: Make IDENTITY_MATRIX final

Bug: 19797138
Change-Id: I127f24b7060a0c4dab401ce8e3057d362c6d6b06

Merge "Frameworks/base: Fix format string in Geofence"

Frameworks/base: Fix format string in Geofence

%p is not a valid conversion in format strings. It is also superfluous,
as it is already known that location is null.

Bug: 19797138
Change-Id: I5784e28b05b4ca9aac57e0fc9da4a7f01d9b3247

Merge "Frameworks/base: Fix format string in Camera"

Frameworks/base: Fix format string in Camera

One cannot print a boolean with %d. That will result in an exception.

Bug: 19797138
Change-Id: I86c42ea834cebebaecff8463637cc9de14d1fc88

Merge "Frameworks/base: Fix precedence bug"

Frameworks/base: Fix precedence bug

Explicit cast has higher precedence than shift.

Bug: 19797138
Change-Id: Ifcf569bf774fbf65ee50c078f736ad167bcc6b8c

Merge "Frameworks/base: Use || instead of |"

Frameworks/base: Use || instead of |

Nothing wrong with | in this case, but || is canonical.

Bug: 19797138
Change-Id: I5f145736a5470f7cde06efce9a217d86eda2135f

Merge "Frameworks/base: Fix trivial equals implementation"

Frameworks/base: Fix trivial equals implementation

The comparator's equal implementation doesn't satisfy the constraints
of an equals method, namely being reflexive. Use the standard Object
implementation instead.

Bug: 19797138
Change-Id: I74f888e99533e1945aab7ab10fe8ee3ded6388f4

[ActivityManager] Avoid killing unrelated processes.

Kernel will reuse process id, when wrap-around happens,
there may be just after a pid is freed, a new thread/process
uses the pid immediately.

For examples, there may be a process with pid 1234 killed
by lowmemorykiller or itself (app.killed=false).
Before the death recipient enters AMS, a new thread/process
is started with pid 1234, then when appDiedLocked executes,
this new pid 1234 will be killed again.

It is especially easy happens to zygote:
During zygote starting process, it will stop 5 daemons threads.
And restart the 5 threads after fork is called, so it has
larger pid range to hit the resue case.
https://code.google.com/p/android/issues/detail?id=160661

Solution:
If the dead event is from binder, it is not necessary
to call killProcessQuiet again, because it should really
be dead.
Keep no checking for killProcessGroup because zygote does
not have record under /acct/, and also the parameter uid
has restriction that will not kill another application
which reuses the pid.

Change-Id: Iec4a4884ae641c4d036f4d024ce463f7a351a17b

hwui : fix memory leak due to duplicate in shadow cache

New ShadowTask with the ShadowDescription key that already
exists in the shadow LruCache will leak as it is not being
added.

Fix adds check for the existing key that is common in the hwui
code but missing for the TessellationCache::precacheShadow
function.

Change-Id: I37fd5ec82f8b8da5d1ec0f2ab9fd04c5f8534367

Merge "Change jit properties to be debug properties"

Merge "Frameworks/base: Fix a comparison"

Change jit properties to be debug properties

Bug: 19735273
Change-Id: I266cf447a7af0595366bf89109d2823094f5cf41

Added copyright notice and formatted to convention

Pure format change. No other changes at all.

Change-Id: I74a53aa67ef0960f70becaf288b85be40066a0ae

Merge "Frameworks/base: Force long computation"

Merge "Frameworks/base: Check before foreach in Script"

Merge "Frameworks/base: Use equals for Integer comparison"

Merge "Revert "Frameworks/base: Fix a constructor""

Revert "Frameworks/base: Fix a constructor"

The constructor was public API, doh. Gotta do this differently.

This reverts commit 33c5b2a62f3e62382c41e24c6b527119978816a0.

Change-Id: Iadca87fe6a8866a8bd9d6f2a91578ec0d4c44691

Merge "Frameworks/base: Fix null-pointer check"

Merge "Frameworks/base: Fix a constructor"

Merge "Frameworks/base: Fix a hashCode implementation"

Merge "Frameworks/base: Fix always-false equals"

Merge "Frameworks/base: Fix missing cast"

Frameworks/base: Force long computation

Ensure that an int-based computation is carried out as long.

Change-Id: I23b10a95600674e8a5a65c0ea349afdc6aa152ae

Frameworks/base: Don't allocate another identity matrix

There is already an identity matrix in Matrix. Don't allocate another
one for VectorDrawable.

Change-Id: I51735f262d6680e043b0009707ec42acb2d0d1ad

Frameworks/base: Fix a comparison

Change-Id: I80d62869920e77110c95f20369ec2631c75f6ed4

Frameworks/base: Check before foreach in Script

According to the if below, ains == null is potentially valid. But
the foreach loop would throw a NullPointerException.

Change-Id: I4460fb1357eaa3abfe0ab9a21effb608f474ab51

Frameworks/base: Use equals for Integer comparison

Integer == is dangerous, as equal objects may not be identical
objects. In fact, MediaFormat.setInteger was creating a new object
every time.

Change MediaFormat.setInteger and setLong to use valueOf, which
may reuse returned objects.

Change-Id: Iedcc6003adbf05c0c870aa4b3ada7f181a5b870e

Frameworks/base: Fix null-pointer check

Change-Id: I715a21c313e909ae654e0c1aa67bdf7bcd89de76

Frameworks/base: Fix a hashCode implementation

Equals uses Arrays.equals. That means two responses are equal if
the content of the data arrays is equal. By convention, the hash
code of those objects should agree. In that case one cannot use
hashCode on the array (which is the identity hash code).

Change-Id: Icce8e2e71e9142421f5dac8a0ee8a211623fb704

Frameworks/base: Fix a constructor

This was meant to be a constructor, according to the comment.

Change-Id: Ief49011b392e58b37d9acb4a3f754f1828b256af

Frameworks/base: Fix always-false equals

Rect != Insets.

Change-Id: I3d4ff890608e446b51f09a1b633af742f0c069d4

Frameworks/base: Fix missing cast

Without a cast, the division is integer division.

Change-Id: I050e53778de8b1591a0be16ebbee8eed70eb1528

Add KM_TAG_AUTH_TOKEN.

Change-Id: Id9a22ad32137f3e0380c2812f790bbecab511d11

Add support for KM_TAG_USER_SECURE_ID.

Change-Id: I629ab2c47ee6d42de20a963ef283e330364c8ee7

Merge "Switch libjnigraphics to sysv-only hash style"

Switch libjnigraphics to sysv-only hash style

Bug: 19059885
Change-Id: I3474974983931c67dce30dfc581a8d28d8b62e6a