Merge "Enable ZSL for Nexus 5 Back Camera, 1 of 2, Redux" into ub-camera-haleakala

Enable ZSL for Nexus 5 Back Camera, 1 of 2, Redux

Allow ZSL implementation to be used with Nexus 5.  We still
need to drop the frame rate, if the current camera is facing back.

Screwed up last submission.  Will always build locally next time
no matter how small the change.

Bug: 18964282
Change-Id: I975129d7262b904f8b0b31a4cdbe95526e88a521

Remove exception when releasing a requested, not yet opened camera

From monkey tests, we see a spike in an exception from new device
which is likely due to slow close performance. If we camera B is
open, and we request camera A, B will be closed then A opened. If
we release A during this process but B is not yet closed, don't
crash, just let the pending underlying open of A happen and release
the request. Preserve the exception for application errors where
truly unrequested cameras are released.

Bug: 17975345
Change-Id: I9aa3b0575e130f516cea6a3df120e4812df55bce

Merge "Revert "Enable ZSL for Nexus 5 Back Camera, 1 of 2"" into ub-camera-haleakala

Revert "Enable ZSL for Nexus 5 Back Camera, 1 of 2"

This reverts commit f7803cb20b39aa48dd399c5768902593a22d3104.

Change-Id: Ic9ebea1d723dc51c07bda01edd83cb2ac23dfe26

Merge "Enable ZSL for Nexus 5 Back Camera, 1 of 2" into ub-camera-haleakala

Enable ZSL for Nexus 5 Back Camera, 1 of 2

Allow ZSL implementation to be used with Nexus 5.  We still
need to drop the frame rate, if the current camera is facing back.

Bug: 18964282
Change-Id: Ia733cbc7bd8766245bf668e8f9141b1ec6d5eaaf

Merge "Ensure button callbacks check lifecycle" into ub-camera-haleakala

Merge "Add HeadingSensor." into ub-camera-haleakala

Add HeadingSensor.

A small step to clean up PhotoModule and CaptureModule...

Bug: 17985028
Change-Id: Ic79dd5e2582479d8ed9705e8150c4438e39116c8

Save media items in burst with sequential timestamps.

Save media items with at least one second difference for burst.

Change-Id: I332a4f19ef754dcf4e82ee9c5725e09ef147bb24

Passing an intermediate preview to the session.

Generalized the image thumbnail preview task to create artifacts of
multiple shape.  Added this task before the full-sized JPEG decode.
Added a field in the TaskInfo to specify the purpose of the imaging
artifacts on the listener, since the processing of an image may result
in multiple uncompressed/compress artifacts and need extra
explicit identification to be properly routed by listener.  Changed the
default behavior of the ImageBackend to produce an intermediate-sized
preview on job submission.  Routed this intermediate-sized preview to
the CaptureSession for an earlier Filmstrip update.

Bug: 18841584
Change-Id: I8db2d3a7c30cddd1e0e9ddf0709b7867f841b056

Ensure button callbacks check lifecycle

The button callbacks are asynchronous behind animators. Most in the codebase
(i.e., all in CaptureModule) do paused checks, but the one in CameraAppUI
for grid lines did not and is causing lots of feedback reports with null
SettingsManagers (done in the activity onDestroy). Audited codebase, these
two instances were the only ones not checking.

Bug: 17905718
Change-Id: I31c26088ef1e7d600b4e3be42950b0ca70613e20

Merge "Change IllegalStateException to a log." into ub-camera-haleakala

Add OneCamera.PictureSaverCallback.

Remove the undesirable dependency between ImageBackend and CameraAppUI.

Create a new interface rather than using PictureCallback since
at some points we might consider retiring it.

Change-Id: I6e02564c9ca9990aa847902d86806f8bbd9cd652

Change IllegalStateException to a log.

Change SurfaceTextureConsumer to ignore new frames when wrapped surface
is null.

Change-Id: I952bdb96c0debc53a0f1548fb66b3221f65f6f7f

Fix placeholder bug.

Bug: 18947927
Change-Id: I08542d614b15f31398150ea52caa6055ec63ccfb

Fix crash that occurs when two consecutive smartburst captures taken.

Change-Id: I771afdd180a0d162b2a8b3c0fc4b8e4cb04baaf0

Merge "Refactor the filmstrip backing data." into ub-camera-haleakala

Fix bug in ReservableTicketPool not being fair

Calls to acquire() by multiple threads could previously violate fairness
and result in starvation in extreme cases.  The associated unit-test
was previously flaky and failed only occasionally.

