Merge "Installd: Move app image A/B artifact" into nyc-mr1-dev

Merge \"Explicitly sets dumpsys timeout.\" into nyc-dev
am: 12af7d51a1

Change-Id: Ieab6b2ad648ff58b3bee7ffc8d4fbc3cfb516225

Merge "Explicitly sets dumpsys timeout." into nyc-dev

Installd: Move app image A/B artifact

In addition to odex, also move art file.

Bug: 25612095
Change-Id: I69f4c2b1e7283c9a9c361f57f5233e66679c5c28

Merge "Installd: Change app image name computation" into nyc-mr1-dev

Explicitly sets dumpsys timeout.

The run_command() function takes a timeout as argument, and dumpsys can
also optionally takes a timeout with the -t argument, hence it makes
sense to pass the same value for both (otherwise dumpsys will timeout in
10s).

BUG: 29042642
Change-Id: Ib6e987d94e966686bab71223d7f910af9e549814

Merge "libagl: Switch from gralloc to GraphicBufferMapper" into nyc-mr1-dev

Merge \"Attach profiles to bug reports\" into nyc-dev
am: 6d98bac4ab

Change-Id: I14404b7188e7be6c426bd550868badc3a8a91a07

Merge "Attach profiles to bug reports" into nyc-dev

Attach profiles to bug reports

Includes the entire '/data/misc/profiles' folder in the bug report
ZIP file if not user build.

Bug: 28610953
Change-Id: I99d46f024e5557cf94e6045a51c46a605a00c52b

Installd: Fix bad merge

Don't cut off the leading dex location character.

Bug: 25612095
Change-Id: Iad556a0bb48c57f3c68ee75ee41f14e88d96b1a8

Installd: Change app image name computation

Refactor computation to take A/B OTAs into account.

Fix a const issue.

Bug: 25612095
Change-Id: I5cf79def532d1eeec4b35bb80014376959083a83

Installd: Refactor dexopt to scoped file descriptor

Refactor the goto-fail cleanup to a unique_fd-like wrapper that
includes a potential cleanup step. In preparation for changes for
A/B OTA.

Bug: 25612095
Change-Id: If6cca85c12e0951bc468cb4f212b2f2d288c6041

Merge "Installd: Remove string-helpers" into nyc-mr1-dev

Merge "SurfaceFlinger: Add mode to apply position with resize." into nyc-dev
am: c3da482bb8

* commit 'c3da482bb8081ee2c19b5965585267b8e21cfc7e':
  SurfaceFlinger: Add mode to apply position with resize.

Change-Id: Icedb7a6e37da9338b1194edaa32a67257640969f

Merge "SurfaceFlinger: Add mode to apply position with resize." into nyc-dev

Merge "Fix Parcelable::readParcelableVector<T>" into nyc-dev
am: 0950fb3d20

* commit '0950fb3d20d100f62961db1cc3eff51b51f9ca6b':
  Fix Parcelable::readParcelableVector<T>

Change-Id: I892d88cfe9fe2df69bf390ce0618c03980b52753

Merge "Fix Parcelable::readParcelableVector<T>" into nyc-dev

SurfaceFlinger: Add mode to apply position with resize.

For some cases, like scaled windows with shadows,
we need to be able to apply the position concurrent with
window resize. This is because the scaling of the shadows causes
the top left coordinate of the non shadow surface content
to change before and after the resize.

Bug: 28899837
Change-Id: I522eacfbbcd79707dc1e5ab71901a263b3004ba9

Installd: Remove string-helpers

Replace string-helpers with equivalent functions from android::base.

Bug: 25612095
Change-Id: I186461b94dffe3590c16455d4dee0c2cbe921bf4

Merge "Enable profman pretty printing" into nyc-dev
am: 572e23d761

* commit '572e23d7614485a06102b00667d003b21b874bf9':
  Enable profman pretty printing

Change-Id: Ib2e450b89b2805de265c44b670ab414af1b8e6ac

Merge "Enable profman pretty printing" into nyc-dev

Merge "SF: Also edit currentState in doTransaction" into nyc-dev
am: d224e6111b

* commit 'd224e6111bf4532fb1c1fc021c20285b38bc80ec':
  SF: Also edit currentState in doTransaction

Change-Id: I6bcd675d033cc1ef46d3498174b2bc52a0072f99

Merge "SF: Also edit currentState in doTransaction" into nyc-dev

Enable profman pretty printing

Bug: 28748264
Change-Id: Ia625a7976d9b95ec3b7e1d03af39af9f39015f3b

