Merge "for intent capture and pano, fix the redo/ok/cancel button sizes so touch ripples show up right." into ub-camera-glacier

Merge "tweaking peephole animation time from 350 to 500, per in-person request" into ub-camera-glacier

tweaking peephole animation time from 350 to 500, per in-person request

Change-Id: I13cd63b00217f8beb079a3ea219d878971756c67

Merge "Avoid building filmstrip item data if activity is destroyed" into ub-camera-glacier

Merge "new share icon" into ub-camera-glacier

Avoid building filmstrip item data if activity is destroyed

Multiple AsyncTasks result in Glide execution after CameraActivity
is destroyed, via FilmstripView.buildItemData. Return early if
destroyed so that no Glide tasks are initiated and no views are
added to the hierarchy.

Bug: 17527470
Bug: 17339652
Change-Id: I7d24197791df812dc012d8b4fadff9246b289b81

Merge "WIP: Add recursive thumbnail call to LocalMediaData" into ub-camera-glacier

new share icon

bug: 17671676
Change-Id: I1b681efdb3129cdd532002d85ad3257cef962d92

Fix crash in VideoModule - Method called after release()

Completely eliminate the runtime exception caused by monkey.
Sample stack trace:
java.lang.RuntimeException: Method called after release()
       at android.media.MediaRecorder.setCamera(Native Method)
       at com.android.camera.VideoModule.initializeRecorder(VideoModule.java:1101)
       at com.android.camera.VideoModule.access$2000(VideoModule.java:88)
       at com.android.camera.VideoModule$10.onStorageUpdateDone(VideoModule.java:1320)
       at com.android.camera.CameraActivity$20.onPostExecute(CameraActivity.java:2014)

Bug: 17313985
Change-Id: I7ecaf375c26ebd1608d7098599ab1528609287ce

Synchronize CaptureModule camera open/close operations

This fixes a race condition in which CaptureModule gets a pause()
immediately after it is asked to open the camera.  pause() MUST
wait/block to close the camera to ensure that we can later open it (a
fast open()->pause() sequence is reproducible via the lockscreen under
certain cases when we get a rapid onResume()->onPause()->onResume),
so this adds the necessary threading logic to wait, if necessary,
for the camera to finish opening so that it can be closed.

Bug: 17642065
Change-Id: Ic88bd32fc5acc2b333de367019085a8cd539c5a5

Merge "Stability: add null check for all CameraProxy obj references in VideoModule." into ub-camera-glacier

Stability: add null check for all CameraProxy obj references in
VideoModule.

Bug: 17657161
Bug: 17653725
Change-Id: Icca2b9dffdfe954e979d9af7e47d78b539dc2b9d

Revert app workaround for bug 17614203 now that HAL is fixed.
This reverts ag/543302.

Bug: 17614203
Bug: 17145703
Change-Id: I3ec21902e59156ea30c181ecd6a7728f7262110a

Merge "Force filmstrip images to generate in JPEG compression format" into ub-camera-glacier

Force filmstrip images to generate in JPEG compression format

Due to https://github.com/bumptech/glide/issues/156, we sometimes
were getting PNG cache image generation which greatly increased
latency and introduced jank. As workaround, explicitly forcing
images to utilize JPEG format.

Bug: 17597861
Change-Id: I49d7266d4ff0173d253877612b3076489a6b385a

Merge "Recreate a OneCameraManager instance for each Activity." into ub-camera-glacier

Stability: add null check for all CameraProxy obj references in PhotoModule.

Bug: 17643705
Bug: 17604028
Change-Id: I6073f32bae286168c040f5667494ede1384ce3fc

Recreate a OneCameraManager instance for each Activity.

Bug: 17484715
Change-Id: If484e3b40ef32324fa77629b1687cd4317466e2e

Skip disk caching for session data

Bring back full color depth but don't bother with a disk cache
for temporary session images, since they will go away and be
replaced by mediastore entries anyway.

Bug: 17642071
Change-Id: I1a8483b71d69967033074fc4d8c69c9dfb17651e

