Merge "Change suggestion window text color based on user feedback." into nyc-dev

Change suggestion window text color based on user feedback.

Changing highlight text color to black alpha=.87 and normal text
to black alpha=.57.

It turned out that alpha value is ignored if we set alpha value
and textColor at the same time. Using #AARRGGBB style instead.

Bug: 27389556
Change-Id: I318379261c66d9da089e8f8bf9f8136c10958511

DO NOT MERGE: Move PinningNetworkCallback out to a new NetworkPinner class.
am: 2a3ca73158  -s ours

* commit '2a3ca73158227751321e98ba707496adf84007a6':
  DO NOT MERGE: Move PinningNetworkCallback out to a new NetworkPinner class.

Merge "Add missing references to VNativeObject" into nyc-dev

Merge changes I2d6f9932,Ib3cc799d into nyc-dev

* changes:
  Rely on SystemService#onSwitchUser() in TSMS.
  Switch to SystemService lifecycle in TSMS.

Add missing references to VNativeObject

Change-Id: I2db4ad94009c0daff4649da6f85e5c084f4c01ea

Rely on SystemService#onSwitchUser() in TSMS.

SystemService class has already provided SystemService#onSwitchUser()
callback event.  We do not need to set up SynchronousUserSwitchObserver
separately in TextServicesManagerService.

Bug: 25816558
Bug: 27456430
Change-Id: I2d6f9932f30b72cf4ae4bc0c41810f75e2667478

Switch to SystemService lifecycle in TSMS.

Like we did this for InputMethodManagerService [1],
TextServicesManagerService (TSMS) needs to be recognized by
SystemServiceManager with SystemService lifecycle mechanism so that we
can receive SystemService#onUnlockUser() event, which is necessary to
make TSMS encryption-aware.

As a preparation, with this CL we only does mechcanical migration to
SystemService lifecycle mechanism in TSMS.  Hence no user-visible
behavior change should occur.

  [1]: Ic17667df60b30e5355b61a3601ad27a000cab3a3
       1e33dc8fdf3f722ecd32cc586b2a9515de24a242

Bug: 27456430
Change-Id: Ib3cc799d384f259b4fa3f5295f2da198df015eb8

Merge "Introduce script matching for enabling default IME subtypes." into nyc-dev

Merge "Change BridgeResources to Resources_Delegate" into nyc-dev

Merge "Animated vector drawable support" into nyc-dev

Merge "Revert "AssetManager: Cache a pre-filtered list of configurations"" into nyc-dev

Merge "Revert "Rename AudioRecordConfiguration to AudioRecordingConfiguration"" into nyc-dev

Revert "Rename AudioRecordConfiguration to AudioRecordingConfiguration"

This reverts commit 99278e1c724e01b792dd27031bc579e7d738edc1.

Change-Id: Ifcb3f17a26df8171cbc21dac093477b98a1333db

Introduce script matching for enabling default IME subtypes.

This is 2nd attempt of I5bb1bd8cdb9096d516d60beb9936e55bf2b757ae

The motivation of this CL is enhance the default IME subtype enabling
algorithm.  The new approach is done by score based algorithm.  The
design of the matching score is determined as follows:
- The matching score for the each two locale is up to 3 and determined
  as follows:
  - Score of 3 : matches all language, script and country.
  - Score of 2 : matches the language and script.
  - Score of 1 : matches the language regardless of country.
  - Score of 0 : doesn't match the language regardless of script and
                 country.
- All locales are fully expanded before matching by addLikelySubtags in
  ICU.

Bug: 27129703
Bug: 27348943
Change-Id: I8fc774154f5175abff2f16e8f12a4847bf5f5b7c

Revert "AssetManager: Cache a pre-filtered list of configurations"

There is a race due to the modification of shared data structures in the framework ResTable.

See b/27499488

This reverts commit d4b169173ad7805369204277580d3942cb08174a.

Change-Id: I3f400a2ad3b5ffc652b84dd5fe777f7cf34b5548

Merge "Rename AudioRecordConfiguration to AudioRecordingConfiguration" into nyc-dev

Making AccessibilityUtils interface multi-user aware

