Closed Bug 1544676 Opened 5 years ago Closed 5 years ago

window.stop() and window.loading = true seems to block document.open()

Categories

(Core :: DOM: Core & HTML, defect, P3)

67 Branch
Unspecified
Android
defect

Tracking

()

RESOLVED INVALID
Webcompat Priority revisit
Tracking Status
firefox68 --- affected

People

(Reporter: karlcow, Unassigned)

References

()

Details

This is https://webcompat.com/issues/28836

  1. With Firefox Android on a pixel 2 device
    Or with Firefox Desktop + Responsive Design Mode + a Mobile user agent
    (example Galaxy S5 in the device menu)
  2. Go to https://supercurioso.com/

Actual:
Page is blank

Expected:
Site is being displayed

At first load of https://supercurioso.com/
we get

<html prefix="og: http://ogp.me/ns#" lang="es">
<!--<![endif]-->

<head>
  <script src="https://cdn.semseoymas.com/hsmobilescript.js?c=supercurioso"></script>
</head>

</html>

This script logic is:

if (mobile_user_agent_string) {

window.stop();

//configs

window.loading = true;

// xhr request preloading a JSON of articles content
// OR window.reload() if not mobile is specified.


// then the document.open

  (r = document.open("text/html", "replace")),
    r.write(
      '<!DOCTYPE html><html><head><meta charset=utf-8><title>HSMobile</title><meta name=viewport content="width=device-width,initial-scale=1,user-scalable=no"><link rel=preload href=https://fonts.gstatic.com/s/sourcesanspro/v11/6xK3dSBYKcSV-LCoeQqfX1RYOo3qOK7lujVj9w.woff2 as=font type=font/woff2 crossorigin><link rel=preload href=https://fonts.gstatic.com/s/sourcesanspro/v11/6xKydSBYKcSV-LCoeQqfX1RYOo3ig4vwlxdu3cOWxw.woff2 as=font type=font/woff2 crossorigin><link rel=preload href=https://fonts.gstatic.com/s/sourcesanspro/v11/6xKwdSBYKcSV-LCoeQqfX1RYOo3qPZZclSds18S0xR41.woff2 as=font type=font/woff2 crossorigin><link rel=preload href=https://fonts.gstatic.com/s/sourcesanspro/v11/6xK1dSBYKcSV-LCoeQqfX1RYOo3qPZ7nsDJB9cme.woff2 as=font type=font/woff2 crossorigin><link href="https://fonts.googleapis.com/css?family=Source+Sans+Pro:400,400i,700,700i" rel=stylesheet><link href=https://cdn.semseoymas.com/dist/static/css/app.80d46841a6ae6123732359e3f48a0f8b.css rel=stylesheet></head><body><div id=app></div><script type=text/javascript src=https://cdn.semseoymas.com/dist/static/js/manifest.4b678d1c358ad66df578.js></script><script type=text/javascript src=https://cdn.semseoymas.com/dist/static/js/vendor.4a6333d13cb7aecf0010.js></script><script type=text/javascript src=https://cdn.semseoymas.com/dist/static/js/app.8bc64f3778bfbfb8f66d.js></script></body></html>');

}

Possibly Bug 1475000

Flags: webcompat?
Depends on: 1475000

Migrating Webcompat whiteboard priorities to project flags. See bug 1547409.

Webcompat Priority: --- → ?

See bug 1547409. Migrating whiteboard priority tags to program flags.

Whiteboard: [webcompat] → [webcompat-revisit]
Webcompat Priority: ? → revisit
Whiteboard: [webcompat-revisit]
Flags: webcompat?

Is this still an issue? I'm not seeing this problem at least in RDM on desktop....

Flags: needinfo?(kdubost)

It doesn't happen anymore because the script is gone. They replaced it by another logic which doesn't depend on window.stop(), window.loading = true

They now define a variable that they reuse later on in the code.

Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(kdubost)
Resolution: --- → INVALID

Great, thank you!

You need to log in before you can comment on or make changes to this bug.