982592 - [Tarako][Browser]Cannot launch YouTube's navigation panel while playing video

Status: RESOLVED INVALID

(Core :: Audio/Video, defect)

Description

[Mike Lien[:mlien]]

[Device]
  Tarako
---------------------------------------------
[Reproduction build] - V1.3T
  Gaia      1f00cb5e533e698c5607bfce668a945270e79944
  Gecko     N/A
  BuildID   20140312075534
  Version   28.0
---------------------------------------------
[Reproduce Steps]
  1. Launch Browser app
  2. Go to "http://www.youtube.com"
  3. Choose any video to play
  4. While playing, tap screen to launch navigation panel

---------------------------------------------
[Expected Result]
  Should launch navigation panel normally to control video (ex: pause/play, enlarge, mute, and scroll timeline)

---------------------------------------------
[Actual Result]
  Cannot launch navigation panel

Comment 1

[Joe Cheng [:jcheng] (please needinfo)]

does it happen on Buri? thanks

Comment 2

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

No it does not happen on 1.3 Buri.  This seems to be Tarako only.

Possible that the touch event is some how eaten? 
I cannot even bring up the navigation panel.  The user agent is the same as buri.

Comment 3

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Noming because of youtube popularity.

Comment 4

[Mike Lien[:mlien]]

I also cannot reproduce this issue on Buri.
For comment 2, it might be Tarako only.

Comment 5

[Joe Cheng [:jcheng] (please needinfo)]

to Vincent for initial investigation, please reassign to appropriate owner after the investigation thanks

Comment 6

[Joe Cheng [:jcheng] (please needinfo)]

after some discussions, this might be in the graphics. 
Peter, is it something you can look into? Thanks

Comment 7

[Peter Chang[:pchang]]

Is this related to Bug 981804? But it looks like not related.

Comment 8

[Joe Cheng [:jcheng] (please needinfo)]

Peter, what do you suggest as next step? 
retest and make sure bug 981804 is included? thanks

Comment 9

[Jason Smith [:jsmith]]