The getEnabledServices and setAccessibilityServiceState now have an
overloaded version to take in a userId parameter.

Bug: 27383941
Change-Id: I3ea648fd5f22cd2e9402686b83ca42976b5306f8
(cherry picked from commit 2d1c0154b1f9aeeb991dc96efb29aa53c76f4957)

DO NOT MERGE: Move PinningNetworkCallback out to a new NetworkPinner class.

Cherry-picked from 531a34430072b9296aaeb47d9e7d04326a93fee4

Bug: 19159232
Change-Id: Ic366b53259ee5944a8e864876425a6558c0a7216

Merge "Fixed IndexOutOfBoundsException when removing child windows" into nyc-dev

Merge "Add error code to MediaCodec.CryptoException" into nyc-dev

Merge "Implement glCreateShaderProgramv JNI" into nyc-dev

Merge "Add "app printer activity" and always keep the print service state updated. Also fiddle with the UI to use more standard values." into nyc-dev

Fixed IndexOutOfBoundsException when removing child windows

The size of the list reduces when a child window is removed from the
list. So, we copy the window list and loop from the last entry to the
first when removing to avoid IndexOutOfBoundsException.

Bug: 27345523
Change-Id: I15986e418d29ee5035dc9d4c4f728ad33bfe6999

Add "app printer activity" and always keep the print service state
updated. Also fiddle with the UI to use more standard values.

To be sure the print service state alwasy updated I changed
PrintManager.getPrintServices to return a loader which just wraps a
registerListener/getList/removeListener combo.

I also added a new function to enabled/disable a print service to be
keep all updating logic inside the PrintManagerService->UserState.

Then I changed all code to use this new interface.

Detailed comments:

PrintServiceInfo:
- I had to add the enabled state to the PrintServiceInfo as some users
  of PrintManager.getPrintServices want all services but then display
  different data depending on the enabled state. Of course I could have
  created two PrintManager.getPrintServices-loaders to load the two
  separate list of services. I think it is much easier to add this
  property though. It is updated every time new data is returned to the
  PrintManager.getPrintServices-loader.

AddPrinterActivity:
- This is shown as a dialog-style overlay to indicate that the user will
  return to the select-printers activity. It contains of three list that
  are updated via separate loaders.
- The recommended services will be added later to keep this path set
  small.

PrintActivity:
- There are two small places where we have to update the data when we
  get a new list of print services.
  - In very, very rare conditions it can happen that the print service
    of the current printer gains or looses the "advancedOptions"
    activity
  - If we have no enabled print services we want to show "Add printer"
    instead of "All printers...".
- Also the print registry is not the only loader anymore, hence we have
  to assign loader ids to it to not conflict with the other loaders in
  this activity.
- Small bug in onPrintersChanged: If a printer is selected and the print
  service of this printer gets disabled the holder goes into "removed"
  state which disables the printer. When the print service is then
  enabled again, we forgot to re-enable the holder.

PrinterRegistry:
- The registry assumed that the FusedPrinterProvider was the only loader
  in the activity. This is not true anymore, hence it has to assign the
  appropriate loader ids.
- The FusedPrinterProvider has an internal loader, hence we have to
  forward a loader Id into it.
- The PrintRegistry is only called backed for a single loader, hence no
  need to check the loader-id.

SelectPrinterActivity:
- The AddPrinterDialog was removed as we now have the
  AddPrinterActivity.
- Added a loader for the enabled services to update the empty state.
- Added dedicated loader Id for the PrinterRegistry again.
- If we have no enabled services, the SelectPrinterActivity chainloads
  the AddPrinterActivity as this is the only thing the user can do
  anyway. "Save a click". This should only happen when the activity is
  create the first time.
- Moved the "add printer" from the menu item to the list of printers as
  suggested by UX and Zach.

PrintManagerService, UserState and IPrintManagerParamtersTest:
- As the only place where the print service state is updated is now the
  userstate, we have no more sychronization problems. Whohoo.
- The users can now register for changes to the print services similar
  as they can register for changes to the print jobs.
- UserState.getPrintServices is the only function can exposes any
  knowledge of the print services to the outside world.

Change-Id: I9be2c7300431e06aaff9bdf7eb36120d869b56ac

