am 4a1fd997: Merge "Fix the build break due to ag/567355." into ub-camera-glacier

* commit '4a1fd9976fc47a732058e764c00a36f538ca430c':
  Fix the build break due to ag/567355.

Merge "Fix the build break due to ag/567355." into ub-camera-glacier

Fix the build break due to ag/567355.

Change-Id: I3f8b657bea9fdd22287115230709dda0a26b8ae4

am 828081f5: Merge "Log camera exception via UsageStatistics." into ub-camera-glacier

* commit '828081f5f947abb8930f084386492efde8c5b83b':
  Log camera exception via UsageStatistics.

Merge "Log camera exception via UsageStatistics." into ub-camera-glacier

am c6a48876: Merge "disable shutter button as soon as possible to avoid double shutter presses" into ub-camera-glacier

* commit 'c6a488761b2b9deb11bdcac051dfeacd212ee90f':
  disable shutter button as soon as possible to avoid double shutter presses

am 54985c97: Merge "Revert "Make CameraDataAdapter background new data loading and post to listener"" into ub-camera-glacier

* commit '54985c973140b896db75f086ac0f904d0228f8aa':
  Revert "Make CameraDataAdapter background new data loading and post to listener"

am 3972da65: Merge "Migrate Settings Fragment to use arguments" into ub-camera-glacier

* commit '3972da65f958edcfed26f1c32e0cf4e8361306e7':
  Migrate Settings Fragment to use arguments

Log camera exception via UsageStatistics.

Change-Id: I2b80338dd489ece87d94e20a303c5e43e972c3b0

Merge "disable shutter button as soon as possible to avoid double shutter presses" into ub-camera-glacier

Merge "Revert "Make CameraDataAdapter background new data loading and post to listener"" into ub-camera-glacier

Revert "Make CameraDataAdapter background new data loading and post to listener"

Reverting out of concerns on thread safety, need to more carefully assess the change. Synchronizing may just reintroduce deadlock for affected user.

This reverts commit cc93255f596b47d1a860461dd0d4be697422964f.

Change-Id: I8b003f978d680942d380cc3563bccd055ab28fe9

Merge "Migrate Settings Fragment to use arguments" into ub-camera-glacier

Migrate Settings Fragment to use arguments

Fix error in settings fragment using a constructor vs argument bundle.

Bug: 17626333
Change-Id: I5842706a3b5060dac1f997ca4035b4cc31ce0287

am 7a3ce60c: Merge "Prevent NullPointerException of PhotoModule in quick resume-pause." into ub-camera-glacier

* commit '7a3ce60cfe7f5a554ab8edeb08c203196b17b164':
  Prevent NullPointerException of PhotoModule in quick resume-pause.

Merge "Prevent NullPointerException of PhotoModule in quick resume-pause." into ub-camera-glacier

Prevent NullPointerException of PhotoModule in quick resume-pause.

Bug: 17984743
Change-Id: Ie597dbddb383004deace8933f51f091029c0a9b4

am 4ed861a7: Preview vs Video Camera Resolution Mismatch

* commit '4ed861a70de7db31a7d4b9227b523bef684d7eed':
  Preview vs Video Camera Resolution Mismatch

Preview vs Video Camera Resolution Mismatch

  Bug: 17458889

This bug is programmatic and reproduces on all devices.   Basically,
the current app defaults to the highest resolution preview that
matches the aspect ratio of the recording, even if the recording
video resolution is lower.  Since it is more important to be WYSIWYG,
in the cases where the preview show at the same resolution of
the recording video resolution, we should preview at same resolution
for truest (but not necessarily best) quality that most faithfully
represents the content that is to be recorded under the constraint
of the UI resolution.

Change-Id: I0ec5a76834cbffa59f0e144f1fe271de25e92523

disable shutter button as soon as possible
to avoid double shutter presses

bug: 17939465
Change-Id: Iac19b8fa78774b525fbaf0d4742d97764d401b78

am e9dedb39: Merge "Don\'t re-open the camera if already open" into ub-camera-glacier

* commit 'e9dedb39b240e87e854918d544de66820b41700c':
  Don't re-open the camera if already open

Merge "Don't re-open the camera if already open" into ub-camera-glacier

am b7378210: Merge "Make CameraDataAdapter background new data loading and post to listener" into ub-camera-glacier

* commit 'b7378210663a1ac9c6692b9a0d8ae5e414da9370':
  Make CameraDataAdapter background new data loading and post to listener

Merge "Make CameraDataAdapter background new data loading and post to listener" into ub-camera-glacier

Make CameraDataAdapter background new data loading and post to listener

To avoid potentially expensive operations in main thread, only call
back to filmstrip on the main thread. Do all processing in background.
Consistently enforce that all listener callbacks are main thread by
posting to a main thread handler.

Bug: 17437217
Change-Id: I6a96690218043cc42644d12a4162322cc2ee7e8f

Don't re-open the camera if already open

Bug: 17870967
Change-Id: Ibafc8c563b6188c4c743a60a3b42b93e107523bc

am a5db8beb: Merge "Refine camera fatal error handling in CameraActivity." into ub-camera-glacier

* commit 'a5db8bebb51ff666b4fa4bbadc3d0457f36041f2':
  Refine camera fatal error handling in CameraActivity.

Merge "Refine camera fatal error handling in CameraActivity." into ub-camera-glacier

am 890ab9be: Merge "Refine in-place conversion semantics for Settings" into ub-camera-glacier

* commit '890ab9beb8642bc9278f9914013be2fd65d740f8':
  Refine in-place conversion semantics for Settings

Merge "Refine in-place conversion semantics for Settings" into ub-camera-glacier

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

am 7839867c: Merge "Update to use real L codename." into ub-camera-glacier

* commit '7839867c4c6eb837d5e7c1c5851741d4933f5fb1':
  Update to use real L codename.

Merge "Update to use real L codename." into ub-camera-glacier

Refine camera fatal error handling in CameraActivity.

Use Log.e() to clearly indicate what action has been taken to handle
the error.

Bug: 17964147
Change-Id: I073e06d99a89e6e1e697766736791ef419339944

am 89ac2e10: Merge "change cahching Drawables to caching ConstantStates, also apply CircelDrawable as necessary at runtime" into ub-camera-glacier

* commit '89ac2e10477b03fbd5addb7da46933626748ed3a':
  change cahching Drawables to caching ConstantStates, also apply CircelDrawable as necessary at runtime

Merge "change cahching Drawables to caching ConstantStates, also apply CircelDrawable as necessary at runtime" into ub-camera-glacier

am eb81de1d: Merge "Native 16x9 viewfinder stream for Gcam to reduce switching jank and eliminate black bar overlay hack." into ub-camera-glacier

* commit 'eb81de1d250a5c61f378428f590238ce5c7e16a4':
  Native 16x9 viewfinder stream for Gcam to reduce switching jank and eliminate black bar overlay hack.

am 6a07363b: Always close the camera

* commit '6a07363be15d11735b0b47059bf8d0e9998693ef':
  Always close the camera

Merge "Native 16x9 viewfinder stream for Gcam to reduce switching jank and eliminate black bar overlay hack." into ub-camera-glacier

Native 16x9 viewfinder stream for Gcam to reduce switching jank and eliminate black bar overlay hack.

companion CL: ag/562183

bug: 17669070
Change-Id: I4d5402df3848423c236108716f7065a87ce95f6d

Always close the camera

Previous logic had a race condition in which the camera may not be
closed if it was in the process of being opened.

Bug: 17939709
Change-Id: Ife9d96b73dac17625b3344fcf8553d181a74024e

change cahching Drawables to caching ConstantStates,
also apply CircelDrawable as necessary at runtime

bug: 17907551

Change-Id: I9bddfc599f15c3e0b09b98f14a9804296d51015b

Merge "Make temp session directory creation more robust and efficient." into ub-camera-haleakala

Make temp session directory creation more robust and efficient.

Also rename some methods for clarity.

Change-Id: Ic6d3e7902ae5396f33231f1471df9eae89234132

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

* commit '4d4585da32d9b5a47bffad1deeb08fc5cd128311':
  Pass NamedEntity to saveFinalPhoto since it can be nulled in pause().

am ac9070ee: Migrate old CameraErrorCallback class to agent\'s CameraExceptionHandler.

* commit 'ac9070ee24d2db6e7804cdd26df578e7ab76895e':
  Migrate old CameraErrorCallback class to agent's CameraExceptionHandler.

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

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

* commit '4fe4680cca29994e0c070b9b5ea5ef9bcf087a65':
  Add verbose logging for os/apk debug builds

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

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

