Merge "Relax strict ActionMode instance checking when finishing action modes" into mnc-dev

Relax strict ActionMode instance checking when finishing action modes

In previous platform versions, finishing an action mode would clean up
the current action mode even if it was not the same ActionMode
instance. Some common shared code inadvertently relied on this
behavior, so stay bug-compatible with it based on targetSdkVersion.
New apps will get the stricter behavior.

Bug 22265882

Change-Id: Id5d6341aefc07a3cb788d5d6d0b531816f761e42

Merge "Bump mms whitelist time up to 1 minute." into mnc-dev

Bump mms whitelist time up to 1 minute.

Change-Id: I09074c440430580fcfb2581275defe61c63c8ae8

Merge "Fix issue #21626564: MMS should be receivied while Dozing" into mnc-dev

Merge "Require ACCESS_COARSE_LOCATION for ACTION_FOUND broadcast" into mnc-dev

Fix issue #21626564: MMS should be receivied while Dozing

We now place whoever is receiving the MMS on the temporary
whitelist while doing so, so they can get network access to
download it.

There was also an issue that needed to be fixed where we
were no longer updating the list of allowed uids while
dozing based on their proc states...  we now do that.

Also did a bit of optimization of the temp white list update
path do the network policy manager, instead of going through
a broadcast we now directly call in to the network policy
manager.  This also allows us to have a synchronous version
of updating the list, so we can know the app has network access
before we tell it to do anything.

Finally added battery stats events for things going on and off
the whitelist so we can diagnose the behavior there.

Change-Id: Ic7fe010af680034d9f8cb014bb135b2addef7455

Merge "AudioManager: fix audio device callback list access" into mnc-dev

Merge "Added sendBroadcastMultiplePermissions method" into mnc-dev

Merge "BatteryStats: Clear power use when removing a UID from uid_cputime kernel module" into mnc-dev

Added sendBroadcastMultiplePermissions method

Added Context.sendBroadcastMultiplePermissions(Intent intent, String[]
receiverPermissions) method, which allows an array of required permissions
to be enforced.

Bug: 21852542
Change-Id: I27c9130e8f004b428452501ebc8a36aabde1f343

BatteryStats: Fix deadlock

Bug:22486756
Change-Id: I20e10cc344b128bfa156c5aada93d3fb2b0fcaa3

BatteryStats: Clear power use when removing a UID from uid_cputime kernel module

Bug:22461683
Change-Id: Iffb847268d2872bd2a843e28dc41fa22b1e36828

AudioManager: fix audio device callback list access

Bug: 22470616.
Change-Id: Ic59e8499557395dabe789b5fec9cb7c86c13d10c

Merge "BatteryStats: Fix units in wakelock distribution calculation" into mnc-dev

Merge "Camera: clarify largest JPEG dimension expectation" into mnc-dev

Merge "Revert "Allow array of required permissions in sendBroadcast"" into mnc-dev

Merge "Need to disable 5GHz bands for WiFi tethering in Japan" into mnc-dev

Revert "Allow array of required permissions in sendBroadcast"

This reverts commit b4e7283c9afd9fb15ebd63f6ce9b75c9c1af658b.

Change-Id: Ie8390964bda5bdfa869cee8f46584043d8e7c664

Merge "Standardize stateToString return values" into mnc-dev

Merge "Allow array of required permissions in sendBroadcast" into mnc-dev

Merge "Better handling of storage paths." into mnc-dev

Merge "Camera: Reconnect after camera service crashes" into mnc-dev

BatteryStats: Fix units in wakelock distribution calculation

This is mostly for checkin stats, this is not used in any computation.

Bug:22468534
Bug:22391540
Change-Id: Ib3d0c86db5044092d3d76acf8f001337272af2ff

Merge "Pass wifi ap state through to tile when receiver triggered." into mnc-dev

am e67416ec: am 4859a612: Merge "docs: Clarified how to find out when a switch is toggled" into mnc-preview-docs