Merge "libgui: Check result in hook_dequeueBuffer_DEPRECATED" into nyc-dev
am: 09e5f59bc9

* commit '09e5f59bc9a371c7b83153bb51543ec759684227':
  libgui: Check result in hook_dequeueBuffer_DEPRECATED

Change-Id: If91c49dfef52a64c3721da737658cf4645c13fc1

Merge "Allow setting alpha to 0 when creating native client buffers." into nyc-dev
am: 02afa28ccd

* commit '02afa28ccdb2cd82a51f938777944c2c8cf857ce':
  Allow setting alpha to 0 when creating native client buffers.

Change-Id: I811af889ed50ebd0b6dd0a676f106a2b78589f26

Merge "libgui: Check result in hook_dequeueBuffer_DEPRECATED" into nyc-dev

Merge "Allow setting alpha to 0 when creating native client buffers." into nyc-dev

libgui: Check result in hook_dequeueBuffer_DEPRECATED

Surface::hook_dequeueBuffer_DEPRECATED calls c->dequeueBuffer without
checking the returned result value.  It should check for failing
result and return on error without using the values that
c->dequeueBuffer did not set.

Bug: 28692406
Change-Id: I10f9a23ad1052c5149040124366e0eb91b616267

libvulkan: reduce log level for swapchain usage errors
am: 42a9eecb23

* commit '42a9eecb23c0b72bbee1eb2ef6b0d6586159d1c3':
  libvulkan: reduce log level for swapchain usage errors

Change-Id: I8b053f14678f9e03497ac2b621e117df581d9779

SF: Also edit currentState in doTransaction

Bug 28899837

Change-Id: I46c8301549c627c5d1c7efa661055aba787a33d6

Allow setting alpha to 0 when creating native client buffers.

Note that RGBA sizes are checked later in the function anyway, so there was no
reason to disallow 0, but it did preclude setting alpha to 0 for 565 buffers.

Fixes: 29126799

Change-Id: Iba11649976221ab385227fc022ae74b14c342203

Fix Parcelable::readParcelableVector<T>

Parcelable::readParcelableVector<T> was apparently never
used. It now is and needed some adjustments to actually
work.

Bug: 22914603
Change-Id: I58ead3b3956299d549cff245588f56f3220563d3

libvulkan: reduce log level for swapchain usage errors

Bug: 26927424
Change-Id: Id9346356436663db83a2d6fa8fefc4ea24d7854b

Merge "Otapreopt: Fix after shared-library work" into nyc-mr1-dev

Otapreopt: Fix after shared-library work

Commit b63d91fd2737680351876406277b6c759f4db33c broke the installd
to otapreopt connection. Fix this. Also refactor a bit to make an
inadvertant break harder.

Bug: 25612095
Change-Id: I8e7b944ecbd5457e4ac4d6ffffdb0468618cf65f

Merge "libvulkan: Slightly better handling of swapchain re-creation" into nyc-dev
am: b2dcc3a122

* commit 'b2dcc3a1220ea5ef6acb5a63659da3c34b1a295a':
  libvulkan: Slightly better handling of swapchain re-creation

Change-Id: I0f66b9e5521388eaf95ce29f5c1f44817780a7b9

Merge "libvulkan: Slightly better handling of swapchain re-creation" into nyc-dev

Merge "vulkan: remove EnumerateDeviceLayerProperties internally" into nyc-dev
am: 691fcb9603

* commit '691fcb96033db2f09030e8c71e86dad7e2fbfd43':
  vulkan: remove EnumerateDeviceLayerProperties internally

Change-Id: I4678a10c62147c7c69bf0f0e81e6e447ed819b2c

Merge "vulkan: remove EnumerateDeviceLayerProperties internally" into nyc-dev

Merge "Camera: Add media/camera NDK documents" into nyc-dev
am: 470241b168

* commit '470241b168a9db7514bc0df775109db591836d4b':
  Camera: Add media/camera NDK documents

Change-Id: Ib66856fba84301834a81f0c361f5244bd152eeb3

Merge "Camera: Add media/camera NDK documents" into nyc-dev

libvulkan: Slightly better handling of swapchain re-creation

Previously we would fail vkCreateSwapchainKHR if
VkSwapchainCreateInfo::oldSwapchain was used, because we were unable
to dequeue all buffers for the new swapchain from an already-used
native window.

