ExtensionTraceDataHandler ignores non-obj perf marks

This code accidentally relied on the details being something that is
`JSON.parse`d as an object, so we can then use the 'in' operator to
search for it.

The fix in this context is to wrap the code in a try catch and return
null if it errors - which means either the details data was not valid
JSON, or it wasn't of the structure we need for the sake of extensions.
Both of these are OK and we just need the extension code to ignore the
event in that case.

Fixed: 341254293
Change-Id: Ic81c006d039b9a240e2c8d7ce8d37d0985fe022b
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5548337
Reviewed-by: Victor Porof <victorporof@chromium.org>
Commit-Queue: Victor Porof <victorporof@chromium.org>
Auto-Submit: Jack Franklin <jacktfranklin@chromium.org>
2 files changed
tree: 5290b9ad5ffa6b950fdbf5c7c5fc10573a332021
  1. .vscode/
  2. build_overrides/
  3. config/
  4. docs/
  5. extension-api/
  6. extensions/
  7. front_end/
  8. inspector_overlay/
  9. node_modules/
  10. scripts/
  11. test/
  12. third_party/
  13. v8/
  14. .clang-format
  15. .editorconfig
  16. .eslintignore
  17. .eslintrc.js
  18. .git-blame-ignore-revs
  19. .gitattributes
  20. .gitignore
  21. .gitmodules
  22. .gn
  23. .mailmap
  24. .npmignore
  25. .npmrc
  26. .style.yapf
  27. .stylelintignore
  28. .stylelintrc.json
  29. AUTHORS
  30. BUILD.gn
  31. codereview.settings
  32. DEPS
  33. LICENSE
  34. OWNERS
  35. package-lock.json
  36. package.json
  37. PRESUBMIT.py
  38. README.md
  39. tsconfig.json
  40. WATCHLISTS
README.md

Chrome DevTools frontend

npm package

The client-side of the Chrome DevTools, including all TypeScript & CSS to run the DevTools webapp.

Source code and documentation

The frontend is available on chromium.googlesource.com. Check out the project documentation for instructions to set up, use, and maintain a DevTools front-end checkout, as well as design guidelines, and architectural documentation.

Additional references

Source mirrors

DevTools frontend repository is mirrored on GitHub.

DevTools frontend is also available on NPM as the chrome-devtools-frontend package. It's not currently available via CJS or ES modules, so consuming this package in other tools may require some effort.

The version number of the npm package (e.g. 1.0.373466) refers to the Chromium commit position of latest frontend git commit. It's incremented with every Chromium commit, however the package is updated roughly daily.

Getting in touch