1045214 - [B2G][Gallery]Launching camera from gallery results in blank viewfinder

Status: VERIFIED FIXED

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

Description

[Ross Kuhlman ([e-mail])]

Tapping the camera button in the lower left corner of the gallery app launches the camera app. If the device is set to 319Mb of memory, the camera app will load, but the viewfinder will only display a black screen. The camera app must be relaunched from homescreen to fix the issue.

If the camera app is already open in the background, then it will load appropriately from the gallery.

Repro Steps:
prerequisite: Set device to 319Mb memory

1) Update a Flame to BuildID: 20140728040209
2) Launch Gallery app
3) Tap Camera icon in lower left corner

Actual:
Camera loads, but black screen is displayed in viewfinder

Expected:
Camera loads and functions as expected

Master M/C Environmental Variables:
Device: Flame Master M/C
BuildID: 20140728040209
Gaia: 295967a0b824a355ae9d57fb08f3632ed2ad18dd
Gecko: a4dcfbebcb58
Version: 34.0a1
Firmware Version: v122


Notes:
Repro frequency: 100%
See attached: logcat, screenshot

Comment 1

[Ross Kuhlman ([e-mail])]

Attachment: GalleryToCameraBlank.txt

This issue sounds very similar to bug #1042028, but the circumstances are different. In his bug, the issue does not occur if the Camera app is already open. In bug #1042028, the issue only occurs if the Camera app is already open.

Comment 2

[Ross Kuhlman ([e-mail])]

Attachment: Blank viewfinder in camera app

Comment 3

[Peter Bylenga [:PBylenga]]

QAWanted for Branch Checks.

Comment 4

[Cody Roesch [:croesch]]

This bug repro's on: Flame 2.1 Master, Buri 2.1 

Actual Results: Choosing the Camera icon from the Gallery, brings up the camera but instead of showing the screen preview, only a black screen appears.

***NOTE: Flame device was tested with 319mb

Environmental Variables:
Device: Flame Master
Build ID: 20140728065013
Gaia: 295967a0b824a355ae9d57fb08f3632ed2ad18dd
Gecko: d77f6a96ff96
Version: 34.0a1 (Master)
Firmware Version: v122
-----------------------------------------------
Environmental Variables:
Device: Buri Master
Build ID: 20140728123640
Gaia: fadfafa17f5175203b8b9457bfb95e5816f54f58
Gecko: 75fe3b8f592c
Version: 34.0a1 (Master)
Firmware Version: v1.2device.cfg

-----------------------------------------------
-----------------------------------------------

This bug does NOT repro on: Flame 2.0, Flame 1.4

Actual Result: No problems seen when going from Gallery to camera.

Environmental Variables:
Device: Flame 2.0
Build ID: 20140728065913
Gaia: b56f023ff247312a344e8f233898e803459788b9
Gecko: e11d0b232ced
Version: 32.0 (2.0)
Firmware Version: v122
-----------------------------------------------
Environmental Variables:
Device: Flame 1.4
Build ID: 20140728061215
Gaia: eb3b185325901d4c04e2d43eb58d90835213bea9
Gecko: aae9112f1fc6
Version: 30.0 (1.4)
Firmware Version: v122

Comment 5

[Joshua Mitchell (Inactive)]

[Blocking Requested - why for this release]: Regression in a major feature (camera)

Comment 6

[Ross Kuhlman ([e-mail])]

I am also seeing this issue when launching the camera from the lockscreen. If the camera is not running in the task manager, then a blank screen will be displayed when camera is launched from lockscreen.

Taking a picture will cause the viewfinder to display correctly.

Master M/C Environmental Variables:
Device: Flame Master M/C MOZ
BuildID: 20140729040211
Gaia: fadfafa17f5175203b8b9457bfb95e5816f54f58
Gecko: b17cad2d1e5e
Version: 34.0a1
Firmware Version: v122

Comment 7

[Ross Kuhlman ([e-mail])]

adding smoketest keyword

Comment 8

[Peter Bylenga [:PBylenga]]

