Merge "Add EXTRA_HANDOVER_FROM_PHONE_ACCOUNT." into oc-dr1-dev

Merge "AOD: Use double tap coordinates to trigger Ambient Indication" into oc-dr1-dev

Merge "Address issues surrounding freezing by display." into oc-dev
am: 8ab4d11145

Change-Id: I5eec6ff604056b7095ac07afd9721a5223a584d8

Merge "Address issues surrounding freezing by display." into oc-dev

Add EXTRA_HANDOVER_FROM_PHONE_ACCOUNT.

Used in handovers to tell the account we're handing over to which account
the call is from.

Test: Manual
Bug: 38027075
Change-Id: I9b61a5fce3e4f7344174a1b81f4964ca02b71ccd

AOD: Use double tap coordinates to trigger Ambient Indication

Bug: 38386446
Test: Double tap on AOD, verify that it still works. On device that supports double tap coordinates, verify that double tapping on ambient music opens ambient music.
Change-Id: Ic75ba731a980c9598c85927e83d2032f01da4822
(cherry picked from commit 44198f5124743f0d603eb4efa3b436bd96f32144)

Address issues surrounding freezing by display.

The first issue is the animating a display as a consequence of the
freezing due to configuration change during construction. This
additional animation is not expected and interferes with tests
expecting for the original display contents to be shown intact. This
CL addresses the issue by not freezing while the display is not ready
(before construction is finished).

The second problem addressed is book-keeping for
DisplayContentsAnimators in WindowAnimator. Currently, a getter
method is used internally to reference these animators, which
generates them if not present and adds them to the animation
iteration. In the case we set an animation on a display that no
longer exists (which can be the case after unfreezing), we end up
recreating this object. This can lead to us trying to animate a
non-existent DisplayContent. This CL prevents creating an animator
for a non-existent display and adjusts the methods using this getter
to handle this case.

Fixes: 62460846
Fixes: 62461229
Bug: 35486733
Bug: 62541591
Test: go/wm-smoke
Test: open and close projected android auto mode repeatedly and
      ensure display correctness
Test: cts-tradefed run singleCommand cts-dev --module CtsMediaTestCases --test android.media.cts.EncodeVirtualDisplayTest#testEncodeVirtualDisplay

Change-Id: I60ade6f97440c8fa01b10e36c019865cf1fd0730

Merge "Eagerly serialize historical sessions" into oc-dev
am: c7ea7effde

Change-Id: Iff55674bc989986e14d0bcea3c71bad5751d4dcb

Merge "Eagerly serialize historical sessions" into oc-dev

Eagerly serialize historical sessions

mHistoricalSessions maintained a strong reference to a PackageInstallerSession,
which in turn kept references to Bitmaps and other heavy-weight objects around.

Since this field is primarily used for debugging, this change replaces it with
a String dump of the session in question. Each dump takes about 600bytes, which
is comparable to the sizes of the un-serialized raw objects.

Bug: 62485552
Test: Manual

Change-Id: I4949a64b538ab4a97384f4f8bc9a6ef155a4b128

Merge "DO NOT MERGE Remove createSocket(Socket, InputStream, boolean) from APIs." into oc-dr1-dev

Merge "Merge "Import translations. DO NOT MERGE" into oc-dev am: 7fc501184a  -s ours" into oc-dr1-dev

Merge "Import translations. DO NOT MERGE" into oc-dr1-dev

Merge "Import translations. DO NOT MERGE" into oc-dev
am: 7fc501184a  -s ours

Change-Id: I418c08d5f5bf997d7e870bafc2a0b68f519dd990

Merge "Import translations. DO NOT MERGE" into oc-dr1-dev

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

Merge "Minimal change to ensure events are propagated to the right DividerView." into oc-dev
am: b1cdc17dfb

Change-Id: I987d923b923e289baa76d5155cf6a7fe4d9ece6a

Merge "[ErrorProne] Fix Error Prone warnings in aware/p2p code" into oc-dr1-dev

Merge "Minimal change to ensure events are propagated to the right DividerView." into oc-dev

