Merge "don't call other services in dump" into nyc-dev

Merge "Fix up label in QS detail" into nyc-dev

Merge "Fix Settings home lifecycle" into nyc-dev

Merge "Change drag label while dragging QS tiles" into nyc-dev

Merge "Add some synchronization to hotspot callbacks" into nyc-dev

Merge "Close overflow when hiding QS editor" into nyc-dev

Merge "Return DNG mimeType for the case kRAW_SkEncodedFormat" into nyc-dev

Return DNG mimeType for the case kRAW_SkEncodedFormat

Bug: 27587823
Change-Id: I0f539034a883083a30f33262edc715714f45e60d

Merge "Load WebView library into the right classloader namespace." into nyc-dev

Merge "Unifying method names and comments to security logging, not device logging" into nyc-dev

Merge changes Idf0cfee8,I27307e2e into nyc-dev

* changes:
  Use native netd binder interface in NetworkManagementService.
  Fix IdleableHandlerThread.

Merge "Change wording of WebView Implementation invalid-choice Toast." into nyc-dev

Merge "Fix forgot adding textEditSuggestionHighlightStyle to dark theme." into nyc-dev

Use native netd binder interface in NetworkManagementService.

Bug: 27239233
Change-Id: Idf0cfee82ec5007fde0237d944ab0c2b53ff4a7b

Fix forgot adding textEditSuggestionHighlightStyle to dark theme.

textEditSuggestionHighlightStyle was introduced
Ieccea12db95c0a040b38680ae794b1cf6971736f but it fogot adding it to
dark theme.

Bug: 27660380
Change-Id: I7b107352b71f273ee5d8d1be8d68c308741e3fbc

Merge "Add a performance test for launching DocumentsUI." into nyc-dev

Merge "Fix: setExtractedText partial span change handling." into nyc-dev

Merge "Fix: setIntegerPart(long) doesn't set ARG_INTEGER_PART." into nyc-dev

Update comment for compareToIgnoreCaseNullable().

Bug: 27286016
Change-Id: I0ff4d59e24e8afe129aa8a2921d3de71d3df7236

Add a performance test for launching DocumentsUI.

The test launches the DocumentsUI as picker, then waits until the
main thread idles, which guarantees that roots are loaded and UI
rendered.

It confirms, that the recent system cache improves cold start
performance by around 24% on my configuration (from 1685ms to 1357ms).

Bug: 27370274
Change-Id: I738202ea434a7bfe7080fc0994f636ef0e7847cd

Merge "Add suffix number when copying a file." into nyc-dev

Add suffix number when copying a file.

If we have an existing file in the destination directory, which has the
same name with the source file, adding suffix number is
DocumentsProvider's responsibility.

Because MTP does not provide a way to check existance of files with
given name, the logic is implemented as try-and error strategy. The CL
lets If we MtpDocumentsProvider assume we have a file that shares the
same name with the source file if it failed to invoke
MtpDevice#sendObjectInfo. In this case MtpDocumentsProvider retry to
invoke sendObjectInfo with new name with suffix number.

BUG=26991190

Change-Id: I223ac5031f079bc91eb27709b0356f621a1ed55b

Merge "Don't clear effects when the panel is fully collapsed." into nyc-dev

Merge "Oops, missed a default Bundle flag." into nyc-dev

Oops, missed a default Bundle flag.

Bug: 27706598
Change-Id: Idde1c25ea1b5944657075cd5f0da65396280f757

Merge changes I068f8821,Ibcf12746,I169ad495 into nyc-dev

* changes:
  Add more @NonNull/@Nullable to TextServicesSettings.
  Remove an unnecessary int to String conversion.
  Add more @NonNull/@Nullable to InputMethodSettings.

Fix IdleableHandlerThread.

Occasionally, ConnectivityServiceTest fails with "BUG: only one
idle handler allowed". I have not been able to reproduce this
consistently, but code inspection reveals an unsafe access to
mIdleHandler inside queueIdle. Wrap that in a synchronized block.