Change-Id: I7ae9bead2ac3cb6d7e81e88076bef2ea08e7d368

Merge "Instantiate a ImageSaver.Builder in CaptureModule." into ub-camera-haleakala

Instantiate a ImageSaver.Builder in CaptureModule.

We will need different image savers for capture intent.
Since we plan to create a separate module for capture intent,
this change allows modules to be able to choose a different builder.

Bug: 18726217
Change-Id: I5871263123b06958d902cc8e3a4d911f14c83e66

Merge "Image Plane Proxy for Image Content Testability" into ub-camera-haleakala

Image Plane Proxy for Image Content Testability

Adds in an Image Plane wrapper level so that image content can be
decoupled from the native Android Image objects whose constructors
are privated and cannot be extended.

Change-Id: Ia16418d18010e672f6c3e5db64bb7f6a9b3b2f3e

Refactor the filmstrip backing data.

- Remove unused fields and code.
- Rename classes and methods to be consistent with usage
- Make data more immutable
- Add Location and Metadata classes for typed data access.
- Use Date instead of long for DateTime representations.
- Filmstrip prefix for filmstrip specific code.
- Cleaner Glide implementations.
- Initialize Glide bitmap pool.
- Eagerly load large image sizes when scroll stops.

Change-Id: I3b51d42416ca076c80bf7db441d257659174b47d

Merge "Stop recording location in onPause" into ub-camera-haleakala

Stop recording location in onPause

This is a legitimate bug where the Legacy provider actually documents a
no-op in disconnect with an intention to stop updates on pause, which
never occurred. This also improves lifecycle of location updates for
other providers.

Bug: 14989298
Change-Id: If963241b01af83ab63bd8c1bccfcf1b5a1a616fa

Merge "Avoid using fast_out_slow_in on K devices." into ub-camera-haleakala

Merge "Fix NPE crash in RoundedThumbnailView." into ub-camera-haleakala

Fix NPE crash in RoundedThumbnailView.

Bug: 18851937
Change-Id: I63f3084cd2ae7cd8bbb73507aefcf4192ed7bfd0

YUV Color conversion fix

Bug: 18902605

More foolproof way of calculating the YUV multipliers at compile time.
Fixes a previous off-by-one error that translated into an incorrect 2x
in chroma magnitude (U,V components) by manual calculation.

Change-Id: I0732ed0c5a1bf4b901f9eb86c6a20e09872134dc

Adjust aspect ratio matching threshold for all devices

Broadening a fix from Glacier now that we have time to test across
devices.

Bug: 18241645
Change-Id: Ic3a65a659009d0301255b608d4540bd15e9bd0fb

Merge "Interface ImageTaskManager for Image-based Tasks" into ub-camera-haleakala

Interface ImageTaskManager for Image-based Tasks

Clearly defined with relationship between derived ImageTasks and the
respective ImageBackend.   Once tasks are derived from
TaskImageContainer, they should only have limited functionality back
to the ImageBackend to spawn new tasks to describe their internal task
graph and manage their image references.  The new class ImageTaskManager
defines this relationship and implies that most of functionality should
be internal to the task itself rather than the ImageBackend.

Also, included clean-up of some unused parameters in ImageConsumer
interface as well.

Change-Id: I6a0a5f72aea55adc8b846eb3cd945b61bc6d2a93

Refactor camera factories to reduce code duplication

Change-Id: I5f5c899f595dbbaa9fa181c5caa092d4bad69f07

Avoid using fast_out_slow_in on K devices.

Bug: 18895268
Change-Id: I9bcd799c5a27b2e05cf45bb5923dcf8e7d0f7123

Fix possible ConcurrentModificationException

Bug: 18895233
Change-Id: Iece6bd467c74082e2ea48df2ff9bb51767b75c17

Merge "Add jsr305 to build" into ub-camera-haleakala

Add jsr305 to build

Change-Id: I0b8a9d07ca834cf2d58b0f4b9398cb2ac31f25b5

Merge "Delete unused/unnecessary interfaces" into ub-camera-haleakala

Merge "Break the dependency from CaptureModuleUI to CaptureModule." into ub-camera-haleakala

Detect slow frames and provide smoothed preview FPS.

Change-Id: Ie1a8feeb8d69f213791cd8b86e0f2e10e5b8cc08

Break the dependency from CaptureModuleUI to CaptureModule.

This doesn't change any logic. Mainly just moving things around.

Change-Id: Ia130f3d0e5821095605219e50cc16008bc7ac143