Merge "Revert temporary session images in filmstrip to RGB_565" into ub-camera-glacier

Revert temporary session images in filmstrip to RGB_565

A simple way to avoid performance issues for now until we get around
the significant load issues seen in writing ARGB_8888 to disk cache.

Bug: 17642071
Change-Id: I6017980fce6e633b52dce93f3f46931e5bcc3d9c

Ensure Camera doesn't attempt access of back camera on Nakasi

Bug: 17647004
Change-Id: I823f98cfdfcc7429b6acf98215e4a08787bc1bb9

Don't show aspect ratio UI if using API2 portability

Avoid preview size changes due to current state of locking in API2
portability implementation.

Bug: 17462976
Change-Id: I9426380699988d67b21ce86850ef00736e84a5d6

new sounds for timer ticker and pano / photosphere intermediate captures

companions CL: ag/534823

bug: 17281234

Change-Id: I9d98d8e869dcefdbd0f7e0b53dea6b8e024a8e3b

for intent capture and pano,
fix the redo/ok/cancel button sizes so touch ripples show up right.

bug: 17505149
Change-Id: I29d8222ff0588d0410cf1f8a94bed963b4bf32f6

Merge "monkey test NPE fix, another null check on mCameraDevice" into ub-camera-glacier

WIP: Add recursive thumbnail call to LocalMediaData

This should allow for flicker free scaling once glide is submitted.

Bug: 17597861
Change-Id: I7a4d874a95f884bc54bea940facc68fd9a1ad7b5

Merge "Fix filmstrip item bounds to filmstrip scale" into ub-camera-glacier

Cleanup gesture UI hide code

Relocate show/hide of zoom UI in the onScaleBegin/onScaleEnd classes.

Bug: 17613719

Change-Id: I400161c95c08c7f3b6b36487de703296236595bb

Clean up unused code

This UI code is completely unused, and has no reference from rest of
app. Delete for cleanup.

Bug: 17613719

Change-Id: I1d9d443ef6642cd434725fe71e59fa0220258789

monkey test NPE fix, another null check on mCameraDevice

bug: 17604028
Change-Id: I7b45c101045b765562146ebe2c779e227a6227bd

Fix filmstrip item bounds to filmstrip scale

There's no reason to scale filmstrip items to anything other
than the fixed filmstrip scale, and doing otherwise interferes
with Glide caching and introduces flickers upon zoom.

Bug: 17597708
Change-Id: Ia143b4d95a48e9e23bbf36ec0edf93599caaa01b

Merge "Import translations. DO NOT MERGE" into ub-camera-glacier

Merge "Changed tap-to-meter rectangle and weights." into ub-camera-glacier

Merge "Fix CameraSettingsActivityHelper" into ub-camera-glacier

Merge "implement new trash icon, plus some asset cleanup" into ub-camera-glacier

Merge "add null check to updateCameraParametersPreferences to apease the monkey" into ub-camera-glacier

add null check to updateCameraParametersPreferences to apease the monkey

bug: 17580046
Change-Id: I4a109c1134684f6d5b2dafdb138137ef36f208f0

portability layer doesn't work well with startPreviewWithCallback()
here's a dirty dirty workaround.

bug: 17576554
Change-Id: Ief40b65ce49712153f6ef604a3513348e493d4d9

mode switch jank fixes

puts many ValueAnimators news into initialize methods
changes startPreview() calls to startPreviewWithCallback()
puts MultToggleImageButton's bitmap combine into an AsyncTask

bug: 17070332
Change-Id: Idea6396264588b345ed27b9f96f9434ff3e7e77c

implement new trash icon, plus some asset cleanup

bug: 17566685
Change-Id: Ia7b55ad3b6fed939c19abcde12cbf34726ba919c

Changed tap-to-meter rectangle and weights.

Bug: 17417649

- Clarified widths in Settings3A.
  - For Gcam only, reduced AE box to 12.25% of the shorter edge of the crop
  rectangle. This matches the setting on Nexus 5.
  - For Gcam only, changed AE weight to 22 after consultations with Ryan.
  - Tested with Gcam and it final shots look great in HDR scenes.