* commit 'e67416eccc6092e2fce3619b2244deda36fabd26':
  docs: Clarified how to find out when a switch is toggled

Merge "Fix status bar bt accessibility message" into mnc-dev

Merge "Move wifi accessibility strings to SettingsLib" into mnc-dev

Fix status bar bt accessibility message

Use the same description for both QS and status bar.

Bug: 20345203
Change-Id: I5789c1385d2240e1ad426a008e6e2192885ddc2b

Merge "BatteryStats: Remove uid_cputime Slog.wtf for high cpu times" into mnc-dev

Better handling of storage paths.

Give more details about why we failed to create storage paths, and
search for underlying volumes using canonical paths.

Bug: 22135060
Change-Id: I75d3584403ece310438b05f5b9fe72d94c9096c6

am 4859a612: Merge "docs: Clarified how to find out when a switch is toggled" into mnc-preview-docs

* commit '4859a6125bcf967cf97339b19f30d9788b971e02':
  docs: Clarified how to find out when a switch is toggled

Merge "docs: Clarified how to find out when a switch is toggled" into mnc-preview-docs

Allow array of required permissions in sendBroadcast

Added Context.sendBroadcast(Intent intent, String[] receiverPermissions)
method, which allows an array of required permissions to be enforced.

Bug: 21852542
Change-Id: I3b8ff258fa9f3249c344bb8093b820b24eef00c0

Merge "Clear binder id when determining external storage state" into mnc-dev

Merge "Allow notification strings to be unredacted in dump output." into mnc-dev

Merge "Make get_accounts a runtime permissions again" into mnc-dev

Standardize stateToString return values

Bug: 20303674
Change-Id: I884029614c8ce36120c1fc039ef33c0e6736ed5c

Require ACCESS_COARSE_LOCATION for ACTION_FOUND broadcast

Receivers of ACTION_FOUND intent are now required to have
ACCESS_COARSE_LOCATION permission.

Bug: 21852542
Change-Id: I8306f7431f067ca36bfc568a912385153fa3d496

BatteryStats: Remove uid_cputime Slog.wtf for high cpu times

High cpu times are expected as multiple cores can be running at the
same time, so comparing against the time between samples is incorrect.

I am reasonable certain that the values we see now are correct, so disabling this
check. However, checking for negative values (overflows) is still enabled and
will remain enabled because there is no case where we will be ok with negative deltas.

Bug:22461683
Change-Id: If9c7cdbb75ceaed059d1e0f4dd83cfdd3e021a93

Clear binder id when determining external storage state

bug:22472435

Change-Id: I5badd51cb9fb2244395880eb1bbff8f4aa0abb8c

Merge "Removed logging for the empty keyguard" into mnc-dev

Merge "Add checks for READ_PRIVILEGED_PHONE_STATE." into mnc-dev

Pass wifi ap state through to tile when receiver triggered.

Bug: 22450078
Change-Id: I9b339aa2238a7427291bfdc2ef6aab58dcdb5cf3

Merge "Fix QS labels on language change" into mnc-dev

Move wifi accessibility strings to SettingsLib

Bug: 22468835
Change-Id: I66e85e8adfb3eda194995eda529422a172f00cbd

Merge "Use original path if translated path doesn't exist" into mnc-dev

Add checks for READ_PRIVILEGED_PHONE_STATE.

Callers will have either PRIVILEGED or regular READ_PHONE_STATE.

Bug: 22468536
Change-Id: I60251f61250a975f4147b027d2539824a728e3b5

Merge "Prevent null allowableAccounts from crashing" into mnc-dev

Merge "Fix unsafety in SkiaShader storage, and texture unit accounting" into mnc-dev

Fix QS labels on language change

Some tiles now won't do updates if the state hasn't changed (rotation) to avoid
breaking the animation state. To handle this, state is cleared when the
configuration changes.

Bug: 22206214
Change-Id: I029fb4c75cc920a6c9907b65e8c48d28c21d529a

Merge "Fix Android Keystore key factories to obey JCA contract." into mnc-dev

Merge "Allow FloatingToolbar to be outside of the attached window." into mnc-dev

Use original path if translated path doesn't exist

Bug: 22376538
Change-Id: I2f6af4fa31c097e2e2d619a068cf83721091c450

Merge "media: Implement getAchievableFrameRatesFor with measured frame rate." into mnc-dev

Prevent null allowableAccounts from crashing

The javadoc for newChooseAccountIntent says that a null
value for the allowableAccounts parameter is valid and
an acceptable default. This CL makes sure that when this
parameter is null, a NullPointerException is not thrown.

Bug: 22475546
Change-Id: Ieb0d67dd02628e1ae5629499b3be3c6382efc9aa

Merge "Fix "Select All" in extracted mode" into mnc-dev

Merge "Retrieve resources for apps, even if not installed for current user." into mnc-dev

Retrieve resources for apps, even if not installed for current user.

Bug: 21448052
Change-Id: Id5db428a6faca4a905684088c47e070752ac2899

Allow FloatingToolbar to be outside of the attached window.

Currently PopupWindow used for the floating toolbar specifies
neither FLAG_LAYOUT_NO_LIMITS nor FLAG_LAYOUT_IN_SCREEN.
As a result, the floating toolbar can overlap the selected
text when the attached window does not have enough height.

Here is the repro code.

    final TextView textView = new TextView(this);
    textView.setLayoutParams(
            new ViewGroup.LayoutParams(
                    ViewGroup.LayoutParams.MATCH_PARENT,
                    ViewGroup.LayoutParams.WRAP_CONTENT));
    textView.setText("A test sentence.");
    textView.setTextIsSelectable(true);
    final AlertDialog dialog = new AlertDialog.Builder(this)
            .setView(textView)
            .create();
    dialog.getWindow().setGravity(Gravity.BOTTOM)
    dialog.show();

If you tap a word in the dialog, the floating toolbar
unintentionally overlaps the selected text due to the limited
height of the AlertDialog.

It also turns out that just calling
  PopupWindow.setClippingEnabled(false)
