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

* commit '0c9af933b7227577003f462ddab12a1ca49208fe':
  Revert "Delay PhotoModule preview cover removal to address jank"

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

am 7351415d: Reset aspect ratio on pause

* commit '7351415ddfaf7f563a4a12c9204e6feb20fdf49d':
  Reset aspect ratio on pause

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

am 87a2300f: Merge "workaround fix" into ub-camera-glacier

* commit '87a2300ff2b49b8bd6a69815d1e4f3db477fcf4d':
  workaround fix

Merge "workaround fix" into ub-camera-glacier

am 3096e672: Merge "Delay PhotoModule preview cover removal to address jank" into ub-camera-glacier

* commit '3096e6729b72a47ee3ed0a68a455bc978036de5c':
  Delay PhotoModule preview cover removal to address jank

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

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

* commit 'd3ee86b7c1f1973119619f8e1b356d9707dd0493':
  Return to filmstrip when video playback finishes.

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

am 5675b8af: Fix the upside-down orienation bug in CaptureModule.

* commit '5675b8af3fddca8d73d0ce0c8aa2f2a38592f556':
  Fix the upside-down orienation bug in CaptureModule.

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

* commit 'd79b38d669e9e61a954b02f3aca63810476f6fc5':
  Remove jank on camera options switching

Fix the upside-down orienation bug in CaptureModule.

Bug: 17369697
Change-Id: I5beb0ec45f2dbf372dda88ffd8a92b1632b3ab90

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

am 9cefdd49: Changed double to float. viewFinder -> viewfinder.

* commit '9cefdd495332719d987ffb942742ec602793d8d4':
  Changed double to float. viewFinder -> viewfinder.

Changed double to float. viewFinder -> viewfinder.

Change-Id: I5f10a5e8dae40f56457b863d5e8c703e7bf68e49

am ffc395d6: Reinitialize shutter button in Camera mode upon preview start

* commit 'ffc395d6e7eaafcdcd283e183188de16aa14aa4c':
  Reinitialize shutter button in Camera mode upon preview start

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

am 5c4bd6b5: Reduce logging for unknown camera scope

* commit '5c4bd6b5348a3464783270ea0bb10d4968d3b0cb':
  Reduce logging for unknown camera scope

Reduce logging for unknown camera scope

Still log a warning, but remove stack trace logging.

Change-Id: I28e47feb5d2700fd05e7ff4e6faeb1d5a9e38de5

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

* commit 'a710f371b8e454d37b8b2be50e001b79e37c67f2':
  This fixes Bitmap.createBitmap offset bug in L, not sure why.

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

am fd97c6da: (-s ours) Merge "DO NOT MERGE: Adjust for camera2 API changes" into ub-camera-glacier

* commit 'fd97c6da954f3e3e6f8547f6f7363cddba6c1b9a':
  DO NOT MERGE: Adjust for camera2 API changes

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

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

* commit '42a5ad5a9c7e660854fd0552b81e025c16db9f05':
  Granting read permission in the result intent for VIDEO_CAPTURE.

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

am 5e3bf4cc: Remove toast. This reverts commit 779414e8bfe23d66de310570ff2314654e45ec9f.

* commit '5e3bf4cc84a186c5e89dfa692f3af95007751877':
  Remove toast. This reverts commit 779414e8bfe23d66de310570ff2314654e45ec9f.

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

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

* commit 'b954b3e8b9fff8adf82808bd6cac1f9a7ac9505a':
  Always match camera orientation with device orientation regardless portrait lock.

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

am 65c00136: Set video quality pref default values at startup.

* commit '65c00136cde07e050d05e3da61876199df90dd18':
  Set video quality pref default values at startup.

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

Merge "Adjust for camera2 API changes" into lmp-dev

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

am 767a1186: Merge "back to bigger HDR+ icons" into ub-camera-glacier

* commit '767a1186a1127f070785d50054a0e1a02975eb4a':
  back to bigger HDR+ icons

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

back to bigger HDR+ icons

bug: 17304734
Change-Id: I9b2b22101003b2d13f386367245622d4e13c0309

Adjust for camera2 API changes

Add frameNumber to CaptureCallback#onCaptureStarted

Bug: 17421092
Change-Id: I6e66bd1abc2ecec4dd8346cf4313dc7261ffa26b

am f974ba71: Additional logging for CAPTURE intent handling

* commit 'f974ba719a1255b6bda1b214842641fb5736a877':
  Additional logging for CAPTURE intent handling

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

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

* commit '6281a7942de9e55386c20c6f41bfb656b126a364':
  In debug_ui mode, show CAF result state, length of AF scan, fps, for camera2 API.

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

Merge "DO NOT MERGE Pipe context through to camera" into lmp-dev

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

am 76215c98: (-s ours) DO NOT MERGE Update Camera2 to use latest APIs

* commit '76215c98a9a11d87b5ce9d2b4198654c2b8bbddf':
  DO NOT MERGE Update Camera2 to use latest APIs

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
(cherry picked from commit 13d07d0ecb3bc808fb6f5b65508b1948ab3149d8)

DO NOT MERGE Update Camera2 to use latest APIs

Bug: 17423266
Change-Id: I1f1d79619d89971b387a78756b8f3b4d4b45d76e

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

