Merge "Adding "attach-agent" to ActivityManagerShellCommand" am: 05dfd10254
am: db0803b4e8

Change-Id: I4493e6b3bd2cf8447e8c3043878ad0f531c0e677

Merge "Adding "attach-agent" to ActivityManagerShellCommand"
am: 05dfd10254

Change-Id: I3623f617319f03bad87f65b61959ec2ce822ac7e

Merge "Adding "attach-agent" to ActivityManagerShellCommand"

Merge "camera case fail for testSceneModes" am: b59cd66e93
am: a22c40839e

Change-Id: I52fca667fcd8b8ff501bb145ee49934ba47f92a3

Merge "camera case fail for testSceneModes"
am: b59cd66e93

Change-Id: I509c87e5b69e4bd26dc97e86063f7a24860e8b3b

Merge "camera case fail for testSceneModes"

Adding "attach-agent" to ActivityManagerShellCommand

This new command is used to attach runtime agents to a running application:

attach-agent <PROCESS> <FILE>
   Attach an agent to the specified <PROCESS>,
   which may be either a process name or a PID.

Test: m test-art-host, manual testing:
    . invalid syntax, missing arguments
    . invalid syntax, extra arguments
    . invalid numeric PID
    . invalid process name
    . valid process, not debuggable
    . valid process, missing agent
    . valid process, valid agent

Bug: 31682382

Change-Id: I61cc8bf20addb1702acc8e7aae65b2f9ed7c5ca0
Merged-In: Ife88dbf23991dde7945d9208e54cd014bb7ecdc6

Merge "Track libcore change d7d594236d66c4d595f7622a264fd51." am: 97ebc29e9c
am: b872486722

Change-Id: Ia7111f2be312494fbdb816efae9a7b465ce6b1a4

Merge "Track libcore change d7d594236d66c4d595f7622a264fd51."
am: 97ebc29e9c

Change-Id: Ied575c14661753a56241f0cea62391b057ab4316

Merge "Track libcore change d7d594236d66c4d595f7622a264fd51."

Merge "TimeZoneBundleInstaller -> TimeZoneDistroInstaller" am: eb82e3de87
am: c1867dedfa

Change-Id: If2f639d07a44861b96b4497cc3dc02364dc85c51

Merge "TimeZoneBundleInstaller -> TimeZoneDistroInstaller"
am: eb82e3de87

Change-Id: I7666ae9c9472f11cafe0df5a2f259df1c499d44b

Merge "TimeZoneBundleInstaller -> TimeZoneDistroInstaller"

Track libcore change d7d594236d66c4d595f7622a264fd51.

Test: make update-api docs
Change-Id: Icfe470034b53afff60469d5bcfd58d959cea2488

Merge changes from topic 'cm_request_to' am: 6fb97e8fd8
am: ede9f94e32

Change-Id: I2720e64e4af88df9d3a7b6209e9fcef4a0340986

[CM] Remove non-functional timeout from request network am: e9aea9d397
am: a6be89aa40

Change-Id: Ie00ac8dee9302fafc0cdad95ec08d376ad908ab9

Merge changes from topic 'cm_request_to'
am: 6fb97e8fd8

Change-Id: I9ca18c0570be831ab37cdf200c28b0778117198d

[CM] Remove non-functional timeout from request network
am: e9aea9d397

Change-Id: Ica347b2c2181fe04a7efb2769d7498a31c888e5a

Merge changes from topic 'cm_request_to'

* changes:
  [CM] Remove maximum timeout value for request network
  [CM] Remove non-functional timeout from request network

[CM] Remove maximum timeout value for request network

The request network with timeout was originally created with a
check of max timeout against a constant of 100 minutes. However,
the API was not public and did not implement a timeout. Any users
were internal and never got any onUnavailable() callback (since
timeout never triggered).

There is no reason to have a max timeout so the constant is
remove.

Bug: 31399536
Test: unit tests and CTS of ConnectivityManager
Change-Id: Icbedfb4299d75b6a7e3e43720111531f1faafd06