Merge "Protect broadcast used in settings for bond cancel" into oc-dev
am: 68a4c91bb4

Change-Id: Ifdb4b625f21dcda5248071263aaf6936158efb39

Merge "Ensure that we use SF Vsync Choreographer for the PiP transition." into oc-dev
am: e1e0db8ca9

Change-Id: I2d854e36c31a8fa1cf74e5c5a536969ee94dc79b

Merge "Fix the issue that uncrypt isn't called under quiescent mode" into oc-dev
am: f40f297546

Change-Id: Id60cf7cf4b82f17fd094fa011bb54a543e636f48

Merge "SurfaceView: Fix missing override for SurfaceControl proxy" into oc-dev
am: 42b3610adc

Change-Id: I3238399a26917f80bd4e33bb20142f62076e6f4f

Merge "Protect broadcast used in settings for bond cancel" into oc-dev

Merge "Attempted fix for navigation bar flicker" into oc-dev
am: bbd7dfc0a4

Change-Id: I80a618eb59f0479060a69ca4f0be8ac883a8d48d

Merge "Improve developer docs for storage APIs." into oc-dev
am: 05b52d8ba5

Change-Id: I282786d353e3dc406c7123daa00d3e7fe43de7bc

Merge "Don't log all instant app resolutions" into oc-dev
am: 3ef0b8260f

Change-Id: I4ef141816c67957b0302be814315e2e739406c31

Merge changes from topic 'sysui-tooltip' into oc-dev
am: 3fbcec5b86

Change-Id: I7affdad6abfb283c0eec6718797f7d8580ea507b

Merge "Import translations. DO NOT MERGE" into oc-dev
am: 7c2a12b658  -s ours

Change-Id: I5f257f746717037d275e025c80581d1f4cccc67d

Merge "Camera2: Cache method list in MethodNameInvoker." into oc-dev
am: 71029bf631

Change-Id: I99c6322d7e06262861eb0bb8be4ce8d4572ffbae

Merge "resolve merge conflicts of 8f1d13300f73 to oc-dr1-dev" into oc-dr1-dev

Import translations. DO NOT MERGE

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

Merge "Active camera apps can defy reserved cache space." into oc-dr1-dev

Import translations. DO NOT MERGE

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

Merge "Ensure that we use SF Vsync Choreographer for the PiP transition." into oc-dev

Merge "Fix the issue that uncrypt isn't called under quiescent mode" into oc-dev

Merge "SurfaceView: Fix missing override for SurfaceControl proxy" into oc-dev

Minimal change to ensure events are propagated to the right DividerView.

- The RecentsDrawnEvent sent when Recents is first drawn can be racey, if
  it is sent before the configuration change in Divider is dispatched, then
  it will be posted for the old DividerView, which prior to ag/2363241 was
  still sending the correct resize calls (the animation is independent of
  the view).  For now, we register the event handler in Divider, and
  instead proxy it to the current view when it is posted.

Bug: 62528361
Test: go/wm-smoke
Test: Play a movie in Play Movies, ensure the activity is resized

Change-Id: I665b0c6af55dea0db1916f5b041589bf72da3baf

Merge "Attempted fix for navigation bar flicker" into oc-dev

Protect broadcast used in settings for bond cancel

Test: cancel pairing from notification tray
Bug: 62094630
Change-Id: Ia83a874c1ea9a1d9d5502609f81e94052bb5759c
(cherry picked from commit 977992f3fc9f4f6bb2d1f9536143327a6021c5d1)

Merge "Improve developer docs for storage APIs." into oc-dev

Merge changes from topic 'ims-handover-api' into oc-dr1-dev

* changes:
  Add callSessionMayHandover API to ImsCallSession.
  IMS: Support for Possible Call Session Handover

Merge "Don't log all instant app resolutions" into oc-dev

Merge "Hide APIs not intended to ship in DR." into oc-dr1-dev

Merge "Updates to NotificationMenuRowPlugin so that long press may be captured" into oc-dr1-dev

Merge changes from topic 'sysui-tooltip' into oc-dev

* changes:
  Pass window token when creating a tooltip popup
  Revert "Use a PopupWindow to show tooltips"

