Google Git
Sign in
chromium / chromium / src / 29859ea14fcddec4f2acad460b57bc3ac601cc8b
commit29859ea14fcddec4f2acad460b57bc3ac601cc8b[log] [tgz]
authorKai Ninomiya <kainino@chromium.org>Fri Nov 03 22:03:39 2023
committerChromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com>Fri Nov 03 22:03:39 2023
treee0814655db49c3b63281e58576bc5ef48ced0dad
parent65e5d48d4ba4c310275c2f74f90142f9d1458322 [diff]
Reject requiredLimits values that are too large for the type

This could have been a 1-character fix, but I preferred to use
CheckedNumeric, and also wanted to update the error message.

Note this doesn't quite match the WebGPU spec - the spec technically
says to allow alignments to be set to powers of 2 >= 2**32, but this
disallows anything higher than 2**31. I'm going to propose to the spec
to tighten this for simplicity of implementation.

Test: https://github.com/gpuweb/cts/pull/3132
Fixed: 1499030
Change-Id: Id2c6e56fcbc326c445b28219b724f6ae325077ed
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5002999
Auto-Submit: Kai Ninomiya <kainino@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Commit-Queue: Kai Ninomiya <kainino@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1219757}
1 file changed
tree: e0814655db49c3b63281e58576bc5ef48ced0dad
  1. android_webview/
  2. apps/
  3. ash/
  4. base/
  5. build/
  6. build_overrides/
  7. buildtools/
  8. cc/
  9. chrome/
  10. chromecast/
  11. chromeos/
  12. codelabs/
  13. components/
  14. content/
  15. courgette/
  16. crypto/
  17. dbus/
  18. device/
  19. docs/
  20. extensions/
  21. fuchsia_web/
  22. gin/
  23. google_apis/
  24. google_update/
  25. gpu/
  26. headless/
  27. infra/
  28. ios/
  29. ipc/
  30. media/
  31. mojo/
  32. native_client_sdk/
  33. net/
  34. pdf/
  35. ppapi/
  36. printing/
  37. remoting/
  38. rlz/
  39. sandbox/
  40. services/
  41. skia/
  42. sql/
  43. storage/
  44. styleguide/
  45. testing/
  46. third_party/
  47. tools/
  48. ui/
  49. url/
  50. webkit/
  51. clank
  52. internal
  53. ios_internal
  54. native_client
  55. signing_keys
  56. v8
  57. .clang-format
  58. .clang-tidy
  59. .eslintrc.js
  60. .git-blame-ignore-revs
  61. .gitattributes
  62. .gitignore
  63. .gitmodules
  64. .gn
  65. .mailmap
  66. .rustfmt.toml
  67. .vpython3
  68. .yapfignore
  69. ATL_OWNERS
  70. AUTHORS
  71. BUILD.gn
  72. CODE_OF_CONDUCT.md
  73. codereview.settings
  74. DEPS
  75. DIR_METADATA
  76. LICENSE
  77. LICENSE.chromium_os
  78. OWNERS
  79. PRESUBMIT.py
  80. PRESUBMIT_test.py
  81. PRESUBMIT_test_mocks.py
  82. README.md
  83. WATCHLISTS
README.md

Logo Chromium

Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.

The project's web site is https://www.chromium.org.

To check out the source code locally, don't use git clone! Instead, follow the instructions on how to get the code.

Documentation in the source is rooted in docs/README.md.

Learn how to Get Around the Chromium Source Code Directory Structure .

For historical reasons, there are some small top level directories. Now the guidance is that new top level directories are for product (e.g. Chrome, Android WebView, Ash). Even if these products have multiple executables, the code should be in subdirectories of the product.

If you found a bug, please file it at https://crbug.com/new.