Merge "OBEX: Set Min maxPacketLength negotiated in CONNECT Operation." am: 3ac3170a4b
am: 1112046506

Change-Id: Ib406bdcd0f388c7fe1f226da2ae17a2c588739c9

Merge "OBEX: Set Min maxPacketLength negotiated in CONNECT Operation."
am: 3ac3170a4b

Change-Id: I477f407d94a43a14083fa68a7b271f5d4fe8445d

Merge "Fix the "Alarm volume" icon is displayed incorrectly" am: e0ba81110a
am: 657abc315d

Change-Id: I295d50e9513775dddb2aea3686275264d7e0f12c

Merge "OBEX: Set Min maxPacketLength negotiated in CONNECT Operation."

Merge "Fix the "Alarm volume" icon is displayed incorrectly"
am: e0ba81110a

Change-Id: Id7996cd16fea9eccbdcd389206e77d7390cd25a2

Merge "Fix the "Alarm volume" icon is displayed incorrectly"

Merge "Captive portal: better detect empty responses" am: c217f7a9f3
am: af8e749820

Change-Id: I0b2cd45fb92e2f8accb53ef4e050d2b6e7f7fd1c

Merge "Captive portal: better detect empty responses"
am: c217f7a9f3

Change-Id: If6229a3a5c0ccc3575d44d11ed9f486cab35eac3

Merge "Captive portal: better detect empty responses"

Captive portal: better detect empty responses

The captive portal detection logic treats empty http responses with a
Content-Length of 0 as not coming from a portal.

However when the Content-Length is missing from the response header, the
empty response is not detected as such and is classified as a captive
portal answer.

When this happens for the http probe, the portal detection logic will
short circuit the detection and wrongly report a portal. The system
validates the network in such conditions only if the https probe
returns a successful connection faster than the http probe.

This patch attempts to better detect empty responses by trying to read
the first byte from the response body when the Content-Length is missing
for 200 responses.

Test: build, flashed + manual tests.
Bug: 33498325

(cherry picked from commit cb4aa4d412c7940386df9b8dd681e0d2efebfd1d)

Change-Id: Ibb9914cba72a4dab3ae76746d8889bbf083be812

Merge "Add extra key for app-specified outgoing call timeout" am: decc6c9d56
am: 2bc24e252e

Change-Id: I1912bfa8856e78f375587d94bf589ef2fd25bee8

Merge "Add extra key for app-specified outgoing call timeout"
am: decc6c9d56

Change-Id: I58a70b265f4b5af735e93c9004616bd44d5c51de

Merge "Add extra key for app-specified outgoing call timeout"

Merge changes from topic 'dual_role_chargers' am: 37b939a180
am: 0a3efc2622

Change-Id: I2c0ba36c46674ff7f9a0b963543e271eec93858c

Merge changes from topic 'dual_role_chargers'
am: 37b939a180

Change-Id: I2b9eda605ae85347eb2e3e4202a6c7ec176778af

Merge changes from topic 'dual_role_chargers'

* changes:
  Introduce debounce to filter rapidly toggling type-c ports
  Fix handling usb dialog for wall chargers.

Introduce debounce to filter rapidly toggling type-c ports

Type-c ports can quickly toggle between connected/disconnected
states. Introduce debounce to prevent sending spurious notifications.

Bug: 34972898
Test: notification should not be queued for a pixel-c charger not connected
to the power outlet.
Change-Id: I4aa19f9f864fe5b77e65f6a07a3184d8aba1f5fc

Fix handling usb dialog for wall chargers.

UsbPort.POWER_ROLE_SINK is orthogonal to the type of the charger
attached. POWER_ROLE_SINK would be the case for AC charging and
USB charging. Therefore query BatteryManager for the charger
type.

Bug: 34972898
Test: Charging notification should not show for pixel-c chargers.
Change-Id: I8dddcd7727b6af973bd173d2c6e325aa4be2ca3a

Add extra key for app-specified outgoing call timeout

