Only call Drawable.setVisible(false) for visible outgoing drawables

In framework views where we're handling the new visibility aggregated
call we only update the drawable visibility when we're attached to a
window. For old outgoing drawables being replaced, gate this on
whether the drawable is already marked visible instead.

This catches a case where views being inflated might set drawables in
in a superclass constructor and have them replaced in a later
constructor. Gating the call into a drawable that might invoke its
callback (the view being constructed) avoids potential problems where
overridden methods are called unexpectedly on a view subclass that has
not finished running its constructor.

This is a better check than isAttachedToWindow, as isAttachedToWindow
will return false if the view has been temporarily detached from its
parent by a view-recycling container. In those cases, the view would
not correctly update the outgoing drawable.

Bug 27461617

Change-Id: I733a2dd3e3df0a8d80d5dc542ca7b30064159d5d

Merge "Make DISK_SCANNED a protected broadcast." into nyc-dev

Merge "Only enable asserts on userdebug/eng builds." into nyc-dev

Merge "Store trace files in app-specific dirs." into nyc-dev

Merge "Delete dead code." into nyc-dev

Merge "Clean up some javadoc issues" into nyc-dev

Clean up some javadoc issues

Bug: 27672974
Bug: 25881314
Change-Id: Ida53e3bf23b2492c26f4a82878fa192110a8794f

Merge "Revert "Open drawer when showing copy/move destination screen"" into nyc-dev

Revert "Open drawer when showing copy/move destination screen"

This reverts commit 4d5f9f6a7f3b9aa8f7fd1a733ba739e5bfe3ed66.
The CL breaks android.appsecurity.cts.DocumentsTest.

BUG: 27746116, 27479369
Change-Id: Ib9426c54e8c9841f3def55a88e07bf4a2cd7dcac

Merge "Notify via TalkBack about selected items." into nyc-dev

Merge "Fix null pointer exception when MTP/PTP device is detached." into nyc-dev

Notify via TalkBack about selected items.

Bug: 27723098
Change-Id: I6927b004b3e5a592325646ae3a4663bb298a65c8

Merge "Support registering a request for the default network" into nyc-dev

Delete dead code.

Change-Id: If8052efaa00f15e5c7eb038435be33206d54276a

Merge "PIP: Increase the PIP menu's char limits for translation" into nyc-dev

Support registering a request for the default network

Change-Id: I079f5be97b585cf5692dd4c7a144b993d168a9a5

Merge "ExifInterface: fix logs" into nyc-dev

Merge "ExifInterface: fix writing STRING and SRATIONAL formats" into nyc-dev

Merge "Remove framework code that calls SIOCKILLADDR" into nyc-dev

ExifInterface: fix logs

- Give more explanations on dealing with files failed to process.
- Log and return immediately when handling lost JPEG files.
- Minor fixes in logging.

Bug: 27583378, Bug: 27587980
Change-Id: Id9a3d137e5f173fa0e6313402e5dc482a5efd57c

Merge "Don't map root documents in DocumentLoader." into nyc-dev

Don't map root documents in DocumentLoader.

Root documents must be added by RootScanner.
Previously DocumentLoader unintentionally tries to load unmapped root
documents, which causes crash.

BUG=27731902

Change-Id: I8674473d80d6e7f758f6636d30148f5d2118bc25

Merge "Add root flag correctly." into nyc-dev

Add root flag correctly.

BUG=27369570

Change-Id: I93b9b662667009a374d879c7ab667b10fb7cbf7c

Merge "Count error document to complete adding documents to the database." into nyc-dev

Count error document to complete adding documents to the database.

Previously DocumentLoader#LoaderTask had a counter to count loaded
documents and completes adding documents to the database.  However it
does not count documents where a MTP device returns an error for
getObjectInfo. The CL fixes the problem to ensure we complete documents
loading.

BUG=27729653

Change-Id: I696eac790a6535f1bd7a1855dc2d6f932e32eae5

PIP: Increase the PIP menu's char limits for translation

Bug: 27699763
Change-Id: I2ae0d44ecb30bf63c2229a9531d226fc5ddcd40d

Remove framework code that calls SIOCKILLADDR

Bug: 26976388
Change-Id: I652361d9e650a77085070d6c26b28b7638ba2dfe

Merge "PIP: Fix NPE when an activity is PIPed again immediately" into nyc-dev

Update api/test-current.txt

Change-Id: I1fa444fbe874862e6296ee6a9751c39a0cbc4b7a

Use handles to identify GATT attributes (4/4)

Bug: 27778668
Change-Id: I01e095939df2f8dc433f14b473957c815ecade74

PIP: Fix NPE when an activity is PIPed again immediately

More proper fix would be not showing Recents after an activity is PIPed,
but NPE shouldn't be happen so let me propose the CL as the first step.

Bug: 27634771
Change-Id: Ie98293b7dd2d206567ded08d54134164aa78ca9c

Change how services are passed up to the stack

Right now we pass all services, characteristics and descriptors one by one.
This patch changes that - now we pass whole GATT database at once.

