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

docs(headless-chrome): switch to prioritize headless #4553

Merged
merged 2 commits into from
Feb 21, 2018

Conversation

ebidel
Copy link
Contributor

@ebidel ebidel commented Feb 15, 2018

Prioritizing headless now that it supports emulation.

Copy link
Collaborator

@patrickhulce patrickhulce left a comment

Choose a reason for hiding this comment

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

nice win!

For now, we recommend running Chrome with xvfb. See below.
## CLI (headless)

> **Note**: Headless Chrome still has a few bugs to work out. For example, [network emulation](https://bugs.chromium.org/p/chromium/issues/detail?id=728451) is not supported yet.
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we remove this caveat now? :D

might be worth still noting there may be some graphics-related perf differences

Copy link
Member

Choose a reason for hiding this comment

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

👍 . offline and throttling are implemented. we can nuke the note.

there should probably be a list somewhere which documents known differences between headless and headful. but there isn't one yet.

Copy link
Member

Choose a reason for hiding this comment

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

ping @ebidel


## CLI (xvfb)

Chrome + xvfb is the stable solution we recommend. These steps worked on Debian Jessie:
Alternativeluy, you can run full Chrome + xvfb instead of headless mode. These steps worked on Debian Jessie:
Copy link
Collaborator

Choose a reason for hiding this comment

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

s/Alternativeluy/Alternatively

@ebidel
Copy link
Contributor Author

ebidel commented Feb 21, 2018

ready to go

@ebidel ebidel closed this Feb 21, 2018
@ebidel ebidel reopened this Feb 21, 2018
@patrickhulce patrickhulce merged commit 7226fcd into master Feb 21, 2018
@patrickhulce patrickhulce deleted the ebidel-patch-3 branch February 21, 2018 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants