Merge "Destroy the reference profile during uninstalls" into nyc-dev
am: 057facf

* commit '057facfdc6b512b4fe43ee19db50b5156cecff84':
  Destroy the reference profile during uninstalls

Change-Id: I1754a78379f46c499e7d54ef011545bb72df5e62

Merge "Do not unlink the profiles if not needed" into nyc-dev
am: 4820e72

* commit '4820e72dbfccad5974dc3d88f2cb462af35c6659':
  Do not unlink the profiles if not needed

Change-Id: I51ba64a994108143805e4d348ccfedfba5251f9e

Merge "Destroy the reference profile during uninstalls" into nyc-dev

Merge "Do not unlink the profiles if not needed" into nyc-dev

Call _exit() instead of exit() on child failure.
am: baa85bd

* commit 'baa85bd893665a48aee73d7ca7b73cd39b48f998':
  Call _exit() instead of exit() on child failure.

Change-Id: I7f78e1f2550cf58467c1a191b8a0c9668277ad1d

Call _exit() instead of exit() on child failure.

As explained on man _exit:

"The  function  _exit()  terminates  the calling process "immediately".
Any open file descriptors belonging to the process are closed;"

If exit() is called instead, the resulting zip file will be corrupted.

BUG: 27900023
BUG: 27804637
Change-Id: I6591749603753d6016f350c629b7082cf6780ae2

Destroy the reference profile during uninstalls

Bug: 27896463
Change-Id: I6b182e2c3276f6228a2ed70ee2c6f0c49c638760

Do not unlink the profiles if not needed

It's enough to just clear the profiles. The app might still run and we
will see misleading warning or we might miss recording profile
information.

Bug: 27895342
Change-Id: I04961f09e0965a3e4af87acb95a7c74773f7a44a

Merge changes I653b84e7,Ieafdc14a into nyc-dev
am: abddf72

* commit 'abddf727041d2ea0e678442b2d7905afc8b33bd4':
  Apply position updates immediately.
  Move crop outside of geometry state.

Change-Id: I7d4b9d49f6807748592c32432e4ab80da59bad37

Merge changes I653b84e7,Ieafdc14a into nyc-dev

* changes:
  Apply position updates immediately.
  Move crop outside of geometry state.

Merge "Define typical sensor sample rate in native code" into nyc-dev
am: a84b43c

* commit 'a84b43c914cdd0e665d7555f5800d5c06b895716':
  Define typical sensor sample rate in native code

Merge "Rename single buffer mode to shared buffer mode" into nyc-dev
am: 59d12e9

* commit '59d12e90ae09171fc8b50fd8fd02e3247c04840c':
  Rename single buffer mode to shared buffer mode

Merge "Dumping a trace isn\'t an error" into nyc-dev
am: 9767bd8

* commit '9767bd89a6f5aa510c649d03e485c086d4ebd614':
  Dumping a trace isn't an error

Increased estimated max progress of bugreports.
am: faf67e3

* commit 'faf67e3fe2451e75d0369906a579d3e1386a5064':
  Increased estimated max progress of bugreports.

resolve merge conflicts of 1ccaed4 to nyc-dev-plus-aosp

Change-Id: Ib30c59ff496ed76ec46f4c8f7d954d571bcbf4ae

Merge "Define typical sensor sample rate in native code" into nyc-dev

Merge "Switch GLES wrappers over to using Clang (and fix inline assembly)."
am: e9aeab8

* commit 'e9aeab858e950fa33062ebabe43ddeb00c08d2f4':
  Switch GLES wrappers over to using Clang (and fix inline assembly).

Merge "Address const issues in preparation for libcxx rebase."

Apply position updates immediately.

Restores the behavior prior to moving transform
in to geometry state. Now geometry state and scaling
mode only control parameters which would cause
buffer scaling.

Bug: 27729195
Bug: 27687126

Change-Id: I653b84e74407f2533c92f7647e2609fc043ed0a4

Rename single buffer mode to shared buffer mode

