Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

video not always scaled to fit window #54

Open
jpiesing opened this issue Sep 24, 2021 · 18 comments
Open

video not always scaled to fit window #54

jpiesing opened this issue Sep 24, 2021 · 18 comments
Assignees
Labels
Deferred Deferred to future work.

Comments

@jpiesing
Copy link

I've seen a couple of times when the video for encrypted content playback is not scaled to fit the window.
Screenshot_20210924_142235
I does the same on a desktop if you make the browser window smaller than the video.
image

@louaybassbouss
Copy link
Collaborator

@FritzHeiden can you check

@FritzHeiden
Copy link
Collaborator

I am unable to reproduce your issue:

image

image

What version do you use (commit id)? Or you did you deploy the test runner?

@jpiesing
Copy link
Author

I was seeing this twice in 3 times when I ran the test.
I just just whatever was the head of the trunk (i.e. ./build.sh --reload-tests --reload-runner).

@FritzHeiden
Copy link
Collaborator

FritzHeiden commented Sep 27, 2021

After taking another look at the screenshots, it seems like there is no css applied to the page. Maybe this is an issue with loading the css file. I'm still not sure what may cause this.

@jpiesing
Copy link
Author

After taking another look at the screenshots, it seems like there is no css applied to the page. Maybe this is an issue with loading the css file. I'm still not sure what may cause this.

Would there be any error messages if this had happened?

@FritzHeiden
Copy link
Collaborator

FritzHeiden commented Sep 27, 2021

There should be an error stating that loading the css file failed. I did not explicitly implement it.

@jpiesing
Copy link
Author

jpiesing commented Oct 6, 2021

I've just tried the encrypted content playback again and I get this issue every time. There's nothing in the messages in the report afterwards. Here are all references to CSS in the docker log.
docker-log-css-references.txt
There's no 403s or 40 anythings. I'll upload a screenshot from the phone while this is happening.

@jpiesing
Copy link
Author

jpiesing commented Oct 6, 2021

Screenshot_20211006-111137
Here's a screen capture from the phone with encrypted content playback running. You can see that the browser graphics are not overlaying the video any more.

@louaybassbouss
Copy link
Collaborator

@jpiesing if you open the same test in Chrome Desktop (via same IP and not localhost) do you see the same issue? if yes can you open the Developer console -> Network Tab and check if there are errors while downloading some files (especially CSS). If the issue only happens on the Android Tablet, is there any chance to get a screenshot of the developer console on the tablet via remote debugging (https://developer.chrome.com/docs/devtools/remote-debugging/)?

@jpiesing
Copy link
Author

jpiesing commented Oct 6, 2021

@jpiesing if you open the same test in Chrome Desktop (via same IP and not localhost) do you see the same issue? if yes can you open the Developer console -> Network Tab and check if there are errors while downloading some files (especially CSS).

Yes - it happens with Chromium on Linux - at least once. The second time, it worked. Once it works the first time, it continues to work. I'm investigating if clearing the cache will make it revert to breaking.
image

@jpiesing
Copy link
Author

jpiesing commented Oct 6, 2021

Clearing the cache seems to make it repeatable.
image

@jpiesing
Copy link
Author

jpiesing commented Oct 6, 2021

This one is off the phone.
image

@jpiesing
Copy link
Author

jpiesing commented Oct 6, 2021

Umm. I wonder if it's this ...

https://www.titanwolf.org/Network/q/51138767-7580-4885-9ec0-a8ceb03703c4/y

which refers to this old Chrome bug - which shows no sign of getting fixed.

https://bugs.chromium.org/p/chromium/issues/detail?id=488043

"The issue is resolved when I use a valid certificate instead of a self-signed one for my web application. Further details can be found by visiting the above URL."

@FritzHeiden
Copy link
Collaborator

Clearing the cache seems to make it repeatable.

I was able to reproduce the behavior for some time. I added a dummy.css before the style.css, hoping the error occurs with the dummy, rather than the actual style. I was unable to test it as I was unable to reproduce the bug after I implemented it. @jpiesing Could try to rerun tests with these changes and see if this may fix the issue?

@jpiesing
Copy link
Author

I was able to reproduce the behavior for some time. I added a dummy.css before the style.css, hoping the error occurs with the dummy, rather than the actual style. I was unable to test it as I was unable to reproduce the bug after I implemented it. @jpiesing Could try to rerun tests with these changes and see if this may fix the issue?

@FritzHeiden Please check that you actually checked this in. When I re-run the tests, the problem is still there.

@jpiesing
Copy link
Author

image

@gitwjr
Copy link

gitwjr commented Nov 8, 2022

Document the issue in Readme as a list of known bugs. Request people doing testing to report if they still see them happening.
@FritzHeiden to document.

@FritzHeiden
Copy link
Collaborator

I added a list of known bugs, including this one here

@gitwjr gitwjr added the Deferred Deferred to future work. label Nov 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Deferred Deferred to future work.
Projects
None yet
Development

No branches or pull requests

4 participants