- Does not fully address b/17417649: ISP AE ignores the weight completely and
  overcompensates for the tap region.
- Changed AutoFocusHelper to take touch points in normalized portrait
  coordinates along with a sensor orientation. The output is a MeteringRectangle
  in sensor pixels.
  - Fixed a bug where a digital-zoomed tap doesn't work when the rect intersects
    edges. It should have been clamp, not check for intersection and return the
    full crop rectangle if it doesn't intersect.

- Submit simultaneously with ag/549706.

Change-Id: If084e64ecbfc4a08ece96a7cebfcaf835d92b9d1

Import translations. DO NOT MERGE

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

Fix CameraSettingsActivityHelper

Previous hack to allow activity up button to work, interfered with
custom preference merging we use to conditonally add items to
Settings. Now fixed by allowing internal control of whether the
full preference tree is returned by the fragment or just the
desired subtree, so that the Helper can access the full tree.

Bug: 17534830
Change-Id: I11510f365b9e212f07009b2233bb5d8036228774

Merge "Don't set mCameraAppUI to null in onDestroy" into ub-camera-glacier

Don't set mCameraAppUI to null in onDestroy

BUG: 17408047

Note that we originally set this to null to address BUG: 12805279
which involved leaking Views.  However, this has regressed and will
require another look regardless.  The number of views leaked
with and without the null assignment is the same, so this change
does not make the regression any worse.

Change-Id: If90aeebb081073eb4f5b928919fc67ef710e1054

set proper alpha on shutter button on mode switch,
eliminates a janky alpha flash

bug: 17487040

Change-Id: I08e66b5be53e58e9e16d32c09ef50081e90e9284

Don't send redundant requests to start the preview

Change-Id: I43d8a2917bbe0140f1e9367cce21f68b19807aa8

Revert layout change for Camera

Changing from LAYOUT_FULLSCREEN to FULLSCREEN interferes with layout of
Action Bar.

Bug: 17477202
Change-Id: I029a84adbe4c5017cc8185c5c0c2601a03196bde

Move launcher icons to mipmap

Bug: 17457682
Change-Id: I6a626c8547f3eea8a06fc47fc70c1a587767b040

Establish synchronous camera closing when app utilizes API1/API2

To ensure API1 and API2 portability implementations coordinate together,
if app uses API2 for any mode, lock both modes into synchronouse camera
closing.

Bug: 17464089
Change-Id: If4cf1eb02053ec9e56b7b8df97c21e012467aa3a

Merge "Import translations. DO NOT MERGE" into ub-camera-glacier

If we are using CaptureModule for Gcam only, turn off flash.

Bug: 17460394

Change-Id: I4743e9c31f516d12537b2ceedc9e1fe35fbd8381

Merge "Revert "Delay PhotoModule preview cover removal to address jank"" into ub-camera-glacier

Revert "Delay PhotoModule preview cover removal to address jank"

This CL introduced PhotoModule state corruption, multiple picture taking was not possible. Need to deal with stopPreview during picture taking vs module initialization.

Bug: 17451185

This reverts commit 49fb93b5517e12606eb747f0a5d218b4d03df821.

Change-Id: I0a1eec8147ce859d2e83ee7bdd17655b4054f904

Reset aspect ratio on pause

When resuming the app we do resize things but the old aspect
ratio is cached, thus skipping a reset of the transform matrices
because the app thinks it is already at the correct aspect ratio.
In certain cases (changing orientation when paused) this leads to
layout bugs.

Bug: 17304671
Change-Id: Ia171190b3c76dc05519bdf4d1947d9664d5fb604

Merge "workaround fix" into ub-camera-glacier

Merge "Delay PhotoModule preview cover removal to address jank" into ub-camera-glacier

Delay PhotoModule preview cover removal to address jank

Cover should stay in place at least until startPreview completes.
Since it is async, we should call handlers in a callback.

Bug: 17451185
Change-Id: I220c49b9bb75ad977d92780c6eea0ff126a7dcb8

workaround fix