Change-Id: Id43d0737d9367981644e498942ebc0077d61038c
(cherry picked from commit 3559fbf93801e2c0d9d8fb246fb9b867a361b464)

Move crop outside of geometry state.

Prior to this CL, if Layer crop is changed while a
surface resize is pending, the crop will not be
applied until a buffer latches at the new size. This
CL makes the crop apply immediately. We can see this
is the desired behavior by looking at the two cases
where a resize is pending.

1. A resize is pending to make the surface smaller.
  In this case we need to be able to immediately update the crop so that
  we can shrink the visible region of the surface at an interactive rate
  with input. The window manager currently uses big surfaces and scaling
  modes to accomplish this.
2. A resize is pending to make the surface larger.
  In this case it doesn't matter. If we expand the crop immediately to
  the new surface size, then we have simply uncropped an area which is
  by definition transparent pixels (as we haven't latched a buffer
  at the new size yet).

This change has conceptual soundness as well. We can see that the
scaling mode will not affect properties that affect scaling (transform,
width/height) but not properties which do not (crop).

Bug: 27729195
Bug: 27687126
Change-Id: Ieafdc14aeecb23085793e3056a746d6f344781df

Merge "Switch GLES wrappers over to using Clang (and fix inline assembly)."

Define typical sensor sample rate in native code

Porting typical sensor sample rate (e.g. SENSOR_DELAY_UI)
defined in Java API to native code.

Change-Id: I8b59216415bc952b4e953ca6e8058beb1dd3c645

Address const issues in preparation for libcxx rebase.

Change-Id: I25a1e2946217049e9205589d6493e06362d2173b

Merge "Rename single buffer mode to shared buffer mode" into nyc-dev

Merge "Dumping a trace isn't an error" into nyc-dev

Dumping a trace isn't an error

Change-Id: I6180c4f72b45b3d64c44a197e7bda3809a69c8fc

Increased estimated max progress of bugreports.

The initial estimate was based on earlier work done before new sections
were added, and it was too slow.

Ideally we should have a per-device estimate, but for N we'll just
increase the common max.

BUG: 26373682
Change-Id: I1a8ce7f09fc6588ac1513a20fa1149aab8c38351

Switch GLES wrappers over to using Clang (and fix inline assembly).

Bug: http://b/18003438

The GLES wrapper implementations make direct calls to their underlying
function without specifying that they are retaining the original
argument register(s). This change makes the retention of these argument
registers more explicit, and then removes the LOCAL_CLANG override,
since Clang no longer optimizes the code into a bad state.

We also switch to "naked" functions, so that we control all stack layout
information. This prevents us from accidentally still corrupting
arguments on x86 (and for other functions with many parameters).

Change-Id: I37f2ef4a697373388950b41f0a292393c9fcbac7

Don\'t report errors when trying to delete a non-existing profile dir
am: b06f98a

* commit 'b06f98aabc5381fd6366526d9b31b5d0345481b6':
  Don't report errors when trying to delete a non-existing profile dir

Don't report errors when trying to delete a non-existing profile dir

Also, put destroy_app_current_profiles behind FLAG_STORAGE_DE flag.

Bug: 27081617
Change-Id: Ib8b9ff292784c659259ceb4339b803e3580da7ed

dumpstate: Add /data/misc/recovery
am: d6ab011

* commit 'd6ab01105bbd80dfa2fc2debc8e31d6422c378ee':
  dumpstate: Add /data/misc/recovery

dumpstate: Add /data/misc/recovery

(cherry picked from commit 23e86e91ff971a00f62954833a17df2dad1a13af)

Bug: 27176738
Change-Id: Ibc2342c6a29e5aebcac4875d7c2eed48fa8af7de

Merge "Remove spurious log" into nyc-dev
am: 7ff357a

* commit '7ff357ab080d0f3330a5642124a352f745d816bb':
  Remove spurious log

Merge "Remove spurious log" into nyc-dev

Remove spurious log

Bug 26696080

Change-Id: I60ce2c1fac653d33a033c1e22f0df87466287fba