Merge "Ensure PopupDecorView is not IME-focusable during exit transition." into nyc-dev

Merge "Reject RenderNodes contents when clip is empty" into nyc-dev

Merge "Do not move PDF content as the content is already moved." into nyc-dev

Merge "Update print document view if printer capabilities changed" into nyc-dev

Merge "Redraw the UI when a print page preview is updated" into nyc-dev

Merge "Fix window insets for settings activities" into nyc-dev

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

Fix window insets for settings activities

Bug: 27276035
Change-Id: I48d8361f38dd7b708c49e71087e3e2ebf3f2df7a

Import translations. DO NOT MERGE

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

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

Import translations. DO NOT MERGE

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

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

Import translations. DO NOT MERGE

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

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

Import translations. DO NOT MERGE

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

Merge "Move QS Edit into QSContainer" into nyc-dev

Move QS Edit into QSContainer

This will let it play nicely with heads up.

 - Move to QS Container.
 - QS Edit is always full height (some layout hacks to do this)
 - Always draw QS customizer on top when animating
 - Block all panel scrolling while QS edit is open (all touches
   go to editing)
 - Instantaneously change the height of the QS container at
   start/end of animation as needed

Bug: 26969293
Change-Id: Iedc6f5aaf659dcc6750972eae5f69cc0cd2df844

Merge "Remove redundant stop() from native audiotrack release." into nyc-dev

Merge "Don't crash if KeyChain can't be bound to." into nyc-dev

Don't crash if KeyChain can't be bound to.

KeyChain can throw an assertion error if
is not around, don't allow that to take down
system.

Bug: 27518175
Change-Id: I99418dfb65c58d3e07cbda91860cdb493b96a836

Import translations. DO NOT MERGE

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

Merge "Do not re-define remote bugreport constants defined in DevicePolicyManager" into nyc-dev

Fix null pointer exception in DocumentLoader.

BUG=27489909

Change-Id: I1ebc9953f6db6639241d0c46257d110c5b0eb5b0

Add support for GradientColor in layoutlib

Change-Id: Ia9a55a9e00d7ddb5263f3dbe46b5da8dde457526

Change BridgeResources to Resources_Delegate

This is a very hacky fix to make layoutlib work with the latest support
lib. Inject a couple of fields in android.content.res.Resources to
allow using most of the earlier code as is.

Bug: 27403642
Bug: http://b.android.com/201934
Change-Id: Ia96a1b4642a1907a77d9ca7a2392f78809b92e38

Merge "Fix wrong metadata conversion in MediaSessionLegacyHelper" into nyc-dev

Ensure PopupDecorView is not IME-focusable during exit transition.

Currently PopupDecorView temporarily gets FLAG_NOT_FOCUSABLE bit during
exit transition.  When doing this we it is important that PopupDecorView
does not have FLAG_ALT_FOCUSABLE_IM bit, because having both
FLAG_NOT_FOCUSABLE and FLAG_ALT_FOCUSABLE_IM means that PopupDecorView
is IME-focusable.

To prevent IMEs from flickering, this CL ensures that PopupDecorView is
not IME-focusable during exit transition.

Bug: 27396330
Change-Id: I0cc81791b826f7d4257f0b6085c028c796fd9536

Merge "Ensure to complete background thread of PipeManager." into nyc-dev

Merge "Use Java7 diamond operator in TSMS." into nyc-dev

Merge "Try to find equivalent SuggestionsSpan when it has been updated." into nyc-dev

Merge "PIP: Apply the latest UI spec for overlay text" into nyc-dev

Use Java7 diamond operator in TSMS.

This CL changes nothing except for deleting redundant type parameters by
using diamond operator in TextServicesManagerService (TSMS).

No behavior change is intended.

Bug: 27456430
Change-Id: I18708fe44d2d9edfef8c1cf7cffef1bb0b5c003a

Revert "Introduce script matching for enabling default IME subtypes."

This reverts commit 9e7a1c9824cffca32fd7d58bb13bd3416ab32091.

Seems that that CL causes ArrayIndexOutOfBoundsException when
initializing InputMethodManagerService, which results in an infinite
boot animation.

Bug: 27129703
Bug: 27348943
Change-Id: I474a87876670ac018c675ac7b4608e90fbb2434b

Merge "Don't make home stack visible if starting activity isn't translucent" into nyc-dev

Don't make home stack visible if starting activity isn't translucent

We allow the home stack to be visible right behind the fullscreen
stack if all the activities fullscreen stack are translucent and
visible. However, if we are starting an acitvity it isn't visible
yet so our check for translucent activities in the fullscreen
stack will return true since there is nothing visible. This will
cause the home stack to the visible and the visiblilty of its app
token set to true in window manager and it been factored into the
transition animation.

In addition to checking if the activity is visible, we now also
check if it is the starting activity when trying to determine if
the stack is translucent.

Bug: 27448511
Change-Id: Icbbd57c0632cc5389c7ac894800a41f75d4bc450

Merge "Introduce script matching for enabling default IME subtypes." into nyc-dev

Introduce script matching for enabling default IME subtypes.

The motivation of this CL is enhance the default IME subtype enabling
algorithm.  The new approach is done by score based algorithm.  The
design of the matching score is determined as follows:
- The matching score for the each two locale is up to 3 and determined
  as follows:
  - Score of 3 : matches all language, script and country.
  - Score of 2 : matches the language and script.
  - Score of 1 : matches the language regardless of country.
  - Score of 0 : doesn't match the language regardless of script and
                 country.
- All locales are fully expanded before matching by addLikelySubtags in
  ICU.

Bug: 27129703
Change-Id: I5bb1bd8cdb9096d516d60beb9936e55bf2b757ae

Merge "Skip publishContentProviders() when we have zero." into nyc-dev

Skip publishContentProviders() when we have zero.

If the list of providers for the application being bound is empty,
don't bother trying to register them.

Bug: 27510621
Change-Id: I921fb70a4c97433fa123fa88e170745ee013ee35

Merge "Set default ringtones through RingtoneManager." into nyc-dev

Merge "fix DocumentsUI tests makefile" into nyc-dev

fix DocumentsUI tests makefile

For instrumentation test apks, it should not statically include
Java libraries that are already present in the app module being
instrumented. They only need to be on classpath during compile
time.

Bug: 26689464
Change-Id: I68d4be95e30ae70bbab23d016a87eb085c10cc50

Set default ringtones through RingtoneManager.

RingtoneManager is the public API that everyone should be going
through, and it now has the side-effect of caching the set ringtone
to make it available before CE storage is unlocked.

Bug: 27435331
Change-Id: I30ed4e2df2ef1e4fd47f947c70845aaa74356384

Merge "Ask RingtonePlayer to open data for caching." into nyc-dev

Ask RingtonePlayer to open data for caching.

When setting default ringtones, RingtoneManager now caches the
selected media for playback before the device is unlocked.  However,
this API hasn't historically required the caller to hold storage
permissions.

To keep this working, we attempt to delegate ringtone access over
through RingtonePlayer, which is what we do for playback.  However,
because we're caching the real ringtone bits now, we need to be much
more careful about the PFDs we're willing to return.  This change
requires that they be in external storage, and that they have the
ringtone/alarm/notification bit set.

Bug: 27366059
Change-Id: I59c2adc1d1250a3eac281f190f35a7cb3119967b

Merge "Add AssetFileDescriptor to MediaExtractor." into nyc-dev

Add AssetFileDescriptor to MediaExtractor.

Mirrors API on MediaPlayer.

Bug: 27506874
Change-Id: Ibc07cf8151c362ff7375b89e7a9044bef1d9e94d

Merge "Kick Bluetooth stack after user is unlocked." into nyc-dev

Kick Bluetooth stack after user is unlocked.

If Bluetooth was enabled at boot time, but the actual Bluetooth
package wasn't encryption aware, we need to kick off another enabled
pass once the user is unlocked.

Bug: 27326711
Change-Id: I511c4fd8567792e5bc4d49fa95929fd87edd0fbf

Merge "Bluetooth should use Slog." into nyc-dev

Bluetooth should use Slog.

It's in the system process, so whatever it has to say is probably
important enough to stick around.