Merge "Import translations. DO NOT MERGE" into oc-dr1-dev

Merge "Import translations. DO NOT MERGE" into oc-dr1-dev

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

resolve merge conflicts of 8f1d13300f73 to oc-dr1-dev

Change-Id: I1a30ba6663041e1ce0a2d347671deb83b1d9d7ff

Merge "Camera2: Cache method list in MethodNameInvoker." into oc-dev

Import translations. DO NOT MERGE

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

Active camera apps can defy reserved cache space.

We normally prevent apps from allocating into the "reserved" cache
space, but this change makes an exception for an active camera app,
since the user is probably trying to capture an important memory.

This change only lets the active camera app clear up to half of the
reserved space, since we don't want to completely destroy the
experience of all other apps.

Test: manual app before/during/after active camera session
Bug: 38267830
Change-Id: Ie9e63884fb2638ca881e10b894629eea84601648

Improve developer docs for storage APIs.

No code changes; only docs.

Test: builds
Bug: 38508833, 37987197, 37978296
Change-Id: Idfeb680480b2f818d18f787cbf20ceab896763a2

Merge "Fix wake-and-unlock animation" into oc-dev

Attempted fix for navigation bar flicker

Not 100% sure why it was flickering, but I do know that the
flicker started happening after putting this into a separate
thread, and that with that change it doesn't flicker anymore,
ever.

Adds 0.5ms while unlocking but at this point this is a good trade-
off to make.

Test: Unlock device with FP
Change-Id: I736a3c64d9b63df9d6cdcb7855bde13a19560a83
Fixes: 62380221

Pass window token when creating a tooltip popup

This allows using tooltips in System UI.

Bug: 62065980
Test: android.view.cts.TooltipTest passes
Change-Id: If0a76d0806aa92efa4be57204c4517242b0ebb99

Merge "Removed deprecated setExtras() method." into oc-dev
am: 7c41ddb975

Change-Id: Id74b4becd0f26d185cad3ecbe539aaa69f7ac89e

Camera2: Cache method list in MethodNameInvoker.

Retrieving list of methods in every invoke() call is very expensive.
Caching the list inside the constructor prevents several unnecessary
calls to Class.getMethods().

Test: Run camera2 CTS
Bug: 62490715

Change-Id: Ib2a93af0f364b055df2eab9bd7870730428429ad

Revert "Use a PopupWindow to show tooltips"

This reverts commit 77e539775b52da55a8b23f1a9765d471ee782013.

Change-Id: Ic4054e0d507ce95ab93282d4da002dd804aeb26d

Merge "Removed deprecated setExtras() method." into oc-dev

Import translations. DO NOT MERGE

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

Fix wake-and-unlock animation

Since okToDisplay was false when we started the keyguard exit
animation, no animation was applied and we didn't create a
starting window, which lead to flickering. We fix this by
allowing animations from mScreenOnEarly.

Furthermore, we synchronize the navigation bar better with the
rest of the animation.

We also need to apply no animation to the status bar window as
we go through performShowLocked because we were waiting for it
to draw.

Test: go/wm-smoke
Test: Wake-and-unlock
Test: Make sure no other regression with screen on experience
Change-Id: I5f264b74cc258e8d7f608978edfb1faa5ead385c
Fixes: 38441599

SurfaceView: Fix missing override for SurfaceControl proxy

I will not pretend classes are interfaces...I will not pretend
classes are interfaces...

Bug: 62444724
Test: cts-tradefed run singleCommand cts -d -o --module CtsViewTestCases --test android.view.cts.SurfaceViewSyncTest#testSurfaceViewBigScale
      go/wm-smoke

Change-Id: Iccca46edcd669430c32f31e9c7d8aba1db254f2d

[ErrorProne] Fix Error Prone warnings in aware/p2p code

Bug: 33104624
Test: builds
Change-Id: Ibf17a02c81aebfa28903fe440a75e3c2e9244a19

Import translations. DO NOT MERGE

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

Fix the issue that uncrypt isn't called under quiescent mode

