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

Add user timings for WebPageTest #748

Merged
merged 1 commit into from
Nov 11, 2016

Conversation

gigabo
Copy link
Contributor

@gigabo gigabo commented Nov 11, 2016

Use the browser's performance.mark API.

Add two timings:

  • react-server.wake
    • When the ClientController is instantiated.
  • react-server.renderAboveTheFold
    • When the content above the fold has been re-rendered (visible page is interactive)
  • react-server.renderComplete
    • When the whole page has been re-rendered (fully interactive)

@gigabo
Copy link
Contributor Author

gigabo commented Nov 11, 2016

Updated to eliminate the change in above-the-fold timing logic. I added a renderComplete performance mark, so the renderAboveTheFold mark isn't useful unless <TheFold/> is actually present.

@gigabo gigabo force-pushed the interactive-performance-mark branch 2 times, most recently from dd24486 to 389d92e Compare November 11, 2016 18:18
Use the browser's `performance.mark` API.

Add two timings:

- `react-server.wake`
    - When the ClientController is instantiated.
- `react-server.renderAboveTheFold`
    - When the content above the fold has been re-rendered (visible page is interactive)
- `react-server.renderComplete`
    - When the whole page has been re-rendered (fully interactive)
@gigabo gigabo merged commit 847032d into redfin:master Nov 11, 2016
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.

3 participants