Merge "Guard against failed array allocation" into nyc-dev
am: 834fde8

* commit '834fde83f9a924f23e920306d9f7590fb6fd0e0b':
  Guard against failed array allocation

Merge "Guard against failed array allocation" into nyc-dev

Guard against failed array allocation

Also sanity check the fd count of incoming Parcels.  Anything
exceeding our actual rlimit is inherently not legit.

Bug 26696080

Change-Id: Ib983dce8e8977d7e33195a4ee471115cdaf1c56b

Merge "Do not clean profiles unconditonally during app data clean up" into nyc-dev
am: d1a6a24

* commit 'd1a6a2495ff82893728571136f4f080164c8dae5':
  Do not clean profiles unconditonally during app data clean up

Merge "Do not clean profiles unconditonally during app data clean up" into nyc-dev

Merge "SF: Add FenceTracker" into nyc-dev
am: 830f500

* commit '830f5004da53a83e23b4102ba9b72e4cb366cb34':
  SF: Add FenceTracker

Merge "SF: Add FenceTracker" into nyc-dev

Merge "Improved (or warned about lack of) error handling." into nyc-dev
am: a207fca

* commit 'a207fca4eda8c17389ea13ec3762ecb2a49d81be':
  Improved (or warned about lack of) error handling.

Merge "Improved (or warned about lack of) error handling." into nyc-dev

Improved (or warned about lack of) error handling.

It would be safer for dumpstate to exit when execvp on a child fails; a
common occurrence is when a list of command arguments is missing NULL.

dumpstate should be more robust to detect those missing NULL-terminated
args, but that will be addressed in a future change.

BUG: 27804637
BUG: 27832567
Change-Id: Ibcbe46041a86b16e365fbb40613b8c4bdf39744c

Merge "Manually fix GLES headers" into nyc-dev
am: a282c18

* commit 'a282c184366bf8b9520ee19fcc1aa7667db7db78':
  Manually fix GLES headers

Merge "Manually fix GLES headers" into nyc-dev

Manually fix GLES headers

Apply the Khronos proposed fix to the GLES headers

Bug 26832318

Change-Id: I3f476d4b8dc4e1819afb9e791da178417e5cb427

Do not clean profiles unconditonally during app data clean up

The goal is to enable profiles to be persisted across OTAs.

This CL makes profile removal a dedicated path which will be called by
PackageManger during an app update or uninstall.

Profiles are cleared when:
- requested to do a full clean up of application data files
- specifically requested to clear them (e.g. app updates)

Profiles are destroyed (includes profile directory removal)
- for each user individually when its application data is destroyed
- for all users if the package gets uninstalled.

Bug: 27081617
Bug: 27688727
Change-Id: Id53c7625011af3cdbe8a58ac0982982fa461e5fd

Merge "Installd: Separate profile merging and compilation" into nyc-dev
am: 950aa7c

* commit '950aa7c853ff43e95f0764f190d1721e1e82dea2':
  Installd: Separate profile merging and compilation

Merge "Installd: Separate profile merging and compilation" into nyc-dev

Merge "Fixed send_broadcast and fork handling issues." into nyc-dev
am: 8e48e0a

* commit '8e48e0aadae765b4ed104890ca33db32ea347a8a':
  Fixed send_broadcast and fork handling issues.

Merge "Fixed send_broadcast and fork handling issues." into nyc-dev

Fixed send_broadcast and fork handling issues.

send_broadcast() was calling run_command_always(char*, bool, int, char*) passing:
    run_command_always(NULL, 20, true, am_args);

I.e., it was passing "true" (!1) as the boolean, and 1 (true) as the
timeout value; the proper call should be:

    run_command_always(NULL, true, 20, am_args);

Also, the code handling failure on child calls was calling _exit(),
which quits dumpstate - it should simply return the function instead.

BUG: 27804637

Change-Id: If8c9b40eddc8b76f92e6d11078dfe446a39ad4d4

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd am: 5ad8d44 am: b38df7b am: 4e067d2 am: 72ca325 am: f6d2288 am: 8c1ee7b
am: 7bd2905

