am 845dadec: (-s ours) Reconcile with klp-release - do not merge

* commit '845dadec2a2cc619d316c44e706b15eba4d75293':

Merge "DO NOT MERGE - Shutter button left handed support" into gb-ub-photos-denali

Merge "Bring back video snapshot" into gb-ub-photos-denali

Bring back video snapshot

Bug: 14220490
Change-Id: Ibf20d169aae9de4eb9c1e2151d9c7335bcf7b3a3

DO NOT MERGE - Shutter button left handed support

Bug: 12823819
Change-Id: I77bdbf17557bdd9f545b98bab58acce68d01b51d
(cherry picked from commit 344320c90639f823451ddac42e8f39b74727787c)

Avoid duplicate entries in the resolutions list.

  Bug: 14277230

Dup entries indicate that a camera doesn't support at least three entries.

Change-Id: Id3c18664637ba9cf80edada924695737863cf3a7

Properly remove the settings for an unavailable camera.

  Bug: 14108948

Change-Id: I34222ab976a481e8e28ed0bbc8b773dd3d49e8bb

DO NOT MERGE - Refactor preview position logic

Centralize the logic of laying out bottom bar and transforming
preview TextureView based on the total space to lay them out
and preview aspect ratio. The layout is updated when either condition
above has changed. Then the rect of the bottom bar and preview
rect can be queried anywhere. This way, bottom bar, mode options
layout, and any view that relies on the preview rect and bottom bar
rect can be laid out without extra layout passes.

Bug: 13795952
Change-Id: I13c38b566b5b5d8f37ec94f325554c089a86ca8e
(cherry picked from commit 28a2950939614a0c1c83787960ce1a218fb69a5e)

Import translations. DO NOT MERGE

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

Don't crash when less than two cameras are present.

  Bug: 14108948

Change-Id: Ib288f9fd2bd86f870df22c1fde03184f00cea219

Import translations. DO NOT MERGE

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

Merge "Fix edge case in supported video quality selection." into gb-ub-photos-denali

Fix edge case in supported video quality selection.

 Bug: 14108743

If less than three distinct video quality levels are supported, it's possible
for getNextSupportedVideoQualityIndex() to return an out-of-range array index.
Instead, just return the previous supported video quality -- this appears to
be the intended behavior.

Change-Id: I089571edf6ff748ce4914f188b4ef844ecb638c2

Do not crash when attempting to refocus a deleted
image.

Bug: 13546660
Change-Id: Ic260811ad2f45a44577609af0dc69a5c55c7fcfd

Don't update metadata on local data insert.

It is possible for insert data to be called in
a loop with many items, and this will cause un-
needed lookups of meta data.

The metadata lookup is already handled but the
filmstrip lazily, so it is not necessary here.

Bug: 13875458
Change-Id: I476eb1d44fa3d089f47b8b16f9be6d3255b4759e

Avoid using stale parameters to get focus mode in video

Bug: 13990664
Change-Id: If6a5b1cc9ce7d1ac67f0558a61036f228601b92b

Merge "Import translations. DO NOT MERGE" into gb-ub-photos-denali

Merge "Set video-size key for Samsung Brand devices." into gb-ub-photos-denali

Set video-size key for Samsung Brand devices.

Bug: 14060121
Change-Id: I5b8e5b0259a4105b8839112b13e58f2222848c7e

Safeguard callback receiver to prevent crashes.

  Bug: 13765929

It is safe to add these checks here. The receiver is only set in
the constructor, and it's only created in one place, which is
onCreate. And the parameter is the Activity itself, so it can
never be null.
This means the NPE can only happen if onCameraOpened is called after
removeCallbackReceiver is called, which is in onDestroy of the activity.
In this case it's safe to exit.

Change-Id: Ibffd6d54679b564a11b3e83319896241f2e306ad

Import translations. DO NOT MERGE

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

Check zoom enabled check and null check on zoom ratio list.

Bug: 13987837
Change-Id: I45d3edc42a240c866f151c5e23431cd50eec7bc6

Merge "Revert "ZoomRatios might not exist, not crash for logging."" into gb-ub-photos-denali

Revert "ZoomRatios might not exist, not crash for logging."

This reverts commit de26138f7821e33cac293e9c78c23b5c4612dcd2.

Change-Id: Ia788ef858cf2bc8891a5bbbf1a538b49d583247d

Merge "DO NOT MERGE Don't call stopPreview after taking a picture on the m8" into gb-ub-photos-denali

DO NOT MERGE Don't call stopPreview after taking a picture on the m8

  Bug: 13966525

The proper solution is to remove the stopPreview call completely,
but as we have only limited time for testing left, let's be
careful and target specific devices only.

Change-Id: I5aa90b7c72068bae5352a8396e39a2c82ac137b9

Merge "ZoomRatios might not exist, not crash for logging." into gb-ub-photos-denali

ZoomRatios might not exist, not crash for logging.

  Bug: 13987837

Change-Id: I77b8863b60cf5311ffd326015e612dfaeb45daa2

Update mode options with hardware spec whenever hdr setting changes.

Bug: 13965048
Change-Id: I7e718bb56cb5dc92c7583bef780d442c04644a56

Shorten the preference string to fit

  Bug: 13962104

Change-Id: Ie0ba57ad1644241748c8d0406c08be1c981b8f63

Merge "Import translations. DO NOT MERGE" into gb-ub-photos-denali

Merge "Tweak log levels for photo, video" into gb-ub-photos-denali

Tweak log levels for photo, video

Adjusting log levels to insure some simple state flow statements
are printed for all clients at info, plus a few other adjustments.

Bug: 13737123
Change-Id: Ic58b2ad44067253960d68678c34d82895f49f6a6

Don't crash when listener registration gets out of sync

 Bug: 13914960

In some cases it can happen that resume/pause sequences happen on
two CameraActivities at the same time, especially when switching
from lock-screen to non-lockscreen activities.

Change-Id: Ic0826fc4a1d601b0ba0e6ac9cef84c022d958ad4

Merge "Fix the layout error in preview" into gb-ub-photos-denali

Fix the layout error in preview

This bug is caused by view configuration and layout rect occasionally
out of sync and bottom bar's incapability of correcting itself. To
be specific, configuration can be out of sync with layout rect (i.e.
configuration.orientation is landscape but onLayout() passes in portrait
coordinates), but when that happens, configuration will soon correct
itself through onConfigurationChanged(), then another layout pass will
happen to make sure everything is laid out correctly. In this case,
when configuration and layout coordinates are inconsistent, bottom bar
lays out itself using the wrong alignment, and then changed the preview
rect accordingly. When another layout pass happens to correct this
inconsistency, bottom bar then lays out itself based on the correct
alignment and a wrong preview rect previously adjusted, rendering
correct alignment but wrong size.

Bug: 13911202
Change-Id: I7cf6dbea582ce3888df11039de8f94687fe75784

Import translations. DO NOT MERGE

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

Merge "Add logging override helper" into gb-ub-photos-denali

Add logging override helper

This CL removes compile-time log suppression with a runtim override.
With a runtime override we can custom configure logging with a single
level, not quite as flexible as a boolean per level but should suit
our needs. Compile-time log pruning could be done with proguard if
desired.

Bug: 13737123

Change-Id: I8bdcaca44b4dc5c7eab8ef2f74e5181bcacd2727

Log memory consumption at end of onCreate (Camera2).
Bug: 13280671

Change-Id: Ifdeadff2c89a08757cc2f3792ba61a2b54d19ae0

Query memory consumption at end of onCreate.
Bug: 13280671

Change-Id: Iedc4c8907a6e726a1835494bd6fad93e6ee8a666

Merge "Cancel touch event forwarding when transitioning from shimmy to scroll state" into gb-ub-photos-denali

Cancel touch event forwarding when transitioning from shimmy to scroll state

Bug: 13903149
Change-Id: I70badb1fdc959b3ce0a9d3385e664938ab7918a9

Merge "Keep lights out even if the system changes it." into gb-ub-photos-denali

Keep lights out even if the system changes it.

  Bug: 13894148

Change-Id: Ie6d5b8fc1d3601082942a5c0843b447cb4805ceb

Merge "Switch visibility on AbstractTutorialOverlay removeOverlay." into gb-ub-photos-denali

