Make sure we're locked

Before calling a method ending with LPw, we need to have the mPackages lock.
Make sure we do that.

Also, ditch the iterator for traversing an ArraySet.

Bug: 29356333
Change-Id: I67df971f2961b577f31153cef46fb153458d6965

Compute AnyMotionDetector angle using atan2. Bug: 29232136 am: db843c913f
am: e46d4fd7ad

Change-Id: I666fc7432cf6140c938423ec4370ca7727996bd6

Merge "Fix a naming typo for source_build" into nyc-dev

Compute AnyMotionDetector angle using atan2. Bug: 29232136
am: db843c913f

Change-Id: I1d2af755dd1a06fc2fef464b6ad5bc5e26f11ff6

Fix a naming typo for source_build

When parsing the last_install, the variable name is supposed to be
*source_build* instead of *source_version*.

Bug: 28658632
Change-Id: I1e0ed7150e04885f904b6a3efa18bd5cfe17cc96

Merge "docs: Updates to DP4 release notes for Android for Work" into nyc-dev

Merge "docs: removing duplicate security config doc" into nyc-dev

Merge "A11y: Fix notification bounds when top notification is scrolled" into nyc-dev

docs: removing duplicate security config doc

current doc: https://developer.android.com/preview/features/security-config.html
The N Preview version of this page will move to this location when N is
officially released.

Change-Id: Idf3bb2ea13b5f08efd3187bb1b1fbb6944519507

Merge "Clear calling identity before calling mountService" into nyc-dev

docs: Updates to DP4 release notes for Android for Work

See first comment for doc stage location.

Change-Id: I20ee0f104ebf2eb274e599ede8bfe5c0c59077fe

A11y: Fix notification bounds when top notification is scrolled

Previously, scrolled notifications would collapse into a stack (the top
stack). With the new design, that's no longer the case, so tall
notifications can now be scrolled out so much that their translation
becomes negative.

Unfortunatly, getBoundsOnScreen then clips the bounds to the parent,
which interferes with our adjustment to the bounds, so we first undo
the clipping.

Fixes: 29368064
Change-Id: I71da054ed1eb6d06868d56e18d0320f764ed86b3

Merge "Fix flashlight dim state" into nyc-dev

Merge "PopupWindow: Resolve measure specs before passing to WM." into nyc-dev

Merge "Revert "boost starred people to at least high importance"" into nyc-dev

Merge "Fix accessibility for autorotate tile." into nyc-dev

Fix flashlight dim state

Change-Id: I7875d4a0c3733ac2462713d7618d1b7595ea2075
Fixes: 29330406

Merge "Update semantics of on-body Keystore API." into nyc-dev

Merge "Make lid switch edge rather than level triggered." into nyc-dev

Merge "Update the DP4 setup instruction for api level 24" into nyc-dev

Update the DP4 setup instruction for api level 24

Change-Id: I8db52d56b1c33e02cffb589a19b1cb1ed20894f4

Merge "Revert "docs: Add Videos section to N Preview landing page"" into nyc-dev

Revert "docs: Add Videos section to N Preview landing page"

Reverting this change since I don't have confirmation that the videos will ship by 10am on 6/15.

This reverts commit a1f02ff4e439d9de58ed4a788a52c2f73cfc36ce.

Change-Id: I854c589436bcc4e146770415898996826f552b21

Merge "docs: Updates for DP4 release" into nyc-dev

docs: Updates for DP4 release

- note final SDK is available
- update links to DP4 release notes

Change-Id: Id56b4bc9673f7ad12212d67045dc18fb9c72a6e8

Merge "Docs: Updated 'NDK Apps Linking to Platform Libraries'" into nyc-dev

Docs: Updated 'NDK Apps Linking to Platform Libraries'

- updated compatibility with future platform releases
- expanded on ways to dignose and debug

Bug: 29215157
Change-Id: Ie63d024eac9914d4553419c734132696fb94d7e5