Now we disconnect and re-connect to the native window in order to
fully reset state, and allow us to dequeue all buffers. Additionally,
we tag the old swapchain as out-of-date, so future image acquires and
presents will fail with VK_ERROR_OUT_OF_DATE_KHR on that swapchain.
This is less than ideal, but better than what we had, and the best we
have time to do for N.

Bug: 26927424
Change-Id: Ifaa5048376f72a63ecb1dca3d1ff85dbee2c24d0

Merge "SF: Re-enable fence tracker by default" into nyc-mr1-dev

Merge "BQ: Don\'t return fences from dequeue in shared buffer mode" into nyc-dev
am: eb6377eb13

* commit 'eb6377eb133f902e737fab370df7679430a1d8d1':
  BQ: Don't return fences from dequeue in shared buffer mode

Change-Id: Ieb075d9598edcaec542ee06663aa292111968474

Merge "Push empty sync transaction before taking screenshot in tests" into nyc-dev
am: 08588dd760

* commit '08588dd76073a8635aa4175d5b98fc5db86afec4':
  Push empty sync transaction before taking screenshot in tests

Change-Id: Id7648a55288f3f8962e6e314584c9690c3f27854

Merge "BQ: Don't return fences from dequeue in shared buffer mode" into nyc-dev

Merge "Push empty sync transaction before taking screenshot in tests" into nyc-dev

Merge "Create primary profile file together with app data folders" into nyc-dev
am: d25aab3ce8

* commit 'd25aab3ce8aab1b95ac60922981c87c6963a0a6b':
  Create primary profile file together with app data folders

Change-Id: Id186b701d0cc755be0093c704ccd2aa8e0899611

Merge "Create primary profile file together with app data folders" into nyc-dev

vulkan: remove EnumerateDeviceLayerProperties internally

We never use it.  Since it is also deprecated, we will never need it.

Bug: 27910383
Change-Id: If4e556bb69890153be2987b165482e43ab06a890

Push empty sync transaction before taking screenshot in tests

Because of ag/1043009 it's now necessary to push an empty sync
transaction before taking a screenshot to ensure that all transactions
have been applied.

Bug 29074285

Change-Id: I9f1b4a5b6b1f39e1ff26d536817aef6f40865177

SF: Re-enable fence tracker by default

- Don't check whether the fences have completed or not unless the data
  has been requested.

Bug 22431447

Change-Id: Id4cb8bfba38547884849ef23c9b6f42f0d4d0750

installd support for profman --dump-info-for
am: 6727c2d0bf

* commit '6727c2d0bfc876c62d6ee60d8a05d26bef1d56ef':
  installd support for profman --dump-info-for

Change-Id: I3df4eee3cb34f6c163e73351180b227b8d6864e8

BQ: Don't return fences from dequeue in shared buffer mode

Bug 29007505

Change-Id: I112ae6bd64ddc9a923ad9080e85a71adb96146af

installd support for profman --dump-info-for

Bug: 28748264
Change-Id: Idcd3ed86294c1d0c75c26ca938b88744a25e387a

Merge "svcmgr: don\'t send reply to one-way txn (part deux)" into nyc-dev
am: e4ecdb66d5

* commit 'e4ecdb66d5c2eed64cbd5b267e0621cab7b8a63b':
  svcmgr: don't send reply to one-way txn (part deux)

Change-Id: Id062b1b3998ab018654bc7830f275d1ac6d64f6d

Merge "svcmgr: don't send reply to one-way txn (part deux)" into nyc-dev

Use explicit path for /vendor/xbin/wlutil
am: 0816520c5c

* commit '0816520c5cd60519d8e221ed92497aa5464e3039':
  Use explicit path for /vendor/xbin/wlutil

Change-Id: I1fd9593c8d9c0b0e3dd87c9a70b6b2e81c9b100a

Use explicit path for /vendor/xbin/wlutil

At this time /vendor/xbin is not in root's path.

Bug: 28850734
Change-Id: Ib6c2b6f8b0285854670c843dcc437cfcc84484a8

Merge "vulkan: fix crash when only non-global layers are enabled" into nyc-dev
am: a101f6a446

* commit 'a101f6a446722697e0ed6afb128829937b90d60a':
  vulkan: fix crash when only non-global layers are enabled

Change-Id: I65baa95e0b00f3fbc136d435c4481256a7e551bd

Merge "vulkan: fix crash when only non-global layers are enabled" into nyc-dev

Camera: Add media/camera NDK documents

Include media/camera ndk headers to codegen source.
Add docs/images directory for image links in generated documents.

