Don\'t optimize out rcUpdateColorBuffer
am: 03031766dd

* commit '03031766dd3978edbb4b6e6d39139d215036921c':
  Don't optimize out rcUpdateColorBuffer

Change-Id: I5d5db64d94139b8c99c37cd85ee5203ff4c5011b

Don't optimize out rcUpdateColorBuffer

bug:28406160

If we want to use rcUpdateColorBuffer to
synchronize on the host in order to remove
out of order webcam frames, we need to call it
if (cb->hostHandle), not just when
the width and height are nonzero.

Otherwise, we get deadlocks.

Change-Id: Ib98774b935e8786e307dd38711a0f2f9ffea440e
(cherry picked from commit 6298285500362b217b73b96def2ce207e206171d)

Merge "Fix const issues in preparation for libcxx rebase." am: c3aab193bf
am: d0347bc119

* commit 'd0347bc119c08c49db485cb0786372e0c2a7902d':
  Fix const issues in preparation for libcxx rebase.

Change-Id: Icdc695f2c42ec0ed97610c31055d1c643aca7b3d

Merge "Fix const issues in preparation for libcxx rebase."
am: c3aab193bf

* commit 'c3aab193bf45fb871e04c0b2096e088b3a2089bb':
  Fix const issues in preparation for libcxx rebase.

Change-Id: If925b6590d99539e270dc96cf32244f3b1541609

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

Fix const issues in preparation for libcxx rebase.

Change-Id: I2a939c87af3ef9881d9c806ba374185bd31a4954

Create and read color buffer when expecting reads
am: ca3b20ff0d

* commit 'ca3b20ff0dd8fed5dae26409c255c68ce73723af':
  Create and read color buffer when expecting reads

Change-Id: I9493b13c8953ca04846d5cb94039608671daf1a0

Create and read color buffer when expecting reads

Make sure color buffers are created when gralloc is allocating buffers
that indicate that software reads are going to happen. In this case
software will require the rendered data to be available from the host at
a later point and that means that rendering has to be stored in a color
buffer. Also make sure that when gralloc locks a buffer for reading it
will actually read the data from the host if the buffer is marked for
reading.

BUG: 28932862

Change-Id: Ief4b7fe34401727cabec9fdd5c2b9812a6c49612
(cherry picked from commit 2637058369fe1ba1a692e5eb9da95624f1bc6f9c)

[phone] Set up the GPU caching parameters for emulator device
am: 6fc69c7d71

* commit '6fc69c7d7126a515f0f92b5da5d576e9c9d0cc4e':
  [phone] Set up the GPU caching parameters for emulator device

Change-Id: I1686aa81daf80a8003e0c5d206406de761332a1a

[phone] Set up the GPU caching parameters for emulator device

Default texture cache is way too small to be able to handle
large background images; this causes huge performance issues
in the emulator;
This CL tweaks the caching parameters to be on par with physical
devices

Bug=https://b.corp.google.com/u/0/issues/28752126

Change-Id: I1852038f7065cc4b00b425fdf5fc35afe4168ace

Emulator: fix fingerprint regression
am: ad525fea6e

* commit 'ad525fea6eae8c4614c7ae25f9e718084513248c':
  Emulator: fix fingerprint regression

Change-Id: Iec22776cfc97e003401be2257255599d831cf98f

Emulator: fix fingerprint regression

bug: 28421052

Now the authenticator_id means something.

Change-Id: Icfca3b672fe00b0c2e40ad58e1facdf2f60281dd

Delete /mnt/sdcard symlink
am: f7f9192

* commit 'f7f9192d71c19fa3c14339316000084f27d21063':
  Delete /mnt/sdcard symlink

Change-Id: Idaaf9182b82187114130ecf511fd313f0ba5da9c

Exit when QemuPipe is corrupted by GL
am: bb75ccc

* commit 'bb75cccc6426826294aee990a095b19060ee8883':
  Exit when QemuPipe is corrupted by GL

Change-Id: I60166c4161a5394e2edadca6b9f10a8f784dc6fa

Delete /mnt/sdcard symlink

Bug: 28108983
Change-Id: I259be7b9ba00950846674bd3b328deb0ba657f1a

