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

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

Reset existing AF scan upon tap-to-focus

 * Interrupts existing, in-progress AF scans upon tap-to-focus
   (triggerFocusAndMeterAtPoint).
 * Removes pending delayed-af-resets upon subsequent tap-to-focus
   to fix the case in which a user triggers AF twice within <3 seconds
   which previously resulted in the AF mode returning to
   continuous_picture shortly after the second AF scan.
 * Removes dependencies on CameraUtil and static Settings3A
   methods, and refactors FrameServerImpl to enable unit-testing.

Bug: 18935538
Change-Id: I0c2d5d92d2b07efdc11de8388e42b931d3ea89d0

Fix ready-state issue when commands are queued

Ensure that the FrameServer is marked as unavailable if a command is
queued, waiting to acquire access.

Bug: 19367334
Change-Id: Id7b1639c6c3a7e65c75002b9f6f8f24c7515e932

Merge "Fix 16:9 start-up dialog selection." into ub-camera-haleakala

Fix 16:9 start-up dialog selection.

  Bug: 19696056

We need to make sure the list from which we choose the highest
resolution size of a given aspect ratio based on a list that is filtered
to be chosen by the user. In case of the N5 the native camera list will
be augmented with a new high resolution 16:9 size which needs to be
selected if the user chooses 16:9 on start-up on the N5.

Change-Id: I03a86aa0a148b6cbabbd51a577202b79b4941d7d

Swap dimensions of artificial 16:9 resolution on N5 on upgrade.

  Bug: 19693226

Since the N5 doesn't natively support a high resolution 16:9 size we need
to artificially add it and then crop the result from the high-resolution
4:3 size. In version 2.4 we unfortunately swapped the dimensions of
ResolutionUtil#NEXUS_5_LARGE_16_BY_9_SIZE, which now causes a few issues
in 2.5. If we detect this case, we will swap the dimensions here to make
sure they are the right way around going forward.

Change-Id: I579e464d177ac86d6cc88e9142c26fd9d6b67a76

Enable shutter button as soon as burst finishes.

Instead of keeping the Capture button disabled, do not wait for burst
results to enable the capture button. Instead enable it as soon as burst
is done.

Change-Id: I19f873d8e461b0c5385bdd2f9fb93f0a9fb7a05b

Import translations. DO NOT MERGE

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

Add Location to EXIF

  Bug: 16961102

Change-Id: I9f2335c0f02bea1286698330483cd30fdb875f45

Add EXIF data to images processed via software JPEG encoder.
bug: 16961102

Change-Id: I3df48a74fedbba6ca5540bf095f5e5eaf93db3ab

Merge "Workaround for future GPS timestamps" into ub-camera-haleakala

Workaround for future GPS timestamps

Some images are known to have incorrect GPS timestamps, which are used
by MediaStore as creation date. If future creation dates are encountered,
fall back to modification date which is not known to have bugs.

Bug: 19565464
Change-Id: I35b5e43c9a3de1f05d9a2f405502a2bb2bf9adde

Add error handling for CameraDevice.StateCallBack when the camera is already opened.

Bug: 19569722
Change-Id: I510931e595d491c2d42710151b05edea89c81094

Merge "Grant read URI permission for playback of video capture" into ub-camera-haleakala

Merge "[CaptureIntent] Detect if flash is supported from camera characteristics." into ub-camera-haleakala

Grant read URI permission for playback of video capture

Current implementation of video capture UI configured MediaRecorder to
output directly into supplied URI, which per recommendation should be
a content: URI. We pass this URI to another app to play back the video,
and should pass along the read permission.

Bug: 19675889
Change-Id: I37f8ae3702a9f1ed9dbaac9416104c0c0bfb6dd0

[CaptureIntent] Detect if flash is supported from camera characteristics.

Bug: 19669175
Change-Id: I04c5ab27d9ee0c8723d9bffab55b89def7311acf

Merge "Limit setting photo item placeholders when session is done." into ub-camera-haleakala

Limit setting photo item placeholders when session is done.

Only add placeholders for an item visible in the filmstrip. If a session
is done for an item not visible, it is less janky for the user to see the
standard gray placeholder while loading the image from disk.

Pruned older methods of determining filmstrip visibility, none of which
worked and relied on older code assuming the camera preview was an item
in the filmstrip, including 'sticky' filmstrip item bits. Added a method
to CameraAppUI that proxies the FilmstripLayout visibility, which is
actually what app uses to show/hide the filmstrip.

Bug: 19517380
Change-Id: I5e2a61b55aa2045bfd09f495dd02cdcb9c65dc68

Merge "Fix zoom for burst." into ub-camera-haleakala

Merge "Remove video portrait orientation prompt" into ub-camera-haleakala

Remove video portrait orientation prompt

No longer display visual reminder for users to rotate device to landscape.

Bug: 19416311
Change-Id: Ia03f54809f64e6034479a9021c87b50f186bab99

Fix zoom for burst.