Adding qaurgent, this breaks the smoke test for launching camera from lockscreen.

Comment 9

[Jayme Mercado [:JMercado] (Inactive 3/4/2016)]

This issue seems to have been caused by bug 1038172.

B2g-inbound Regression Window

Last Working 
Environmental Variables:
Device: Flame Master
BuildID: 20140723085706
Gaia: 15c84c943e41ad834640a45e1e1c2ac804168af7
Gecko: dfc1543c9c50
Version: 34.0a1 (Master) 
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0


First Broken 
Environmental Variables:
Device: Flame Master
BuildID: 20140723123306
Gaia: 5458f73e319759543fddf7e96d7ece4d78318e32
Gecko: 5b53be9551bb
Version: 34.0a1 (Master) 
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0



Last working gaia / First broken gecko - Issue does NOT occur
Gaia: 15c84c943e41ad834640a45e1e1c2ac804168af7
Gecko: 5b53be9551bb

First broken gaia / Last working gecko - Issue DOES occur
Gaia: 5458f73e319759543fddf7e96d7ece4d78318e32
Gecko: dfc1543c9c50


Gaia Pushlog:  https://github.com/mozilla-b2g/gaia/compare/15c84c943e41ad834640a45e1e1c2ac804168af7...5458f73e319759543fddf7e96d7ece4d78318e32

Comment 10

[Joshua Mitchell (Inactive)]

broken by bug 1038172 ? Wilson, can you take a look?

Comment 11

[Wilson Page [:wilsonpage]]

Attachment: pull-request (master)

Comment 12

[Wilson Page [:wilsonpage]]

[Blocking Requested - why for this release]:

This will need uplifting to 'v2.0' once bug 1038172 lands.

Comment 13

[Wilson Page [:wilsonpage]]

Landed on 'master' 

https://github.com/mozilla-b2g/gaia/commit/efd6f69414f80e587c59e6bd2d381e30dd288d2e

Comment 14

[Jason Smith [:jsmith]]

Wilson - Given comment 13 has this landed on master, can we close this?

Comment 15

[Diego Marcos [:dmarcos]]

This is apparently a regression introduced by bug 1038172 that has not yet landed on 2.0.

Once bug 1038172 lands on 2.0 we need to uplift this one as well.

Should we still mark it as closed?

Comment 16

[Jason Smith [:jsmith]]

(In reply to Diego Marcos [:dmarcos] from comment #15)
> This is apparently a regression introduced by bug 1038172 that has not yet
> landed on 2.0.
> 
> Once bug 1038172 lands on 2.0 we need to uplift this one as well.
> 
> Should we still mark it as closed?

Yeah, I think we can do that. We'll just need to make sure that we indicate that bug 1038172 and this bug need to land together to avoid having the smoketest regression appear on 2.0.

Comment 17

[Hema Koka [:hema]]

Blocking because of regression from patch in https://bugzilla.mozilla.org/show_bug.cgi?id=1038172

Comment 18

[Martin Shuman [:Marty]]

Verified as fixed in the latest 2.1 Flame Build:

Environmental Variables:
Device: Flame Master
Build ID: 20140801040326
Gaia: 04ea7e1a4034a50d4a7a4f5b95a04a2ed8313908
Gecko: 104254bd1fc8
Version: 34.0a1 (Master)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Camera app loads from the Gallery and functions as expected.

Comment 19

[Ryan VanderMeulen [:RyanVM]]

Adding NO_UPLIFT to match bug 1038172.

Comment 20

[Ryan VanderMeulen [:RyanVM]]

wontfix on v2.0 per bug 1038172 comment 104.

Comment 21

[Martin Shuman [:Marty]]

Verified as Fixed in latest Flame 2.1 build.
Camera app opens from the gallery and functions properly.

Environmental Variables:
Device: Flame 2.1
BuildID: 20141021001201
Gaia: e458f5804c0851eb4e93c9eb143fe044988cecda
Gecko: ee86921a986f
Gonk: 05aa7b98d3f891b334031dc710d48d0d6b82ec1d
Version: 34.0 (2.1)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0