OTA fails on fugu under quiescent mode because the reboot reason changes
from "recovery-update" to "recovery-update,quiescent". The new reason
isn't checked in shutdown thread so that shutdown thread doesn't call
uncrypt properly before rebooting into recovery.

Bug: 62324707
Test: Recreated and fixed the "block.map" missing failure on fugu.

Change-Id: I110653cd64dbbdc71e89ead2197bf023a7c054e8

Updates to NotificationMenuRowPlugin so that long press may be captured

This will allow the NotificationMenuRow plugin to do something else
rather than showing a menu when the view is long pressed.

- Allow menu to intercept touch events
- If the menu item provided to long press is null, don’t show it
- Also passes the status bar notification when the menu is created
  or updated

Bug: 62534409
Test: manual, have no plugin and long press -> see notification info
      have a plugin that returns null for long press -> nothing happens
Change-Id: I384c5217d5343e990f4c3506eb2477b015705e5c

Import translations. DO NOT MERGE

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

Don't log all instant app resolutions

we only want to log the successes

Change-Id: I31c79a1c964088ee67fd7527ca7fea16b0d29830
Fixes: 36563095
Test: Manual; run and see that only the success events are in the eventlog

Ensure that we use SF Vsync Choreographer for the PiP transition.

- Move the bounds animation onto the animation thread
- Remove existing code referencing the old sf-vsync choreographer
- Add ability for ValueAnimator subclasses to reference a different
  AnimationHandler, which uses a different FrameCallbackProvider with the
  sf-vsync choreographer in the animations that require it
- Ensure that PiP touch events are batched and sent aligned with the
  sf-vsync
- Move GC onto its own thread to not block other BackgroundThread calls

Bug: 36371375
Test: android.server.cts.ActivityManagerPinnedStackTests
Test: bit FrameworksServicesTests:com.android.server.wm.BoundsAnimationControllerTests
Test: go/wm-smoke

Change-Id: I6a41b35a4e4d4d6dbea82c2673452825fe3ffa58

Merge "Change FlakyTest to Ignores" into oc-dr1-dev

Merge "Work on issue #62390590: SecurityException in JobIntentService$..." into oc-dev
am: 4eb76b2db8

Change-Id: I3aa5bbccbc669499a8ab6b12f180a7f5841164c3

Merge "Work on issue #62390590: SecurityException in JobIntentService$..." into oc-dev

Merge "Prevent ConcurrentModificationException in updateAllRecognitions" into oc-dev
am: 484fb951e0  -s ours

Change-Id: I97dc24bc38e194b508a1b485ec4d7e1f8b62285d

Hide APIs not intended to ship in DR.

This CL will be reverted once it merges into the next release branch
to re-introduce the proposed APIs.

This CL also makes the following code changes to preserve API
compatibility:

-- It keeps the recently added RemoteViews.clone() synchronized logic
intact, but moves it inside the method to avoid changing the API
signature.

-- It reverts the RttCall.read() behavior to the oc-dev logic, since
we can't throw IOException until we get an API level bump.  (The
original logic returned null instead of throwing.)

Test: builds, boots
Bug: 62427252, 62431886, 62427329
Bug: 62468911, 62431162, 62428935, 62429096
Bug: 37290820, 37359238, 36886243
Bug: 30143923, 35761231, 62192626, 29829689
Change-Id: I83d723f598cb0ee1fe198e65debd86ef7fd0420c

Merge "Prevent ConcurrentModificationException in updateAllRecognitions" into oc-dev

Merge "Round the slashes of SlashDrawable" into oc-dr1-dev

Change FlakyTest to Ignores

These are failing reliably, no need for FlakyTest

Bug: 62475085
Test: runtest systemui
Change-Id: Ib9abedda149976cffe07ea75c7034dd290d1f715

Change the multipath hint default to 0.
am: d1e7d3d73e

Change-Id: Iab590e33a1ae64701f0023970d53b23470533962

Change the multipath hint default to 0.

It was set to 3 (handover+reliability) during the development
cycle, but should not be set to a nonzero value on user devices
until the returned value takes into account the amount of
non-default network data usage.