to specify FLAG_LAYOUT_NO_LIMITS is not sufficient and ends up
showing the toolbar on the NavBar because we have mistakenly
compared bounds in window-local coordinates
(e.g. FloatingActionModemContentRectOnWindow) with bounds in
screen coordinates (e.g. FloatingActionMode#mScreenRect).
Hence the confusion of window-local coordinates and screen
coordinates in FloatingToolbar and FloatingToolbar also needs
to be addresses.

To summarize here are the notable changes in this CL:
  - Specify FLAG_LAYOUT_NO_LIMITS so that the floating
    toolbar can be placed outside of the attached window.
    (We do this with PopupWindow#setClippingEnabled)
  - Switch to use screen coordinates from window-local
    coordiantes in FloatingToolbar and FloatingActionMode
    because some system components such as WindowManager
    prefer screen coordinates.
  - Put -OnScreen suffix for Rect and Point variables
    as long as they are in screen coordinates.

Bug: 22335001
Change-Id: I71a8d356e868dc7715b030ca1078da4ec39368c3

Fix "Select All" in extracted mode

Stop calling onExtractTextContextMenuItem if "Select All"
is selected (the action does not modify text thus does
not need batch editing). Editor#finishBatchEdit reports
that text changed which in turn can stop the action mode
and the selection after it was started by onPreDraw.

Bug: 22059417
Change-Id: I5354cbe4bae374e0d5f3de39616336170ee33b92

Merge "Fix insertion handle disappearing in extract mode" into mnc-dev

Merge "Log NetworkRequestInfo to get more detailed history" into mnc-dev

Fix BluetoothSocket file descriptor leak

Sockets accepted on a server socket didn't populate
the mPfd field, which is used to close out the java
end of the native-and-java communication socket when
the overall rfcomm socket is closed. #badnewsbears

b/21398841

Change-Id: I3adb0a9965f83d0f3006fa4f79ea4abeab5c9a17

Log NetworkRequestInfo to get more detailed history

Bug: 21641983
Bug: 22235775
Bug: 22299819
Bug: 22358164
Change-Id: I48a2d162dd499a3590f06d805267a63822856969

Fix build

Change-Id: I2c97e405eaec7ee1569fcddc872ff2a98c829efe

Make get_accounts a runtime permissions again

The permission was lowered in protection level to normal
allowing apps to assess migration strategy. This change
makes the permission dangerous again allowing apps to
implement this strategy.

bug:22385299

Change-Id: I3694c06236a33b97f63761b48745b1df654a2bf4

Merge "Teach storage appops." into mnc-dev

Teach storage appops.

For modern apps targeting M SDK and up the external storage state
is deterined by granted permissions. For apps targeting older SDK
the storage access is determined by app ops correspning to the
storage permissions as the latter are always granted.

When app ops change we do not remount as we kill the app process
in both cases enabling and disabling an app op since legacy code
is not prepared for dynamic behavior where an operation that failed
may next succeed. Hence, we remount when we start the app.

For modern apps we don't kill the app process on a permission
grant, therefore we synchronously remount the app storage.

bug:22104923

Change-Id: I601c19c764a74c2d15bea6630d0f5fdc52bf6a5a

Merge "Default browser should not supersede intent filter priorities" into mnc-dev

Default browser should not supersede intent filter priorities

Bug 22304850

Change-Id: Ie4deffea2621fdc03a9bfe35690ecd109eed1a04

Merge "Check signatures of privileged persistent apps for granting defailt  permissions" into mnc-dev

Merge "Prioritize most-recently-enabled link-handling app" into mnc-dev

Merge "Fix two grant default permissions edge cases." into mnc-dev

Fix two grant default permissions edge cases.

1. When querying for components to which to grant default
   permissions we are now getting disabled components as
   well as we want such packages to get default pemrissions
   on upgrade as disabled components may get reenabled.

2. When resolving activities to whose packages to grant default
   permissions we skip the corss-profile and skip-profile filters
   as they are set by device policy which should not affect
   implicitly default permission grants. Policy has explicit
   APIs for permission management.

bug:22405963

Change-Id: Idf56d35e567a6fd6e097cd45988d1561422ee6fc

Camera: Reconnect after camera service crashes

Schedule a reconnection after camera service crashes if any client
registered a camera availability callback or torch status callback.

Bug: 407755
Change-Id: Iabe0bf713863898ca468cff59d4c97c66a802630

Need to disable 5GHz bands for WiFi tethering in Japan

b/22396383

Change-Id: I61efef1babaab756425e6958843dfb9ad47b46cd

am 70d39c28: am 2b856100: Merge "docs: Removed references to lockscreen widgets, added note to Android 5.0 changes page" into mnc-preview-docs

* commit '70d39c2836f3c05a93035439df0082b5c471c010':
  docs: Removed references to lockscreen widgets, added note to Android 5.0 changes page

am 2b856100: Merge "docs: Removed references to lockscreen widgets, added note to Android 5.0 changes page" into mnc-preview-docs

* commit '2b856100f67da1505eac30e2d989b84a2d68f20a':
  docs: Removed references to lockscreen widgets, added note to Android 5.0 changes page

Merge "Add a dumpsys for KeyguardUpdateMonitor" into mnc-dev

Prioritize most-recently-enabled link-handling app

In the case when multiple apps handle a given web-link action,
all of which have been marked as "launch the app instead of a
browser" and so are otherwise ambiguous, always prefer the app
that was most recently placed into the always-handle-links state.

Bug 22051035

Change-Id: I3f43c19b0d7b74e9843445e41971bb5433affb1c

am 28006a3b: am 7182f075: Merge "docs: Changed text as requested in Android TV advertising bug, with minor cleanup" into mnc-preview-docs

* commit '28006a3b31d1cd3765568ceed211f8bc8de9c50c':
  docs: Changed text as requested in Android TV advertising bug, with minor cleanup

am 1962b730: am 3430bec7: Merge "docs: Removed parenthetic phrase that made explanation unnecessarily confusing" into mnc-preview-docs

* commit '1962b730067d5b9ad9666ee3a7d1e3f90004c7d7':
  docs: Removed parenthetic phrase that made explanation unnecessarily confusing

am 5c3bf584: am e0f0fabb: am ccb8a9af: am b510225e: Merge "docs: replacing command line tool instructions with Studio and Monitor steps" into lmp-docs

* commit '5c3bf5847c98b23a525288ed04f77b2a68ec6870':
  docs: replacing command line tool instructions with Studio and Monitor steps

am f13e914d: am 8cce3984: Merge "Adding a link to what is currently our most popular "100 Days" video." into mnc-preview-docs

* commit 'f13e914d23a47809d978183a8e6b6ab3a5d51098':
  Adding a link to what is currently our most popular "100 Days" video.

Merge "Perform One touch play upon pressing home key" into mnc-dev

Merge "docs: Removed references to lockscreen widgets, added note to Android 5.0 changes page" into mnc-preview-docs

Removed logging for the empty keyguard

Only logging the states in the bugreport now.

Bug: 21701756
Change-Id: I8252583300d0162b878dd217c8098c1d6b970ea4

am 7182f075: Merge "docs: Changed text as requested in Android TV advertising bug, with minor cleanup" into mnc-preview-docs

* commit '7182f075fad7865eb8c1ce58d90327addabc7c4b':
  docs: Changed text as requested in Android TV advertising bug, with minor cleanup

am 3430bec7: Merge "docs: Removed parenthetic phrase that made explanation unnecessarily confusing" into mnc-preview-docs

* commit '3430bec7e0b2f7a0552a66a523e478cd2ecd9043':
  docs: Removed parenthetic phrase that made explanation unnecessarily confusing

docs: Removed references to lockscreen widgets, added note to Android 5.0
changes page

Removed one line from source code sample as well.

Bug: 20826638
Change-Id: I3c27353db0f0092c21962f9beec1bf1da52ee9d5

Merge "Catch & log security exception in telephonymanager" into mnc-dev

Merge "docs: Changed text as requested in Android TV advertising bug, with minor cleanup" into mnc-preview-docs

Merge "docs: Removed parenthetic phrase that made explanation unnecessarily confusing" into mnc-preview-docs

Fix Android Keystore key factories to obey JCA contract.

Android Keystore provider's KeyFactory and SecretKeyFactory
implementations were throwing UnsupportedOperationException instead of
InvalidKeyException/InvalidKeySpecException from their
translateKey/generateKey methods.

Bug: 22459811
Change-Id: I6d5a5dc1bed724e858ad324d558b7480b9b848da

Merge "Do not check runtime READ_PHONE_STATE if caller has the privileged one." into mnc-dev

Merge "Improve resource loading by ~3x" into mnc-dev

Improve resource loading by ~3x

Bug: 22392651

ColorStateLists were never cached because the lazy-create
of the constant state had a typo.

Resource caching in general was broken because ThemeKey did not
clone the hash code, so all keys in the cache had a hashCode
of 0 which did not match the real, uncloned ThemeKeys hash code
so the binary search in ArrayMap based off of hash code was failing.

Change-Id: I9df1628b226bfa797bed97875354c19bf64f41ad

Add a dumpsys for KeyguardUpdateMonitor

Include the info used when determining carrier text display.

Bug: 22212693
Change-Id: I12f01683c259ff11c7f27eec5d03d53f0b290d0b

Merge "Finish already paused activity if it should be finished after pausing" into mnc-dev