Merge "docs: Wording change for Support Lib 24.0 release notes" into nyc-dev

Merge "docs: Updating N preview release notes for DP4" into nyc-dev

Merge "Docs: Updated localizations for N Developer Preview docs." into nyc-dev

Docs: Updated localizations for N Developer Preview docs.

Bug: 28912213

Change-Id: I8ff0dd5989863ce4097e2290ff9a10e4cc5789ec

Merge "Update CTS shims" into nyc-dev

docs: Wording change for Support Lib 24.0 release notes

See first comment for doc stage location.

bug: 29215447
Change-Id: I75b1dc88e19e557c9ffee6336bfc342a17d1271f

docs: Updating N preview release notes for DP4

See first comment for doc stage location.

bug: 28982873
Change-Id: I692b361f4a0d2d3627fa8cea19082fc24306f470

Merge "Only skip trimming task if the stacks are not null." into nyc-dev

Merge "Allow apps with CREATE_USERS permission to call UM.getUserInfo." into nyc-dev

Merge "Docs: Adds SurfaceView to API Overview page for Android-N DP4." into nyc-dev

Merge "Toast timeout: Just strip FLAG_KEEP_SCREEN_ON" into nyc-dev

Merge "Unbreak Javadoc on getAttestationChallenge()" into nyc-dev

Docs: Adds SurfaceView to API Overview page for Android-N DP4.

Change-Id: I29854c82ef788b00c1ebc305bc5faf463c2d8dc8

Merge "Add API 24 to reference selector." into nyc-dev

Merge "Doc change: add diffs for 23->24." into nyc-dev

Only skip trimming task if the stacks are not null.

Bug: 29121819, 29359871
Change-Id: I8f8602796e316ed5b36a033e4eaa020650064f74

Update semantics of on-body Keystore API.

Architectural changes from the Wear team have required changes to the
semantics of the Keystore on-body feature. Although no devices will
actually implement this feature until the release of new Wear devices
with the necessary sensors, we need to get the API documentation right
now.

Bug: 29369151
Change-Id: I857e00928f994a1337313123eae7e00534c53c3c

Add API 24 to reference selector.

Change-Id: I153245ef814e27bf50f9d972ad1aa34fd92e6b8a

Allow apps with CREATE_USERS permission to call UM.getUserInfo.

Bug: 29355382
Change-Id: Iaf73c453156e3a989660d05124b0c0938619abd7

Merge "Make sure task exists in WM before trying to set the resizing state" into nyc-dev

Merge "Fixed another case where the systemUI could crashloop" into nyc-dev

Unbreak Javadoc on getAttestationChallenge()

Bug: 28844154
Change-Id: I8402a39f366291cd2b5d646536cd2c2bab8556d2

Merge "docs: Noted "Crypto" removal" into nyc-dev

Toast timeout: Just strip FLAG_KEEP_SCREEN_ON

We added the toast timeout to fix b/21693547. Where the client
leaks non visible toasts which end up keeping the screen on.
Simply removing the toast windows as we were doing previously,
isn't very good though. If the timeout fires when an app could
still use the toast (e.g. due to long pause under debugger),
the app could crash. It's not obvious how (if even possible
without API semantic/guarantee change) to prevent this issue
from the client side. Furthermore, #33 at b/21693457 implies that
we need to respect FLAG_KEEP_SCREEN_ON even for windows that
never submitted a buffer. With time constraints as they are,
the only solution I can think of that solves both without
chance of regression is just stripping FLAG_KEEP_SCREEN_ON when the
timeout fires.

Bug: 29105388
Change-Id: Ic2df897f86acf814ea3d4f911a0532d65af0c19f

Merge "Quick workaround for a performance regression in IME APIs." into nyc-dev

Merge "Version PrintRecommendationService." into nyc-dev

Merge "Revert "Don't update task to return to type for last task over home"" into nyc-dev