Bug: 27102995
Change-Id: I9c75e2ceba3458c9962c26cf786e1402fd963696

Add "ip maddr" to dumpstate output for debugging.
am: f7f2b0fd05

* commit 'f7f2b0fd059bc26e27c022f3fb446fc74eab6c88':
  Add "ip maddr" to dumpstate output for debugging.

Change-Id: Ic3898db87bc7b726079d922ff24eaa1f8201f288

vulkan: fix crash when only non-global layers are enabled

We fail to set up get_instance_proc_addr_ and get_device_proc_addr_ when
there are layers but only non-global ones.

Bug: 28990561
Change-Id: I82704f946deb5a76640f9dfb792e6fd118c78aa7

Add "ip maddr" to dumpstate output for debugging.

Bug: 28966611
Change-Id: I8fdab94fbd41448e52f2fed406e8f43edf7f18ad

Merge "Check the existence of raft dir." into nyc-mr1-dev

svcmgr: don't send reply to one-way txn (part deux)

Do not split transaction into two when sending synchronous responses.

Partially revert changes associated with binder_send_reply from
previous incantations in commit 51f592c7bad13d3d8e8f7cfec25edfef42901802
that resulted in a performance regression.

Signed-off-by: Mark Salyzyn <salyzyn@google.com>
Bug: 27111612
Bug: 28696050
Bug: 28946245
Change-Id: I12edc10a2d1408a71507a7983afa7f17b5a6a81b

Mark the multiwindow divider as a trusted overlay.
am: b9be0cf397

* commit 'b9be0cf397eb6053fb5b472d91cf2d6a22c6693f':
  Mark the multiwindow divider as a trusted overlay.

Change-Id: I1906ca39ce35a436ba65c3815e0bdda8a3c00e9e

Mark the multiwindow divider as a trusted overlay.

Having it be untrusted breaks the permissions granting dialog in
multiwindow mode.

Bug: 28828041
Change-Id: I9193dbce7315ebaa205673edd4f406d0b72665c8

Create primary profile file together with app data folders

Bug: 28785119
Bug: 28625993
Change-Id: I12c0de85bc3dc35199acf32c10fe69f93e76d7fc

Check the existence of raft dir.

b/28430346

Change-Id: I6ceb54f60c766d7b73ae93daa81f57b5a229e291

Merge "Write compressed and encoded raft logs to stdout if not zip_writer" into nyc-mr1-dev

Write compressed and encoded raft logs to stdout if not zip_writer

Change-Id: Ib9948bebf9d5e2ef823e08b48fb67c36e0079a45

Merge "BufferQueue/SF: Add OccupancyTracker" into nyc-mr1-dev

Merge "SF: Force empty sync transactions to apply anyway" into nyc-dev
am: 7443c1f253

* commit '7443c1f253008acd868af1e35449dd5eb3139c8d':
  SF: Force empty sync transactions to apply anyway

Change-Id: Iaa91350ba6e499d145e7adfcc1874601fff33775

Merge "SF: Force empty sync transactions to apply anyway" into nyc-dev

SF: Force empty sync transactions to apply anyway

Following the change to not flush transactions in the screen shot
path, this enables the use of empty synchronous transactions as a
flush mechanism for previous asynchronous transactions.

Bug: 27098060
Change-Id: I282fdfffb72db148edcbf1ded65b15816fc714a2

Merge "SF: Remove forced transaction on screenshot" into nyc-dev
am: 854071a38f

* commit '854071a38f250bd0638f1dfdb99b9ccc0010e7d0':
  SF: Remove forced transaction on screenshot

Change-Id: I79297918957661befdec24345bdfff2720433188

Merge "SF: Remove forced transaction on screenshot" into nyc-dev

Merge "HWC2: Fix breakage from header change" into nyc-dev
am: 80bc52aefa

* commit '80bc52aefa56e590f05bc30ff0182d860f00b128':
  HWC2: Fix breakage from header change

Change-Id: I02f8b45ded3d00dc2d34689f5b902d77ee74ca1e

Merge "HWC2: Fix breakage from header change" into nyc-dev

stagefright: define mpeg2 HighP level constant
am: 9e332babb9

* commit '9e332babb92d329c3c9b2e58fef77252cc11e52b':
  stagefright: define mpeg2 HighP level constant

Change-Id: I4db62224022bccc89ee4fdda2e32813a1f1c074d

stagefright: define mpeg2 HighP level constant