Change-Id: Iaeca99d6b9c4881c53ab89216b5128e345a5ff0f

Merge "Move more PM calls to ParceledListSlice." into nyc-dev

Add error code to MediaCodec.CryptoException

Added a code to represent unsupported operations

bug: 25929554
Change-Id: Id8bfa092f29de073db19c4d7444f89249ea2cdd5

Remove accidentally committed core dump file

Change-Id: I610abfbce62585c25e98b5c36ac9f496861e66e7

Merge "AAPT2: Emit proper doclava comments in R.java" into nyc-dev

Merge "AAPT2: Support --no-version-vectors" into nyc-dev

AAPT2: Emit proper doclava comments in R.java

Bug:25958912
Change-Id: I663f2eb5bd54e3c3288ce9bc186c928f0a014f93

Merge "Fixed the crash caused by type mismatch" into nyc-dev

Merge "Don't crash legacy apps with revoked "Draw over apps access"" into nyc-dev

Merge changes I5c414d67,I83d74c94 into nyc-dev

* changes:
  Just return the task to the pool if it is not in the current stack.
  Refactoring to update stack in onResume.

AAPT2: Support --no-version-vectors

Change-Id: I8029b2f9e2f7ab8e37a49565f95210f47a7be609

Fixed the crash caused by type mismatch

Bug: 27503442
Change-Id: I019afc1f3dd872c8f9d79e9b6e589a77acd0cddc

Just return the task to the pool if it is not in the current stack.

Bug: 27467025
Change-Id: I5c414d674fb0db152b77c2025b4885d340e81880

Refactoring to update stack in onResume.

This CL fixes a long-standing issue in which Recents is not updated
correctly if it is not completely hidden and shown.  In particular, it
would cause animation issues when launching into a non-fullscreen
activity or if the user quickly toggles between recent tasks.  It
contains several fixes:

- The visual state in Recents is no longer reset until the activity is
  fully hidden (onStop() is called), and the task stack state is saved
  allowing us to return to the same initial state.  When restarting the
  activity, we propagate whether the activity was hidden down the view
  hierarchy, so that each task can decide whether to reset itself.
- When the recents activity is started, we now merge the new stack with
  the current stack instead of replacing it completely.  This unifies
  the logic when dismissing multi-window while Recents is open, and this
  CL fixes an issue with the merging where onStackTaskAdded() was called
  before the stack was updated with the new task.  As a result of this
  change, we can just rebind the task views without having to return and
  pick them up from the view pool.
- This CL also fixes an issue with flashing when the screen turns off.
  The activity onStop() can be called before the activity is fully
  hidden, which would trigger a reset(), which would return all views to
  the pool.

Bug: 23815609
Bug: 25411120
Bug: 27186407
Change-Id: I83d74c947f9b47766d6778b7f5c421bb6df833e9

Merge "Changed breadcrumb arrow to match mocks" into nyc-dev

Reject RenderNodes contents when clip is empty

bug:22630332

Change-Id: I326123de3154f655e35d649c242daea3b4f0fda0

Don't crash legacy apps with revoked "Draw over apps access"

bug:26396161

Change-Id: I8b86e175b4815e58ace78c37a86141c5e4dbe183

Merge "Support GPU profiling vis in new pipeline" into nyc-dev

Support GPU profiling vis in new pipeline

bug:27353099

Change-Id: I905c1a998d9a9e2097c047dab9de87a70d7a370e

Changed breadcrumb arrow to match mocks

Bug: 27136601
Change-Id: I03b3d789c1edf8de7f3d92238045cca123ce5e72

Merge "AAPT2: Support building, linking, and merging static libraries" into nyc-dev

Merge "Fix BiDi issues in app error dialogs" into nyc-dev

Move more PM calls to ParceledListSlice.

Since the data returned by these calls can grow unbounded based on
various GET flags, we need to switch 'em over.

Bug: 27391893
Change-Id: Ie849ca30dbaaa91158da4c83675657715629a0ee

Log bouncer prompt reason

Bug: 27434873
Change-Id: I6918507b66506cf95b20f95fe4a5cb19c899122f

Merge "API changes for audio recording notifications" into nyc-dev