Merge "Make sure picture sizes are cached on start-up, if they are not already." into ub-camera-haleakala

Make sure picture sizes are cached on start-up, if they are not already.

  Bug: 20015834

We cache picture sizes on disk, keyed by OS build version number. This makes sure
we update this cache if we start the app on an updated OS. Refocus requires these
sizes and loads them from cache.

Change-Id: Ib8aa9a37481b9a6be441122be41ab81917a5bd37

Add support for dynamic ring-buffer max size

Enables specifying a maximum ring-buffer size.
Setting the ring-buffer size to a value smaller than
ImageReader#getMaxImages reduces the number of android.media.Images held
open at a steady-state at application startup.

Because ImageReader images are allocated on-demand, this mitigates
stuttering in the preview-stream caused by massive allocations that
would otherwise occur as the ring buffer expands.

Bug: 19989536
Change-Id: I5623d0cd0da6542599f430f0dcbf97d581ec8df1

Merge "Report jank for simple capture module cameras." into ub-camera-haleakala

Report jank for simple capture module cameras.

This clarifies the jank statistics by explicitly including a
report if the jank detector is installed.

Bug: 19987252

Change-Id: Ie8e4b9826eb0e3152d7f8358728019d1a81e787b

Fix build by adding missing imports.

Change-Id: Ic98b815cd9aa956d1d36db70f85d8bc29d5b8cf6

Merge "Add support for new independent Lens Blur quality setting." into ub-camera-haleakala

Merge "Collect and log jank statistics in the Camera." into ub-camera-haleakala

Merge "Reduce startup preview jank by preventing Glide pre-allocation." into ub-camera-haleakala

Add support for new independent Lens Blur quality setting.

  Bug: 19424875

This CL and its sibling do two things:
- Bring back the Lens Blur setting
- Populate the setting with up to three native picture resolutions.
- If needed, choose the default as being the second highest resolution.
  (The list is not a full list, but filterd by what we show to the user).
- Make sure that after a new install a the default is set.
- Extra check in RefocusModule to make sure the default is set if old
  setting is not valid. This is important for upgrades.
- To make the size logic re-usable, I wrapped it into a PictureSizeLoader
  which also makes the CameraSettingsActivity a bit easier to read.

Change-Id: I53fcac8304f19be4f2c5ac053363f6805b933d89

Collect and log jank statistics in the Camera.

This detects and logs sequences of camera timestamps where the
delta between the the current and last frame exceeds the
previous delta by a percentage threshold. This should only occur
when there are visible janks in the camera preview.

Bug: 19987252

Change-Id: Icced48be8232bba625aa501b2b1f5c111dbb5573

Merge "Detect jank in the preview by measuring sensor timestamps." into ub-camera-haleakala

Detect jank in the preview by measuring sensor timestamps.

This detects frames dropped by the camera (Which results in a
preview that skips and jitters) by tracking the delta between
frames and logging when that delta increases by more than 120%.

Bug: 19076469
Change-Id: Icc9f08174097587c850c06ae88007ce784775851

Import translations. DO NOT MERGE

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

Merge "Enable exposure compensation in capture intent." into ub-camera-haleakala

Enable exposure compensation in capture intent.

Bug: 19964499
Change-Id: I2f7d43aa421dba0fa873eaf47c46d955136662e5

Merge "Don't use undersore for variable names." into ub-camera-haleakala

Import translations. DO NOT MERGE

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

Don't use undersore for variable names.

Gets rid of the following compiler warning:
"use of '_' as an identifier might not be supported in releases after Java SE 8)"

Change-Id: Icb7eec3093acdd629f1e5b9994bd157a01fd8781

Merge "Sync Video torch setting with on-screen indicator" into ub-camera-haleakala

Hiding manual exposure button in video module.

Manual exposure button should never appear in non-camera modes.
This fix explicitly ensures that it will never appear in either
disabled or greyed out states in video module.

Bug: 19954837
Change-Id: Ia6730da9e3acaa62e2a30f594f344d86e2b25c69

Sync Video torch setting with on-screen indicator

bug: 19954583
Change-Id: Ia491b63f31d81038ca8d8841b1f72f5ee76bb64f

Merge "Include HDR on/off correctly in event." into ub-camera-haleakala

Include HDR on/off correctly in event.

Bug: 19900458
Change-Id: Ifd9a585a61801cd1abe16bdbe47a56d96399b84e

Make sure N4 uses the correct texture view transform.

  Bug: 19948775

Change-Id: Ib489a5a66cf4a667914c8516f71f54219660fc09

Prevent countdown cancelation from starting the camera on pause.