Bug: 28671284
Change-Id: Ic97fd9c3cb4ce04f9cfcd18a4daf68db3349694b

vulkan: advertise VK_EXT_debug_report
am: 319382517a

* commit '319382517afe913285df5bbf76a8a484e6adfbd0':
  vulkan: advertise VK_EXT_debug_report

Change-Id: I904abeac1234e146f73b14a817c42adc9b6bc7bb

vulkan: add a wrapper for hwvulkan_device_t
am: 31b2e4f01d

* commit '31b2e4f01d31aae107340cbc4221b8d49fb42767':
  vulkan: add a wrapper for hwvulkan_device_t

Change-Id: I4d64d13427218c437babb31d85982c438ed7f2af

vulkan: advertise VK_EXT_debug_report

We use VK_EXT_debug_report to report most messages and we should always
advertise the extension.  This is complicated however because the HAL may
also advertise the extension.

Do the minimal and cache whether the HAL supports VK_EXT_debug_report in
class Hal.  More refactorings will be required if we want to be more
flexible.

Bug: 28753291
Change-Id: I0298b2c3024e2ed059411128dbbe264c0b289a4b

vulkan: add a wrapper for hwvulkan_device_t

Add class Hal as a wrapper for hwvulkan_device_t.

Bug: 28753291
Change-Id: I9612ee5515c7f2266cfc1ef0c8583fa98c82dd86

Merge "vulkan: do not use exported functions internally" into nyc-dev
am: e6759c3d88

* commit 'e6759c3d886491f01dd2295f71d22609b3b9b259':
  vulkan: do not use exported functions internally

Change-Id: I138aa42e7098b69d00517d9bac3d811b9b56e8a7

Merge "vulkan: do not use exported functions internally" into nyc-dev

Resolve merge conflicts of 6108f5d to nyc-mr1-dev

BUG: 28867848
Change-Id: I99392f1706af5fb9268ad1b32aa7f4b10e397d84

Merge "Expose functions used to add entries to the zip file." into nyc-dev

Expose functions used to add entries to the zip file.

Currently only dumpstate.cpp is using the new functions and variables
used to add an entry to the zip file so they're defined locally, but
they should be exposed through the .h so they can be used by the
device-specific dumpstate_board() implementations.

This CL moves the following definitions to the .h:

- bugreport_dir
- ZIP_ROOT_DIR
- add_zip_entry()
- add_zip_entry_from_fd()

It also refactored functions that were taking a specific file location
to use bugreport_dir instead, to avoid pollution on main().

BUG: 28867848
Change-Id: Ic5faa203b0863bc950d640f22fb91977e52b6bfc

omx: add asString for new enumerations and fix some issues
am: 1b9c8069f4

* commit '1b9c8069f423f25c4dc166618d9fbe8d94d7f7c0':
  omx: add asString for new enumerations and fix some issues

Change-Id: I774af98ced6dd8bf7aac8873e3d93655fd5c6f2a

omx: add asString for new enumerations and fix some issues

Bug: 28671284
Change-Id: I54e98d727f974544d51b2a14b565ad24912ab242

Merge "omx: add missing MPEG4 levels" into nyc-dev
am: 6472c8c660

* commit '6472c8c660232300a0d2bf8721ec028a2db229a9':
  omx: add missing MPEG4 levels

Change-Id: If2a70773c6fcc4975de4a6fbc161e544f5d40715

Merge "omx: add missing MPEG4 levels" into nyc-dev

vulkan: do not use exported functions internally

Our vkGet*ProcAddr uses the exported functions.  They will break when any
of the exported functions are overridden (e.g., through LD_PRELOAD).

Unexport and move all exported functions to vulkan::api namespace.
Re-export them by having vkFoo as a wrapper to vulkan::api::Foo.

Another option is to re-export vulkan::api::Foo by having vkFoo as an
alias using __attribute__((alias)).  That results in smaller binaries.
But we will not be able to catch mismatches between vulkan.h and
vulkan.api.

To avoid future breakage, define VK_NO_PROTOTYPES for all files except
api_gen.cpp.

Bug: 28886971
Change-Id: I08fde7ebb247f8c7e040ccf812b40b02094d3c7f

Merge "Ensure memory ordering around libagl and EGL refcount operations" into nyc-dev
am: bac438bbb6

* commit 'bac438bbb682b09b618f47f2e36628a464afc8ea':
  Ensure memory ordering around libagl and EGL refcount operations

Change-Id: Ie2643fd243a9aa66080457d318a35550c502fd67