Skip to content

Process events from other devtools pages #1517

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

Closed
annagrin opened this issue Feb 18, 2022 · 1 comment
Closed

Process events from other devtools pages #1517

annagrin opened this issue Feb 18, 2022 · 1 comment

Comments

@annagrin
Copy link
Contributor

Dwds uses Devtools pageReady event from the debugger page to determine the load time of Devtools. For flutter apps, the first event would need to come from the Inspector page instead. Change dwds to record the first pageReady event from any page instead:

if (screen == 'debugger' && action == 'pageReady') {

annagrin pushed a commit to annagrin/webdev that referenced this issue Feb 19, 2022
- Store dwdsStats on appServices for an app to make sure we are
  updating an reporting the same stats for an app. Previously,
  if the app services already existed, we would use incorrect
  object to report the stats, causing incorrect times and missing
  reports.
- Set the times on dwdsStats when debug request arrives to make
  sure times are recorded only from the time when the user starts
  debugging.
- Process `pageReady` events from any DevTools page, as DevTools
  does not always show the debugger page first.
- Enabled previously skipped tests.

Closes:dart-lang#1517
annagrin pushed a commit that referenced this issue Mar 7, 2022
* Fix issues in DevTools load time reporting

- Store dwdsStats on appServices for an app to make sure we are
  updating an reporting the same stats for an app. Previously,
  if the app services already existed, we would use incorrect
  object to report the stats, causing incorrect times and missing
  reports.
- Set the times on dwdsStats when debug request arrives to make
  sure times are recorded only from the time when the user starts
  debugging.
- Process `pageReady` events from any DevTools page, as DevTools
  does not always show the debugger page first.
- Enabled previously skipped tests.

Closes:#1517

* Format
@bkonyi bkonyi closed this as completed Dec 4, 2024
@jyameo
Copy link
Contributor

jyameo commented Dec 4, 2024

related: #2394

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

No branches or pull requests

3 participants