Make sure task exists in WM before trying to set the resizing state

It is possible for a task to be removed from the WM and stack in AM
while resizing (e.g. app crashes). In this case we want to make sure
the task exist in WM before we try to set the dock resizing state vs.
depending on the fact that the task is in recent list in AM.

Bug: 29325623
Change-Id: Ib89a7bfe12fe1f2c4c6c56ac9e13b0052ea2b4ec

Docs: Added overview of Custom Pointer API

Bug: 28241984
Change-Id: Iffa2a0cc46f71377b79f4ac792d2f469353cc395

Quick workaround for a performance regression in IME APIs.

It turns out that the performance of
InputMethodManager#getCurrentInputMethodSubtype() is regressed from ~1ms
to ~20ms when
 - Settings.Secure.SELECTED_INPUT_METHOD_SUBTYPE == -1 and
 - The active IME supports many subtypes (~100)
because we try to find a fallback subtype based on the system locales
every time when IMM#getCurrentInputMethodSubtype() is called.

This could be contributing UI janks because spell checker clients
running in the UI thread indirectly depend on that method.

Fortunatelly the critical path is in
InputMethodUtils#getImplicitlyApplicableSubtypesLockedImpl(), which is
basically a state-less method. We can easily and safely cache its
result by using LocaleList and InputMethod as cache keys.

With this CL the performance basically recovers to the Android M level.

Bug: 28889203
Change-Id: I5ed16c7f14cc18052854f4fd6c9bae8550c332ee

Fix accessibility for autorotate tile.

The tile should be 'on' when rotation is not locked.

Change-Id: I084be4ffb759bf20efe80358b5d103fec8892f76
Fixes: 29102740

docs: Noted "Crypto" removal

See first comment for doc stage location.

bug: 28972616
Change-Id: I891af3d2e0b84f2daa207221e477a09efafc9d18

Only reload callback on configuration change.

So it'll get zen changes when the volume dialog is not showing.

Change-Id: I0d4220d0a000d74c6994b07b2d5b8562a6a52585
Fixes: 29355577

Merge "Fixing several issues with accessibility" into nyc-dev

Merge "Revert "Announce access point changes."" into nyc-dev

Merge "Note null alarm worksource names under the alarm's package" into nyc-dev

Merge "docs: Updated support library revisions page for 24.0.0" into nyc-dev

Version PrintRecommendationService.

Fixes: 28908572
Change-Id: I8df106df887cacc8db37f781ade97678960cee04

Revert "Don't update task to return to type for last task over home"

This reverts commit 21893824c1d15a1a46cb75725ad951620569c3c6.

Original commit revealed issue b/29342752 when launch logic was applied
to incorrect stack while moving to PiP mode.

Bug: 29342752
Bug: 29237751
Change-Id: I88d75399eb4b9daf4de87a9a560eb0df0b515173

Fixed another case where the systemUI could crashloop

Because it's not guaranteed to use any builder at all,
there is another case where the SmallIcon could remain null.
We are now checking this lazily instead of ahead of time.

Change-Id: I7a0feff6911b2bce6707427259d3423131a26e32
Fixes: 29255365

Merge \\"docs: Clarifying direct boot notification messages\\" into mnc-io-docs am: 120fd583d5
am: 4c5e5ec6d5

Change-Id: I2bfe82b79c928d10b7725315d42b00a1994bbd52

Merge \"docs: Clarifying direct boot notification messages\" into mnc-io-docs
am: 120fd583d5

Change-Id: I076e69e0a0497ed461dc576b72419152e89bb010

Merge "docs: Clarifying direct boot notification messages" into mnc-io-docs

Update CTS shims

* define a update hash such that com.android.cts.priv.ctsshim can't
  be updated
* create a new APK to verify that <restrict-update> works as expected

Bug: 29311441
Change-Id: I61f3ae3dbd9c79b2dcdfbc30757af0f9a669fce6