Change-Id: I27307e2e55fa8d937d7f043bd436894091c3c667

Merge "Adding a default implementation for the new Window.Callback method" into nyc-dev

Add more @NonNull/@Nullable to TextServicesSettings.

We need to fix TextServicesSettings as well as InputMethodSettings,
because regarding Bug 27687531 the same issue was copied from
InputMethodSettings to TextServicesSettings by my CL [1].

This is basically a mirror of the fix for InputMethodSettings [2].

  [1]: Ie3d61458648df469abe149b7aaad8087c531a675
       49ed14017697f8f19b8d49d55462713396a0588e
  [2]: I169ad4957e68b65c64251b0849056195b8ca4911

Bug: 27687531
Change-Id: I068f8821a32d2e199fbf6239bbe833da8c5a4e5e

Remove an unnecessary int to String conversion.

This is a safe refactoring to remove an unnecessary int to String
conversion in TextServicesSettings.

Settings.Secure.SELECTED_SPELL_CHECKER_SUBTYPE is a integer value that
indicates subtype ID (or SpellCheckerSubtype#hashCode() if the subtype
ID is not specified), and we can just rely on
Settings.Secure#putIntForUser() rather than converting int to String
by ourselves then pass it to Settings.Secure#putStringForUser().

Note that this change is still OK for existing users because
Settings.Secure#putIntForUser() has been internally doing exactly the
same thing.

Bug: 27687531
Change-Id: Ibcf12746f1295c12bec095300ea7f6ced0a51d09

Add more @NonNull/@Nullable to InputMethodSettings.

This follows up to a previous CL [1] for Bug 26279466.

It turns out that we have not clearly defined how nonexistent key should
be handled in InputMethodSettings#getEnabledInputMethodsStr(), e.g. it
returns "" for when mCopyOnWrite is true but returns null when
mCopyOnWrite is false.

Also, since InputMethodSettings now can revert changes made during
mCopyOnWrite is true, the caller may also start receiving null in the
following scenario.
  1. call mSettings.switchCurrentUser(userId, true).
  2. call mSettings.putEnabledInputMethodsStr(str) where str is non-null.
  3. call mSettings.switchCurrentUser(userId, false).
  4. call mSettings.getEnabledInputMethodsStr().
If the caller of getEnabledInputMethodsStr() has assumed that it would
never return null, then it would start crashing due to NPE.

With this CL, getEnabledInputMethodsStr() is marked to be @NonNull we
should no longer see such kind of NPE.

  [1]: I9c6f9bb3d51174198e5f73588637f87ea0d90e11
       68645a638ad1bfb734b2b0f56b17fe206bb891c5

Bug: 27687531
Change-Id: I169ad4957e68b65c64251b0849056195b8ca4911

Adding a default implementation for the new Window.Callback method

Bug: 27702972
Change-Id: I6b8c1766ecf871801006d4dc47796a819e49bb57

Merge "Refinement for onVisibilityAggregated" into nyc-dev

Merge "Flush package restriction changes for a user." into nyc-dev

Merge "Clean up BoringLayout#isBoring() a little" into nyc-dev

Merge "Apply temp inset bounds to child window if it's not empty" into nyc-dev

Merge "Chaser patch for AppOps restriction changes." into nyc-dev

Apply temp inset bounds to child window if it's not empty

bug: 27676101
bug: 27687126
Change-Id: Iab1129cc0224194e7a9d11d0454bc7af6897a6e8

Merge "Disallow disabling of the package installer" into nyc-dev

Merge "Add emoji-related tests for the font chain" into nyc-dev

Merge "DocsUI: Don't animate selection changes on all bind events." into nyc-dev

Merge "Add new key in CarrierConfigManager for ICCID display." into nyc-dev

Merge "Add delete key support." into nyc-dev