Bug: 34474757
Test: manual
Change-Id: Ia93d68d6a01308b0b93b88977daf983a0ed3a22c

Merge "ConnectivityManager: fix potential NPE" am: b688dd18ab
am: ab453cd565

Change-Id: I72e114037ce6a0ec53863749b8ffa34513c9fdce

Merge "ConnectivityManager: fix potential NPE"
am: b688dd18ab

Change-Id: If562fc6a921569ad9d3141cac72b6f395a30da9e

Merge "Split chooseUpstreamType() into two separate functions" am: 7a42d3ad83
am: f6d55d7e9e

Change-Id: Ibd6b1fae52ec31ae488484be727b5261820e6ecb

Merge "ConnectivityManager: fix potential NPE"

Merge "Slightly simplify mTryCell and requestUpstreamMobileConnection() usage" am: 63445824d6
am: 4650762557

Change-Id: Ie004da6fe1252ddec90204637ff17c3ed73f2923

Merge "Split chooseUpstreamType() into two separate functions"
am: 7a42d3ad83

Change-Id: I77649e11e74140ec985dcdadab226e3e73e67642

Merge "Slightly simplify mTryCell and requestUpstreamMobileConnection() usage"
am: 63445824d6

Change-Id: Ie921ee1a2f1dbdd338c9f0700b24808e1cb6435e

Merge "Split chooseUpstreamType() into two separate functions"

Merge "Slightly simplify mTryCell and requestUpstreamMobileConnection() usage"

Merge "Pass targetSdkVersion specifier for SELinux labeling" am: 4516240b47
am: f8cefed995

Change-Id: Ie71368e7b4571d209d8c3a824bbf1a8c738f56a8

Merge "Pass targetSdkVersion specifier for SELinux labeling"
am: 4516240b47

Change-Id: Ic61c4dc30f075d01a85d8b60cfe4ae1de773aa09

Merge "Pass targetSdkVersion specifier for SELinux labeling"

Merge "Fix incorrect values for VelocityTracker in SwipeDismissLayout." into cw-f-dev
am: c61cd1a635

Change-Id: I02c70d3be50c00927e78282c9f3635b7ed9ae197

Merge "Fix incorrect values for VelocityTracker in SwipeDismissLayout." into cw-f-dev

[CM] Remove non-functional timeout from request network

Until now the request network with timeout was non-functional
(see b/31402633). Any caller which was using the timeout getting
the request network without a timeout.

While the timeout version has now been implemented, a max
timeout value is being removed since the large number used
(100 minutes) had no rationale for existing.

Updating the code to reflect the functionality which was
actually used - without a timeout.

Bug: 31399536
Bug: 35354391
Test: unit tests and CTS of ConnectivityManager
Change-Id: I0ebf0917a4478036a5ae532dd1e0784479057eb8

Fix incorrect values for VelocityTracker in SwipeDismissLayout.

The offset used to adjust MotionEvents for swipe velocity tracking
was incorrect, and caused issues when touch points where close
together. Fixed the offset used, which resolved swiping issues.

Bug: 34673753
Change-Id: Ide6060b511510bcf299e3db778e6ffc6afda5e19

Merge "Use asm-5.2 prebuilt" am: 6b32770d6d
am: 6c1ac4dd80

Change-Id: Icc01bcd92c5b23198bb39ff382610c72b4bf1867

Merge "Use asm-5.2 prebuilt"
am: 6b32770d6d

Change-Id: I7dc856be1a10ea7a65c7b3f889b37d9afaadbd5c

Merge "Use asm-5.2 prebuilt"

Pass targetSdkVersion specifier for SELinux labeling

Motivation:
Provide the ability to phase in new security policies by
applying them to apps with a minimum targetSdkVersion

Test: Marlin builds and boots. Apps targeting targetSdkVersion<=25
run in untrusted_app_25 domain. Apps targeting the current development
build >=26 run in the untrusted_app domain with fewer permissions. No
new denials observed during testing.
Bug: 34115651
Change-Id: Ib9f6ded9bd2f426861a6d843861b4074084253b0