Merge "Improve action bar transition" into nyc-dev

Merge "Fix jittery action bar during transitions" into nyc-dev

Merge "Make RemoteView margins density-change safe" into nyc-dev

Merge "Update SyncManager min period to match JobScheduler" into nyc-dev

Merge "MediaPlayer: fix java doc for setSurface and setDisplay." into nyc-dev

Merge "docs: Updating N download pages for DP4" into nyc-dev

Merge "Don't take the DPMS lock in DPMI.createPackageSuspendedDialogIntent" into nyc-dev

Merge "Docs: Changed a command in Keyboard Shortcuts" into nyc-dev

MediaPlayer: fix java doc for setSurface and setDisplay.

Bug: 29149415
Change-Id: I281b6328c068255387ca01a987ff136c3be5e25b

Merge \\"Fix broken {@docRoot} links\\" into mnc-io-docs am: a0fa50e40b
am: 423fe7db6a

Change-Id: I192297bc254d199e25249952591b3b2a1fc2b617

Revert "Announce access point changes."

Fixes: 29151018

This reverts commit 7ed62eb7a214126d3dbfd6abd7d6a7db73cc9f3c.

Change-Id: If40669b21cc2a2db7e2bb27c7c981d889cb094ed

Merge \"Fix broken {@docRoot} links\" into mnc-io-docs
am: a0fa50e40b

Change-Id: I7dd53f31f44262e09624d23d820723893aaba5c7

Note null alarm worksource names under the alarm's package

And log a nastygram because it implies the app is misbehaving.

Bug 29333122

Change-Id: Ia683304b8a9eafb234df81e874ee3f0636c4e56a

Merge "Fix broken {@docRoot} links" into mnc-io-docs

Merge "Add a key for setting carrier's CDMA roaming mode" into nyc-dev

Merge "Fixed a bug with a legacy notification API" into nyc-dev

Don't take the DPMS lock in DPMI.createPackageSuspendedDialogIntent

This method is called by AM with the lock held, so can't take the
DPMS lock.

It still takes a different lock, but we don't call into the external
world while holding this lock, so this is fine.

Bug 29242568

Change-Id: Idbecdd7d97385ca66c693903443471fdbae833e6

docs: Clarifying direct boot notification messages

Adding details on which notification message to listen for
when developing apps that need to interact with direct boot mode.
Note: This is a doc fix for the N direct boot feature docs, but
not specific to any preview release.

Bug: 28729775
Change-Id: If6b239e7aa92bced735cfa1e72d8891d193a420c

Merge \\"Docs: Watch face performance update\\" into mnc-io-docs am: 99677fb42d
am: 106af82850

Change-Id: Ia9fa224a4ebb6e51a3c6141751036504d202e137

Merge \"Docs: Watch face performance update\" into mnc-io-docs
am: 99677fb42d

Change-Id: I41b6ee0628e561cfb7f711d07ac1e41ff71fcb60

Merge "Docs: Watch face performance update" into mnc-io-docs

Update SyncManager min period to match JobScheduler

Use JobScheduler's minimum period in SyncManager.
Syncs will respect JobScheduler's min period anyway,
this change ensures that dumpsys of both services
show the same period.

Bug: 29253834
Change-Id: I0cd7f85da56af967963573a048c3a71436ed9c5b

Merge "Redraw if dirty during stopped when resumed" into nyc-dev

Merge "Import translations. DO NOT MERGE" into nyc-dev

Merge "Import translations. DO NOT MERGE" into nyc-dev

Merge "Show sign-in to network notification to all users, not just owner." into nyc-dev

Import translations. DO NOT MERGE

Change-Id: I4aafe42447a82d42a1b3c013f046a4a264dafcc4
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: Ied2403cbd3d46b70771f137bd4b5aab6fb7150b5
Auto-generated-cl: translation import

Merge "Make CotentResolver#openInputStream handle errors from reliable pipes." into nyc-dev