Refine in-place conversion semantics for Settings

Since settings are now stored as strings, avoid app crashes
for settings which are still around as other types, and just
treat it as if those settings are absent (return default). This
requires re-working the version number tracker itself, which
is done by making it a new key and migrating just the old
key in the getLastVersion override. This is somewhat cleaner
and overall the change should increase stability given the
change to use in place values of different types.

Bug: 17949433
Change-Id: Icaf0d9653eabb9e144f3946a51b4038d3ed48ea9

Merge "Pass NamedEntity to saveFinalPhoto since it can be nulled in pause()." into ub-camera-glacier

Migrate old CameraErrorCallback class to agent's CameraExceptionHandler.

Per ag/565603, clean up the old CameraErrorCallback.

Bug: 17931297
Change-Id: Id750581850680ed6cf85738daf5ebdebf67f4800

Pass NamedEntity to saveFinalPhoto since it can be nulled in pause().

  Bug: 15289634

Change-Id: I551f73f5f0ceecf2d8a1e3b379dd34fd06b044d8

Merge "Add verbose logging for os/apk debug builds" into ub-camera-glacier

Add verbose logging for os/apk debug builds

Relax VERBOSE screening so that by default, we log more when not
in a release configuration.

Bug: 17950234
Change-Id: I30250974456607b4fb64c15d6ecfa288ceb2f133

Merge "change text color to white for tutorial and onboarding screens." into ub-camera-glacier

change text color to white for tutorial and onboarding screens.

Change-Id: If431ce0b321991e15f33f850acc3cc6c4474ffb4

Use a separate activity for capture intents

Bug: 17911061
Change-Id: Id5d3e5347c1830dc43b8962c1494bb4bf5195136

Make sure a null mimetype is not crashing the app.

  Bug: 17398971

Though at this point something else is already horribly wrong with
mediaserver, otherwise this cannot happen.

Change-Id: I837dfd5992611ded0c34ea7cc1e677ac1456c0c0

Revert "More lockscreen tuning"

This reverts commit 7ac13f5f3644c8de53ebd625f195d757166bf9c9.

Change-Id: I3c59f4d71efe80290731e55421011c16e5dd4a41

reverting b/559979

Change-Id: If4e58c67d4424ade68198a16c1d2d21595504efd

Merge "Tweaked constants for Gcam." into ub-camera-glacier

Adopt the new error handling mechanism in portability.

Show camera fatal dialog to guide users exiting the app when
1) camera framework throws RTE or
2) DispatchThread job is timeout.

If users hit home button while error dialog is shown, just exit
the app.

Bug: 16189216
Change-Id: Ia0f1c64d452735ec62549f50ab62646d8d873c65

Merge "Catch error where framework is broken and returns no cameras." into ub-camera-glacier

Catch error where framework is broken and returns no cameras.

  Bug: 17868174

Exit the app with the "Cannot connect to camera" error.

Change-Id: Ic5a6e0af222a3740b8bd3f6e66ee575f9443008e

Merge "More lockscreen tuning" into ub-camera-glacier

More lockscreen tuning

Delay all onResume calls since we cannot always determine if we
will be immediately paused due to a lockscreen start.

Bug: 16035858
Change-Id: I96928814708eb8026bea8d0087341fda4b04bcbc

Merge "adjust margin on TextViews and ImageViews in onboarding and tutorial screens" into ub-camera-glacier

adjust margin on TextViews and ImageViews in onboarding and tutorial screens

bug: 17654418
Change-Id: I10633495ae04fd00ac97c055094a6097f5f7c0a0

Lockscreen workaround tuning

 * Remove delay of onStartTasks.  This fixes b/17868175, and appears to
   result in ~50ms improvements to cold starts from the lockscreen and
   warm starts from the secure lockscreen.
 * Disabled double-onResume() workaround for non-secure lockscreen launches
   since this no longer happens when singleTask is enabled.
 * Reduced the delay for the double-onResume() workaround by eliminating
   the delay on the second call to onResume()

Bug:17868175
Bug:16035858
Change-Id: I7f7611901133e429def9a03d4009e1a99caaabd0

Merge "disable all user interactions (including mode switching) on SD Card errors" into ub-camera-glacier

disable all user interactions (including mode switching) on SD Card errors