(In reply to Joe Cheng [:jcheng] from comment #8)
> Peter, what do you suggest as next step? 
> retest and make sure bug 981804 is included? thanks

Note - bug 981804 only landed on 1.4. It didn't land on 1.3T.

Comment 10

[Peter Chang[:pchang]]

(In reply to Joe Cheng [:jcheng] from comment #8)
> Peter, what do you suggest as next step? 
> retest and make sure bug 981804 is included? thanks

Bug 977757 which caused bug 981804 was not landed in 1.3T.

Based on comment 2, it only happenes on Tarako. 
vliu, do you find the root cause related to gfx?

Comment 11

[yang.zhao]

Maybe Bug 979147 is the same as this one.There are some patches for Tarako only,I found if I reverted these patches ,the music controller would display.But I don't know which patch is the one affected this issue.

Comment 12

[Vincent Liu[:vliu]]

The issue I saw is when playing youtube, even touching the playing screen, the navigation panel never pops up to operate.
After did some works and I still can't confirm if it is relative to gfx. But one thing I can confirm that changing the gecko to v1.3 branch, the issue is gone. In other words, the navigation panel can pops up and operate it during playing youtube.

Comment 13

[Joe Cheng [:jcheng] (please needinfo)]

Hi Yang Zhao, is it something you can help looking into? as to which uplift break this? Thanks 

and can we have QA to also help looking for the regression window? Thanks

Comment 14

[Jason Smith [:jsmith]]

(In reply to Joe Cheng [:jcheng] from comment #13)
> Hi Yang Zhao, is it something you can help looking into? as to which uplift
> break this? Thanks 
> 
> and can we have QA to also help looking for the regression window? Thanks

It's too expensive right now to do regression windows on Tarako, as we have too few devices to do it within a reasonable amount of time. I'm also not convinced this is a regression based on comment 11.

Comment 15

[yang.zhao]

I just revert these patches and build an image to verify Bug 979147 and find it ok.The names of the patches I reverted here are:

gecko : 0000-Bug-xxxxxx-set-default-network-type.patch
gecko : 0005-Bug-903149-Minify-the-chrome-js-code.patch
gecko : 0006-Bug-956982-Clean-up-omni.ja-content.patch
gecko : 0007-Bug-957241-Don-t-package-the-full-sdk-when-we-don-t-.patch
gecko : 0008-Bug-957509-Reduce-USS-of-Nuwa-and-preallocated-proce.patch
gecko : 0012-Bug-959966-avoid-preallocated-process-from-being-kil.patch
gecko : 0014-BUG-944659-Considerable-amounts-of-script-sources.patch
gecko : 0019-Bug-974308-Study-memory-thrashing-and-find-out-a-pro.patch
gecko : 0020-bug-976656-Add-debug-log-for-DrawTargetCairo-CreateS.patch

I don't know whether it is the same to this issue.I add a test app which include a <video></video> label,but it doesn't have a control bar when launch it.

Just like this link http://218.211.38.207/Download_Test/DownloadAudio/audio.html
Open the link in a computer browser ,there will be a control bar on it ,but open it on the phone browser,there will not be a control bar on it.

Comment 17

[[:fabrice] Fabrice Desré]

Attachment: 2014-03-18-16-31-43.png

I don't reproduce that at all on my tarako. Note that I *don't* have the additional patches from queue.

I suspect that this bug is caused by one of these:
gecko : 0006-Bug-956982-Clean-up-omni.ja-content.patch
gecko : 0007-Bug-957241-Don-t-package-the-full-sdk-when-we-don-t-.patch

PLEASE STOP DOING BUILDS THAT DON'T MATCH WHAT WE WILL SHIP. WE ARE DOING USELESS QA THERE.

Comment 18

[Jason Smith [:jsmith]]

(In reply to Fabrice Desré [:fabrice] from comment #17)
> Created attachment 8393228 [details]
> 2014-03-18-16-31-43.png
> 
> I don't reproduce that at all on my tarako. Note that I *don't* have the
> additional patches from queue.
> 
> I suspect that this bug is caused by one of these:
> gecko : 0006-Bug-956982-Clean-up-omni.ja-content.patch
> gecko : 0007-Bug-957241-Don-t-package-the-full-sdk-when-we-don-t-.patch
> 
> PLEASE STOP DOING BUILDS THAT DON'T MATCH WHAT WE WILL SHIP. WE ARE DOING
> USELESS QA THERE.

So this is invalid then, right? Since the regressing patch isn't in the 1.3T branch.

Comment 19

[Joe Cheng [:jcheng] (please needinfo)]

Hi Yang Zhao, can you confirm if it is this Bug 956982 - Clean up omni.ja content that's causing the issue? thanks

Comment 20

[bhavana bajaj [:bajaj]]

Per comment #17 marking this as invalid as this is not a production build and was a build with not a valid patch queue..

Comment 21

[yang.zhao]

(In reply to Joe Cheng [:jcheng] from comment #19)
> Hi Yang Zhao, can you confirm if it is this Bug 956982 - Clean up omni.ja
> content that's causing the issue? thanks
In the latest version ,This patch ->gecko : 0006-Bug-956982-Clean-up-omni.ja-content.patch has already removed .And  Bug 979147 is ok in this version.But after I apply the patch again,Bug 979147 is NOT ok again.
So,I think you're right,this patch ->gecko : 0006-Bug-956982-Clean-up-omni.ja-content.patch resulted the issue.But now it has already removed ,so the issue is invalid.

Since I couldn't open the link "http://www.youtube.com",so the verify is all about Bug 956982.

Comment 22

[yang.zhao]

(In reply to yang.zhao from comment #21)
> (In reply to Joe Cheng [:jcheng] from comment #19)
> > Hi Yang Zhao, can you confirm if it is this Bug 956982 - Clean up omni.ja
> > content that's causing the issue? thanks
> In the latest version ,This patch ->gecko :
> 0006-Bug-956982-Clean-up-omni.ja-content.patch has already removed .And  Bug
> 979147 is ok in this version.But after I apply the patch again,Bug 979147 is
> NOT ok again.
> So,I think you're right,this patch ->gecko :
> 0006-Bug-956982-Clean-up-omni.ja-content.patch resulted the issue.But now it
> has already removed ,so the issue is invalid.
> 
> Since I couldn't open the link "http://www.youtube.com",so the verify is all
> about Bug 956982.

Sorry!The last sentence should be "so the verify is all about Bug 979147"

Comment 23

[James Zhang (Spreadtrum)]

We have removed all WIP patch now.
Please don't provide any WIP patch for tarako, all patch should be uplift to v1.3t, mozilla and spreadtrum should use the same code base, thanks.