Instead of passing the root request builder, use the request builder
configured by the Camera factory.

BUG: 19637871

Change-Id: Ibae1a85c47b0b31e07c91c4a15ec138ad021449e

Merge "Fix a typo in Camera capture session." into ub-camera-haleakala

Fix a typo in Camera capture session.

Fix a typo that leads to recursive call.

Change-Id: Iad9fe9335169f1d7b211d6d6f5355229a08f797b

Merge "Hide "Advanced" settings screen on N6." into ub-camera-haleakala

Hide "Advanced" settings screen on N6.

The N6 currently doesn't support manual exposure. That's the only item
under Advanced, so hiding that option entirely.

Bug: 18523154
Change-Id: I1b8238a9b9f0e04f8b0d2047a444b81fcecc4767

Revert "Import translations. DO NOT MERGE"

This reverts commit be4ba3e3f1285ad76445fa5c56271b9940097f74.

Change-Id: I16cac16e134e88629fdd6916f096d16e65780a34

Import translations. DO NOT MERGE

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

CaptureSession passes on message IDs instead of strings.

  Bug: 18867164

Change-Id: Ie601a0fbaf64873c56c6a9b2767a2fa6a04a11d9

Added more error handling for camera access and picture storage issues.

Implemented onFailure() method in the OpenCallback interface and
onPictureTakingFailed() in the PictureCallback interface.
OnFailure() routes through showErrorAndFinish() in the AppController.

Bug: 19452391

Change-Id: I3c19a274cc7664f7d1834899a486c57d0585dc9a

Merge "Fix units in video duration calculation" into ub-camera-haleakala

Merge "Fix Version Codes from L to LOLLIPOP" into ub-camera-haleakala

Fix Version Codes from L to LOLLIPOP

Just so that Intellij IDE don't barf on these files.

Change-Id: I405e41da920d15bd4b0726fb90b859a8741bfef7

Merge "Add black-list for resolutions." into ub-camera-haleakala

Add black-list for resolutions.

  Bug: 19611738

Change-Id: I5147bcd925c28e3db0e1e50c8a013cc6750fb47e

Fix units in video duration calculation

Convert milliseconds coming out of media table, to seconds
for details formatting.

Bug: 19305203
Change-Id: I4fafae9b3d09d23e60906c2fb3c3f305faee1339

Merge "Fix exif dimensions for cropped images" into ub-camera-haleakala

Fix exif dimensions for cropped images

Bug: 19323062
Change-Id: Ieb2905bc44324f2e9f7f4498fd1e1882924e4e92

Merge "Fix flash mode by changing the setting scope to be correct." into ub-camera-haleakala

Fix flash mode by changing the setting scope to be correct.

Bug: 19630464
Change-Id: I27abc70041002ff8f9134f249b70d318e05620b9

Merge "Add support for negative rotations" into ub-camera-haleakala

Merge " Change interface of BurstMediaItem to provide filepath." into ub-camera-haleakala

Fix flash mode by changing the setting scope to be correct.

Bug: 19630464

Change-Id: Id1257c23fef1152792a9fb81572768a4d91e22e5

Add support for negative rotations

Bug: 19636843
Change-Id: I03a30b433942cacd4978c7788e2ae7d0046479a7

 Change interface of BurstMediaItem to provide filepath.

- Instead of passing a byte array, use a file path for the burst media
item.
- Change the save operation in Stack saver to simply rename the image.
- The burst controller is now responsible for managing captured images and
implementations should free the captured images as soon as possible.

Change-Id: I6762972c68865d6ab15bd6fa20b60fb275f103f1

Merge "Update stats." into ub-camera-haleakala

Merge "[CaptureIntent] Fix 16x9 stretch problem on Nexus 4." into ub-camera-haleakala

Update stats.

Bug: 18913439
Change-Id: Iadca6b527c76b55504ce21e9f37f7c29dbb71c57

Merge "Add partial crop support to ImageBackend" into ub-camera-haleakala

[CaptureIntent] Fix 16x9 stretch problem on Nexus 4.

2. Also, more clean up for unit tests.

Unfortunately I lump two things together is because there are some nice
things about the clean up that helps deliver the fix.

Bug: 19616722
Change-Id: I632e9f679db6e8a68179912ee4e29c1394084324

Merge "Fix CameraCaptureSession exception handling" into ub-camera-haleakala

Fix CameraCaptureSession exception handling

Bug: 19484657
Change-Id: I475444ad5f52d790460fa3c7232f1571c432d54e

Merge "Fix jpeg compression without rotation" into ub-camera-haleakala

Merge "Custom checkbox drawable for K devices" into ub-camera-haleakala

Custom checkbox drawable for K devices

L is already taken care of in a v21 style, with the new buttonTint
property. For K, use the given assets, which happen to be in a
Material style as provided by UX.

Bug: 19083903
Change-Id: I9392edf6460a3a9c7c00e1b62cc0f61f9e2b4231