change View.SYSTEM_UI_FLAG_LAYOUT_FULLSCREEN to View.SYSTEM_UI_FLAG_FULLSCREEN

on L, with View.SYSTEM_UI_FLAG_LAYOUT_FULLSCREEN the entire camera app window is shifted down by the height of the status bar.

with View.SYSTEM_UI_FLAG_FULLSCREEN thw window lays out correctly.

note: with either flag on KitKat the window lays out correctly.

bug: 17457492

Change-Id: I6534fcedaafbbb525967289db7fb759ba50fca42

Merge "Return to filmstrip when video playback finishes." into ub-camera-glacier

Return to filmstrip when video playback finishes.

Bug: 17091348
Change-Id: I9703d014029bf29431b099b129b1ed8bce0ffa03

Fix the upside-down orienation bug in CaptureModule.

Bug: 17369697
Change-Id: I5beb0ec45f2dbf372dda88ffd8a92b1632b3ab90

Merge "Remove jank on camera options switching" into ub-camera-glacier

Changed double to float. viewFinder -> viewfinder.

Change-Id: I5f10a5e8dae40f56457b863d5e8c703e7bf68e49

Reinitialize shutter button in Camera mode upon preview start

VideoModule already does this here, and by adding to PhotoModule we
address a bug where pausing when capturing leaves the button
disabled on resume.

Bug: 17426670
Change-Id: I8333b52a72c0da8b58070d689406975332ae632e

Reduce logging for unknown camera scope

Still log a warning, but remove stack trace logging.

Change-Id: I28e47feb5d2700fd05e7ff4e6faeb1d5a9e38de5

Merge "This fixes Bitmap.createBitmap offset bug in L, not sure why." into ub-camera-glacier

Merge "DO NOT MERGE: Adjust for camera2 API changes" into ub-camera-glacier

Merge "Granting read permission in the result intent for VIDEO_CAPTURE." into ub-camera-glacier

Remove jank on camera options switching

There are two main causes of jank on encountered on options switching:

1. When requesting camera in requestCamera, a portability refactor
introduced a regression where we blocked in the main thread to close
the camera. This main thread blockage is unacceptable as it caused a
preview freeze in addition to the freeze we were trying to create during
the switch, leading to a two-frame preview freeze.

2. The mode options animations which delay the button state transition,
also resulted in preview stutters. Unsure of cause but reverting first
and will assess fixes and reintroduce.

Bug: 17414652
Change-Id: Ib72e331f1bbb5d3b608d8bdc5656c2ff2836ed42

Remove toast.
This reverts commit 779414e8bfe23d66de310570ff2314654e45ec9f.

Change-Id: Idf04e0bbb69822792327fd17634a4f95f7eb2092

Merge "Always match camera orientation with device orientation regardless portrait lock." into ub-camera-glacier

Set video quality pref default values at startup.

The problem of the old approach is the default values never got set
until switching on video module.

The caveat of the this CL is that the default value tweak is only applied
for Shamu. Rather than basing on device model string, detecting
CamcorderProfile capability is a better way to get this job done. However,
|CamcorderProfile.hasProfile| needs camera id info so I'd argue this CL
is good enough for the Shamu/L release.

Bug: 17060480
Change-Id: Ia53ff6aecad44fe7a32257fa1415163f167c09bf

Always match camera orientation with device orientation regardless
portrait lock.

Verified that this CL fixes the orientation issue for portrait/landscape
photos taken in PhotoModule with all configuration combinations such as
Camera API1/API2 devices, K/L devices, front/back camera.

Bug: 17360176
Change-Id: If84244726cf7e2ade470c231b8ac16e9b80366fe

This fixes Bitmap.createBitmap offset bug in L, not sure why.

Bug: 17145703
Change-Id: I18a673a5cac4400a2d6e80a851b1fb12381dc995

Granting read permission in the result intent for VIDEO_CAPTURE.

BUG:17437266
Change-Id: I84d1a69a3d314a05a7e3b5c949c896700426644e

DO NOT MERGE: Adjust for camera2 API changes