Merge "Add "pm set-home-activity"" into nyc-dev

Merge "DocsUI: Don't crash on invalid window sizes." into nyc-dev

DocsUI: Don't crash on invalid window sizes.

Our layout code asserts when passed an (invalid) width of 0.  The assert
is unnecessary as we already handle this situation by always laying out
the content pane with a column count of at least 1.

Also, per Steve, bump the minimum column count in grid mode to 2.

BUG=27150284

Change-Id: I8aa87beaaf4907ef5248999d0c590deef732589f

Merge "Adjust graphics dataspace references to new standard" into nyc-dev

Add "pm set-home-activity"

This is needed to change the default launcher on CTS for
ShortcutManager.

- When the default set with DPM.addPersistentPreferredActivity() is
still stronger.

- Ideally we should unify the code with HomeSettings, but I couldn't
find a good way to do this.

- I didn't make it to work to set the default for any intents
because building an intent filter from a command line would be
painful.

Bug 27548047

Change-Id: I66b153c20c7796147f77214b483112629f9d80c5

Add new key in CarrierConfigManager for ICCID display.

Bug: 27574058
Change-Id: I74445c68f58e64a5baa87cd05fea9e91e59b0201

[NAN] Refactor session lifecycle to clarify API & eliminate race conditions [DO NOT MERGE]
am: 2a73c7f  -s ours

* commit '2a73c7fb2a4aba4dd75341414f1583e5fb1f4613':
  [NAN] Refactor session lifecycle to clarify API & eliminate race conditions [DO NOT MERGE]

Merge "Update ContextHubManager's System APIs." into nyc-dev

Clean up BoringLayout#isBoring() a little

No functional change. Just some renames and clarifications.

Bug: 23641693
Bug: 27702584
Change-Id: I14efe290a27c695ab79bbb18f9f069c8a2d3ffd6

Merge "Dynamic audio policies: device type and address" into nyc-dev

Merge "Add fillType support to VectorDrawable" into nyc-dev

Merge "AudioService: do not persist mic mute and master mute" into nyc-dev

Update ContextHubManager's System APIs.

- Consolidates the use of handles in context hub, and nano apps.
- Quick fixes to typos.
- Update the signature to find nano apps to use primitive types

Change-Id: Icffa6fa86e0e33a536516d98bc4674c0a39b2163

Add emoji-related tests for the font chain

1. Check that all emoji characters are supported in emoji fonts.

2. Check that no default-emoji characters are included in any font
that can get a higher score then the color emoji font, either by
being before it or by having a language/script assigned.

Bug: 26187231

Change-Id: Ic76bf59f1c6f9d2d290b1b64128d725ae08dcb5c

Merge "Remove references to Google packages." into nyc-dev

Merge "Revert "Precompute cursor indexes in DocumentsUI and improve perf by 2.7%." It ia causing tests failures and exception at the app start." into nyc-dev

Remove references to Google packages.

Bug: 26441633
Change-Id: Iba6566215e2b6d224fd3b9c11d086f5c2db87dca

Add delete key support.

- Pull the action mode up from the ActionMode listener into the
  DirectoryFragment.  There is no way to globally check on action mode,
  so fragment-level coordination on the action mode requires a
  fragment-level variable.

- Add a key handling clause for the delete key.  Unfortunately this
  can't be set up as a shortcut key, because shortcuts all have to be
  modified with the "Ctrl" key.

BUG=27409328

Change-Id: I48d1c87613ee7434728828e3b745fadb3bc915c1

Merge "Defuse Bundles parsed by the system process." into nyc-dev

Revert "Precompute cursor indexes in DocumentsUI and improve perf by 2.7%."
It ia causing tests failures and exception at the app start.

This reverts commit d35a974b76cb0b3387aff7780d101e3f7de2ebd3.

Change-Id: I2577f723a7e25d4dcc12050791c65a900ac41f7d

