Merge "Trap camera close exceptions on pause" into lmp-dev

Merge "Avoid porting null keys when copying preferences into new system" into lmp-dev

Only set preview size before starting preview

Remove recursive call to startPreview when updating preview size.
Avoid invoking preview size parameter updates for all calls to
updateCameraParametersPreference, and only adjust preview size
immediately before starting preview. Also added info logging
for all call paths into startPreview() to better diagnose
sequence of events in logs if we still see any races or
ordering issues.

Bug: 17809876
Change-Id: I626c913bb154c54aaffb451c24a63c6750553a52
(cherry picked from commit 19597371a18921bb03337e4fa24c28dd2d8f06ed)

Avoid porting null keys when copying preferences into new system

Testing on Fishlake shows certain preferences are of 'null' type
in shared preferences. Avoid an NPE on porting a null. From testing,
this can happen with focus mode, best to just skip porting the key
and let the app use a default.

Bug: 17829562
Change-Id: I483cb5beb160d3fb7fc8a42aa5085531e3fccb63
(cherry picked from commit d560aabf7baef81a14e78341af81994efc1d4f17)

Trap camera close exceptions on pause

Bug: 17759456

Change-Id: I5e5fd39829bd0ebf3f37b354ca380cf6396bef55
(cherry picked from commit 48d4c8f1856503d88f4e2e42fa729f2556173694)

Use NEW_DOCUMENT vs NEW_TASK when starting Activities from Camera

To ensure items are placed in separate affinity in document mode.

Bug: 17758840
Change-Id: I7ea1c9c841f3e60271307ed67e760be494835855

Fix error in settings key from refactor

PANO_ORIENTATION indicator icon was using an
incorrect key to sync state.

Bug: 17752154
Change-Id: I587b2ee7eeab3758b05386540f62b79ca48299f2
(cherry picked from commit 5c9d7f34039ac96b3a136ccd6b16162e3b220848)

Only allow one draw call pending at a time

If modules call draw aggressively, there should only be one pending
at a time. If we get another, just drop it, there will be a draw
momentarily.

Bug: 17703978
Change-Id: I60b3b9e257ae4e064287a97973b1de878ce66de1
(cherry picked from commit 5db8e20db77bf86eba25ec713c76da783dd98b19)

Stability: call stopPreview() before switching off video module.

If the preview is not stopped, the following camera operations posted
by other modules such as "apply settings" will turn camera object
in turmoil. The camera object could start throwing exception and eventually
hang.

Bug: 16300704
Bug: 17403384
Change-Id: Idd2360869d51b9dfc06309f8ac185a65add8d0c9
(cherry picked from commit 36e045a43d9b8949d9a219b84cf894cf21f8e98a)

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

Import translations. DO NOT MERGE

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

am 2d3475ae: Fix crash in VideoModule - Method called after release()

* commit '2d3475ae1f045a2062119f352bec874cd78e9d3b':
  Fix crash in VideoModule - Method called after release()

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

am 022ce61e: Synchronize CaptureModule camera open/close operations

* commit '022ce61e1c4f338a79585e8656ab57990ecd32c1':
  Synchronize CaptureModule camera open/close operations

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

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

* commit 'feb07ea4e32c7dcc4488515220a4a1881b5e6585':
  Stability: add null check for all CameraProxy obj references in VideoModule.

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

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

* commit 'ad12363ecaba370c2425677eef6e562bf7c98e3e':
  Revert app workaround for bug 17614203 now that HAL is fixed. This reverts ag/543302.

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

Bug: 17614203
Bug: 17145703
Change-Id: I3ec21902e59156ea30c181ecd6a7728f7262110a

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

* commit '47c4474ea9e51a58f2ba1795de90c9aaf135f442':
  Force filmstrip images to generate in JPEG compression format

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

am 4d2f3bf4: Merge "Recreate a OneCameraManager instance for each Activity." into ub-camera-glacier

* commit '4d2f3bf448d8546920a3d94ce15033357388d2cc':
  Recreate a OneCameraManager instance for each Activity.

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

* commit 'ee3123ba4b4cc107423441bb8d687c14a2dc4b9a':
  Stability: add null check for all CameraProxy obj references in PhotoModule.

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

am da6f3acc: Skip disk caching for session data

* commit 'da6f3accc02c942b1f2dd1c248edb6a0bd66ee10':
  Skip disk caching for session data

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

am 6b2d1e0b: Merge "Revert temporary session images in filmstrip to RGB_565" into ub-camera-glacier

* commit '6b2d1e0b660e931c6cbc91a25ec85df070712e65':
  Revert temporary session images in filmstrip to RGB_565

am 347dccbe: Ensure Camera doesn\'t attempt access of back camera on Nakasi

* commit '347dccbe1ed10ceec190af4d3797a197426f1caf':
  Ensure Camera doesn't attempt access of back camera on Nakasi

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