Bug: 19937825
Change-Id: I13e774afe2da9342b9defcc0f1d304032d931b12

[CaptureIntent] Fix NPE crash when open camera.

Change-Id: I09cddae3ce935c92d6829924372a4082bb69f48a

Merge "Remove the setFocusRatio verbose logspam." into ub-camera-haleakala

Merge "Change CaptureSession.saveAndFinish to return a future for the URI." into ub-camera-haleakala

Change CaptureSession.saveAndFinish to return a future for the URI.

  Bug: 19665531

Most callers of saveAndFinish never provided the callback. In addition
it's unclear on what thread the callback is called. This cleans this up
a bit and moves the responsibility to the caller to decide on which
thread the URI should be received. Callers that don't care don't have to
do anything.

Also fixes a bug where we wouldn't even supply a URI for when there was
no placeholder.

Change-Id: I1a186b470d2cca8f34a674e9eb5c533fc01a4409

Remove the setFocusRatio verbose logspam.

Bug: 19931564
Change-Id: Ib4d9768a95e47879f945f9a869ce1631d43cbbc8

Fix build error. Mirror API change to Camera2.

  Bug: 19899007

Change-Id: I51d3e02b42a0bff1bea85c6cd910192e6c239293

Merge "Catch HAL error that prevents retrieval of resolutions." into ub-camera-haleakala

Merge "Cancel in-progress capture when paused" into ub-camera-haleakala

Catch HAL error that prevents retrieval of resolutions.

  Bug: 19899007

Change-Id: I822510c03ecc1d48f607bf8f765831f1a39e3074

Cancel in-progress capture when paused