Exit when QemuPipe is corrupted by GL

Previously when QemuPipe is corrupted, it prints an error message, and
the application will continue to use it, which leads to random behaviour
in GL on the host.

This patch crashes the APP and pops up the crash dialog when the pipe is
(going to be) corrupted. It would happen if the user asks GL to write /
read a NULL pointer (i.e. passing NULL to glGenBuffers). It might also
happen for other unknown reasons.

Change-Id: I303772029bb72a845fa0d4c79b0d38dba1fca2c7
(cherry picked from commit 5bc2615d233a2a5de39f1033be4f063d0ae8f5e4)

Merge "Workaround: disable checksum for a problem in glTexSubImage2D" into nyc-dev
am: 6dfd183

* commit '6dfd183133a94c6e38982fc2756cb79ed5b6eae5':
  Workaround: disable checksum for a problem in glTexSubImage2D

Change-Id: I4b60269382ad0bb6926acccb5c4b08830d3dcfbb

Merge "Workaround: disable checksum for a problem in glTexSubImage2D" into nyc-dev

Workaround: disable checksum for a problem in glTexSubImage2D

This is a temporary workaround for a problem in glTexSubImage2D. It
disables the GL checksum to avoid crashes. Please revert this CL when
the root cause is solved.

Change-Id: I8401449e42da87f580f119982a1bcb1b6b9a4c8b

Merge "Stop aggressive validation in glVertexAttribPointer" into nyc-dev
am: 3bf9ef4

* commit '3bf9ef45c19b85eec827cd8acef1bcdcf26f4182':
  Stop aggressive validation in glVertexAttribPointer

Change-Id: Ib5fff9b967f9de0efbab325d92d5de22de93d04f

Merge "Stop aggressive validation in glVertexAttribPointer" into nyc-dev

Emulator: Fix camera crash
am: d81ee62

* commit 'd81ee62349fc40c82200803077b7a9550d44fe19':
  Emulator: Fix camera crash

Change-Id: I28cd5c3445951944acf28033cd5b34f1729b7540

Emulator: Fix camera crash

Because the camera runs a thread without giving it a name.
bug: 28074344

Change-Id: I85fd0d0786e82305d6615e7f50b7394785e96df7

Stop aggressive validation in glVertexAttribPointer

This breaks Antutu 3D benchmark.

Change-Id: I72ef2fe25f841b851bcc947b707019d94c88a8e1

Merge changes from topic \'GL pipe checksum (nyc)\' into nyc-dev
am: 6f05412

* commit '6f054128fe33fd98d929c43aa4b8d8af630f73ad':
  Checksum for GLES messages from host to guest
  Checksum for GL pipe communication

Merge changes from topic 'GL pipe checksum (nyc)' into nyc-dev

* changes:
  Checksum for GLES messages from host to guest
  Checksum for GL pipe communication

Checksum for GLES messages from host to guest

Add checksum for GLES messages from host to guest. This patch contains
generated code (gl_enc.cpp, gl2_enc.cpp, renderControl_enc.cpp) and
shared code with the host (ChecksumCalculator.cpp,
ChecksumCalculator.h).

(cherry picked from commit eee828509ce3d8fac4f962a68e8d381e83223f8a)

Change-Id: Ie95128c50ad4536ecdcf5f075889313169d677d2

Checksum for GL pipe communication

This is the guest side implementation for cl go/oag/c/207292.

It initializes checksum whenever a renderControl is created. Each
HostConnection owns a checksum structure.

The list contains mostly auto-generated code by emugen.

(cherry picked from commit bfd420ab5f9e0704012b6796522ef78cd56a3ac4)

Change-Id: I2eacc654522f048a82a77803f71116ba58a39e86

Fix a typo in GLES encoder headers
am: eaf0c97

* commit 'eaf0c975eee97b004b32e1243fa6a49bc89c81fd':
  Fix a typo in GLES encoder headers

Fix a typo in GLES encoder headers

Unimportant but let's fix it.

Change-Id: I360ba0dc0b676238adaeb5d4360c10a9ec8c2371
(cherry picked from commit 10c23934975b8c053e1991af4e5af06c5d4a6573)

