Skip to content
This repository was archived by the owner on Feb 29, 2020. It is now read-only.

fix(prerender): Check top before loading more scripts#4128

Merged
Mardak merged 1 commit intomozilla:masterfrom
Mardak:check-top
May 1, 2018
Merged

fix(prerender): Check top before loading more scripts#4128
Mardak merged 1 commit intomozilla:masterfrom
Mardak:check-top

Conversation

@Mardak
Copy link
Member

@Mardak Mardak commented Apr 30, 2018

r?@k88hudson Just short circuit the rest of the top level script

Copy link
Contributor

@k88hudson k88hudson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this on prerendered/non-prerendered pages and I don't see a difference – can you maybe just clarify what the reasoning for this / when this condition actually occurs?

@k88hudson
Copy link
Contributor

Ok, this works as expected 👍

@Mardak Mardak merged commit 7be9401 into mozilla:master May 1, 2018
@Mardak Mardak deleted the check-top branch May 1, 2018 18:06
Mardak added a commit that referenced this pull request May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants