-
Notifications
You must be signed in to change notification settings - Fork 35
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
bump to latest devtools #25
base: master
Are you sure you want to change the base?
Conversation
Hi @paulirish Debugging devtools-timeline-model in master and latestbump branches I have reached these conclusions: In this test:
You expected that the events of the main thread should be 7721 but it result in 7234. Debugging the test, I have seen that these events are conformed by the addition of Events, JSFrames and JSSamples. Between master and latetsbump there is a difference between JSFrames (1193 in master and 709 in latest, using the same report './test/assets/devtools-homepage-w-screenshots-trace.json'). So I think that the json cannot be the same between versions due to incompatibilities in chrome-devtools-frontend. I also think that these test are testing that chrome-devtools-frontend is generating the model properly so in my opinion I suppose that should be this project instead devtools-timeline-model which has to test the model generation from file because devtools-timeline-model is simply using the API provided by chrom-devtools-frontend. So, ¿Is there any other approach to test this project like test that the API calls are returning values instead an exact value? For example:
Thanks in advance! |
This PR is awesome, and very worthy to merge. @paulirish, please merge it. |
@paulirish when would you like to merge this PR? it would be very helpful for us so this great tool can support timeline file generated by latest Chrome. |
Looks like it's abandoned. Anyone interests to fork this repo and merge this PR? |
the bottom-up and topdown results aren't sorted correctly. aside from that, this is in good shape.