device/generic/goldfish/camera: Make Thread::run threadName argument required
am: 3d1bfd57c1

* commit '3d1bfd57c186daca1e06c362b0ba09729ddee88f':
  device/generic/goldfish/camera: Make Thread::run threadName argument required

device/generic/goldfish/camera: Make Thread::run threadName argument required

Bug: 27557176
Change-Id: If15d0574a53a7d48ac87d6d0865a43f3cc6debbc

fingerprint: Don\'t call notify on fingerprit_cancel.
am: 80b6241d91

* commit '80b6241d91920fa56270e29b132cbd113001004d':
  fingerprint: Don't call notify on fingerprit_cancel.

fingerprint: Don't call notify on fingerprit_cancel.

Fingerprint cancel doesn't expect notify to be called as a result of
calling fingerprint_cancel. In fact, doing so will cause it to crash
when fingerprint_remove is called.

BUG:27548519
Change-Id: Ib4467756f8128ab74754e358f3bef3966dcf6d9f

Merge "Settings: Show on-screen keyboard when hardware keyboard is attached." into nyc-dev
am: 9940731ed3

* commit '9940731ed3a495337093bc9ab57c9089bcbe4b3a':
  Settings: Show on-screen keyboard when hardware keyboard is attached.

Merge "Settings: Show on-screen keyboard when hardware keyboard is attached." into nyc-dev

Merge "Settings: Start with screen unlocked after boot." into nyc-dev
am: 01d7db0cc5

* commit '01d7db0cc50b4f0b735c70a7a620decfcc7a5378':
  Settings: Start with screen unlocked after boot.

Merge "Settings: Start with screen unlocked after boot." into nyc-dev

Settings: Show on-screen keyboard when hardware keyboard is attached.

Changes the default behvaiour of hiding the on-screen keyboard when a
hardware keyboard is attached. We want both for the users of emulator.

BUG: 27484734
Change-Id: I8c8f0183cbbe17d857dea8ed36a1855be95ab25a

Settings: Start with screen unlocked after boot.

This CL adds a product overlay for goldfish and overrides the default
property used by SettingsProvider to unlock screen on boot.

BUG: 2748734
Change-Id: I222a8b2781169603e33147c787692a893d32c1e0

Emulator: remove expensive glCheckFramebufferStatus
am: 4df5506384

* commit '4df5506384ebefc4c12b3cda389d95c228476a60':
  Emulator: remove expensive glCheckFramebufferStatus

Emulator: remove expensive glCheckFramebufferStatus

The additional calls of glCheckFramebufferStatus in glDraw*
functions have very negative impact on framerate.

More specifically, the glCheckFramebufferStatus will flush
the write buffer of qemu-pipe and then do a block read until
the host finishes all the gl calls before this call and send
back results. This turns out to be costly, probably partially
due to the relatively longer latency (than real GPU) we have
in the GPU emulation layer -- it would be nice to hammer
that latency down in the near future.

bug: 27418143

Change-Id: Ic6d9e739562c6bd0b688d62805fda4d469b98759

Cache glGetIntegerv(GL_MAX_VERTEX_ATTRIBS, int*).
am: a31a031015

* commit 'a31a0310154660e3105ab6a1955d3667efda3630':
  Cache glGetIntegerv(GL_MAX_VERTEX_ATTRIBS, int*).

Cache glGetIntegerv(GL_MAX_VERTEX_ATTRIBS, int*).

BUG:27418143

If we don't, the chatter over the pipe will
kill framerate.

I believe this is OK for the emulator,
as the parameter is dependent on only the
GPU hardware.

Change-Id: I9bd74de6317316902bddf54b915ef509625cee52

Merge "Revert "Fix glActiveTexture invalid parameter bug"" into nyc-dev
am: 24db87f763

* commit '24db87f76308c1eb933dbb32f312fe53f945cbc9':
  Revert "Fix glActiveTexture invalid parameter bug"

Merge "Revert "Fix glActiveTexture invalid parameter bug"" into nyc-dev

ranchu: Enable an adoptable SD card
am: 61d3a7e611