Switch visibility on AbstractTutorialOverlay removeOverlay.

  Bug: 13337436
  Bug: 12440059

Change-Id: I34dd96df02cedf33be72ae5563bbad3d4b307a12

Merge "Import translations. DO NOT MERGE" into gb-ub-photos-denali

Merge "Replace isDebugging compiletime logging override with runtime property" into gb-ub-photos-denali

Merge "Don't show the tutorial wrapper all the time." into gb-ub-photos-denali

Don't show the tutorial wrapper all the time.

  Bug: 13337436
  Bug: 12440059

Change-Id: I10a969a4c550b6ebbc101f7cdce8ef12c3029137

Replace isDebugging compiletime logging override with runtime property

This CL accomplishes the same goal as the compile-time isDebuggable,
with a runtime property check. This is useful so that we opt in to
logging everything at a given level with one setprop line.

Note that either with this new runtime property or with isDebuggable,
the Log proxy wouldn't log if a particular Configuration value is false,
since that level is instead routed to SILENT_LOGGER.

Bug: 13737123
Change-Id: I78ce629620f420a08a2cc013865847674941fb9b

Merge "Make tutorials render in smaller layout on tablets." into gb-ub-photos-denali

Make tutorials render in smaller layout on tablets.

  Bug: 13337436
  Bug: 12440059

Change-Id: I7886a86189a94483775a5535c69e22090cec1530

Shorten ExternalViewerButton log

While examining logs, noticed our Log proxy rejected this log line
as too long.

Bug: 13737123
Change-Id: I8ca2cf6be79539db0fd262ffd13a317f6fe20dcb

Merge "Hide mode switcher in selected state when window is no longer visible" into gb-ub-photos-denali

Hide mode switcher in selected state when window is no longer visible

Bug: 13751149
Change-Id: Id479eeed06ec8965df2836c4825de6212c7efa7a

Migrate LocationManager to use camera Logger

This is the last remaining non-test class in Camera2 using
android.util.Log directly.

Bug: 13737123
Change-Id: Ia90d1d91b651cace026d9fdd5867d556aa8bad72

Import translations. DO NOT MERGE

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

Merge "Make removeOverlay publicly available." into gb-ub-photos-denali

Merge "Resize mode drawer content for tablet to 1.5x of that on phones" into gb-ub-photos-denali

Remove unneeded log line.

Change-Id: Ibf22b2389ed898f68f7f9217f0d899dfae8aa767

Resize mode drawer content for tablet to 1.5x of that on phones

Bug: 13768342
Change-Id: I4c9017b9f003d5093e7153bf31a378d818caf788

Merge "Workaround for black sqaure issue caused by incorrectly recreated HW layer" into gb-ub-photos-denali

Make removeOverlay publicly available.

  Bug: 13337436

Change-Id: I222f3353230ad734db304f2068f853bdf8ef9de3

exposure compensation control is back

bug: 13607051
Change-Id: Ib4920ba3f8f3a548651a2d07613ccfdc28ca1d6e

Workaround for black sqaure issue caused by incorrectly recreated HW layer

Bug: 12594617
Change-Id: I9209d4bc0d6ebd045b957b3c79eabdca0a4f7cc4

Merge "Fix Settings Activity" into gb-ub-photos-denali

Fix Settings Activity

Bug: 13817516
Change-Id: Idb8b95c3501b52519ae97b6d603ee59e8cc50ce4

Merge "Make the camera id setting have static possible values." into gb-ub-photos-denali

Merge "Add context to parameters passed to ExtraSettings" into gb-ub-photos-denali

Add context to parameters passed to ExtraSettings

Bug: 13791528
Change-Id: Ibcc4db1ab07b274f18f1ab369747d759180f155b

Merge "Add resources for tutorials." into gb-ub-photos-denali

Merge "Fix hdr check." into gb-ub-photos-denali

Fix hdr check.

Change-Id: I7ebd06a985fbc0661d71b189fe1fdbf620f1c341

Merge "Use only filename from path as reference." into gb-ub-photos-denali

Merge "Import translations. DO NOT MERGE" into gb-ub-photos-denali