* commit '7bd290578888ae572e95e800357a64573b23f385':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd am: 5ad8d44 am: b38df7b am: 4e067d2 am: 72ca325 am: f6d2288
am: 8c1ee7b

* commit '8c1ee7bbecf3dd96c21ba4b0663bcddb25c0a287':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd am: 5ad8d44 am: b38df7b am: 4e067d2 am: 72ca325
am: f6d2288

* commit 'f6d2288e5893b17639dbd36338ffc4a84a184425':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd am: 5ad8d44 am: b38df7b am: 4e067d2
am: 72ca325

* commit '72ca3255ba44366d93a4e9bf3cbf1672086ef4c6':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd am: 5ad8d44 am: b38df7b
am: 4e067d2

* commit '4e067d26119e6b995b795eb2b0a921b24595ab47':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2 am: b4f0f33 am: e564a36 am: 1e2ce52 am: 3d38d6c am: 6e89bd1 am: e07748c
am: 0836986

* commit '083698611fdaa80c856959563c86732d42989fac':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd am: 5ad8d44
am: b38df7b

* commit 'b38df7b4e9d93686c62d9ec654bba3be6e561d56':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2 am: b4f0f33 am: e564a36 am: 1e2ce52 am: 3d38d6c am: 6e89bd1
am: e07748c

* commit 'e07748c9520d8a21bde134789c5b4087b02bffef':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd
am: 5ad8d44

* commit '5ad8d443776e74a56e0e461ae6d333cf52e4f591':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2 am: b4f0f33 am: e564a36 am: 1e2ce52 am: 3d38d6c
am: 6e89bd1

* commit '6e89bd1aa1844999dccbfa716da82cfef21d434c':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2 am: b4f0f33 am: e564a36 am: 1e2ce52
am: 3d38d6c

* commit '3d38d6ca5cc95f9ab82a8dba72065edeba527a62':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours
am: 162fecd

* commit '162fecdc2c98844faaa6cb43a4fbe854f511b077':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2 am: b4f0f33 am: e564a36
am: 1e2ce52

* commit '1e2ce52e97a27f5d9cd68c809f63e56847c658ac':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: ce156f6 am: a202396 am: df806c5 am: ecc7934
am: 4a16310

* commit '4a16310e7fe924d0451f49e4fba5c205575499cf':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17
am: 74d2c4b  -s ours

* commit '74d2c4b4c97dbbcf43a9f8870007593d61beb547':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2 am: b4f0f33
am: e564a36

* commit 'e564a36b5ca8ef3cacaebb4707b0e8e38b6d7d2d':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: ce156f6 am: a202396 am: df806c5
am: ecc7934

* commit 'ecc7934d98959a615a8d7cd6c685b7851e1f02f3':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2
am: b4f0f33

* commit 'b4f0f339c714be74cf0ae17e4c88620130b02121':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: ce156f6 am: a202396
am: df806c5

* commit 'df806c5c9a488f02d385e03bf6ec261bf8ffd879':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: ce156f6
am: a202396

* commit 'a202396ca27e8d5a658585861cfcd4277d978362':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder
am: 41e7b17

* commit '41e7b1780f106d2eb4304b1f9cf060ce44177cae':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder
am: ce156f6

* commit 'ce156f60adb32c7cbebe50e313f51e564f93fe99':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder
am: 12bafe2

* commit '12bafe205c236da50f34ed85adf62a5631525ab6':
  Fix issue #27252896: Security Vulnerability -- weak binder

Installd: Separate profile merging and compilation

Create a new merge_profiles command to check and merge profiles.

(Almost) always pass a profile to a dexopt pass.

Require compilation filter input to dexopt and use it for dex2oat
(except for hard overrides like safe-mode).

Bug: 27689078
Change-Id: I1257857cc15c17e2271d1261ea4cc80752270fcb

Fix issue #27252896: Security Vulnerability -- weak binder

