Merge "Fix Fingerprint confusing personal and work fingerprints when unified" into nyc-dev

Merge "Restrict the cases when we profile guided compile an apk" into nyc-dev

Merge "Check task.isResizeable() instead of the resize mode directly" into nyc-dev

Merge "Add setAlwaysOpaque and setTint to BarTransitions" into nyc-dev

Merge "Display Toast with linker warnings after Activity.onStart()" into nyc-dev

Merge "Fixed handling of blockUninstall setting" into nyc-dev

Merge "Correct state loss tracking for FragmentTransaction.commitNow" into nyc-dev

Track change 02072325e0133d78cc575732a0762ed3da0242a7

Bug: 27404545
Change-Id: I51b1e7239d456f0ec1d0bdc6b54761aeb44368d9

Merge "Regenerate API after b8f53ee812b75b526c3b481b62334e45609fa70e." into nyc-dev

Regenerate API after b8f53ee812b75b526c3b481b62334e45609fa70e.

bug: 26704777
Change-Id: Ib448d90fdf0200d77df9cbb8930a904fbe3374ca

Merge "Update Android mediamuxer documentation." into nyc-dev

Merge "Heed user shell restriction for pm calls" into nyc-dev

Correct state loss tracking for FragmentTransaction.commitNow

Booleans are hard.

Bug 27419920

Change-Id: I91a746aa14cad36bf77ec6224ffc13b00dc39ae7

Merge "AAPT2: Fix segfault when showing error in 9-patch" into nyc-dev

AAPT2: Fix segfault when showing error in 9-patch

Change-Id: I8f457212efc27480758a7157455d98b5e316f772

Merge "media: define HEVCProfileMain10HDR10" into nyc-dev

Merge "Adding SecurityException information to all device logging DPM public methods" into nyc-dev

Fix animation bug in chooser activity

Make sure to set the status color to 0 (fully translucent), so
there is no ugly line travelling across the screen when opening
the activity.

Bug: 27285627
Change-Id: If8aa1059958f3e2a45a48f6ba276560244f284cd

Adding SecurityException information to all device logging DPM public methods

Bug: 22860162
Change-Id: I6cdfad2dcdf14896c855debef34964def190e110

Merge "Api review: AutomaticZenRule" into nyc-dev

Fix QS icon animations once and for all

Bug: 27250295
Change-Id: I6449b9cea19acf120dfc02d146fe608da46fe790

Merge "Fix crash in isChildDocument when parent is in zip, and child not." into nyc-dev

Fix crash in isChildDocument when parent is in zip, and child not.

There is no regular file representation for archive documents, so
the call to getFileForDocId(archivedDocId) was throwing an exception.

Bug: 27383423
Change-Id: Icbc1e8ecdddc7698cfdb79b41da6e1d3cecf11ae

Fix popup window calculation for multi-window

If ignoreBottomDecorations=true, the display size was extracted from
the resources. However, this didn't work if the parent window was in
multi-window, as all the calculations went wrong. Instead, introduce
View.getWindowDisplayFrame which returns the "full" frame of the task
the window is currently in, without any insets, and use that to
calculate the bottom edge.

Bug: 26255254
Change-Id: I8b235b335775022ae399ee082d1200aa76cc047c

Merge "Fix argument type in MtpDatabase#getObjectPropertyValue." into nyc-dev

Merge "Fix RootScanner to execute scan at least once before cancelling it." into nyc-dev

Fix argument type in MtpDatabase#getObjectPropertyValue.

In MTP code, we use jint for ID (e.g. object handle) and code (property
code). But getObjectPropertyValue takes jlong and it causes missing
property values in Java code. Previously we passed the long value to
Map<Integer, MtpPropertyGroup>#get(Integer) and the compiler somehow
accepted the code. However it actually takes Long at runtime, so the
properties are never found.

BUG=26437284
Change-Id: I79defd325ea5d20c4dce84d891d984e24abcacc5

Add constructor with more inputs for dependency injection.
am: b9df11099f

* commit 'b9df11099f16ca816a69e06f1327eb1f86723578':
  Add constructor with more inputs for dependency injection.

Add constructor with more inputs for dependency injection.

Bug: 26991160
Bug: 27406552
Change-Id: If4a0a2f1afb1da4ad59df9eb31c540047c84659b

Fix RootScanner to execute scan at least once before cancelling it.

Previously RootScanner is implemented by FutureTask, and if it is
cancelled before it starts running in background thread, the scan will
be never executed.

The CL stops using FutureTask and introduces CountDownLatch to control
UpdateRootsRunnable.

