Merge "IllegalArgumentException in MagnificationGestureHandler" am: 7aff46195e
am: f124c72600

Change-Id: I664ad0b3d09036bb2bbb0f0ce2e56d22ca042e7e

Merge "IllegalArgumentException in MagnificationGestureHandler"
am: 7aff46195e

Change-Id: I3822533c2d7d01c7d804cfd6ad914d36f5ef482f

Merge "IllegalArgumentException in MagnificationGestureHandler"

Merge "Fix errorprone build" am: 3abddeca75  -s ours
am: 069915cd1c  -s ours

Change-Id: Ifb99a846c309f83f5d31cc81a88ca339787cf68b

Merge "Fix errorprone build"
am: 3abddeca75  -s ours

Change-Id: I9eb51d858194869763c94260beeec05aac6578ee

Merge "Fix errorprone build"

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

Change-Id: Ifb8aae76ed66234e0e39d538d84e2567f9e26003

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

Fix errorprone build

Fixes:
frameworks/base/tests/net/java/com/android/server/connectivity/tethering/OffloadControllerTest.java:149: error: [JUnit4TestNotRun] Test method will not be run; please add @Test annotation
    public void testNoSettingsValueDefaultDisabledDoesNotStart() throws Exception {
                ^
    (see http://errorprone.info/bugpattern/JUnit4TestNotRun)
  Did you mean '@Test'?
frameworks/base/tests/net/java/com/android/server/connectivity/tethering/OffloadControllerTest.java:170: error: [JUnit4TestNotRun] Test method will not be run; please add @Test annotation
    public void testNoSettingsValueDefaultEnabledDoesStart() throws Exception {
                ^
    (see http://errorprone.info/bugpattern/JUnit4TestNotRun)
  Did you mean '@Test'?

Bug: 64489631
Test: m -j RUN_ERROR_PRONE=true javac-check
Merged-In: Ib32489d07778465134bca52c589baddbd78ab129
Change-Id: I41dfdf87529e532df385617fa05c7006a7a14c86

Merge "Fix QS user icon in guest mode" am: 7d4e9b81b4
am: 526cd8923f

Change-Id: I91836c0d28cefbe5965586609d6eb521c8c2158a

Merge "Fix QS user icon in guest mode"
am: 7d4e9b81b4

Change-Id: I144ab6fea2e9ece6f01004c03c85da72a8c069f2

Merge "Fix QS user icon in guest mode"

Merge "Adapt ICU 59 API update to Doclava changes." am: e048261656
am: 0fc0cc9dd8

Change-Id: I93b14c129711fad75fee3aa34f3a0fed04eb1182

Merge "Adapt ICU 59 API update to Doclava changes."
am: e048261656

Change-Id: Iea671a7029e083bffee2bcaacfaa0df41f45c1cc

Merge "Adapt ICU 59 API update to Doclava changes."

Fix QS user icon in guest mode

When the profile is updated in guest mode, the user icon in QS footer
is not displayed correctly. To fix this issue, the tinting of
guest icon is applied only when the drawable is not UserIconDrawable.

Fixes: 65148947
Test: manual - switch to guest and update the profile.

Change-Id: I9690e9fd286703a25ee08d877c3dd7f4729e2b31

Adapt ICU 59 API update to Doclava changes.

Add methods that were missing from the previous API update because the
update-api version used for that had different behaviour from the
current one.

Specifically overrides for methods that differ only in sychronization
are now considered API-worthy and weren't before (b/62576297).

All methods added in this commit are methods that exist in the base
class without synchronization and are synchronized in the class they are
now being added in.

Bug: 64807220
Test: m checkapi
Change-Id: I9bf1b3d02e6a8f04b93ec36c0adffb7eedad3ad3
Merged-In: Iff584fb72dc8d651f9482b573f4b8cec89840ca6

Merge "Fix GATT client leakage when scan is throttled (1/2)" am: 3cc1ad87f0
am: d019cce887

Change-Id: I24d4dbf1344b13c558aa6974dfee6fd0e8549056

Merge "Fix GATT client leakage when scan is throttled (1/2)"
am: 3cc1ad87f0

Change-Id: Ieda555f6f8bda35201327c6c6e0c97621b88671e

Merge "Fix GATT client leakage when scan is throttled (1/2)"

Merge "Nat464Xlat: interface notification handler on ConnectivityService" am: 91817b3129
am: 02f6cfa49e

Change-Id: Ib224392c9a185f6bd79fd60cd5cb5549f2a7851e

Merge "Nat464Xlat: interface notification handler on ConnectivityService"
am: 91817b3129

Change-Id: I2f002cd197e2eeaaadadd747a6b33d264cd34433

Merge "Nat464Xlat: interface notification handler on ConnectivityService"

Merge "Use guava 21.0" am: edb2e0c722
am: f1c99fb47f

Change-Id: I8349115cf2b7678ac6e24ea2aa06ed59bffa5930

Merge "Use guava 21.0"
am: edb2e0c722

Change-Id: Icf8873b268677a631659a7ee0f7816495f6acd38

Nat464Xlat: interface notification handler on ConnectivityService

This patch adds a layer of asynchonicity to the NetworkBaseObserver
callbacks implemented by Nat464Xlat in order to allow these callbacks
to run on the main ConnectivityService handler.

This allows to run interfaceLinkStateChanged and interfaceRemoved
callbacks in the same thread context as other Nat464Xlat methods and
solves the following issues:
  - NPE risk due to race between fixupLinkProperties called on the
    ConnectivityService thread and interfaceRemoved called as a
    callback by NetworkManagementService.
  - stale LinkProperties reads in both callbacks not called on
    ConnectivityService handler.
  - removes the race between stop() and interfaceRemoved().

This patch also:
  - removes/simplifies comments related to the threading
    model which are no obsolete.
  - extract clatd management logic from ConnectivityService into
    NetworkAgentInfo

Bug: 62997041
Bug: 64571917
Test:  runtest frameworks-net
       manually connected to ipv6 network and went to test-ipv6.com
Change-Id: I889d98e47423ff3d4746d6ed8015b265286e7c52

Merge "Use guava 21.0"

Merge "Nat464Xlat: internal state guards cleanup + state enum" am: aee703e7e4
am: 20d4df8cc1

Change-Id: Ic2246a97618c596dbdbf48cda39c2f5b66e3bfb6

Merge "Nat464Xlat: internal state guards cleanup + state enum"
am: aee703e7e4

Change-Id: I188ac4c367db11cb33b67fe92df3a120e3c6fbce

Merge "Fixed mimetype error of PPT file" am: 3c28e2b280
am: 148414cd2c

Change-Id: I73f527fd006a39f00dafbe0b8c85cd583cf283f7

Merge "Nat464Xlat: internal state guards cleanup + state enum"

Merge "Fixed mimetype error of PPT file"
am: 3c28e2b280

Change-Id: I24b9dcf6506c53e26b901f74442f64bb28198611

Merge "Fixed mimetype error of PPT file"

Import translations. DO NOT MERGE
am: e0fec63824  -s ours

Change-Id: Iac255ef2407c7e3ff393fb01124d9a052bcde728

Fixed mimetype error of PPT file

Change mimetype of PPT file to match MS specification.
(MimeUtils.java matched the MS specification)

Test: Check mimetype of DB after saving PPT file.

Change-Id: Ie559499ae39bc527fcc656ba37f28845af209e6c
Signed-off-by: bokyung.kim <bokyung.kim@lge.com>

Merge "Ensure the generation is not changed before put value to cache" am: 11ae580f81
am: 489f2c2c48

Change-Id: Idab1a2d1f4ee8cc28a38c3c82ea9a8ad7f9d3bf4

Merge "Ensure the generation is not changed before put value to cache"
am: 11ae580f81

Change-Id: I5bf2c557f6f46f1455204761a9045b8b90d7ea32

Merge "Ensure the generation is not changed before put value to cache"

Fix GATT client leakage when scan is throttled (1/2)

Currently, scan throttling happens after client is registered, but
before the scan is started. This might lead to scan client being leaked.
This patch fixed that by moving check before client registration.

Bug: 64887233
Test: manual
Change-Id: I22ae624a0c51110cb69679f796926e3b2b36d0ac

IllegalArgumentException in MagnificationGestureHandler

When enabling the magnifier feature it is possible to trigger
a system crash.

Problem:
On ACTION_DOWN a MESSAGE_ON_ACTION_TAP_AND_HOLD is sent to the
handler with a delay based on long-press timeout.  If the user
then removes their finger while outside of the magnification area,
(ACTION_UP) this message is not removed.  Then when the message
is handled on long-press timeout, it puts the state into
STATE_VIEWPORT_DRAGGING.  If the user then taps the screen again
it is handled by the StateViewportDraggingHandler resulting in:

 IllegalArgumentException: Unexpected event type: ACTION_DOWN

and a system server crash.

Bug: 65101895
Test: Manual:
1. Enable "Magnify with button" from accessibility settings
2. Tap accessibility button on the right navbar
3. Touch & hold bottom part of application area
4. Quickly drag down to the navbar and release
5. Touch anywhere
With this patch it should not crash

Change-Id: I7df8d2185f6db9d148d24d2b155cb357b3637a71

Merge "Trivial change to test expectations." am: c0d8300b32
am: 3dd0137ee5

Change-Id: I40e6535b22688b47845e6c57d3f13963bc8a6201

Merge "Trivial change to test expectations."
am: c0d8300b32

Change-Id: I58155e3edb424ea6918a3d2bb8e284e61346765f

Merge "Trivial change to test expectations."

Merge "Nat464Xlat: clat management cleanup" am: 81b179cfdd
am: b6805dc02f

Change-Id: I1a19e6fbb0cb13262e90b171d861062469078fb6

Merge "Nat464Xlat: clat management cleanup"
am: 81b179cfdd

Change-Id: Ic3808a1afe48efac745b1b378fb12cc5678918ec

Merge "Nat464Xlat: clat management cleanup"

Trivial change to test expectations.

(cherry picked from commit 0fee732ca579bbc4320ade35f238bdfa1aa500f5)

Bug: 32163131
Test: runtest frameworks-net
Change-Id: Iec8af5191b4da495e47ef97d70a03a5d2e75cfe2

Merge "switch to native_handle_create() for passing ownership" am: 5fe8e51904
am: 639808f1c7

Change-Id: I3bd8f99cc3c1a6bc435721c7dc18e061f61fc175

Merge "switch to native_handle_create() for passing ownership"
am: 5fe8e51904

Change-Id: I101de0e7e76f61bd3f05183f4a13ac94c79ae1aa

Merge "switch to native_handle_create() for passing ownership"

Nat464Xlat: internal state guards cleanup + state enum

This patch does some cleanup of Nat464Xlat internal state guards
against the Nat464Xlat state Idle | Started | Running, which reduces
code nesting.

It also replaces introspection of internal state for distinguishing
between different stages in 464xlat lifecycle with an enum explicitly
introducing these three Idle | Started | Running states.

Bug: 62997041
Bug: 64571917
Test:  runtest frameworks-net
       manually connected to ipv6 network and went to test-ipv6.com
Change-Id: I6efc9fed2420ca488731a2b9b9c3c025b16eca10

Nat464Xlat: clat management cleanup

This patch does some minor refactoring of clat starting/stopping code:
 - remove unused LinkProperties arguments in updateClat
 - remove unused Context argument in Nat464Xlat ctor
 - introduce ensureClatIsStarted and ensureClatIsStopped methods and
   simplify updateClat
 - add clatd to NetworkAgentInfo toString() method
 - clarify some comments

This changes prepare for moving BaseNetworkObserver callbacks to
ConnectivityService.

Bug: 62997041
Bug: 64571917
Test: runtest frameworks-net
      manually connected to IPv6 only network and went to test-ipv6.com
Change-Id: Idb204784614cfe700f73255a7a7b78c5e9ee6eca

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Ida2b6138972c18c15b86e7822d010c4a675ab762

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Ia33326f08f0440ae65f9820b071b32a762c83965

Merge "DeviceInfoUtils.java: support to show kernel version compiled with clang" am: b5b1308f2f
am: c54162d3ff

Change-Id: Ie9225108f7ac354212a186d2e26d1f278d9fb7e8

Merge "DeviceInfoUtils.java: support to show kernel version compiled with clang"
am: b5b1308f2f

Change-Id: I05f37f0ca0a18bd291aea8d49e2747b40d30b572

Merge "DeviceInfoUtils.java: support to show kernel version compiled with clang"

Merge "Clarified that getAutofillValue() must return same value passed to autofill()." into oc-dev am: 7d29b6f2df
am: da31220015

Change-Id: I2a928f1e1c0e7fba03c1ebc8c61a182ecebb32c0

Merge "AOD: Only play wakeup animation when actually waking up" into oc-dr1-dev
am: feb69bf5a7

Change-Id: I14d244bc7cc413ff204d37dee6d621af9e0e8e29

Merge "Clarified that getAutofillValue() must return same value passed to autofill()." into oc-dev
am: 7d29b6f2df

Change-Id: Ibfabc9d32aa0da80e636a27eebf1b50bdd9692b3

Merge "Clarified that getAutofillValue() must return same value passed to autofill()." into oc-dev

Merge "AOD: Only play wakeup animation when actually waking up" into oc-dr1-dev

Merge "Emergency redial implementation" am: b3b09849bc
am: f5a457244c

Change-Id: Ied2aa7255546df647b01884b6c1689c6192e1d67

Merge "Emergency redial implementation"
am: b3b09849bc

Change-Id: If7fe179da4eb4c639dad1e2957f091bbee9e54f9

Merge "Emergency redial implementation"

Merge "Minor Autofill javadoc fixes." into oc-dev am: 560221752e
am: 948e9ff4d3

Change-Id: I90a42739dfa01c49fdbf799c49e3d569abcedf11

Merge "Minor Autofill javadoc fixes." into oc-dev
am: 560221752e

Change-Id: Ib2924dae99bfe132a56a1583ecf4a88f833d3937

Merge "Minor Autofill javadoc fixes." into oc-dev

Merge "Add new error code in SmsManager" am: 387ed22fef
am: 782395561d

Change-Id: I12924c377167e11889942f2bfba555a07b45d9ac

Use guava 21.0

guava 20.0 is being removed.

Test: m -j checkbuild
Change-Id: I48ac4f641da3567985339c061dad5d60829aeee0

Merge "Add new error code in SmsManager"
am: 387ed22fef

Change-Id: Iff96995b90ecd6ae7fcceebf6b2017344038be72

Merge "Add new error code in SmsManager"

Emergency redial implementation

Define connection event to notify
Telecom/InCallUi about change in
account handle after redial and extra
for emergency phone handle.

Bug: 27059146
Change-Id: Ie72ab2901ec05d972204ed11f115a05b79173c1d

Merge "Switch /data/misc/reboot/last_reboot_reason to persistent property" am: 0172ce8dbf
am: cf5fa80fbf

Change-Id: Ica26cd9240a977d9140c72f5606c8b9d874001d9

Merge "Switch /data/misc/reboot/last_reboot_reason to persistent property"
am: 0172ce8dbf

Change-Id: I69a9c17fc8594dee564e8c64e9d85f2afe60607b

Merge "Switch /data/misc/reboot/last_reboot_reason to persistent property"

switch to native_handle_create() for passing ownership

Test: builds
Bug: 29337859
Bug: 32163131
Bug: 64976634
Change-Id: Ifd064736c9fcb633b72e56e93b7baa5f4a5ff051

Ensure the generation is not changed before put value to cache

It may get old value from mValues, so check the generation
is not changed before update cache

Bug: https://issuetracker.google.com/issues/64995655
Test: Build
Change-Id: Ia7ae18baa269d0590e36f186e2f14b0bfbab3504
Signed-off-by: xulicheng <xulicheng@xiaomi.com>

Merge "Track conference start in system elapsed time." am: 3b166fba8a
am: 49042dc67c

Change-Id: I90aa59e2889973dd1bd1d443b7bf5a11cbf46695

Merge "Track conference start in system elapsed time."
am: 3b166fba8a

Change-Id: I8b78aaa447adcc1bfce0fec8f50857daa37d0575

Merge "Track conference start in system elapsed time."

Merge "Add new telephony disconnect causes for low battery scenario." am: 573948c49c
am: 3da596d291

Change-Id: I9c9f4898b74d7f0a4cbef0a8f85d94deb683b9dc

Merge changes from topic "bt-fix-checkstyle-errors" am: 838ab94a0f
am: 8ae5462199

Change-Id: I44636b8f052b6e48f562f53869dcdfeffbcc0fc9

Merge "Add new telephony disconnect causes for low battery scenario."
am: 573948c49c

Change-Id: I73255cdc492992b03633831a2d9240b933f21064

Merge changes from topic "bt-fix-checkstyle-errors"
am: 838ab94a0f

Change-Id: I4e619cb17582a6cc954814654820cd945f66e1cc

Add new error code in SmsManager

Design doc: https://docs.google.com/a/google.com/document/d
/1YPTNoM2Atjz8oISGAsRYlFVSA1NcYP8MbqYQIzLcJ9E/edit?usp=sharing

Test: compile/flash code; test locally
Bug: 34773406
Change-Id: I0b227e76048dd46d9c8023d37647db763e1e38e1
Merged-In: I0b227e76048dd46d9c8023d37647db763e1e38e1

Merge "Add new telephony disconnect causes for low battery scenario."

Merge changes from topic "bt-fix-checkstyle-errors"

* changes:
  Fix checkstyle errors (2/2)
  Fix checkstyle errors (1/2)

Add new telephony disconnect causes for low battery scenario.

LOW_BATTERY is for disconnect mid-call due to low battery.
DIAL_LOW_BATTERY is for failure while dialing due to low battery.

Test: Manual.
Merged-In: Ife7a4ca5645b3fb42ab20effb4d8d4a8fef8116b
Change-Id: Ife7a4ca5645b3fb42ab20effb4d8d4a8fef8116b
Fixes: 63936576

Fix checkstyle errors (2/2)

* Manual style corrections with IDE assistance
* Variable name refactors are done through IDE
* Corrected general style errors such as:
  - "final private var" -> "private final var"
  - "&&", "+", "||" should not be at the end of line
  - Non-static private variable should be like "mVar"
  - Private static variable should be like "sVar"
  - Code file should always end with newline
  - Inherited methods should be annotated with @Override
    and no @hide tags
  - Public methods should always have a JavaDoc entry
  - "int[] array" is preferred over "int array[]"
  - private methods should be accessed without "this."
    when there is no name collisions.
  - "boolean ? true : false" -> boolean
  - "boolean ? false : true" -> !boolean
  - "boolean == true" OR "boolean != false" -> boolean
  - "boolean != true" OR "boolean == false" -> !boolean

Bug: 63596319
Test: make checkbuild, no functional changes
Change-Id: Iabdc2be912a32dd63a53213d175cf1bfef268ccd

Fix checkstyle errors (1/2)

* Automatic style corrections through IDE

Bug: 63596319
Test: make checkbuild, no manual changes, no functional changes
Change-Id: I2397d55abc34c9b7a9b748bec6137778df3421a7

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

Change-Id: I45956469aa428467ffb7039288cd99a177566083

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

DeviceInfoUtils.java: support to show kernel version compiled with clang

The kernel version displayed in settings is only shown correctly
when built with gcc, this change adds support for a kernel built
with clang.

Test: manually on hikey with following command:
adb shell am instrument -w -e class com.android.settings.DeviceInfoSettingsTest com.android.settings.tests.unit/android.support.test.runner.AndroidJUnitRunner

Change-Id: I9b7991681cb5abdf44e7d1d1bff84c7c866be2ed
Signed-off-by: Yongqin Liu <yongqin.liu@linaro.org>

Import translations. DO NOT MERGE

Auto-generated-cl: translation import
Exempt-From-Owner-Approval: translation import

Bug: 64712476
Change-Id: Ied6de9cbe69d8c72f9ba8c0095451a175f02d4dd

Import translations. DO NOT MERGE
am: b4f16b10af  -s ours

Change-Id: I0b3cf2e8204f78f342754b06988bd63344e72fa7

Merge changes from topic "offload_cherrypick" am: b09cf3fbf1
am: df2a880775

Change-Id: I3db23fef19fc66a4eee158fb3367f45078b30872

Merge changes from topic "offload_cherrypick"
am: b09cf3fbf1

Change-Id: Icb4332b34bc8b940a2c30beab143f7136e0178de

Merge changes from topic "offload_cherrypick"

* changes:
  Send add/removeDownstream info to offload HAL
  Don't completely stop offload if setting data limit fails.

Merge "Make NetworkManagementServiceTest pass again." am: 29b1826ff2
am: d3d6ddea94

Change-Id: I3bd2e7242a8100590311cca40369adf45b06e52f