Bug: 27455533
Change-Id: Ie42cd80072538e411904b9c9b011a978f26158b9

Fix null pointer exception when MTP/PTP device is detached.

FIXED=27743872

Change-Id: I9b754ba7efbc4958a47aafe332f1909bc0d50457

Merge "Fix the logic to add suffix when copying." into nyc-dev

Merge "Clean up and rename TypefaceImpl" into nyc-dev

Fix the logic to add suffix when copying.

Previously the logic add '.' even if the extension is empty.

BUG=27729309

Change-Id: I3fa4b09fea977b5c716ec5cbbbc7d9f4ab96db08

Merge "Add USB_HOST_CONNECTED extra to ACTION_USB_STATE sticky intent." into nyc-dev

Merge "Change the title of DocumentsUI from "Files" to "Downloads"." into nyc-dev

Merge "PIP: Prevent PipOverlayActivity from restarting" into nyc-dev

Merge "Hide system UI activity with FLAG_ACTIVITY_EXCLUDE_FROM_RECENTS in Recents" into nyc-dev

Merge "Reland: Move text logic from jni to hwui level" into nyc-dev

Merge "ShortcutManager: Clean up for uninstalled packages." into nyc-dev

Merge "Add API to read the current wallpaper's ID" into nyc-dev

Clean up and rename TypefaceImpl

bug:25865834
Change-Id: I77e8a627163e040a5c25865054a8a936052af367

Make DISK_SCANNED a protected broadcast.

Bug: 27763115
Change-Id: Ib8b40ff9f4c1f929ac4f07a9d5460a0a593ba080

Only enable asserts on userdebug/eng builds.

Bug: 27699194
Change-Id: Id20f808448ae94371da62d38387f90ad97e7cbd3

Reland: Move text logic from jni to hwui level

Initial CL: https://googleplex-android-review.git.corp.google.com/#/c/886854/

Change-Id: I9dfd85fe1d2a2c44f4360c8a29fd58d80e6f31c8

ShortcutManager: Clean up for uninstalled packages.

Bug 27548047

Change-Id: I95ca832af496fbf71023eb27a7e2647c124ffb86

Store trace files in app-specific dirs.

The previous default location of "/sdcard" became painful to use
starting in M, because it required a runtime permission.  So now we
default to storing trace files in app-specific directories on shared
storage, which apps always have write access to with no additional
permissions.

Update docs to be consistent between all overloads.

Bug: 22807654
Change-Id: If4feca7c8778dfdf4ccce8cfb68418dc416260b5

Merge "Don't finish resolver activity onStop if it is for home intent" into nyc-dev

Merge "Add a comment about ResultReceiver lifetime." into nyc-dev

Merge "Add guard around getXmlFileParser() call" into nyc-dev

Add API to read the current wallpaper's ID

Also regularize cross-user call handling throughout.

Bug 27743435

Change-Id: Ibc530d0576d657723a613888357a6ea71b482763

Merge "Don't mix UI and print job state" into nyc-dev

Merge "Fix string equality comparison" into nyc-dev

Merge "Ignore wildcard network policies." into nyc-dev

Merge "Skip app widget config changes when user locked." into nyc-dev

Merge "Check "Secure start-up" before toggling emulation." into nyc-dev

Merge "Mark even more Bundles as defusable." into nyc-dev

Add guard around getXmlFileParser() call

Prevents the need to update the LayoutLib API level.

Bug: http://b.android.com/202942
Change-Id: I507239148ab8d6d45406624dba569c8b7197f979

Merge "Revert "Move text logic from jni to hwui level"" into nyc-dev

Revert "Move text logic from jni to hwui level"

This reverts commit a7f6bba1a3565c19715e878dfe7f0e01022944ff.

Change-Id: If4f36f87a85411b6128fd92d391313803ccaf9dd

Merge "Set drawable callbacks last" into nyc-dev

Merge "Move text logic from jni to hwui level" into nyc-dev

Merge "Added support to specify animation duration when resizing stack" into nyc-dev

Don't mix UI and print job state

In PrintActivity.OnPause we set the print job state based on the
PrintActivity state. The issue was that FINISHED was interpreted as
"cancel print job". Actually with FINISHED does not tell what state the
job should be in.

The only purpose of the finished state was to guard against
double-invocation of doFinish().

Hence remove this recently introduced state and use a simple boolean to
guard against doule invocation of doFinish().

Bug: 27760331
Change-Id: I0767c2b0b9c28e8904647cc832c236f67f92bcfe

Merge "PowerManagerService: Adds a VrStateListener to track Vr Mode state changes." into nyc-dev

Merge "Move setup to earlier boot phase." into nyc-dev

Merge "Audio ENCODING_IEC61937: enforce use of CHANNEL_OUT_STEREO" into nyc-dev

Merge "Account for top and bottom padding in ListPopupWindow" into nyc-dev

Merge "Handle unbounded drawPaint/drawGLFunction operations safely" into nyc-dev

Ignore wildcard network policies.

They never should have been persisted to disk in the first place.

