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

Simulate mobile CPU #3027

Closed
pankajparashar opened this issue Jan 20, 2024 · 3 comments · Fixed by #3032
Closed

Simulate mobile CPU #3027

pankajparashar opened this issue Jan 20, 2024 · 3 comments · Fixed by #3032
Labels
ADO cat: devtools DevTools-related content.

Comments

@pankajparashar
Copy link
Contributor

pankajparashar commented Jan 20, 2024

Link to the documentation page:
https://learn.microsoft.com/en-us/microsoft-edge/devtools-guide-chromium/evaluate-performance/#simulate-a-mobile-cpu

Description of the issue:
Reading the docs on how to simulate mobile CPU, it's not clear why Step 2 (Select the Screenshots checkbox) is required. I was able to test the demo page to slow the down the CPU (4x/6x) successfully without needing Step 2.

I think the text below the screenshot which says (The demo doesn't work well with 6x slowdown, so it just uses 4x slowdown for instructional purposes.) may also be not relevant anymore, as I was able to throttle the CPU to 6x slower and the demo still worked fine, albeit a bit slow (which is expected).

Screenshots:
image

Microsoft Edge version:
Version 120.0.2210.144 (Official build) (arm64)

AB#48606170

@captainbrosset captainbrosset added cat: devtools DevTools-related content. and removed triage-needed labels Jan 22, 2024
@captainbrosset
Copy link
Contributor

Thanks for filing @pankajparashar. Adding to our backlog to fix.

@pankajparashar
Copy link
Contributor Author

I'm fine to submit a PR to remove these two sections (highlighted), if you agree that the observations are correct.

@captainbrosset
Copy link
Contributor

Yes that would be great! While I haven't tested yet, I agree with your observations and would welcome a PR on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ADO cat: devtools DevTools-related content.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants