Skip to content
This repository was archived by the owner on Jul 22, 2024. It is now read-only.

Commit 970212e

Browse files
authored
Set youtube viewport before DOMContentLoaded (#2596)
* Set youtube viewport before DOMContentLoaded * Restore Youtube UA overrides * Move youtube reload code
1 parent 4ccc999 commit 970212e

File tree

3 files changed

+10
-7
lines changed

3 files changed

+10
-7
lines changed

app/src/common/shared/org/mozilla/vrbrowser/browser/engine/Session.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -386,16 +386,15 @@ private void restore() {
386386
loadDefaultPage();
387387
} else if (mState.mSessionState != null) {
388388
mState.mSession.restoreState(mState.mSessionState);
389+
if (mState.mUri != null && mState.mUri.contains(".youtube.com")) {
390+
mState.mSession.loadUri(mState.mUri, GeckoSession.LOAD_FLAGS_REPLACE_HISTORY);
391+
}
389392
} else if (mState.mUri != null) {
390393
mState.mSession.loadUri(mState.mUri);
391394
} else {
392395
loadDefaultPage();
393396
}
394397

395-
if (mState.mUri != null && mState.mUri.contains(".youtube.com")) {
396-
mState.mSession.loadUri(mState.mUri, GeckoSession.LOAD_FLAGS_REPLACE_HISTORY);
397-
}
398-
399398
dumpAllState();
400399
mState.setActive(true);
401400
}
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,6 @@
11
{
22
"744723238cc3c00582d7ff9e8ece83e6503a3926d8c07491672d5da3634be41e1d41580dd3694b663c0d0a5e64280a2b411a1346e6ed0d53d694835781cf6436": "Mozilla/5.0 (X11; Linux x86_64; rv:69) Gecko/20100101 Firefox/69.0",
3+
"ae0755740e4354ac67025056e775ad06d8a529ae4f37244fbb02d72199e2c780311e47aa9895079b980ec4bfa676f1f39c4ab41ea995c524e52bde9a73623da2": "Mozilla/5.0 (Linux; Android 7.1.1; Quest) AppleWebKit/537.36 (KHTML, like Gecko) OculusBrowser/7.0.13.186866463 SamsungBrowser/4.0 Chrome/77.0.3865.126 Mobile VR Safari/537.36",
4+
"e6137b4c2f49a3917c2c90a50fb270a5eebb962f2c72344ae2e29e321bb21891e5ca4fec06cae78e14f4a8510473e934234e9ec3f60e8415f5f6da754c55b9b1": "Mozilla/5.0 (Linux; Android 7.1.1; Quest) AppleWebKit/537.36 (KHTML, like Gecko) OculusBrowser/7.0.13.186866463 SamsungBrowser/4.0 Chrome/77.0.3865.126 Mobile VR Safari/537.36"
35
}
6+

app/src/main/assets/web_extensions/webcompat_youtube/main.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,10 @@ class YoutubeExtension {
3131
if (viewport) {
3232
viewport.setAttribute('content', content);
3333
} else {
34-
document.head.insertAdjacentHTML('afterbegin', `<meta name="viewport" content="${content}"/>`);
34+
const container = document.head || document.documentElement;
35+
container.insertAdjacentHTML('afterbegin', `<meta name="viewport" content="${content}"/>`);
3536
}
36-
logDebug(`Youtube viewport updated`);
37+
logDebug(`Youtube viewport updated: ${window.innerWidth}x${window.innerHeight} `);
3738
}
3839

3940
// Select a better youtube video quality
@@ -203,7 +204,7 @@ class YoutubeExtension {
203204
logDebug(`Initializing youtube extension in frame: ${window.location.href}`);
204205
const youtube = new YoutubeExtension();
205206
youtube.overrideUA();
206-
window.addEventListener('DOMContentLoaded', () => youtube.overrideViewport());
207+
youtube.overrideViewport();
207208
window.addEventListener('load', () => {
208209
logDebug('page load');
209210
youtube.overrideVideoProjection();

0 commit comments

Comments
 (0)