* commit '954b9b1ce7cd3e7b4a4c0fa35f8ca873fba5ac14':
  wait for filmstrip bottom panel buttons to finish animating to show clings

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

am 13d07d0e: (-s ours) DO NOT MERGE Pipe context through to camera

* commit '13d07d0ecb3bc808fb6f5b65508b1948ab3149d8':
  DO NOT MERGE Pipe context through to camera

am bcc384ea: Revert "Pipe context through to camera"

* commit 'bcc384ea15fc9d25ee3efd934c2ac7ec7a059a52':
  Revert "Pipe context through to camera"

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

am f6bf65e9: Pipe context through to camera

* commit 'f6bf65e9442f2fb7042e37ec21bf1c6c4fbd9bb6':
  Pipe context through to camera

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

am 9eb0eb55: (-s ours) Merge "DO NOT MERGE Revert "Update camera2 API usage"" into ub-camera-glacier

* commit '9eb0eb55e686c84d0003cdcd60c76e08ed369468':
  DO NOT MERGE Revert "Update camera2 API usage"

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

* commit '200d5f833a70616465fecb200d45bc2726bd1499':
  Close the camera when the async open callback runs after pause

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

am 82465c1c: Merge "Update camera2 API usage" into ub-camera-glacier

* commit '82465c1c603d3d76bf8c7487779766d5ea38a5cd':
  Update camera2 API usage

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

am 29a009c9: Add additional lifecycle logging to Camera

* commit '29a009c974f153f72cd5e28851a8cb85a691e215':
  Add additional lifecycle logging to Camera

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

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

* commit 'dc0369f37b07adb4355113540af3c1f7c82a404e':
  Load Gcam OneCamera in CaptureModule when it's also doing normal shots.

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

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

* commit 'eacf94608e5358a028c64dd1523270e61093e8b2':
  remove stray bool resource for determining if OS is L or not, moved to more universal ApiHelper for that.

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

remove stray bool resource for determining if OS is L or not,
moved to more universal ApiHelper for that.

this brings back the fade-in on shutter button touchDown.

bug: 17389859

Change-Id: I7c7d24db30bfbb149f84e6f5acf3e211c282c1ec

am 17d9084e: Add short flash animation and shutter sound to ZSL camera.

* commit '17d9084e38e64f06a9d411d64fe3c2739d299cda':
  Add short flash animation and shutter sound to ZSL camera.

Add short flash animation and shutter sound to ZSL camera.

Bug: 17217567
Change-Id: I9a75fdd15e03f47029ef4a631287899718794c5c

am f096e898: (-s ours) Merge "Import translations. DO NOT MERGE" into ub-camera-glacier

* commit 'f096e898055178994d372525139fd47f625aa85f':
  Import translations. DO NOT MERGE

Merge "Import translations. DO NOT MERGE" into lmp-dev

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

am f1e3dc86: Merge "Added log for first preview frame" into ub-camera-glacier

* commit 'f1e3dc865aa7dfcb3fee584b8e37fd24c23e78c0':
  Added log for first preview frame

Merge "Added log for first preview frame" into ub-camera-glacier

Added log for first preview frame

Bug: 17386943
Change-Id: I17907402c9d1cd4678634e2ceb8cbf921f010d9f

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

am ae6e94f5: Merge "Re-enable shutter-button when switching between front and back cameras." into ub-camera-glacier

* commit 'ae6e94f570a8436715384993f2e06b2193915b78':
  Re-enable shutter-button when switching between front and back cameras.

am 2ef0f9dc: Fix unnecessary string cast

* commit '2ef0f9dc95527a354846940bae675741c464e6ae':
  Fix unnecessary string cast

Merge "Re-enable shutter-button when switching between front and back cameras." into ub-camera-glacier

Fix unnecessary string cast

Code was taking a setting as a string, which when empty returns null,
then tried to parseInt it. Instead ask for an int, which will default
to zero, and things will work.

Bug: 16827108
Change-Id: I5bbed091ec29fc98257a5eb18948891c43caa29f

am 2b084ec5: Merge "Fixing settings upgrades" into ub-camera-glacier

* commit '2b084ec5c7c262003581be3584825300269adae2':
  Fixing settings upgrades

Merge "Fixing settings upgrades" into ub-camera-glacier

Fixing settings upgrades

Did a full audit of upgrade scenarios and performed the following fixes.
* restored pre-existing location upgrade code that handled old upgrades.
preserves location choice from C straight to G.
* added string type format conversion for "old global" values that were
not implemented in the refactor : hdr, hdr+, grid buttons
* moved a "reset" that existed in upgraders that put the user in Camera
mode on upgrade. it should run for everglades upgraders along with size
changes.

Bug: 17317510
Change-Id: I87db511bfa89676525ee727620cb546c180a8ef6

Re-enable shutter-button when switching between front and back cameras.

Bug: 16961251
Change-Id: Ie3075bf711ab44dfe5e6cdd6b3645dd6583708b5

am 4d6d676c: Refactor default picture size selection in zsl cam.

* commit '4d6d676c368b70e8c0fcf462e51de1d59fe34028':
  Refactor default picture size selection in zsl cam.