BUG=27369585

Change-Id: Ica8799faba0a8e5ca91a6b8be36dc4f5118d6333

Add setAlwaysOpaque and setTint to BarTransitions

Allow the opacity and color of the NavigationBarView
to be controlled.

Bug: 26425669

Change-Id: I364f183bf0ef90c446ae68987ccc3b6bbf09aa67

Merge "Add javadocs to MediaCodec.CryptoInfo methods" into nyc-dev

Merge "Fix settings icon row consuming the click when tapping the notification" into nyc-dev

Check task.isResizeable() instead of the resize mode directly

The resize mode doesn't take into account developer settings override.

Bug: 27390258
Change-Id: Id161ceb4f143f982f6bec3f915655bb66949ddd3

Fix Fingerprint confusing personal and work fingerprints when unified

When the personal and work profile have unified locks, FingerprintService
was retrieving the credential owner and returning the personal data
even when explicitely asked for the work profile.
Bug: 27408217

Change-Id: I67d20f92e0016e4ccd38f8b19edebc543830df4a

Fixed handling of blockUninstall setting

Previously, if one user has blockUninstall, it will block uninstalling
the package for all users. Now the package is uninstalled for all users,
excluding the ones that have blockUninstall setting.

Bug: 24616123
Change-Id: Ie4a68ef97fbef4711b18ccd311e39f32ab4b8efe

Merge "Proper grant default permissions to default SMS and Phone" into nyc-dev

Merge "Remove revealLauncher and revealApp from tests." into nyc-dev

Heed user shell restriction for pm calls

Make sure that a restricted user cannot use adb shell commands
to grant/revoke runtime permissions.

Bug: 27418014
Change-Id: I518b09f56ca7d77cc2bfa1d0b4faaf129e19000d

Proper grant default permissions to default SMS and Phone

We grant default permissions to the default SMS and
Phone apps when they are selected as such to ensure
they can do their job. We don't grant default permissions
to apps if the version on the system image does not
declare them. This is correct for default grants on
first boot or a new user creation. This is a problem
for default Phone and SMS as we want to grant them
the permission as a result of a deliberate user action
and therefore should not care if the version on the
system image declares the permission.

As a result if an SMS app that ships as a stub to reduce
image size is later updated to the full version and
made the default SMS it would not get the default grants
while an installed third-party app that did not ship
on the system image would get the grants.

This change ensures the default SMS and Phone get
their default grants regardless whether the verion on
the system image declares the permissions.

bug:25288760

Change-Id: I2fecf5cbce4ebabb145a3f29abb555bcb65d54d6

Merge "Allow system to bypass DISALLOW_SHARE_LOCATION restriction" into nyc-dev

Merge "Unlock Keystore/Keymaster separately for Work Challenge" into nyc-dev

Merge "Make bugreport details dialog confirm to Material Guidelines for Dialogs." into nyc-dev

Merge "Explicitly check for updated system apps" into nyc-dev

Make bugreport details dialog confirm to Material Guidelines for Dialogs.

Changes:
- Removed hints.
- Added TextViews for field labels.
- Added padding for inner dialog
- Adedd autoCorrect and capSentences to title and summary
- Changed strings.
- Set name to be selectAllOnFocus initially.

Also improved some logging statements.

BUG: 26324085
Change-Id: I32597a7c2839ca706dbbcf13660e976469ab8dd0

Merge "media: switch to using MediaImage2" into nyc-dev

Merge "Fix issue #27385109: control activity behavior without triggering..." into nyc-dev

Fixed a bug where group expansion size calculation was wrong

On the lockscreen, the minheight was always based on the intrinsic
height and not actually the minHeight of the children. Because
children were userlocked, they were now reporting their small size
instead of the single line size.
This meant that the expand motion on the lockscreen wasn't following
the finger at all.

Bug: 27418617
Change-Id: I81daaf87887de1adc014cb4c6d46f4ef118061e5

Merge "Reintroduce per-app scheduled job limits" into nyc-dev

Merge "Notification listener and ranker callbacks on binder threads." into nyc-dev

Notification listener and ranker callbacks on binder threads.

The callbacks for the notification listener and notification
ranker were delivered on binder threads which is problematic
becuase: 1) permission checks and app ops checks would fail
unless the app developer knows to clear binder calling id and
restore it after that; 2) developers need to synchronize their
implementation as they get callbacks on different threads (
arguably callbacks should not be concurrent); 3) this doesn't
follow the pattern in the platform;