* commit '61d3a7e61111ea06f7173635e1118a2648a6db80':
  ranchu: Enable an adoptable SD card

ranchu: Enable an adoptable SD card

This CL is only applicable to API 23 and above. It is the ranchu version
of https://android-review.googlesource.com/195220. See also a related CL
that works for APIs 21 and 22:

 https://android-review.googlesource.com/145850

It requires the following CLs:

 https://android-review.googlesource.com/195057
 https://android-review.googlesource.com/195240

Change-Id: Id77a3bc3d0d9a054f7e9c0f70cce678f2627912a
Signed-off-by: Yu Ning <yu.ning@intel.com>
(cherry picked from commit f08ecd12ddcde47deee418dfdb652fde5d38a16d)

[camera] Be permissive about no-op metadata request.
am: 294e955a1c

* commit '294e955a1cbb89366a9224d1d01338d14a91cd58':
  [camera] Be permissive about no-op metadata request.

[camera] Be permissive about no-op metadata request.

Instead of returning error, silently agree if emulated camera is
requested to not include metadata in the video. We don't support
metadata, but if we return an error when requested to not include
metadata, framework treats it as an error, and video recording fails.

BUG=b.android.com/201678

Change-Id: I3ccf8de76ae150a881d82da5e13f87aa60f49aa0

Revert "Fix glActiveTexture invalid parameter bug"
am: 276fa6bd1d

* commit '276fa6bd1d188bd606c7f006ff5fa30710b7b19d':
  Revert "Fix glActiveTexture invalid parameter bug"

Revert "Fix glActiveTexture invalid parameter bug"

This reverts commit ac3f6d133db03e03dbbd15754d0d342748a81967.

Because somehow, the init is not called and the check failed
everytime.
bug: 27324004

Change-Id: Id667926cf40e94ea666fbec534a4b54277657aea

Revert "Fix glActiveTexture invalid parameter bug"

This reverts commit ac3f6d133db03e03dbbd15754d0d342748a81967.

Because somehow, the init is not called and the check failed
everytime.

Emulator: use FINGERPRINT_FILENAME
am: 2d354db529

* commit '2d354db5299a0d4c1f61abd06202ec180a95276a':
  Emulator: use FINGERPRINT_FILENAME

Emulator: use FINGERPRINT_FILENAME

instead of hardcoded filename

Change-Id: Ie046904160c34eedaa04fe22caa799d52376013e

Emulator: fix fingerprint regression
am: 27c1e6cd69

* commit '27c1e6cd697493e3c7e3ee7479080a58249cd72e':
  Emulator: fix fingerprint regression

Emulator: fix fingerprint regression

It cannot save fingerprints or load it at restart.
bug: 27337363

Fixed by (1) correcting the filename (2) loading
fingerprints _after_ active group is set.

Change-Id: I3f8fabdecb8ebe1ebad91bd8c5d42dd8095a0c71

GL encoder null pointer check before write
am: e11bc58ade

* commit 'e11bc58adeae10786220c16e499e4078f112cf0b':
  GL encoder null pointer check before write

GL encoder null pointer check before write

BUG=27273457

Previously when encoding some functions, if the user gives us a null
pointer, the encoder will write a header with some data length while not
actually writing any data. This patch fix it by writing 0 into the data
length field for all those functions.

The changed functions include:
glBufferSubData
glCompressedSubTexImage2D
glTexSubImage3DOES
glCompressedTexImage3DOES
glCompressedTexSubImage3DOES

Other functions do not directly write user's pointer, so they don't need
to be updated.

Since this is auto-generated code, there would be another patch in
emu-master-dev to update the code generator.

Change-Id: I05fca078dfcf01e20fa757f1050820b21ca080b7
(cherry picked from commit 82b3ebce97851f019991965301ab6063ba7a1fee)

goldfish: Make SD card adoptable
am: 4b25713034

* commit '4b25713034cd4eeed734812aef1fe41dce963621':
  goldfish: Make SD card adoptable

goldfish: Make SD card adoptable

The current storage configuration for the classic emulator (goldfish)
follows the "Android 6.0: Physical primary only" example per:

 https://source.android.com/devices/storage/config-example.html