Defuse Bundles parsed by the system process.

It's easy for apps to throw custom Parcelables into Bundles, but
if the system tries peeking inside one of these Bundles, it triggers
a BadParcelableException.  If that Bundle was passed away from the
Binder thread that delivered it into the system, we end up with a
nasty runtime restart.

This change mitigates this trouble by "defusing" any Bundles parsed by
the system server.  That is, if it encounters BadParcelableException
while unpacking a Bundle, it logs and delivers an empty Bundle as
the result.

Simultaneously, to help catch the system process sticking its
fingers into Bundles that are destined for other processes, a Bundle
now tracks if it's "defusable."  For example, any Intents delivered
through ActivityThread are marked as being defusable, since they've
arrived at their final destination.  Any other Bundles are considered
to be "in transit" and we log if the system tries unparceling them.

Merges several Parcel boolean fields into a flags int.  Add better
docs to several classes.

Bug: 27581063
Change-Id: I28cf3e7439503b5dc9a429bafae5eb48f21f0d93

Merge "Converting empty graphic to vector drawable." into nyc-dev

Refinement for onVisibilityAggregated

Change the args for onVisibilityAggregated to a single boolean for
visibility to the user. Also fix an ordering bug that could trigger a
view background drawable call to setVisible before the view would
respond true to verifyDrawable, which caused issues with some apps.

Bug 27461617
Bug 27689884

Change-Id: I58bdc7c4364264f7fa0863689ba2b76df904ef81

Merge "DO NOT MERGE Fix conditional when checking for confirm key." into nyc-dev

Merge "Add support for dpads." into nyc-dev

Merge "Remove unused include" into nyc-dev

Close overflow when hiding QS editor

Bug: 27625965
Change-Id: Iab29a7ab879ce879f772b759a645e598f3b70328

Merge "Fix band selection problems." into nyc-dev

Merge "Don't allow selection in copy dest picker." into nyc-dev

Remove unused include

This include breaks compilation when HWUI_NEW_OPS=false

Change-Id: Ia3167f1550167f96de83af4b2f25abd6903c4723

don't call other services in dump

Currently the ranker config file is synced to app opps.
Also sync App Opps to the config file.
Then use that data to inform the dump, instead of polling.

Bug: 25424580
Change-Id: I6089532aa8618a31bfc3a36bee630c04c3ef3798

Merge "Fix NC caused by race condition in SystemSensorManager" into nyc-dev

Add some synchronization to hotspot callbacks

Bug: 27497199
Change-Id: I792a246c0e2387e8db1fd528a7c2f20844cdd1a0

Dynamic audio policies: device type and address

Store device type and address in audio mixes regardless of the
  mix route type, no "registration" field.
For LOOP_BACK mixes, assign the REMOTE_SUBMIX device type.

Bug 25448664

Change-Id: I8d59b2fd3e05b1dcf1081ad826f985f5e3e74769

DocsUI: Don't animate selection changes on all bind events.

When directory contents update, this causes all items to be bound, which
can cause flickering even though item selection state hasn't changed.

Add a boolean var to trigger UI animations only when selection changes
are triggered by the selection manager.

BUG=27680153

Change-Id: I892d8111cf3ba7e1610d94af4a1a646f1cdb5546

Don't clear effects when the panel is fully collapsed.

Otherwise, a FLAG_SHOW_WHEN_LOCKED activity would be unable
to make noise (particularly in conjunction with a long
sound, or FLAG_INSISTENT) because the mere presence of the
status bar on screen would generate an updateVisibleToUser()
call, which would in turn use onPanelRevealed() to call
clearEffects().

Instead, we have to clear effects from deep in the stack
scroller; that's apparently the only place right now where
we know with 100% certainty that notifications are actually
visible to the user.

We also no longer clear effects just because the keyguard is
shown. That's silly.

