900501 - [Browser] [Youtube] Size of video is not proper after changing orientation multiple times

Status: RESOLVED FIXED

(Tech Evangelism Graveyard :: Preinstalled B2G Apps, defect, P1)

Description

[Leo]

STR

1. Open browser
2. Navigate to you tube.
3. Open a random video and start playing in landscape mode
4. During video play rotate the screen number of times.

Expected result:
The video size should be properly adjusted during orientation changes.

Actual Result:
Some time in portrait mode, video size is similar to landscape mode.

** Please see the attached images for better understanding

Comment 1

[Leo]

Attachment: Video outside the screen

Comment 2

[Leo]

Attachment: Video outside the screen

Comment 3

[Jason Smith [:jsmith]]

Confirmed I can reproduce.

Chris - Is this a site bug or a core bug?

Comment 4

[Sotaro Ikeda [:sotaro]]

I did no debugging. This bug reminds me Bug 883024 as similar symptom.

Comment 5

[cajbir (:cajbir)]

(In reply to Jason Smith [:jsmith] from comment #3)
> 
> Chris - Is this a site bug or a core bug?

I don't know. Edwin may be able to investigate.

Comment 6

[Robert O'Callahan (:roc) (email my personal email if necessary)]

This is probably not a video bug. More likely to be a layout or layers bug.

This bug was filed against trunk. Does it happen on trunk, or was it actually tested on b2g18?

Comment 7

[Jason Smith [:jsmith]]

(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #6)
> This is probably not a video bug. More likely to be a layout or layers bug.
> 
> This bug was filed against trunk. Does it happen on trunk, or was it
> actually tested on b2g18?

This was originally tested on b2g18 on a leo partner build by leo. Fixing version on bug as such.

Comment 8

[Jason Smith [:jsmith]]

This is on leo's signoff list for bugs to resolve for YouTube, so I'm putting this into the nom queue to see if Mozilla/Leo wants to block on this.

Comment 9

[Sotaro Ikeda [:sotaro]]

I am going to think it is youtube site's bug. nsVideoFrame's width and height styles was eStyleUnit_Coord(20).

Comment 10

[Sotaro Ikeda [:sotaro]]

Attachment: temporary patch - add logout to nsVideoFrame

The patch add logout that related to nsVideoFrame's style infos.

Comment 11

[Sotaro Ikeda [:sotaro]]

Attachment: logout - start playback portrait then landscape

The logcat of youtube playback of following
-[1] Start youtube video playback at youtube web site.
   hamachi is in portrait state
-[2] change hamachi to landscape state after start the video playback

Comment 12

[Sotaro Ikeda [:sotaro]]

Attachment: logout - start playback landscape then portrait

The logcat of youtube playback of following STR
-[1] Start youtube video playback at youtube web site.
   hamachi is in landscape state
-[2] change hamachi to portrait state after start the video playback

Comment 13

[Wayne Chang [:wchang]]

Triage - Partner will not block release on this bug, but this should be fixed to optimize user experience and the youtube story.

Comment 14

[Jason Smith [:jsmith]]

Fair enough - if leo is okay with this and allows YouTube to remain in the build, then I'll pull the nom.

Comment 15

[Leo]

Leo quality team raise this issue. So we should fix it till this week.

Comment 16

[Sotaro Ikeda [:sotaro]]

Edwin, what is your current state?

Comment 17

[Sotaro Ikeda [:sotaro]]

Doublec, how do you think about comment #9?

Comment 18

[Edwin Flores [inactive from 2016-12-01] [:eflores] [:edwin]]

(In reply to Sotaro Ikeda [:sotaro] from comment #16)
> Edwin, what is your current state?

Haven't had time to give this a proper look. I'll look at it today.

Comment 19

[Hema Koka [:hema]]

Candice Serran deleted the linked story in Pivotal Tracker

Comment 20

[Hema Koka [:hema]]

Candice Serran deleted the linked story in Pivotal Tracker

Comment 21

[cajbir (:cajbir)]

(In reply to Sotaro Ikeda [:sotaro] from comment #17)
> Doublec, how do you think about comment #9?

I don't know sorry. I defer to Edwin since he's looking at it according to comment 18.

Comment 22

[Edwin Flores [inactive from 2016-12-01] [:eflores] [:edwin]]

Reproduced on desktop by setting b2g26 user agent and using the responsive design view. It doesn't look like the width or height attributes are being changed.

Site bug.

Comment 23

[Jason Smith [:jsmith]]

Vishy - Can you outreach to youtube about this bug?

Comment 24

[Donovan Preston [:fzzzy]]

I already mentioned it to them... but Vishy can outreach again ;-)

Comment 25

[leo.bugzilla.gecko]

Could you find any different point the Youtube contents and working correct contents?

Comment 26

[Sotaro Ikeda [:sotaro]]

I confirmed how the youtube site works by using PC Firefox nightly by the following.
- Change User Agent to "Mozilla/5.0 (Mobile; rv:14.0) Gecko/14.0 Firefox/14.0"
       by using User Agent Switcher add-on
- Change the view size by using the responsive design view(640*360).
- change media.windows-media-foundation.enabled to "true" at about:config
- visite youtube web site and start a video playback.
- Check a css of the video by using inspector

When the view size was 640*360, the video tag's attribute was followng. It was not changed when change the rotation.
 - controls="true" style="width: 640px; height: 360px; margin-left: 0px; margin-right: 0px;

n.b. There were cases that the firefox browser shows the youtube web site for android and can not playback HTML5 video in browser.

Comment 27

[leo.bugzilla.gecko]

We should fix this issue by this week because of Leo's Q process deadline
So I want to know if youtube is modifying this issue.
Donovan, Can you contact to youtube and share info with us?

Comment 28

[Jason Smith [:jsmith]]

(In reply to leo.bugzilla.gecko from comment #27)
> We should fix this issue by this week because of Leo's Q process deadline
> So I want to know if youtube is modifying this issue.
> Donovan, Can you contact to youtube and share info with us?

Donovan is on PTO, so sending this over to Vishy.

Comment 29

[vkrishnamoorthy]

Youtube is now looking at this issue, I will update with details as I get them

Comment 30

[vkrishnamoorthy]

The problem is in the youtube content and they are actively debugging this.

Comment 31

[jongsoo.oh]

Do you have any update from youtube?

Comment 32

[Donovan Preston [:fzzzy]]

I will ask them for a status update.

Comment 33

[Donovan Preston [:fzzzy]]

YT says a fix is being tested and should be deployed later this week.

Comment 34

[Donovan Preston [:fzzzy]]

This was fixed in YT's deploy last week.

I tested it and verified that it is fixed.