Add frameNumber to CaptureCallback#onCaptureStarted

Bug: 17421092
Change-Id: I6e66bd1abc2ecec4dd8346cf4313dc7261ffa26b

Merge "back to bigger HDR+ icons" into ub-camera-glacier

back to bigger HDR+ icons

bug: 17304734
Change-Id: I9b2b22101003b2d13f386367245622d4e13c0309

Import translations. DO NOT MERGE

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

Additional logging for CAPTURE intent handling

Adds verbose logging and some exception log warnings to help diagnose issues
in CAPTURE intents.

Bug: 17421140
Change-Id: Ib0afe4219418b1e7ee5ca4879e39a4448ec13cc0

Merge "In debug_ui mode, show CAF result state, length of AF scan, fps, for camera2 API." into ub-camera-glacier

In debug_ui mode, show CAF result state, length of AF scan, fps, for camera2 API.

Bug: 17397777
Bug: 17028594
Bug: 17391241
Change-Id: Id5a7fe748f988755cb2d786a2103e375eb97b08f

DO NOT MERGE Update Camera2 to use latest APIs

Bug: 17423266
Change-Id: I1f1d79619d89971b387a78756b8f3b4d4b45d76e

Merge "wait for filmstrip bottom panel buttons to finish animating to show clings" into ub-camera-glacier

DO NOT MERGE Pipe context through to camera

  Bug: 17203825

Not merging since a CL depending on this one cannot be merged cleanly into lmp-dev right now: http://ag/541468

Will also apply this one to haleakala.

Change-Id: I7a386b92cf6168185bd28bd02a78d87c5826639a

Revert "Pipe context through to camera"

Second half of the CL didn't make it through to lmp-dev due to Camera API change craziness. Reverting this one to unbreak the build and will submit again to G and H Camera branches only.

This reverts commit f6bf65e9442f2fb7042e37ec21bf1c6c4fbd9bb6.

Change-Id: I5e11c5c1b29586110ed504fbb1be2dfe76accab0

Pipe context through to camera

  Bug: 17203825

Also don't just activate the shutter after a picture has been taken
since some cameras might not be ready for another picture yet.
This decision should be up to the camera.

Change-Id: I5991da1d8fc942b48fe085b18ec93b07bfafe889

Merge "DO NOT MERGE Revert "Update camera2 API usage"" into ub-camera-glacier

Merge "Close the camera when the async open callback runs after pause" into ub-camera-glacier

DO NOT MERGE Revert "Update camera2 API usage"

This temporarily reverts commit for unbundled branch 9c84812bba91831e1759c23e8c92082a8a5f44fd.

Change-Id: Ic4767cd27373c36dd77df4469f5006ced8746f9f

wait for filmstrip bottom panel buttons to finish animating
to show clings

bug: 17321626

Change-Id: Ib810157a2821ed40b2d1e30bee80bf1cc243ee59

Close the camera when the async open callback runs after pause

Current implementation leaves the camera device open if callback
returns after app is paused, since activity/module pausing is
already complete before the camera opened. This CL adds a close
if the opened callback returns when the activity is already paused.

Bug: 17397753
Change-Id: Iec5d16445201b4e308b16e9237fd98ed8599fddc

Merge "Update camera2 API usage" into ub-camera-glacier

Add additional lifecycle logging to Camera

To assist in diagnosing startup issues, added some logging.

Bug: 17397753
Change-Id: Icc47ec278c0b68c8834bfb52554425ff7f371ea0

Update camera2 API usage

Rename listeners to callbacks

Bug: 17389922
Change-Id: I33affcaeec331eca2d084ded4eb199f86fa58765

Merge "Load Gcam OneCamera in CaptureModule when it's also doing normal shots." into ub-camera-glacier

Load Gcam OneCamera in CaptureModule when it's also doing normal shots.

  Bug: 16654225

Change-Id: I939e4793125cc996e26bbd5a13579f2f4ebab663

Merge "remove stray bool resource for determining if OS is L or not, moved to more universal ApiHelper for that." into ub-camera-glacier