Delete unused/unnecessary interfaces

Change-Id: Ic3cdad63236cf98db5eaeb45287393d4ec16fc0a

Fix race condition in which close() may deadlock

Change-Id: I422471ed47216dafd7f3114fc3d1ea7f47d281c3

Replace Pollable with Guava Supplier

Change-Id: Ibd4359cf5f0c7dbacb534dcb098d77a1f2f36133

Merge "Replace FutureResult with guava SettableFuture" into ub-camera-haleakala

Merge "Fix bug in CountableBufferQueue.getNext" into ub-camera-haleakala

Replace FutureResult with guava SettableFuture

Change-Id: I2439909e2c6dd663b1bc31a828e7744e18a6daaf

Fix bug in CountableBufferQueue.getNext

Change-Id: I2eb19a692acb9e41da0c172364fdb285061de2e6

Only run non-zsl fallback if the ring buffer is empty

Change-Id: I5d85aa1a3bbb0035189fa0f178a6a0077740ae46

Add Guava to Camera2 build

Change-Id: I7a1f49df884a7660fbbd45322f135a0fb15dd78d

Merge "Rename to RequestTemplate" into ub-camera-haleakala

Merge "Remove unused parameter to OneCamera.close" into ub-camera-haleakala

Merge "Simplify ImageSaver interface" into ub-camera-haleakala

'Fix' ANR in holiday build happening frequently from lockscreen.

  Bug: 18854352

Note: This is more of a hack around the already pretty hacky camera
open-close lock. Proper fix is to refactor the code to not require this
lock anymore. But to get the holiday droidfood going further, this is
a quick for now.

Change-Id: Ib69821d4e6cb3b82f438aeee28c807cc7c33dd25

Simplify ImageSaver interface

Change-Id: I92bdade418f39d852e972f18f58e5620d38067aa

Revert "'Fix' ANR in holiday build happening frequently from lockscreen."

This doesn't work well and introduces an issue with HDR+.

This reverts commit 7b81135c88f01cb2ac7a958dcb66c0d2d2bde7c4.

Change-Id: Ib7e412221c216d0cf6116817599d28a4d4afa3ef

Remove unused parameter to OneCamera.close

Change-Id: Iea5dcb41eb5c387a7f19db280ea9c681e2aabc7e

'Fix' ANR in holiday build happening frequently from lockscreen.

  Bug: 18854352

Note: This is more of a hack around the already pretty hacky camera
open-close lock. Proper fix is to refactor the code to not require this
lock anymore. But to get the holiday droidfood going further, this is
a quick for now.

Change-Id: Iad974e91aae6ede3897ce3bb7d69f8764d9fd08a

Rename to RequestTemplate

Change-Id: I4f1dd9ac41d8c35c50eaead4be211d3bb108bc73

Make sure session is copied into ImageSaver.

This prevents references to CaptureSession being basically randomly called.

Change-Id: Ic63308a23c22a21ffe7572397697772c3f65ccfa

Rotate images before storing them so that rotation for reading is always 0.

This also fixed the filmstrip loading slowness issue.

Change-Id: I920f5f136576c6812522d99bfed5063357593206

Merge "Fix bug in FirstRunDialog detection and re-enable CaptureModule on N9." into ub-camera-haleakala

Fix bug in FirstRunDialog detection and re-enable CaptureModule on N9.

  Bug: 18840830

Change-Id: I22abae32c5e55f779b5c645324f1f3a8421e4bde

Merge "Add logging to track the total number of open images." into ub-camera-haleakala

Add logging to track the total number of open images.

Change-Id: I62e4cf54c856c537bebd557efc7215481ac6e383

Merge "CaptureModule is not yet working on N9, so switch it to PhotoModule." into ub-camera-haleakala

CaptureModule is not yet working on N9, so switch it to PhotoModule.

Change-Id: I58aee4b7704840b0d89c8c2c0d96f54a2347c42b

Added shutter sound to CaptureModule for non-HDR cameras

Change-Id: I11beb99633e116b8e0328f0021e09100e6d031e6

Merge "Fix NPE crash in RoundedThumbnailView." into ub-camera-haleakala

Merge "Store ZSL images in correct orientation and show progress in filmstrip." into ub-camera-haleakala

Fix NPE crash in RoundedThumbnailView.

Bug: 18835171
Change-Id: Ib77e15cfeb93e3d73cf14e2459dfcdbe54d92073

Store ZSL images in correct orientation and show progress in filmstrip.