Also the code delivering callbacks was catching Throwable which
we shouldn't do in general and also masks bugs in the listener
or ranker implementation. Now that the callbacks are offloaded
to the main listener/ranker thread system code should not be
guarding against Throwable to handle exceptions propagated
over binder calls.

bug:26704777

Change-Id: I171fb41bbe25e6105dd05e4166193dbcec594f82

Explicitly check for updated system apps

Unbundled apps should not be allowed to share
linker-namespaces with default namespace; it implies
that they will not have unlimited access to private
native libs once updated.

Bug: http://b/27189432
Change-Id: I60295b9b31d700016add93537418fdcce5a77719

Merge "Only copyLayer if valid rendertarget" into nyc-dev

Merge "Skip boot prompt for both native and emulated FBE." into nyc-dev

Skip boot prompt for both native and emulated FBE.

Bug: 27412530
Change-Id: I53d39392ccab54cb79cf6cc1e51923ad7b5a3b0c

Merge "Updates to BlockedNumberContract from API review." into nyc-dev

Merge "AudioService: add support for ENCODED_SURROUND_OUTPUT" into nyc-dev

Merge "Need to clear identity when rebooting for FBE." into nyc-dev

Merge "Parcelable classes should always be final." into nyc-dev

Need to clear identity when rebooting for FBE.

Bug: 27412530
Change-Id: I6d5b925a16daefd1b6906ce036faeb6244d503fb

Only copyLayer if valid rendertarget

Bug: 27407637

Texture layers will set a GL_NONE rendertarget
until they have managed to get the real rendertarget
from SurfaceTexture. If, however, copyLayer is called
prior to any calls to updateTexImage() (for example, if
no frames have yet been produced), this will
result in trying to use a Texture with a GL_NONE rendertarget
which is bad.

Change-Id: Ib7285d799cb94ebd44f07a89f2ef2075d79ef71f

Add javadocs to MediaCodec.CryptoInfo methods

bug:27383993

Change-Id: I11bba4a2690d112e4210add41910f14c5ba6a927

Parcelable classes should always be final.

Also hide ConnectivityMetricsEvent which isn't being used yet.

Bug: 27415331
Change-Id: Iacdccddda504f3f669185f807b4f35b8dc2b0212

Fixed a bug where group children where not expandable

Bug: 27417777
Change-Id: I57368f18b810bc9f6fc7c08c0f7fb038743b528d

Updates to BlockedNumberContract from API review.

Javadoc updates are also included.

BUG: 27364137
Change-Id: Id99b5c0fadd2a62a33217b44c7f30dd2f3beb9be

Merge "More lint checks." into nyc-dev

More lint checks.

-- Error if Parcelable classes aren't final.
-- Warn that MIN_/MAX_ constants that should be dynamic.
-- Error if throwing raw RemoteException.
-- Warn if methods accept File without stream variants.
-- Ignore upstream ICU project.

Bug: 27410989
Change-Id: I12d843e63078dc696c1798a20e088608f1ba9ed1

Merge "Fix glGetTransformFeedbackVarying JNI" into nyc-dev

Merge "Update preconditions to be asserts..." into nyc-dev

Fix settings icon row consuming the click when tapping the notification

This CL makes the settings icon row invisible when it's not displaying,
this way it won't get the click event on the notification row before
the parent.

Bug: 27389571

Update preconditions to be asserts...

Which are now supported by Jack.
Retain a few preconditions checks that enforce contract.
Disable the one assert we had...because it failed in tests.

Change-Id: I2f6cb01e1af1534be4428ce4a3ee625a163dd163

Merge "Use SkFILEStream directly when possible" into nyc-dev

Merge "Use realActivity name when comparing if intents are the same" into nyc-dev

Use realActivity name when comparing if intents are the same

The TaskRecord intent (usually the intent of the root activity) component
names are based on the realActivity (the activity we are actually launching
and not the input alias) and the ActivityRecord intent is based on the input
component name which can be an alias. This leads to issues when we are trying
to compare the intent of a task and an activity to see if they resolve to
the same thing since the component names will be different in the case of
aliasing.
We know base the activity intent component name on the realActivity before
comparing with the task record intent.

Bug: 27403679
Bug: 27112965
Change-Id: I196e03bb018582cbac977fb3ad45354f00f51578

Merge "Add dumpsys meminfo --unreachable" into nyc-dev

Merge "Fix matrix mapping of negative rects" into nyc-dev

Merge "Fix arc positioning" into nyc-dev

Merge "Revert "InputConnectionWrapper never supports null target."" into nyc-dev

Merge "Fix dirtyMask in AnimateFunctor" into nyc-dev

Merge "Fix setBluetoothContactSharingDisabled does not persist" into nyc-dev