Bug: 34630278
Test: marlin builds and boots
Test: ConnectivityServiceTest passes
Change-Id: I2b09a6e09c497886862cbaaaf3b283dedfbfa94c

Merge "Add additional mocking for Context innards" into oc-dr1-dev

Merge "Convert IControlsTethering from interface into callback" into oc-dr1-dev

Prevent ConcurrentModificationException in updateAllRecognitions

When invoking updateAllRecognitions, if a callback binder was determined
to have died, an internal function would go and remove it from
mModelDataMap. However, updateAllRecognitions was iterating over this
map, so it would then explode. By first making a copy of the model datas
before iterating over all of them, this problem is avoided.

(As part of trying to figure out what was happening, also updated all
the method names that implicitly assumed they had a lock, and double
checked that everything with a Locked suffix is actually locked)

Bug: 62487479
Test: Use the sound trigger test app to load and start two models, force
kill the app (so the dangling binders hang around), then enable power
save (which triggers the call to updateAllRecognitions)

Change-Id: I87b9dfc1b2af5e294050b146737916ccaad882c1

Merge "Prevent ConcurrentModificationException in updateAllRecognitions" into oc-dr1-dev

Add additional mocking for Context innards

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net passes
    - cherry-picked to master and verified there as well
Bug: 32163131
Bug: 32561414

Merged-In: I2695841bfc31280060754132e589af1ca95911da
Merged-In: I9a05f34035a15b233a44d517f2b2426481679974
Change-Id: I0b5b1b12d55547d08c332c7d274f19f0023a7b07
(cherry picked from commit ec37275ec9702b56bf328031675906a5483b89d4)

Convert IControlsTethering from interface into callback

Additionally:
    - add updateLinkProperties() method to new callback
    - skeletally connect LinkProperties updates through
      to OffloadController

TODOs liberally sprinkled through out.

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net passes
Bug: 29337859
Bug: 32163131
Merged-In: I178130e65db4fef26d62ea7eaa9a52f7e8103432
Merged-In: I832d77747de7517513c22d36fff4a2fa2bb95154
Change-Id: I631d17b26be153534551a1615931fc98b598b953
(cherry picked from commit ab6439b0f8dd0a67c81235a3fc4d9c149de3780d)

Prevent ConcurrentModificationException in updateAllRecognitions

When invoking updateAllRecognitions, if a callback binder was determined
to have died, an internal function would go and remove it from
mModelDataMap. However, updateAllRecognitions was iterating over this
map, so it would then explode. By first making a copy of the model datas
before iterating over all of them, this problem is avoided.

(As part of trying to figure out what was happening, also updated all
the method names that implicitly assumed they had a lock, and double
checked that everything with a Locked suffix is actually locked)

Bug: 62487479
Test: Use the sound trigger test app to load and start two models, force
kill the app (so the dangling binders hang around), then enable power
save (which triggers the call to updateAllRecognitions)

Change-Id: I87b9dfc1b2af5e294050b146737916ccaad882c1

Merge "Added 'dumpsys activity lastanr'" into oc-dev
am: b32fecd6e0

Change-Id: Iede7911ea687b7890833ee967da010db6041fb49

Merge "Added 'dumpsys activity lastanr'" into oc-dev

Merge "Import translations. DO NOT MERGE" into oc-dr1-dev

Update captive portal notif for Android TV
am: 6ab655227a

Change-Id: I60a67b0f22aa186e8b738b8a9ee002b2b5b04e42

Merge "Import translations. DO NOT MERGE" into oc-dr1-dev

Merge "Import translations. DO NOT MERGE" into oc-dr1-dev

Merge "Import translations. DO NOT MERGE" into oc-dr1-dev

Update captive portal notif for Android TV

Test: Connect to captive portal, notification appears in side panel

BUG:38453402
Change-Id: I39baf90897f2f3d67b8b03c31dd74fa1d39d39de

Merge "Preemptively create Bitmap from GraphicBuffer" into oc-dev
am: a304564081

Change-Id: I916e692fae56ed79a60d37b471b6e07457d2509d

Merge "Preemptively create Bitmap from GraphicBuffer" into oc-dev

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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