Make the camera id setting have static possible values.

This is important because the dynamic hook for querying the
supported number of cameras is currently only based on the camera 1 api.

We want to make sure we can set the enabled/disabled state of back
and front facing camera in the mode options, so it's necessasry to
always have both possible values.

Bug: 13769609
Change-Id: I9d526315dbe15e008542832f9b2c4c9d23d34021

Use only filename from path as reference.

Change-Id: Ifa3dd1a8c89cab5b135953eebca70f4ed53c1102

Merge "Logging revisions." into gb-ub-photos-denali

Logging revisions.

Bug: 13747555

Change-Id: Ic5f735de89f9a69d981cdeab209b3966532697c7

Fix layout error during mode switch

Cause for the incorrect layout: the two modes have a preview
of the same aspect ratio, and therefore bottom bar did not need to re-layout
after the mode switch and it incidentally skipped adjusting preview position.

Bug: 13210080

Change-Id: Iad722abd682885633885ff0e8d9fba4efb4e5e6a

Merge "Move SettingsManager back into Activity." into gb-ub-photos-denali

Fix for visibilty of cling out of sync with the external viewer button

Due to no visiblity changed listener provided from Framework, a subclass
of the image button was created to track the visiblity change of itself
and more importantly its ancesters. This way cling is gauranteed to be
in sync with the visility of the button.

Also fixed preference type of showing refocus viewer cling.

Bug: 13754204
Bug: 13752484
Change-Id: Iaf38c076d5b1e251f3d5d4b4ccba1bd8447999aa

Move SettingsManager back into Activity.

We cannot easily share one instance between Activities as the manager
needs the current module index. So this can cause crashes if two
Activity instances are around and one gets destroyed.

Change-Id: I964b14460669c4fbe3a51ab80cd385d5dfa0a16d

Merge "Add API to query the current auto-rotation setting" into gb-ub-photos-denali

Turn Lights-Out mode back on.

  Bug: 13765919

This is reverting http://ag/436107

Change-Id: Id40645f2555d3133b0e866de57c8c951d8c3b86d

Merge "Remove unused ComboPreferences cruft." into gb-ub-photos-denali

Remove unused ComboPreferences cruft.

Change-Id: Ib2e005e00316f8840e29fb559e34c6a5d9f28950

Add resources for tutorials.

  Bug: 13337436

Change-Id: I51eeb1def89d4bb0c0e6388454f65ca43d4afbb9

Avoid a race condition causing dup'd images.

When a session is finished, it is written to the
media store.

We now have a listener looking for new images in
the media store.  If that happens to fire before
the session manager has updated the filmstrip,
the photo ends up being added twice.

Now, the listener checks to see if it is a photo
already being handled by a session

Bug: 13644124

Change-Id: I8a3c4a161ad0c326d9295f6714f6c6cbbcc6764e

When we reset app ui, re-enable shutter button.

This allows all modules to have a known state on
resume.

Change-Id: I8efab600b7c6df611e522bb20c8a164006a27a9d

Merge "Fix crash related to deleted items being processed" into gb-ub-photos-denali

Fix crash related to deleted items being processed

If an item from the media store was being edited
there is a race condition where the item may be
deleted from the media store before it is finished

The resulting update to the media store cannot
find the original item, and therefore results in
0 rows being updated, which is what is desired,
as the item was deleted by the user.

Bug: 11598186
Change-Id: I1b2945f638676ccd63ab70ba201d2488df920eb3

Import translations. DO NOT MERGE

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

Merge "Move disk IO from first time use background task." into gb-ub-photos-denali

Merge "Make SettingsManager accessible on app-level, not Activity-level." into gb-ub-photos-denali

Merge "Fix broken shutter button enabling in Photo and Video." into gb-ub-photos-denali

Add API to query the current auto-rotation setting

bug:13527779
Change-Id: I1ebcab1108bb03a997508033befeec26fb9a6b7e

Make SettingsManager accessible on app-level, not Activity-level.

  Bug: 13754773

This also cleans up the SettingsManager and removes all constructor
parameters except the context, which is needed.

Change-Id: I60a8d5582771e5a1f53a761897f4d41c30e50166