Merge "Update the animation when showing / closing inline controls" into nyc-dev

Fix dirtyMask in AnimateFunctor

Bug: 27313379
Change-Id: I0c0cffe7e1940914f0143714643ede539b86216f

Merge "Reset pointer icon when view is not present" into nyc-dev

Revert "InputConnectionWrapper never supports null target."

This reverts commit 90bd36363c5738b3f526aa1f1d44f432236300a0.

Seems that the semantics of InputConnectionWrapper#setTarget() is more
complicated than I thought.  At least the following cases have worked
fine.

  case 1:
    InputConnectionWrapper wrapper =
            new InputConnectionWrapper(null, false);
    wrapper.SetTarget(ic);
    ...

  case 2:
    InputConnectionWrapper wrapper =
            new InputConnectionWrapper(null, true);
    wrapper.SetTarget(ic);
    ...

  case 3:
    InputConnectionWrapper wrapper =
            new InputConnectionWrapper(ic, true);
    wrapper.SetTarget(null);
    wrapper.SetTarget(ic2);
    ...

The previous code did not intended to break existing code.  Let's revert
it we decide how to deal with above cases.

Bug: 27407697
Change-Id: I8bc84d484ab0b27a02e74f11110430f70646e69a

Fix arc positioning

bug:27389290

Change-Id: I479a34ea079f1fce1c308aa809a57885ffa1d004

Fix setBluetoothContactSharingDisabled does not persist

The default value of bluetotoh contact sharing is true.
So we should save when it is false.

Bug: 27410265
Change-Id: Icaf4ceeda09eca46d160acfecc53834819b66a18

Merge "InputConnectionWrapper never supports null target." into nyc-dev

Fix matrix mapping of negative rects

bug:27381362

Also rejects ops with empty clip at record time, and short circuits clip
intersection, when one is empty.

Change-Id: I842612da14ad8fd9f1ba5e9e4fd027ba4e08d365

Merge "Frameworks/base: Refactor Paint & Canvas native allocation support" into nyc-dev

InputConnectionWrapper never supports null target.

This CL makes it clear that InputConnectionWrapper does not support null
target.  In other words, the semantics of null InputConnection can never
be emulated by a non-null InputConnectionWrapper.

This is particularly problematic when app developers are just forwarding
the return value of super.onCreateInputConnection() to
InputConnectionWrapper or its subclass, because there are many chance
that super.onCreateInputConnection() starts returning null, e.g. when:
  A. the application is extending a Framework class, and the Framework
     class is updated by OTA.
  B. the application is extending system WebView, and the WebView is
     updated.
  C. the application is extending a 3rd party library, and the app
     developer creates a new build with a new version of the 3rd party
     library.

To make it easy to catch these kind of bugs, this CL lets the
constructor of InputMethodWrapper throw NullPointerException when target
is null.  Bugs like crbug.com/571229 should be caught by developers
more easily.

Bug: 27407697
Change-Id: I83875bea886d4784f9507c930050efc29708d9db

Fix uninstalling apps with damaged/gone APK files

bug:27394497

Change-Id: I0d0317f2d7168be4412a24d34a9f23f04c6d8601

Merge "Fix flashlight availability." into nyc-dev

Fix issue #27385109: control activity behavior without triggering...

...isUserAMonkey for testing purpose

Add an argument for the caller to specify if they are a poo flinging
monkey.

Change-Id: I0e149a8d78776abaf07517bd4ae886047b7f4252

Merge "Fixes in QS customization" into nyc-dev

Merge "Add new API to create an intent to manage blocked numbers." into nyc-dev

Merge "Add new API for creating intent to manage blocked numbers." into nyc-dev

AudioService: add support for ENCODED_SURROUND_OUTPUT

Preference allows override of AudioPolicyManager.
Lets user force or prevent use of AC3 and DTS regardless
of what the EDID says.

Bug: 26373761
Change-Id: I21440f2b90af9a369a36b7b07724e992501bce6d
Signed-off-by: Phil Burk <philburk@google.com>

Merge "Use asset cookie in XML block cache key" into nyc-dev

Fixes in QS customization

 - Revert to long-press to drag
 - X -> <-
 - Always save

Bug: 27397111
Bug: 27364013
Bug: 27396440
Change-Id: Ib19096b0ed65fb1112386735dffe9308e57c3d5a

Add new API to create an intent to manage blocked numbers.

This replaces the launchManageBlockedNumbersActivity API.

BUG: 27364137

Change-Id: I5d5a20d25f0dea8b8e0e998b51f80658ff404e2c