1189105 - [Aries][Camera] Zoom level bar still appearing for front face camera.

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::Camera, defect)

Description

[John Thomas [:Johnt]]

Attachment: logcat_20150729_1517.txt

Description:
The zoom level bar for the front facing camera is still appearing even though it has no functionality.


Repro Steps:
1) Update a Aries to 20150702210513
2) Select Camera
3) Change Camera settings from back facing to front facing.
4) Attempt to Zoom camera.


Actual:
Zooming level bar appears with no additional functionality.

Expected:
It is expected that if the zoom bar is going to appear for the front facing camera that it will have the option to adjust the zoom level OR that the zoom level bar will be removed for front facing camera entirely.

Environmental Variables:
Device: Flame 2.5
Build ID: 20150702010204
Gaia: b901c8b7be2119f4df42781aac1401ed12765460
Gecko: f5e3bacfb60e
Gonk: a4f6f31d1fe213ac935ca8ede7d05e47324101a4
Version: 42.0a1 (2.5)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:42.0) Gecko/42.0 Firefox/42.0


Repro frequency: (2/3, 100%, etc.)
See attached: Logcat & Video

Video: https://youtu.be/xXZ_WxwGe-g

Comment 1

[John Thomas [:Johnt]]

This bug is related to Bug 1180056

This issue does NOT occur on the Flame 2.5

Result: Front facing camera zoom level bar can be adjusted accordingly.

Environmental Variables:
Device: Flame 2.5 Kk Fullflash (319mb)
BuildID: 20150729030209
Gaia: 21256d7665f972255d198f8af81a8df4bd0e0fc4
Gecko: 2ee9895e032c
Gonk: 41d3e221039d1c4486fc13ff26793a7a39226423
Version: 42.0a1 (2.5) 
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:42.0) Gecko/42.0 Firefox/42.0

Comment 2

[KTucker [:KTucker][Inactive 3/4/2016]]

[Blocking Requested - why for this release]:

This bar needs to be removed since we are not supporting pinch to zoom on Aries for the front facing camera. Nominating this 2.5?

Comment 3

[No-Jun Park [:njpark]]

Agree with Comment 2, we can't have slider that doesn't do anything.

Comment 4

[Michael:mtreese]

bad user experience, please fix.

Comment 5

[GitHub Autolander Bot]

Attachment: [gaia] aosmond:bug1189105 > mozilla-b2g:master

Comment 6

[Justin D'Arcangelo [:justindarc]]

Comment on attachment 8647012 [details] [review]
[gaia] aosmond:bug1189105 > mozilla-b2g:master

One minor nit about the new method name. Otherwise, looks good.

Comment 7

[Ryan VanderMeulen [:RyanVM]]

Master: https://github.com/mozilla-b2g/gaia/commit/d06be46d187ff97b475ea04ff2afb588ec9602bf

Comment 8

[Annabelle]

Attachment: Verified_Aries_v2.5.3gp

This bug has been verified as pass on latest Nightly Flame KK v2.5 and Aries KK v2.5 by the STR in comment 0.
See attachment: Verified_Aries_v2.5.3gp & Verified_Flame_v2.5.3gp
Reproduce rate: 0/6

Actual results:
  [Flame kk v2.5]If change Camera settings from back facing to front facing and zoom,there is a Zooming level bar on Flame kk v2.5, it  has the option to adjust the zoom level

  [Aries kk v2.5]If change Camera settings from back facing to front facing and zoom,there is not a Zooming level bar.

Device: Flame KK v2.5 (Pass)
Build ID               20150819030215
Gaia Revision          1e1197e0e8e64307aa382ffba4711d1c661de7ca
Gaia Date              2015-08-18 16:54:35
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/f384789a29dcfd514d25d4a16a97ec5309612d78
Gecko Version          43.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150819.063618
Firmware Date          Wed Aug 19 06:36:31 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

Device: Aries KK v2.5 (Pass)
Build ID               20150820005800
Gaia Revision          89e0096a3de0378e3eda77e6a2a0bb5ca03eb8bb
Gaia Date              2015-08-19 18:28:14
Gecko Revision         https://hg.mozilla.org/integration/mozilla-inbound/rev/29b2df16e961fbe9a379362ecba6f888d1754bc3
Gecko Version          43.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20150820.002839
Firmware Date          Thu Aug 20 00:28:46 UTC 2015
Bootloader             s1

Comment 9

[Annabelle]

Attachment: Verified_Flame_v2.5.3gp