It only allows the virtual SD card to be used as external storage. A
better configuration scheme would be "Android 6.0: Emulated primary,
physical secondary", which gives the user the choice to format the
virtual SD card as internal storage, if they so prefer.

This CL implements the new configuration. It is only applicable to API
23 and above, and requires the following CL:

 https://android-review.googlesource.com/195057

BUG=27273457

Change-Id: Idd2b51da5e90680b54d3534e5fed1424dc7e2a60
Signed-off-by: Yu Ning <yu.ning@intel.com>
(cherry picked from commit ec1e5323e59192dd9641695248a16049939aa8e0)

GLESv2: Correctly handle location -1 in shared groups.
am: c675b2a62b

* commit 'c675b2a62b59ed17c5a6b97401798ed39063fba2':
  GLESv2: Correctly handle location -1 in shared groups.

GLESv2: Correctly handle location -1 in shared groups.

Before this CL, getHostLocation left an out parameter undefined when
accessing location == -1.
It now sets it to -1, consistent with the rest of the SharedGroup
library.

BUG=b.android.com/199855
BUG=27273807

Change-Id: Ibbc234e22895083b2057fe5feb34dbd6d7b02be3

resolve merge conflicts of 935f252c6e to nyc-dev-plus-aosp

Change-Id: I0a452ab019db8d5d06550f9f7f942561f9c08599

Merge "Build camera related goldfish HALs as brilloemulator for Brillo"

Add check for data partition
am: 70bef5f034

* commit '70bef5f03409de145fa56e720f4ac30d2cc0bce8':
  Add check for data partition

Add check for data partition

bug:
b.android.com/201150
When the data partition is corrupted, emulator will fail
to boot. Adding check to reduce the possibility of boot
failure due to corrupted data partition.

Note: fstab.golfish has check already. ranchu seems
should have it as well.
Note: I don't see much boot time degradation either.
Note: need to give e2fsck SELINUX permission to do the check
bug: https://b2.corp.google.com/u/0/issues/27224003

Change-Id: I9b040ecd2009a3003c5b6c4c6e4b30c7d03fa71a

Don\'t mis-parse the Emulator\'s GPS altitude
am: af4659f02d

* commit 'af4659f02d250020317a42056a73665745e5041f':
  Don't mis-parse the Emulator's GPS altitude

Don't mis-parse the Emulator's GPS altitude

When tokenizing an NMEA string, don't skip null input fields;
turn them into null tokens.

By skipping null fields, the subsequent values become mis-aligned.
This bug prevented the code from handling the altitude in the
NMEA 0183 GPGGA string.

b.android.com/200940

Change-Id: I774de342cd9ebfb85dcc6d72b4033e1c3e101434
(cherry picked from commit 40bbf5b37a20d5fdcc392ad30edb9a475fc6344e)

Add support for new sensors in the QEMU-specific sensors HAL module
am: 737bb41ef2

* commit '737bb41ef2946730b0b0ec7b702ec3445047863b':
  Add support for new sensors in the QEMU-specific sensors HAL module

Add support for new sensors in the QEMU-specific sensors HAL module

Change-Id: I596aa8e90ecdb81d5984d6c2eb33410ced0fd861
(cherry picked from commit d9c1ea0d820055ab0e262958938ff9e5e4fd91de)

Run fingerprintd as system user.
am: 33f6216acd

* commit '33f6216acd47589ec6cc20c58e6b4db998860e48':
  Run fingerprintd as system user.

Enable fingerprint daemon on ranchu
am: 2a107261ad

* commit '2a107261ad52ef36cc802e060d1a09e02b6c59ec':
  Enable fingerprint daemon on ranchu

Run fingerprintd as system user.

Address the following denials:
type=1400 audit(1449001264.880:5): avc: denied { dac_override } for pid=1264 comm="fingerprintd" capability=1 scontext=u:r:fingerprintd:s0 tcontext=u:r:fingerprintd:s0 tclass=capability permissive=0
type=1400 audit(1449001264.880:6): avc: denied { dac_read_search } for pid=1264 comm="fingerprintd" capability=2   scontext=u:r:fingerprintd:s0 tcontext=u:r:fingerprintd:s0 tclass=capability permissive=0