am 2631a15a: Don\'t show aspect ratio UI if using API2 portability

* commit '2631a15a37b890f4ee720c5a4bbe3e654524deb6':
  Don't show aspect ratio UI if using API2 portability

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

am eeeed4fa: new sounds for timer ticker and pano / photosphere intermediate captures

* commit 'eeeed4fa00ed842050bc7e0edd3b37c262802a5b':
  new sounds for timer ticker and pano / photosphere intermediate captures

new sounds for timer ticker and pano / photosphere intermediate captures

companions CL: ag/534823

bug: 17281234

Change-Id: I9d98d8e869dcefdbd0f7e0b53dea6b8e024a8e3b

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

* commit '7d1fb593ae26e4f327e47bcbb857057da4ea23a0':
  monkey test NPE fix, another null check on mCameraDevice

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

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

* commit 'b60af188e3bc712a7f06700c591c428834237dff':
  Fix filmstrip item bounds to filmstrip scale

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

am 5f86f7fd: Cleanup gesture UI hide code

* commit '5f86f7fdd19dad9cdf670e0504cfaddefd4a89fc':
  Cleanup gesture UI hide code

am b2b0e02c: Clean up unused code

* commit 'b2b0e02cb5f0aba8fed9389193fa98150da75efc':
  Clean up unused code

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

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

* commit '02758e4c60ea1ff5a78c6d7d8d77ee353a2bf6f5':
  Import translations. DO NOT MERGE

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

am 30ae8c64: Merge "Changed tap-to-meter rectangle and weights." into ub-camera-glacier

* commit '30ae8c64cec9313fc050daf94ae3fbddedf80b36':
  Changed tap-to-meter rectangle and weights.

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

am e684ea54: Merge "Fix CameraSettingsActivityHelper" into ub-camera-glacier

* commit 'e684ea54121ef885a0aa84cd3bea50c5a345d55e':
  Fix CameraSettingsActivityHelper

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

* commit '19942a171844d6bd361026a6b8df2eebaa0661db':
  implement new trash icon, plus some asset cleanup

Merge "Fix CameraSettingsActivityHelper" into ub-camera-glacier

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

am 8bbb5122: Merge "add null check to updateCameraParametersPreferences to apease the monkey" into ub-camera-glacier

* commit '8bbb51221106c12b0cf7e933fc2af2a63b84b1c1':
  add null check to updateCameraParametersPreferences to apease the monkey

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

am 963775c3: portability layer doesn\'t work well with startPreviewWithCallback() here\'s a dirty dirty workaround.

* commit '963775c3092e2553ae8157df9e5d307d4f711de1':
  portability layer doesn't work well with startPreviewWithCallback() here's a dirty dirty workaround.

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

bug: 17576554
Change-Id: Ief40b65ce49712153f6ef604a3513348e493d4d9

am 3c3b31df: mode switch jank fixes

* commit '3c3b31dfd5ec7a43e89cd4695cc8c498afc3a91d':
  mode switch jank fixes

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 "Import translations. DO NOT MERGE" into lmp-dev

am 6f1dfe69: Merge "Don\'t set mCameraAppUI to null in onDestroy" into ub-camera-glacier

* commit '6f1dfe69b93bfb42c11d456901e9d3ffee826cce':
  Don't set mCameraAppUI to null in onDestroy

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

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

* commit 'a16219d51937bda5f3e69c0e77171ae0fb893823':
  set proper alpha on shutter button on mode switch, eliminates a janky alpha flash

Import translations. DO NOT MERGE

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

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

bug: 17487040

Change-Id: I08e66b5be53e58e9e16d32c09ef50081e90e9284

am 70a96525: Don\'t send redundant requests to start the preview

* commit '70a96525f16ca6c6abb9a7a9dab26ba1e61235dd':
  Don't send redundant requests to start the preview

Don't send redundant requests to start the preview

Change-Id: I43d8a2917bbe0140f1e9367cce21f68b19807aa8

am 486ba1da: Revert layout change for Camera

* commit '486ba1da145f70b07235617e5f1f64186daa0fc0':
  Revert layout change for Camera

am ca5173d8: Move launcher icons to mipmap

* commit 'ca5173d84722383d2d9ef49dd85fba28af035429':
  Move launcher icons to mipmap

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

am 45f316bc: Establish synchronous camera closing when app utilizes API1/API2

* commit '45f316bcecb8713cbb217507981fd657fcdaa274':
  Establish synchronous camera closing when app utilizes API1/API2

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

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

* commit '0c5fc1d311eaa82dff09cee6332140fda21d473e':
  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 e6ebf3d7: If we are using CaptureModule for Gcam only, turn off flash.

* commit 'e6ebf3d73bb3f337413bdde437fbb8fde6cee436':
  If we are using CaptureModule for Gcam only, turn off flash.

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

Bug: 17460394

Change-Id: I4743e9c31f516d12537b2ceedc9e1fe35fbd8381

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