Merge "Import CallSite classes and BootstrapMethodError class from ojdk8u60" am: 3023906b1b
am: 49e0cad3e1

Change-Id: I16775763cb63e18d35765b807acb91fc57286b85

Merge "Import CallSite classes and BootstrapMethodError class from ojdk8u60"
am: 3023906b1b

Change-Id: If332034df3b7afac1f7fdb92ca40adfb420ba768

Merge "Import CallSite classes and BootstrapMethodError class from ojdk8u60"

Merge "[NotificationManagerService]: Fixes the thread-safe issue." am: d530d1bf18
am: f69a73793d

Change-Id: Ifcce0699f9434cf490f9b4b596ba7604208275c6

Merge "[NotificationManagerService]: Fixes the thread-safe issue."
am: d530d1bf18

Change-Id: I661207bb52d84b1928d7ff3ad262cfe89e71e60b

Merge "[NotificationManagerService]: Fixes the thread-safe issue."

Split chooseUpstreamType() into two separate functions

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - runtest frameworks-net passes
Bug: 32163131
Change-Id: If892864ff59a6f38340791597c2d9e5b4022bbfa

Slightly simplify mTryCell and requestUpstreamMobileConnection() usage

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - runtest frameworks-net passes
Bug: 32163131

Change-Id: I46fbd176a71e61552d7d92660c1debe6487a1edf

Merge "Verify duplicate upstream notifications are ignored" am: 3d57eb0bdc
am: 02e71f6245

Change-Id: I0e62e4f6f371aa43eae1a0e5c62787c4e84ddd9a

Merge "Verify duplicate upstream notifications are ignored"
am: 3d57eb0bdc

Change-Id: Iff84a1a12714047b20408f03e3bb2252a463b74b

Merge "Verify duplicate upstream notifications are ignored"

ConnectivityManager: fix potential NPE

ConnectivityManager static sCallbackHandler is referenced and directly
used in a way that is not ensuring its proper initialization.

This patch fixes this potential NPE by using getHandler() instead.

Also this patch changes sendRequestForNetwork's signature to only accept
the subtype CallbackHandler instead of Handler: without using
CallbackHandler the NetworkCallbacks are not triggered properly and
bookkeeping of sCallbacks does not happen. sendRequestForNetwork's
signature now makes this explicit.

This step prepares the addition of overloaded versions of
registerNetworkCallback and cie that takes custom Handlers.

Test: build, flashed, manually checked connectivity
Change-Id: I52e8a2cb5075e7aef7b35e30c9845cacba927d13

Verify duplicate upstream notifications are ignored

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - runtest .../tethering/TetherInterfaceStateMachineTest.java passes
Bug: 32163131

Change-Id: I3c097d766fe4c416a8e3f02640809fe1199c6ef2

Merge "Cleanup in the face of upstream error" am: ed6d75d5f3
am: 76e6f790db

Change-Id: I097318f3edc5257288d582a0f17744e1f984856c

Merge "Cleanup in the face of upstream error"
am: ed6d75d5f3

Change-Id: If8dbf882bd8c4a8d53ae026d8f83a00a52590830

Merge "Cleanup in the face of upstream error"

Cleanup in the face of upstream error

If either enableNat() or startInterfaceForwarding() fail, be sure
to cleanup any commands that might have succeeded.

Most of this change is a refactoring of cleanupUpstreamIface() into
two methods, one of which (cleanupUpstreamInterface()) is reused
in error handling scenarios.

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - runtest -x .../tethering/TetherInterfaceStateMachineTest.java passes
Bug: 32031803
Bug: 32163131

Change-Id: Ia4d56e03beeab1908d8b8c2202e94992f1aa58a4

Merge "Create createFromByteArray() for WifiSsid" am: fb19d8d7c1
am: 18b3a7f586

Change-Id: Iea1e8282f0a9d2b58d48ab67dd856fafce95daf0