Bug: 24972775
Change-Id: If9254f74455823f9fda9545fdf543fc7250a4016

Move setup to earlier boot phase.

Bug: 27764094
Change-Id: I2a01b47129a3ef2a4ae37fbf98eb9ebe81061e62

Skip app widget config changes when user locked.

Bug: 27377050
Change-Id: Ic63b06159cf59106b4e7bc62ec5f3e89d47321c5

Merge "Don't copy Bitmaps when cloning RemoteViews" into nyc-dev

Check "Secure start-up" before toggling emulation.

To fully emulate FBE we need to avoid the Cryptkeeper bounce, so
check to make sure user has disabled "Secure start-up" so vold can
unlock early during boot.

Bug: 27595501
Change-Id: Ic0d03f94982534381671e1039f14d1fd06bc0e4e

Merge "Add guard around getXmlFileParser() call" into mnc-ub-dev
am: 8662173  -s ours

* commit '86621737fc1ba2d6ce7cb97e61fc8ce186eb851a':
  Add guard around getXmlFileParser() call

Merge "Add guard around getXmlFileParser() call" into mnc-ub-dev

Fix string equality comparison

Don't use "==" to compare strings.

Bug: 25624963
Change-Id: Id25696e4fdcbcf4d48ec74e8ed65c1a33716b30c

Merge "Update A11y IME switch button content description." into nyc-dev

Mark even more Bundles as defusable.

Bug: 27766193
Change-Id: Ib027ac7b40c7a575a76f289faabde9655338865e

Merge "Correct naming issues with video calling carrier config items." into nyc-dev

Merge "Fix deadlock when full data backup times out" into nyc-dev

Merge "Work-around paletted image decoding bug in SDK_JELLYBEAN" into nyc-dev

Merge "API review: Hide ComplexColor and GradientColor" into nyc-dev

Added support to specify animation duration when resizing stack

Needed for sys-ui to control the duration of various Pip transitions.

Bug: 27674339
Change-Id: I7bad27aaa19755a73c594e88b88b56db033e1a45

Audio ENCODING_IEC61937: enforce use of CHANNEL_OUT_STEREO

Prevent developers from using a channelMask that will fail.

Bug: 27721209
Change-Id: I87ad67caedaedf6ed2230165cb1687a2a0b74703
Signed-off-by: Phil Burk <philburk@google.com>

Merge "Clarify docs about event recycling." into nyc-dev

API review: Hide ComplexColor and GradientColor

b/27532267

Change-Id: I01582e067ec8f4897c34dcec0f7c14700c40e02b

Merge "Disallow guest user from changing Wifi settings" into nyc-dev

Fix deadlock when full data backup times out

The code was attempting to let a reported error in the app <-> engine
surface take precedence over apparent success at the engine <-> transport
handoff surface.  However, in the case of timeout, this is inappropriate.
It was leading to deadlock because the engine runs free, with socket-closed
as its shutdown signals for determinism.  In this case that means that
having accidentally asked it to finish and report the final result, we
locked up forever since the data it was writing dutifully to the engine
was no longer being consumed, and the actual teardown signals were never
sent.

The fix is to properly express the error-state hierarchy: only when the
engine <-> transport layer is not issuing its own abort is the app-data-
moving layer consulted about errors detected at that surface.

Bug 22348852

Change-Id: I8987be0c4f708116dfeb08098d7222241ed317f3

Account for top and bottom padding in ListPopupWindow

Bug: 27550635
Change-Id: Icbd3190c35dfeb915cfb9f0b08a2f9d2b1307b24

Don't copy Bitmaps when cloning RemoteViews

Bug: 27385859
Fixes: 27385859

Change-Id: I9f0768b20e6e71d5366eb21df76c984bfe6989b1

Set drawable callbacks last

Bug: 27554777
Change-Id: I3ef9981bfafcb2a674304c9da10e4f5b1e66f932

Merge "Document the cache bitmap behavior for VD" into nyc-dev

New Linkify.addLinks function with set of known schemes

A new addLinks function is added that accepts a default scheme and a
set of known schemes. Default scheme is applied whenever the link found
does not start with one of the given known schemes.

Moreover, previously JavaDoc for addLinks functions with a single scheme
parameter described that the scheme attribute will be prepended to the
links that do not have 'a' scheme. The code actually checks if the link
starts with the given scheme and prepends if not. JavaDoc updated
accordingly.

Bug: 26985901
Change-Id: I94ea81dcf83ba7a6b6cd47c10fe8fb277964eb15

Ril Error codes: Voice related messages
am: 499a8da

* commit '499a8da1daa82f29eecd59f99721a1cdd52db716':
  Ril Error codes: Voice related messages

Clarify docs about event recycling.

Bug: 27454838
Change-Id: I65d0ced3198dc6ad45b99e4932a0d6e32306cf10

Ril Error codes: Voice related messages

This change adds new error codes for messages
related to voice

Bug: 27038563
Change-Id: Id3a212c0b12e1feb26b0836fad3f9399d6dec19c