* commit 'a9731952fb824f78d033ae360ff8324ae6cf2d6a':
  change text color to white for tutorial and onboarding screens.

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

Merge "resolved conflicts for merge of d0927687 to lmp-mr1-dev-plus-aosp" into ub-camera-haleakala

am 59d337b4: Use a separate activity for capture intents

* commit '59d337b422cc373666609fb0ef7fcc422884b683':
  Use a separate activity for capture intents

Use a separate activity for capture intents

Bug: 17911061
Change-Id: Id5d3e5347c1830dc43b8962c1494bb4bf5195136

Update to use real L codename.

Change-Id: If6e5f02f693848f6cb0014084dccf910be4c958d

am e11ad6d6: Make sure a null mimetype is not crashing the app.

* commit 'e11ad6d67ad95a9ecab8a594bcb55a05708abc0f':
  Make sure a null mimetype is not crashing the app.

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

am 0d4cdba2: Revert "More lockscreen tuning"

* commit '0d4cdba26b5fe4b7ca03796c5a2644a09e2c7144':
  Revert "More lockscreen tuning"

Revert "More lockscreen tuning"

This reverts commit 7ac13f5f3644c8de53ebd625f195d757166bf9c9.

Change-Id: I3c59f4d71efe80290731e55421011c16e5dd4a41

am fe7eb164: reverting b/559979

* commit 'fe7eb1640b1aca8500b2cab152648a699833d6fd':
  reverting b/559979

reverting b/559979

Change-Id: If4e58c67d4424ade68198a16c1d2d21595504efd

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

* commit 'fd8c804c9429f150f6028c6f371d34352934f964':
  Tweaked constants for Gcam.

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

am 2a0e18e2: Adopt the new error handling mechanism in portability.

* commit '2a0e18e280dfe541d4740345511ad299099cda79':
  Adopt the new error handling mechanism in portability.

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

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

* commit '1b19bc806b42ad405a3ea60e5e6c4e278e0aaee2':
  Catch error where framework is broken and returns no cameras.

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

am b2f15652: Merge "More lockscreen tuning" into ub-camera-glacier

* commit 'b2f156527da437c1c4901c03bd004eb49eae23f6':
  More lockscreen tuning

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

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

* commit 'c9a3303761d72edd579406bdf6b6a1f29a16a8bc':
  adjust margin on TextViews and ImageViews in onboarding and tutorial screens

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

am 3e236cc8: Lockscreen workaround tuning

* commit '3e236cc8e33be40e434e73f7777bd81446277512':
  Lockscreen workaround tuning

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

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

* commit '0dd0d5ef549b489d7711d60ea1988834313851a5':
  disable all user interactions (including mode switching) on SD Card errors

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

am 97563a79: color changes for modes, onboarding screens, and tutorial screens

* commit '97563a7926f87ac350bff7f644d43209fe874185':
  color changes for modes, onboarding screens, and tutorial screens

color changes for modes, onboarding screens, and tutorial screens

bug: 17785834

Change-Id: Idaf26517399a07471f2b6a98fdb9629baf41afcf

am 39fe5d96: Merge "implement proper mode color for cancel button" into ub-camera-glacier

* commit '39fe5d96cc0787f0ea3f8eb0b18108ef2abd3be6':
  implement proper mode color for cancel button

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

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

* commit '2dcf912d17f2ed7197c783d1273ff6200709e6d3':
  Make Camera V2 creation more robust agains failures.

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

am 877c2b65: Unload timer sounds on pause in PhotoModule; release SoundPLayer on destroy.

* commit '877c2b65415ad43eb424f8a4a27bbeee7771d5bf':
  Unload timer sounds on pause in PhotoModule; release SoundPLayer on destroy.

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

am 5381549f: Re-run settings files migrations

* commit '5381549fe58cd3c5a28dc18e2416c37d3c2d300b':
  Re-run settings files migrations

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

am 90672aa0: Fix preference copy migration and incorrect HDR+ sticky bit

* commit '90672aa08379e598426c12b529a86afd4b61011e':
  Fix preference copy migration and incorrect HDR+ sticky bit

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

am 19597371: Only set preview size before starting preview

* commit '19597371a18921bb03337e4fa24c28dd2d8f06ed':
  Only set preview size before starting preview

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

am 4032a773: Fix incorrect key translation in Settings refactor

* commit '4032a773c4605894a2b6137590c7124ea16b7ab2':
  Fix incorrect key translation in Settings refactor