Sending transaction to freed BBinder through weak handle
can cause use of a (mostly) freed object.  We need to try to
safely promote to a strong reference first.

Change-Id: Ic9c6940fa824980472e94ed2dfeca52a6b0fd342
(manually cherry picked and resolved conflicts from commit
c11146106f94e07016e8e26e4f8628f9a0c73199)

Merge "Flushes stdout when execvp fails." into nyc-dev
am: d97e18c

* commit 'd97e18c038f23cecbe3d5e95fabf4dcd2bfaac20':
  Flushes stdout when execvp fails.

Merge "Flushes stdout when execvp fails." into nyc-dev

Flushes stdout when execvp fails.

BUG: 27804637
Change-Id: Idf414a1d08bf51854475c74767588a4814c8d806

Merge "dumpsys: fix error condition" into nyc-dev
am: eff45d0

* commit 'eff45d06902c2a9dc523848a652d2456dfa19d7a':
  dumpsys: fix error condition

Merge "dumpsys: fix error condition" into nyc-dev

dumpsys: fix error condition

Bug: 27809529
Change-Id: Ia2bdac83a367e1dd550ea7a5b50eac14d84f66b2
Signed-off-by: Thierry Strudel <tstrudel@google.com>

Merge "Handled failure of execvp and increases am timeout." into nyc-dev
am: 9b02ecf

* commit '9b02ecfe308302eae98d50fa165b1faae695f3df':
  Handled failure of execvp and increases am timeout.

Merge "Handled failure of execvp and increases am timeout." into nyc-dev

Handled failure of execvp and increases am timeout.

The run_command_always() call fork and runs the command in the child
code, and user to have the above logic:

    if (pid == 0) {
        // boiling code
        execvp(command, (char**) args);
        // execvp's result will be handled after waitpid_with_timeout() below...
        _exit(-1); // ...but it doesn't hurt to force exit, just in case
    }

Recently, the _exit(-1) line was removed and there's been reports of
dumpstate hanging after, because it tries to wait for pid 0 to
finish (and in some cases it even tries to kill it).

Also increased the timeout when running am to send broadcasts.

BUG: 27804637
BUG: 27809534
Change-Id: Ia129253e367e5886d305947e56a643fd660a33bd

Merge "Add final crop implementation" into nyc-dev
am: 41371bf

* commit '41371bfa0dad4a4332a8cc30b066d95546e4c4fc':
  Add final crop implementation

Merge "Add final crop implementation" into nyc-dev

Merge "Delete the entire profile directory when the app is uninstalled" into nyc-dev
am: 5b81754

* commit '5b817549fcea6f58bbeaaffbf654ddd8751ebbdc':
  Delete the entire profile directory when the app is uninstalled

Merge "Delete the entire profile directory when the app is uninstalled" into nyc-dev

Merge "sensorservice: Strengthen compiler warnings." into nyc-dev
am: 2376b4e

* commit '2376b4e1cb9a24d0d7dcd1d57458261610818ef5':
  sensorservice: Strengthen compiler warnings.

Merge "sensorservice: Strengthen compiler warnings." into nyc-dev

Merge "media: add HDRStaticInfo and DescribeHDRStaticInfoParams" into nyc-dev
am: b706c05

* commit 'b706c054f9aa3366c6c41f2ddaa985fb5ba6a00a':
  media: add HDRStaticInfo and DescribeHDRStaticInfoParams

Merge "media: add HDRStaticInfo and DescribeHDRStaticInfoParams" into nyc-dev

Merge "dumpsys: make timeout value an argument" into nyc-dev
am: ca9b292

* commit 'ca9b29213820ac399f9e2b9aac827377fa7e6622':
  dumpsys: make timeout value an argument

Merge "dumpsys: make timeout value an argument" into nyc-dev

dumpsys: make timeout value an argument

Also rework the args parsing using getopt_long.

Bug: 27788406
Change-Id: I60fd8e614616334d47b8de574f3c81a121a6ac0a
Signed-off-by: Thierry Strudel <tstrudel@google.com>