Change exception-handling behavior to cancel an in-progress capture (via
CaptureSession#cancel()) if the app is paused before any image is
acquired from the device.  Previous behavior launched an error dialog to the
user.

Bug: 19913612
Change-Id: If40dceabc4adbd901fbdac1a0d97f805389c778d

Suppress HDR 16:9 on N4

Add logic to CaptureModule HardwareSpec to avoid
support for HDR if in 16:9. This takes care of UI,
also wire HardwareSpec into OneCameraCaptureSetting
to ensure we only use the stored setting when HDR
is supported.

Bug: 19697027
Change-Id: Idcd02ea9910974d8afda5f81c45eb928b9df89af

Merge "Show error dialog when camera is unavailable." into ub-camera-haleakala

Show error dialog when camera is unavailable.

Bug: 19888394
Change-Id: I6d52d1be7ecac6ece85845bc3f06b1c7e0963d6f

Reduce startup preview jank by preventing Glide pre-allocation.

Several systraces and app runs seem to show that removing the
Glide bitmap pre-allocation for full size images slightly reduces
the startup preview jank on the N6. There did not appear to be
any significant regressions in the filmstrip jank.

Bug: 19076469

Change-Id: Ia01d7d09eedfc6ac78611c0f433cdcad8c15c559

Merge "[CaptureIntent] Log CaptureDone event to Clearcut." into ub-camera-haleakala

[CaptureIntent] Log CaptureDone event to Clearcut.

Bug: 19035503
Change-Id: I786f834beb45d519047aaa06a7838720c0188459

DO NOT MERGE Disable ZSL for N5.

 Bug: 19565931

Data showed the preview freeze is happening and WAR is not working.

Change-Id: I9e49bfe7cdf447ef5fe57a1a393ba8d96d1b051b

Send AF_TRIGGER_CANCEL before every AF_TRIGGER_START

Implements a workaround for bug on N6 in which the camera driver may get
stuck if two adjacent AF_TRIGGER_START are sent in a sequence.

Bug: 19892741
Change-Id: I729d7a5ca865cd5b3ba8ddd5862e422470c16671

Merge "Fix the camera2 build and import the correct Android metadata." into ub-camera-haleakala

Fix the camera2 build and import the correct Android metadata.

Change-Id: Id7f6d630ec7eb8db2e43c97025b5cbb172ed9192

Merge "Hiding manual exposure button in N6. The button was previously disabled." into ub-camera-haleakala

Read and enable face detection when available in capture module.

Bug: 19715640

Change-Id: I719cc92800a73ec3eaa27044fda09f64e1b9d946

Hiding manual exposure button in N6. The button was previously disabled.

Bug: 18523154
Change-Id: Id08cef46fc4a67e50ce4b1acd185c5ae6efa94ad

Merge "Limit maximum display size to prevent out of memory problems." into ub-camera-haleakala

Limit maximum display size to prevent out of memory problems.

Change-Id: Id681ab9f66d6a5f10f9df8fb6ae57ce90f5e25d8

Merge "Add Faces and Lens Dist. info to photoCaptureDone" into ub-camera-haleakala

Merge "OneCameraOpener should inform ActiveCameraDeviceTracker about camera open." into ub-camera-haleakala

Merge "Clean up HDR+ button state logic." into ub-camera-haleakala

OneCameraOpener should inform ActiveCameraDeviceTracker about camera open.

Therefore, people use OneCameraOpener to open a camera don't need to worry
about ActiveCameraDeviceTracker.

To make people don't need to depend on CameraActivity/AppController, move
setting scope generation into two utility methods:
  * SettingsManager.getCameraSettingScope()
  * SettingsManager.getModuleSettingScope()

Bug: 19892712
Change-Id: I04caae24ea72a41d7d8e0786a99912098c2cf1ef

Add Faces and Lens Dist. info to photoCaptureDone

Added the plumbing for getting values off of the
TotalCaptureResult and putting them into the
photoCaptureDoneEvent.

Bug: 19805467
Change-Id: Id0d16623ffe7eef78b23fd15a4edfeeca5a66eaa

Merge "Compute the max texture size from the EGL context" into ub-camera-haleakala

Merge "Clamp pixels and bounds for filmstrip images." into ub-camera-haleakala

Compute the max texture size from the EGL context

Initialize and read the GL_MAX_TEXTURE_SIZE property by opening
and initializing an Open GL display, context, and surface and
using the initialized objects to read out the texture size.

Bug: 19871547
Change-Id: If74045e00d99e7268b78dcca0ad0a22f4d581f0e

Clean up HDR+ button state logic.

CameraAppUI#applyModuleSpecs should honor whatever HardwareSpec and
BottomBarUISpec provided by modules. UI should not change behavior by
poking setting by itself. This can cause huge pain when debugging.

Bug: 19657871
Change-Id: Ief153f6c92a12ab1a72732a7a0908059fd91167e

Clamp pixels and bounds for filmstrip images.

This change does a more extensive set of checks and constraints
based on the maximum area (to prevent jank) and adds clamping for
the bounds of the image based on the maximum supported texture
size of the device to prevent blank frames.

Bug: 19871547
Change-Id: Ib33e6e591638fb4dfefb60ddea488818e985495d

Check for null on N5 resolution upgrade

In case of first run with no existing resolution setting, ignore
the upgrade step.

Bug: 19889422
Change-Id: I518eaf089468c8255dedfea371f68763d62d777a

Import translations. DO NOT MERGE

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

Merge "Display error dialog and log error when video recording fails to start." into ub-camera-haleakala

Merge "Catch memory allocation errors JPEG ImageTask" into ub-camera-haleakala

Display error dialog and log error when video recording fails to start.

Bug: 19814676
Change-Id: Iec3fde247720db4471da8f40a5c0f55d706d52cd

Merge "Re-arrange camera initialization." into ub-camera-haleakala

Re-arrange camera initialization.

This change re-arranges several things to make it possible to know
the CameraId you are about to open before you actually open the
camera device. It also adds an explicit CameraId class for handling
both legacy and camera2 id comparisons, an active camera tracker,
and moves the camera id query mechanism into a hardware manager
class in prep for the subsequent camera release.

Bug: 19679820

Change-Id: I4c09b9a12a1ea5badcac3a6a073a56764f9a7eba

Catch memory allocation errors JPEG ImageTask

Bug: 19354210

Although rare, we should fail gracefully in the case
where we cannot allocate a large memory buffer in
TaskCompressImageToJpeg.

Change-Id: I1f16ba16a785b32869030a5f8ec2116fbbc900d3

Rename CaptureSession#finalize  to not collide with Object#finalize.

  Bug: 19855781

Change-Id: Ie32024a9d2b7c04a27cc1ff7eee62df0e6857031

Merge "Hide flash icon if not supported by camera hardware." into ub-camera-haleakala

Merge "don't mess with the options buttons in transistionToCapture and transitionToCancel" into ub-camera-haleakala

Hide flash icon if not supported by camera hardware.

bug: 19199856
Change-Id: I5e4f7842c61e6c260b7a35c455f2caaa4d8e5e53

don't mess with the options buttons in transistionToCapture and transitionToCancel

bug: 19366720
Change-Id: I0620d1ceb80c2c45de95336be18ccf212ace6f69

Import translations. DO NOT MERGE

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

Pass crop and implement crop in JPEG ImageSaver

Bug: 19660460

Allows for the JpegImageBackendSaver to accept crop
regions and then implements this crop within the
JPEG compression task.

Change-Id: I165b272376c3d5c919c1c41b55bf0b8e551520be

Merge "Add processing time to capture done event for CaptureModule captures." into ub-camera-haleakala

Merge "Detect and log missing images" into ub-camera-haleakala

Detect and log missing images

Adds logic to detect and log when the camera framework
fails to deliver an image, but successfully delivers
subsequent images.

Bug: 19430369
Change-Id: I404888882879ac021ba84d725732e9c9a9c61f05

Merge "Optimize ZSL for auto-flash mode" into ub-camera-haleakala

Optimize ZSL for auto-flash mode

Cache the most recent converged AE state to enable more
auto-flash pictures to be taken with zero shutter lag.

Bug: 19151363
Change-Id: I7d4bd5a67d4cddb7a1f733844e1a080acfacf7f4

Merge "Fix bug where return value for thumbnail can be null." into ub-camera-haleakala

Optimize shot-to-shot time for camera2 legacy

Optimize shot-to-shot time for devices which only support
the camera2 LEGACY api by releasing the exclusive lock on the
FrameServer as soon as each image is exposed, allowing the user
to immediately submit a request for the next shot.

Bug: 19365933
Change-Id: I8935611fae666dde1fb3a1c4aecf95c1cc5a7ef8

Add processing time to capture done event for CaptureModule captures.

Bug: 18913439
Bug: 19035503
Change-Id: Ib3768d93a2240ccfbf5f5a750b66939c6930ed0e

Fix bug where return value for thumbnail can be null.

  Bug: 19779596

Change-Id: I4ccf39b0e19956c6873868bc02be836b547fa1c5

Merge "Add PhotoCaptureEvent to Capture Module" into ub-camera-haleakala

Add PhotoCaptureEvent to Capture Module

Adds a CaptureSession collector class so that
we can combine stats information from both the
CaptureModule and ImageBackend to create a
complete PhotoCaptureDone event.

Bug: 19035503
Change-Id: Ide5f708c5f3efb710fa704f3f30c52ac7a8c3fe4

Import translations. DO NOT MERGE

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

[CaptureIntent] clean up extra PreviewAreaChangedListener.

A simple cleanup that we register two PreviewAreaChangedListener for
CaptureIntentModuleUI which is kind of silly. This CL consolidate it
into one.

Change-Id: I1c5b29a77021a21cd6d8c023b4b98906430c305e

Merge "Don't take picture if camera is closed." into ub-camera-haleakala

Automated ImageProcessorListener registration

I've added the ImageConsumer interface and implementation
in ImageBackend to support the automatic registering and
unregistering of the ImageProcessingListener.  This
new functionality avoids the issues of reference leaking,
so that the transient listener implementations are NOT
held by the image backend.  Also more general purpose
interface has been created such that any passed Runnable
will be executed when the associated set of tasks that have
been enqueued with ImageBackend::ReceiveImage have finished
processing.   This functionality solves the referencing
leaking issue of ImageProcessingListener in the current
implementation.  Also, removed all references to Main
thread executor so that ImageBackend is not a direct
suspect in jank issues.

Bug: 19517380
Change-Id: I806582f13157030ed6bc23ad70a4efd93339d9ff

Don't take picture if camera is closed.

  Bug: 19712310

Change-Id: Iaaae7a1c784e4149ca837e9bda496f734be56890

Merge "Implementation of Crop in Thumbnail Creation" into ub-camera-haleakala

Fix build

Change-Id: I36c5f2bf963bfe7d63941777ecedce7354aa0212

Merge "Reset existing AF scan upon tap-to-focus" into ub-camera-haleakala

Merge "Disable manual exposure button in HDR+ mode." into ub-camera-haleakala

Disable manual exposure button in HDR+ mode.

Bug: 19641530
Change-Id: I70d47a678eb0c5b77040ff071af14dd35c1e73a8

Implementation of Crop in Thumbnail Creation

Propogated the crop information into the thumbnail creation
code.  Required generalization of the thubmnail creator to
accept a crop region, and implementation of consistent
handling of the crop parameter into the TaskInfo.

Tests for this newer functionality are available in ag/653259.

Bug: 19323062
Change-Id: I135ae16459eedf7f2631925526c0e64184f6a859

If JPEG data already has some EXIF data, carry it over during EXIF population
bug: 16961102

Change-Id: I887726dda0af62c7eaecbde4297079fc447c39fa

add null check protections to ExifUtil to prevent Nexus 4 crashes

bug: 19701200
Change-Id: Icea7d5d57e5c692d214745f83efd4b06f3517740