Bug: 25951089
Change-Id: I72adbfd1565b555aec5193afe8114d2d67a1de9e
(cherry picked from commit 7e9c67b73f93977f739a6e641a0bf5163b721533)

Enable fingerprint daemon on ranchu

Change-Id: I0813e9e0d248acb964491cfc8c28ed8b46e5d182
(cherry picked from commit 1a681387f53ff1fe3855d48129ebd259366aefe3)

Merge "gps: Add ranchu support." into nyc-dev
am: 3713f93da8

* commit '3713f93da8a6db140b30a19a70e8d283c4b5a122':
  gps: Add ranchu support.

Merge "gps: Add ranchu support." into nyc-dev

Turn off dhcp since radio is working on ranchu
am: 6fc29096a9

* commit '6fc29096a91146ec628ed59d9abc37141a0643f7':
  Turn off dhcp since radio is working on ranchu

Turn off dhcp since radio is working on ranchu

Change-Id: Ia1550bf009fbb5f57213cac07d98360b3b789a71
(cherry picked from commit c249484b0928b5e6c2c34eacb342845e7e22c0a4)

gps: Add ranchu support.

Compile the code as gps.ranchu.so, in addition to
gps.goldfish.so to enable the gps on the ranchu
board as well as on the goldfish board.

Change-Id: I2e7c77d6352d300a1b3a172f944a4dd78bebcd20
(cherry picked from commit 61eb21dace6df5bedca7aab11c9c362e203642c6)

Compile camera for ranchu as well as goldfish
am: 834da40409

* commit '834da404097017ddfb49af4d13d75bdb73249b4f':
  Compile camera for ranchu as well as goldfish

Compile camera for ranchu as well as goldfish

This is for camera support in QEMU2
with the new emulator.

Previously, only camera.goldfish.*.so's
were being built, so the camera did not work in QEMU2.

Building camera.ranchu.*.so's seems to have fixed
the problem on guest side.

Change-Id: I9cc16ed0a48729d9a17bd5ce8a0a04bb6e031415
(cherry picked from commit e249aa0ce28c5cf33fee39eae7323d50fee0591e)

Fingerprint: Add set_active_group()
am: c0d37d7743

* commit 'c0d37d77439be7c91276ebc1063003f5b45f52f0':
  Fingerprint: Add set_active_group()

Fingerprint: Add set_active_group()

Bug: 21109822

Change-Id: Ie925dcb95f39be758de4fb88a58d1d0e48373277

Fix crash when forgot to call glVertexAttribPointer
am: c2fa3dca08

* commit 'c2fa3dca08d30325238d9aa24c0d2450490003e1':
  Fix crash when forgot to call glVertexAttribPointer

Support glDrawElements with GL_UNSIGNED_INT in the encoder
am: 49435c0ad0

* commit '49435c0ad04bb4698b3e553ec413250322a3346a':
  Support glDrawElements with GL_UNSIGNED_INT in the encoder

Fix the GLES version number in the emulator encoder
am: 10665c8e74

* commit '10665c8e74cbbef8444242ec6723816d6641d00f':
  Fix the GLES version number in the emulator encoder

Clean up minor opengl errors/warnings.
am: b95d6e60ad

* commit 'b95d6e60ad4ef0b5f6d036ac6810c3fcb044ba37':
  Clean up minor opengl errors/warnings.

Build camera related goldfish HALs as brilloemulator for Brillo

Brillo runs a reduced subset of the graphics stack and a different
system configuration from goldfish.  However, many of these HALs still
make sense for Brillo.  Build the HALs with a board name that will allow
the camera and graphics subsystems to pick them up.

Bug: 26936651
Test: brilloemulator is able to pick up and use these HALs to take
      fake pictures.

Change-Id: I79265ada33a5a97c6f670b79f9c239606465c9fc

Merge "Support generic buffers in EmulatedFakeCamera3"
am: 9ac3628e4d

* commit '9ac3628e4d457307e78c5708aa3000e868d44e1a':
  Support generic buffers in EmulatedFakeCamera3

