933572 - [v1.3-only][Gallery][Video] Bluetooth pairing dialog appears to be cut off when invoked from Gallery/Video

Status: RESOLVED DUPLICATE of bug 948596

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)

Description

[Xinhe Yan]

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/535.19 (KHTML, like Gecko) Ubuntu/11.04 Chromium/18.0.1025.151 Chrome/18.0.1025.151 Safari/535.19

Steps to reproduce:

1 Enter Gallery app
2 Choose one photo
3 Click share icon
4 Choose Bluetooth Transfer


Actual results:

Can not see Bluetooth devices UI. Reenter Gallery app, Bluetooth UI appear. But choose one paired device, shows "Bluetooth file transfer failed"

Comment 1

[James Zhang (Spreadtrum)]

hamachi also has this issue, take it easy.

Comment 2

[Steven Yang [:styang]]

Evelyn, looks like it's a gallery issue, could you check this out first?

Comment 3

[[e-mail], [e-mail]]

I'm able to reproduce the issue in Video app too. But Music app is working fine for Bluetooth files sending.

Comment 4

[John Hu [:johnhu][:johu][:醬糊小弟]]

I can reproduce it, too.

But I feel this is not related to gallery or video apps. When we remove the setting, "fullscreen": true, from manifest.webapp, it works.

So, I put needinfo to alive to check if this is a bug from system app.

Comment 5

[[e-mail], [e-mail]]

Looks like the problem could be between fullscreen app to inline activity.

Comment 6

[(inactive after 6/18) Alive Kuo [:alive]]

Does this repro on v1.2? I see koi?

Comment 7

[Xinhe Yan]

this bug was found in v1.2 branch, but I did not test on master branch

Comment 8

[John Hu [:johnhu][:johu][:醬糊小弟]]

Is bug 928010 related to this one??

It also complains about share in gallery not working but the difference is that there is no share options showing. And the music still works, too.

Comment 9

[(inactive after 6/18) Alive Kuo [:alive]]

Yeah, this is really bad. koi+ without doubt.

Comment 10

[Preeti Raghunath(:Preeti)]

Hema

This seems to be a blocker for Gallery.

Comment 11

[Hema Koka [:hema]]

Hi Alive, 

Could you please check if this is system related? From comment 4 and 5, it looks like when John turned off the fullscreen option, and it is working fine. (Also since you have taken the ownership of the bug, may be you are already looking into it)

Thanks!
Hema

Comment 12

[(inactive after 6/18) Alive Kuo [:alive]]

Yes, a system bug indeed. Any system platform/frontend guy could steal this from me. Thanks.

Comment 14

[Chris Kreinbring [:CKreinbring]]

Regression window found on Buri 1.2 commercial RIL.

The last version that the Bluetooth page appears after the option is selected:
BuildID: 20131026004003
Gaia: de70f4d9b7a8dd3ba2565184735094eaa2bd3d18
Gecko: 780de7966fc0
Version: 26.0a2 

The bug began appearing on the following build:
BuildID: 20131027004003
Gaia: 04ee9e4430b25ba2c38752d3897f0ee5e2a6ab80
Gecko: 52f24889dccc
Version: 26.0a2

Comment 15

[[e-mail], [e-mail]]

Attachment: the size of window height is incorrect

Hi Alive,

I use a little bit time for tracking the issue. It could be reproduced on v1.2 and master. It might be relative with resize issue since we revised orientation from 'portrait-primary' to 'default'. And system app already support default orientation. If I revert the property to 'portrait-primary', it could show the inline activity as the attached screen shot. I'm not sure the root cause yet. I suspect the resize issue is between fullScreen app to inline activity. Any suggested direction for us. Thanks.

Comment 17

[bhavana bajaj [:bajaj]]

(In reply to Ian Liu [:ianliu] from comment #15)
> Created attachment 829160 [details]
> the size of window height is incorrect
> 
> Hi Alive,
> 
> I use a little bit time for tracking the issue. It could be reproduced on
> v1.2 and master. It might be relative with resize issue since we revised
> orientation from 'portrait-primary' to 'default'. And system app already
> support default orientation. If I revert the property to 'portrait-primary',
> it could show the inline activity as the attached screen shot. I'm not sure
> the root cause yet. I suspect the resize issue is between fullScreen app to
> inline activity. Any suggested direction for us. Thanks.

NI :alive to see if he help here as requested.

Comment 18

[(inactive after 6/18) Alive Kuo [:alive]]

I'm think about backing out 908601 in v1.2 and leave this bug opened for master.

I think Fred already back them out in 1.2, please try again.

Comment 19

[[e-mail], [e-mail]]

I have make sure the issue not be reproduced in v1.2. So I leave the flag from koi+. But we have to trace it for master. It's a blocking user story bug.

Gaia:     a6484b1e6fc07cf6bd8d6fcf9aeebb14b7e8869d
Gecko:    http://hg.mozilla.org/releases/mozilla-b2g26_v1_2/rev/948e30162eac
BuildID   20131114004004
Version   26.0

Comment 20

[[e-mail], [e-mail]]

As comment 19, nominating 1.3+..

Comment 21

[Jason Smith [:jsmith]]

(In reply to Ian Liu [:ianliu] from comment #20)
> As comment 19, nominating 1.3+..

I do not understand why this has been set to 1.3? - the regression here is still valid for 1.2 - bug 908601 hasn't been backed out yet.

Ian - Can you clarify?

Comment 22

[[e-mail], [e-mail]]

Jason, I see bug 908601 has been backed out via https://github.com/mozilla-b2g/gaia/commit/3259ac8a9813740ffc43a5e4fdff8e4a024a56c8

Comment 23

[Jason Smith [:jsmith]]

(In reply to Ian Liu [:ianliu] from comment #22)
> Jason, I see bug 908601 has been backed out via
> https://github.com/mozilla-b2g/gaia/commit/
> 3259ac8a9813740ffc43a5e4fdff8e4a024a56c8

Okay - I updated the bugs with the backout commit shown. I'll move this to 1.3+ since this regression no longer applies to 1.2 with that backout.

Comment 25

[[e-mail], [e-mail]]

Take over the issue.:)

Comment 26

[[e-mail], [e-mail]]

Hi Jason,

Unagi is working fine for the issue in v1.3. According to revised title, is the issue only on Fugu? Thanks.

Gaia  	59b89715c92d0383fb3d09941db93af5dfd05bc8
Gecko 	http://hg.mozilla.org/releases/mozilla-aurora/rev/cc2ab73484b2
BuildID   20131215004003
Version   28.0a2
ro.build.version.incremental=eng.cltbld.20131212.040310
ro.build.date=Thu Dec 12 04:24:12 EST 2013

Comment 27

[Jason Smith [:jsmith]]

It shouldn't reproduce on Fugu, as the patch was backed out from 1.2.

QA Wanted - Can someone confirm the above analysis that this bug no longer reproduces on 1.3 & 1.4?

Comment 28

[[e-mail], [e-mail]]

Just make sure the issue is not in 1.4 unagi. I want to distinguish a device dependancy issue or not.

Gaia  	358cd74fd2b2ef5d541f71a5d53d65d6a7335424
Gecko 	http://hg.mozilla.org/mozilla-central/rev/f67feb33a974
BuildID   20131216040201
Version   29.0a1
ro.build.version.incremental=eng.cltbld.20131212.040310
ro.build.date=Thu Dec 12 04:24:12 EST 2013

Comment 29

[Sarah Parsons]

Unable to reproduce this issue on the Buri 1.3 Build ID: 20131223004001

Gaia   01e9da49be2cc4bc134eeefc434740d572ec2246
SourceStamp af28fe58e263
BuildID 20131223004001
Version 28.0a2

Also, unable to reproduce this issue on the Buri 1.4 Build ID: 20131223040210

Gaia   5c4bb7e9dcdf6f168701508922c232a67cd8ea1f
SourceStamp c0f85061c7d3
BuildID 20131223040210
Version 29.0a1

Comment 30

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

Reopening this because the window height issue is still occurring on the latest buri v 1.3.0 Mozilla RIL build.

Environmental Variables
Device: Buri v 1.3.0 Mozilla RIL
Build ID: 20140106004001
Gecko: http://hg.mozilla.org/releases/mozilla-aurora/rev/a43cb4b322d3
Gaia: 35a60b82f8cf2d759939a350e2dadbb9d8b2f5dc
Platform Version: 28.0a2

The bluetooth confirmation window is the incorrect height and appears cutoff at the top. 


This issue does not occur on the latest Buri v 1.4.0 Mozilla RIL build. 

Environmental Variables
Device: Buri v 1.4.0 Mozilla RIL 
Build ID: 20140106040201
Gecko: http://hg.mozilla.org/mozilla-central/rev/14ac61461f2a
Gaia: 9a222ac02db176e47299bb37112ae40aeadbeca7
Platform Version: 29.0a1

The bluetooth confirmation window height appears correctly.





The bluetooth window height appears correctly.

Comment 31

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Revise title based on comment 30.

Ian, do you have time to work on this 1.3+ bug?

Comment 32

[[e-mail], [e-mail]]

Yes, will take a look for the issue. We're able to reproduce it in Gallery app while goes into inline activity.

Comment 33

[[e-mail], [e-mail]]

This is a layout issue in inline activity. I'm not sure the latest definition for the margin top of inline activity frame. There is no top property in Gaia/master. But we still have the top property in v1.3.

Alive, could you please help me to confirm the layout definition? Then, I'm able to start to fix it. Thanks.

Gaia/v1.3 system.css
https://github.com/mozilla-b2g/gaia/blame/v1.3/apps/system/style/system/system.css#L452-L454

Comment 34

[(inactive after 6/18) Alive Kuo [:alive]]

Read AppWindow.resize and ActivityWindow.resize part.
Some sizing is predefined in window.css(system.css in v1.3)

Comment 35

[[e-mail], [e-mail]]

Alive,

I was surprised that the issue is fixed with my additional log only while I'm debugging. In fact, I'm not able to reproduce the issue here after you uplifted bug 948596 to v1.3. It's a duplicated work.