Added an ImageRotationCalculator which calcuates the image rotation
based on camera characteristics and the device orientation at the time
of the shot.

Also moves the image backend code into its own sub-package.

Change-Id: I4b6b83e3e3cebece83215c19d0aff826ff853352

Added thread-safe SettingsManager wrapper

Change-Id: Ie2d2bb14e0dd4760e8f8d47e7a988ab20c8567fe

Until Camera2-based JPEG camera is ready, N4 uses PhotoModule.

Change-Id: I3fc9503ae254a4d3975d9387f13f78e387ab0361

Merge "Do not disableModeOptions() on shutter click" into ub-camera-haleakala

Merge "Changes to support complete disabling of Burst." into ub-camera-haleakala

Do not disableModeOptions() on shutter click

Change-Id: Ib4b226417ab992358231cbe96cf7d46c27926c7e

Added non-zsl YUV OneCamera implementation

Change-Id: Iea389f274ea453b81c3fba39c65e0ec2dc1ee5c6

Full end-to-end integration for ZSL

Full end-to-end functionality using the ZSL integrated with
the ImageBackend.  Still some outstanding issues regarding image
orientation, but the full pipeline is plumbed for ZSL.

Change-Id: I636944fbd57e7d1bf6b1ef24e1e2c1317afdd28d

Changes to support complete disabling of Burst.

Have the stub burst controller retain a reference to the preview surface
texture.

Change-Id: Iaf86f8cd30ac862b1f29f9edeca65fd20e825b52

Merge "Revert "Changes to support complete disabling of Burst."" into ub-camera-haleakala

Revert "Changes to support complete disabling of Burst."

This reverts commit 8565f7695f41f0150ba096a75591ad7b2fe5de81.

This crashes pre-MR1 devices. Will fix and re-submit.

Change-Id: Ie645a63270e73b451d0a484928c98cf5104e320a

Merge "Implement callbacks for focus indicator on zsl" into ub-camera-haleakala

Implement callbacks for focus indicator on zsl

Change-Id: I5348192a76222ac41923b680acb58ac6480affcf

Merge "Changes to support complete disabling of Burst." into ub-camera-haleakala

Changes to support complete disabling of Burst.

Have the stub burst controller retain a reference to the preview surface
texture.

Change-Id: I9a966a87aac2c21cd6050f511ea4e648b6267744

Merge "Restore activity lifecycle when OneCameraManager fails to initialize" into ub-camera-haleakala

Restore activity lifecycle when OneCameraManager fails to initialize

The return in onCreateTasks when OneCameraManager catches an exception, cause
camera crashes instead of showing the failure dialog as intended. The exception
is an NPE in PanoramaViewHelper but is due to assumptions of things init'ed
in onCreate not holding due to the return when onStart is executed. While we
could proof out every object to assume lifecycle events exit, I prefer to
maintain lifecycle and remove the return, and make anything that relies on
OneCameraManager know that it could be null. This also moves the handling of the
error dialog down to CaptureModule, in the unlikely case an API1 module would
still work this enables it while preserving the same intended behavior for
CaptureModule.

Bug: 18048591
Change-Id: Ib81f8b42eaa26daae2d3354bae1ef5127a9cbcf1

Merge "Refactor the interface of CaptureModule to Burst to allow disabling." into ub-camera-haleakala

Refactor the interface of CaptureModule to Burst to allow disabling.

Change-Id: I87164c0cd8961ad929a63ff34248a12690f7d4d1

Enable AE and fix metering regions for ZSL camera

Change-Id: Icaf3bfe9b004e032e43ae600fb3e10ce201ce545

Merge "Fix potential NPE in ConcurrentBufferQueue.getNext()" into ub-camera-haleakala

Fix potential NPE in ConcurrentBufferQueue.getNext()

Change-Id: Ieb6101d0cf9466f3ec3e1bff32797a247b0ecba3

Fix ConcurrentModificationException in ImageDistributor

Change-Id: Ibf250795189d2113b00f89a9d58bb78581c0dc0f

Fix bug in which double close() increased ticket count

Change-Id: Ic534cb436755cbb3fada8769e0f7f81e6167eb5f

Merge "Added zero-shutter-lag capture implementation over the FrameServer" into ub-camera-haleakala

Don't use glide if we have a bitmap already as a placeholder.

Change-Id: I8e9676675c025cbfcd36e8c1b899a33d1b20f2d1

Change placholder API to use Bitmap's instead of JPEG bytes.

Change-Id: Id6f3b1e604bee56f3372b891bc44d8716b06478d