Merge "Refactor gralloc HAL to not repeat defintions"
am: 509d98b429

* commit '509d98b429fb95ca2d56ffb8689b6493c84d92f8':
  Refactor gralloc HAL to not repeat defintions

Merge "Support generic buffers in EmulatedFakeCamera3"

Merge "Refactor gralloc HAL to not repeat defintions"

Support generic buffers in EmulatedFakeCamera3

When we're using a gralloc HAL other than the goldfish version (e.g.
gralloc.default, which gralloc.goldfish will fall back on), we
are not given cb_handle_t.  Check for this and fall back appropriately.

Bug: 26936651
Test: brilloemulator stops complaining about a format of 0.

Change-Id: I7a84c775ee6c49e23c82afb8f8b1f490505a55c5

Refactor gralloc HAL to not repeat defintions

Bug: 26936651
Change-Id: If11a2af0496cdaf86e3bc87ded37fc8d93b7fa16

Fix crash when forgot to call glVertexAttribPointer

Calling glDraw* will crash the emulatorif one forgot to call
glVertexAttribPointer first. This patch fixes it.

There are two situations: (1) user forgot to enable any vertex
attribute arrays, or (2) user enabled a vertex attribute array but
forgot to bind any data to it. This patch fix both situations. After the
patch, in (1) user will get an error message, in (2) user will get an
error message as well as GL error code GL_INVALID_OPERATION. These are
similar behaviors as running it in swift shader or on a real device.

(Running it on a real device with (1) will give nothing and with (2)
will crash the app.)

Change-Id: Id045c0c81a489a7c6ee4b6efc91f5ab148432f35
(cherry picked from commit 44dae11a1030d9d67f426eb3de4e9640a6efad7d)
(cherry picked with conflicts)

Support glDrawElements with GL_UNSIGNED_INT in the encoder

Currently glDrawElements with type GL_UNSIGNED_INT is not supported by
our encoder and decoder. The emulator will crash if someone uses it.
This patch add the support in the encoder side.

The related issue is here: b.android.com/200338
The patch on the decoder:
https://android-review.googlesource.com/#/c/202101

Change-Id: I9a6cde9983d34f39cfcc8c66a7ccf021dd5963eb
(cherry picked from commit e988586a6db9c7c3a7e6805f693d3a40913841a1)

Fix the GLES version number in the emulator encoder

The native code, emulator encoder and emulator decoder each keeps its
own GLES version number. Previously they are inconsistent when
EGL_CONTEXT_CLIENT_VERSION=3. In this patch, because our emulator
currently doesn't support GLES3, we return EGL_NO_CONTEXT with an
error code EGL_BAD_CONFIG when the user tries to use GLES 3.

In future when we impelment GLES3, please remmeber to change the
version number here, as well as those in native code and the emulator
decoder.

Release Note:
Report error code when setting up GLES3

Change-Id: I1d1083310d28bb57f42625daaed710711495a3ed
(cherry picked from commit 51278f0da350513e0826e32e83e27453193fa720)

Clean up minor opengl errors/warnings.

There are always disconcerting messages printed to logcat
whenever developing apps on the emulator.

glUtilsParamSize and other functions
often fail with "unknow param 0x00..."

These are missing GLenums and types that for
some reason are not in glUtils.cpp
in OpenglCodecCommon.

There also has been a long-standing warning
that eglSurfaceAttrib is not implemented.

It seems we don't need to implement that to get things done,
so we will give a no-op implementation that suppresses
the warning.

Change-Id: Ie44fbdd10422dd824c9c3c3ea70c386023663969
(cherry picked from commit 215bb77438519bcf1b952e4d5b621abb0bf3ed82)

Fingerprint emulation: fix unrecognized fingerid

When restarting AVD after registering a finger id, it is
not recognized, on OSX.

It is caused by not properly initializing fingerprint file
when fingerprint id is saved. Fixed.

bug: 26920715
(cherry picked from commit 1754cb33c3e2495f65fb2ed162ce059b87683e27)

Change-Id: Ia9b29e7d8e58094be54e4509d5e27ba79a794a45