bug: 15841887

Change-Id: I8e7e92a884dda54d9b8385b29d9ef4b7bb7fbd14

color changes for modes, onboarding screens, and tutorial screens

bug: 17785834

Change-Id: Idaf26517399a07471f2b6a98fdb9629baf41afcf

Merge "implement proper mode color for cancel button" into ub-camera-glacier

Merge "Make Camera V2 creation more robust agains failures." into ub-camera-glacier

Unload timer sounds on pause in PhotoModule; release SoundPLayer on destroy.

Bug: 17740679
Change-Id: Ia15749971f45f6579a19c379ee95c83c9ce20ccc

Tweaked constants for Gcam.

Bug: 17629128
Bug: 17550818

- In HDR+ mode only, change tap-to-expose/focus rectangle to 12% of the
  viewfinder - works much better and matches final output.
- Changed post-shot return-to-continuous-AF constant to 1 second (3 seconds was
  too long).

Submit together with ag/554414.

Change-Id: I56afb19e4378dbfa25f38376b05293fdb3908d47

Re-run settings files migrations

Increment version number for upgrade and re-run copying. This will
be a no-op for a first time glacier upgrader, copying runs in the
same sequence for this case. For alpha users who hit crashes, this
will re-run the copy, potentially overwriting some values but
ensuring all values are valid. Tested both N4 and N5 upgrade
scenarios that had previously seen issues, this addresses issues
without the need to clear data.

Bug: 17875895
Change-Id: I37c469c1b03151d03ee0d32ce0ba62983af32a36

Make Camera V2 creation more robust agains failures.

  Bug: 17878708

Change-Id: I9e64f89a1b3eb56e7f3afc774d8823a7666d8208

Fix preference copy migration and incorrect HDR+ sticky bit

A sticky bit upgrader particularly for HDR+ was looking in
globals but was actually a module setting. The copy migrator
was not properly handling booleans! Modified the copy migrator
to properly handle supported types and removed the custom
incorrect upgrader.

Bug: 17875895
Change-Id: Id36abc429d12a77c5462e5c49ef7786e3d092f2c

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

Fix incorrect key translation in Settings refactor

Bug: 17832052
Change-Id: Ie3ac25a1b9b813295b2bbd6ce723669d30cdb652

Merge "Avoid porting null keys when copying preferences into new system" into ub-camera-glacier

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

Merge "Use singleTask to improve unsecure lockscreen startup" into ub-camera-glacier

Use singleTask to improve unsecure lockscreen startup

This enables warm-starts from the lockscreen by reusing the same
activity if possible.

Bug: 16035858
Change-Id: I4b1ce0c290e209a5098e3f3e71509e0ca86e1845

Suppress 'help feedback' message pre-L in secure camera

Becase we see issues when starting activities above lockscreen
from filmstrip menu in K, disable unless we are on L.

Bug: 17767821
Change-Id: Ic59defd3fef5e283b356e94a952a2b75f4e9f1d6

Merge "Add zoom value and timer duration to capture parameters, for logging." into ub-camera-glacier

Merge "Close dangling camera proxy" into ub-camera-glacier

Close dangling camera proxy

Bug: 17677971
Change-Id: I2cf2df7a12a44b3f9f33d0d9eac479c5281da356

Add zoom value and timer duration to capture parameters, for logging.

Bug: 17204955
Change-Id: I13e087a8e87c9722b83b83d90af491966ff30c9e

implement proper mode color for cancel button

bug: 17780517
Change-Id: I522fb223b4aa78d1acd8f18ebf20dbfe20ea8c99

Trap CameraAgentFactory RTEs during recycle

Bug: 17759456
Change-Id: I5d6577f4175963168bb4186d70dd7cee68b5466e

Trap camera close exceptions on pause

Bug: 17759456

Change-Id: I5e5fd39829bd0ebf3f37b354ca380cf6396bef55

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

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

Merge "Stability: call stopPreview() before switching off video module." into ub-camera-glacier

Let VideoModule override default mode options enable/disable

bug: 17553759
Change-Id: Iadb2690d81e3efce4e84c8c49f6e684a700c247b

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

Cleanup:  Move api check to ApiHelper

Change-Id: Ifc0b080713329d09cdeaeec5ccdf88d8515e5543

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