Bug: 26863782
Change-Id: I9ec53024f4be060da203b835d31da81104e8c8e6

Don't allow selection in copy dest picker.

Bug: 27490262
Change-Id: I1938a900cfd3693739c2ac10b5eea3176a4ca82c

Change drag label while dragging QS tiles

Bug: 27316050
Change-Id: I20b4f3c94f1362e9990cebefdfa900a23636a938

Converting empty graphic to vector drawable.

Bug: 27154293
Change-Id: I2ee46bf5aa9ab7ad05c054363d53fdda0a3069dc

Fix Settings home lifecycle

Bug: 26268813
Bug: 27330440
Change-Id: I6f5ab4a412b129728db3e8dff134c04272909997

Merge "Document that Rect right/bottom are exclusive" into nyc-dev

Merge changes I3eba2cdb,Ie529f27e,I4f0290fa,I7fb1f704,I32b9dd59, ... into nyc-dev

* changes:
  Scroll to the launch-task when toggling overview.
  Ensure that recents toggles even in the empty state.
  Removing paging tuner setting.
  Updating empty state graphic.
  Fixing issue with the stack layout not being updated on config change.
  Ensuring that we evict ActivityInfos from the cache.

Add fillType support to VectorDrawable

Default as non-zero, which is the same as SVG.
b/27533958

Change-Id: Id20e6d3493bb4d2b4b65d7f6cdb13586631c40e4

Fix band selection problems.

BUG=27364174
BUG=27645942

Change-Id: Ib14680db708f1ddf1f5dc9dff8b93b655fa3f123

Scroll to the launch-task when toggling overview.

Bug: 27680710
Change-Id: I3eba2cdbe0a92ef05dd6882a5e41db2a5875c98a

Ensure that recents toggles even in the empty state.

Bug: 27604213
Change-Id: Ie529f27e7e8110be8e86224bbad7622300fa87a7

Removing paging tuner setting.

Bug: 26962865
Change-Id: I4f0290fa334b6153cc160cb5dbe6e333a002334c

Updating empty state graphic.

Change-Id: I7fb1f704dfbf0a300e00805d75c08b2fdabf7260

Fixing issue with the stack layout not being updated on config change.

Bug: 27376567
Change-Id: I32b9dd59a6187c87387785335dd66cc4f28372b3

Ensuring that we evict ActivityInfos from the cache.

- Once the data that we depend on the ActivityInfos for are evicted,
  also remove them from the cache so that they are not stale the next
  time we might want to use them.

Bug: 27495264
Change-Id: I615842df4f48deb12a051ef241991a51a10cf383

Merge "TIF: Unhide sendAppPrivateCommand/onAppPrivateCommand" into nyc-dev

TIF: Unhide sendAppPrivateCommand/onAppPrivateCommand

Bug: 25726685
Change-Id: Iad6417b4da0543043a7fa17287487313a7670209

Fix NC caused by race condition in SystemSensorManager

This change fixes a race condition when two SystemSensorManager
instances are initialized the same time.

Bug: 27334914
Change-Id: I7a886abcbff2ed6f8abf87556a78d1a4afb71f91

Merge "Apply stack crop to docked stack during animation" into nyc-dev

Apply stack crop to docked stack during animation

If device goes to sleep with docked stack visible, the docked stack
will be minimized after unlock. Due to "force hide enter" animation
and the entering animation of the app itself, the app will be animating
for the initial few frames. Not applying the crop causes the app to
be seen in full briefly, then minimized.

Docked divider doesn't use a final stack bounds but changes stack
bounds continously during docked stack resize, so we can apply the
bounds even during animation.

bug: 27592767
Change-Id: I12f4eb46a95c9a7dd2147b68c362e17609a8b410

Fix up label in QS detail

Bug: 27601234
Change-Id: Ib21166b84b45118ee1aa574e5a6ea5c73e189490

Merge "Exposing Context Hub service." into nyc-dev