Merge "Create createFromByteArray() for WifiSsid"
am: fb19d8d7c1

Change-Id: I8b88c2259950b81b452452a2b56fd0ceaf2b7180

Merge "Create createFromByteArray() for WifiSsid"

Merge "Add simActivationState to PhoneStateListener" am: 132842efc9
am: b206f5b382

Change-Id: I3feca80a72ee4a7ef3fe1583aa563451b91a68f4

Merge "Add simActivationState to PhoneStateListener"
am: 132842efc9

Change-Id: I0b35d7d8f5b6237ff55aaab0cab0b86d75699a7a

Merge "Add simActivationState to PhoneStateListener"

Merge "DO NOT MERGE : Revert "DO NOT MERGE. No direct Uri grants from system."" into nyc-mr1-dev am: 7556690cf5
am: c510e19d18

Change-Id: I532b675a10ae9e21f28f3aa38d0f2e85ac080faa

Merge "DO NOT MERGE : Revert "DO NOT MERGE. No direct Uri grants from system."" into nyc-mr1-dev
am: 7556690cf5

Change-Id: Ib0446301e56751f6a72403155c576516cd7e559c

Merge "Only modify window layout params when necessary." into cw-f-dev
am: 9fbd981f23

Change-Id: I36aa427a3023dcba2a29eccfc0b7aac43748e534

Merge "DO NOT MERGE : Revert "DO NOT MERGE. No direct Uri grants from system."" into nyc-mr1-dev

Merge "Only modify window layout params when necessary." into cw-f-dev

DO NOT MERGE - Check exported status in chooser
am: 7bae18232e

Change-Id: Id29e65bd455c704738618f1dacfcad53b8d0b773

Create createFromByteArray() for WifiSsid

Bug: 35318006
Test: compile, unit tests
Change-Id: I0138bd67c0dec37419897318401b18ee4445a111

DO NOT MERGE - Check exported status in chooser

Relevant change in master is ag/1844170

bug: 34147053
Change-Id: Ic9d9d65c66325904c95ab3966bdcc16e7668278b
(cherry picked from commit e3286c3145d54564e783333ac2a2bb89b9ffdc49)

DO NOT MERGE : Revert "DO NOT MERGE. No direct Uri grants from system."

Reason: Breaking setting the user avatar in Settings.
Bug: 35158271
This reverts commit b61338ecb307a2f595a8f78e290b5287a6503aaa.

Change-Id: Ia08c12be3da06b0d02e71635b46350c8ff50ec19

Only modify window layout params when necessary.

Cancelling swipe-to-dismiss will trigger a check to ensure the window
is reset to its original state. Ensure that the reset is actually
required before setting the new layout attributes.

Bug: 34816397
Change-Id: Idf26ce7c8b63dc44a76effefcb32eb8d8665f605

Use asm-5.2 prebuilt

Test: m -j checkbuild tests cts
Change-Id: I37d40cc2e0406679394012aacd49f7ca9e51e3f6

Merge "hotspot2: fix typo in the getter functions" am: d3146f3c3c
am: 5644c190a0

Change-Id: Idac3a70dc9c8a10e82076014078d7b5af3acac72

Merge "hotspot2: fix typo in the getter functions"
am: d3146f3c3c

Change-Id: I8fc4042c26f896d899c090dd7afbe16a44d431eb

Merge "hotspot2: fix typo in the getter functions"

Add tweaks to smooth out swipe to dismiss.
am: 5758a9a94c

Change-Id: I64b9e88a7efb164b0e0160f8b25f770fb986efd6

Merge "Add test coverage for cleanup bug to be fixed." am: 89e6bf0532
am: 0cca15ff88

Change-Id: Ib5d1a6d6616b99a9e4f7cf0286ebc0a11d62bff8

Merge "Add test coverage for cleanup bug to be fixed."
am: 89e6bf0532

Change-Id: I33347c8fc2bec5660b8e9c14a316f6d14a6cf8b1

Merge "Add test coverage for cleanup bug to be fixed."