| Commit-Queue | +1 |
Hi reviewers. PTAL as follows:
- @sk...@chromium.org: third_party/cros-components/ and third_party/material_web_components/
- @joelh...@chromium.org: ui/file_manager/
- @rbpotter: Everything else
Thanks!
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Commit-Queue | +1 |
Hi reviewers. PTAL as follows:
- @sk...@chromium.org: third_party/cros-components/ and third_party/material_web_components/
- @joelh...@chromium.org: ui/file_manager/
- @rbpotter: Everything else
Thanks!
There is still an issue with `ios-simulator`, whih I thought had already been fixed. Pinged https://issues.chromium.org/issues/487338357#comment17 for help on fixing. Will resolve this comment once the iOS issue is addressed.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Commit-Queue | +1 |
Demetrios PapadopoulosHi reviewers. PTAL as follows:
- @sk...@chromium.org: third_party/cros-components/ and third_party/material_web_components/
- @joelh...@chromium.org: ui/file_manager/
- @rbpotter: Everything else
Thanks!
Demetrios PapadopoulosThere is still an issue with `ios-simulator`, whih I thought had already been fixed. Pinged https://issues.chromium.org/issues/487338357#comment17 for help on fixing. Will resolve this comment once the iOS issue is addressed.
There is still an issue with ios-simulator
This is now resolved.
@skau: third_party/cros-components/ and third_party/material_web_components/
Redirecting to other reviweers since skau@ is OOO.
@jhaw...@chromium.org: PTAL at third_party/cros-components/tsconfig_base.json
@camb...@google.com: PTAL at third_party/material_web_components/tsconfig_base.json.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Code-Review | +1 |
Demetrios PapadopoulosHi reviewers. PTAL as follows:
- @sk...@chromium.org: third_party/cros-components/ and third_party/material_web_components/
- @joelh...@chromium.org: ui/file_manager/
- @rbpotter: Everything else
Thanks!
Demetrios PapadopoulosThere is still an issue with `ios-simulator`, whih I thought had already been fixed. Pinged https://issues.chromium.org/issues/487338357#comment17 for help on fixing. Will resolve this comment once the iOS issue is addressed.
There is still an issue with ios-simulator
This is now resolved.
@skau: third_party/cros-components/ and third_party/material_web_components/
Redirecting to other reviweers since skau@ is OOO.
@jhaw...@chromium.org: PTAL at third_party/cros-components/tsconfig_base.json
@camb...@google.com: PTAL at third_party/material_web_components/tsconfig_base.json.
Acknowledged
@thakis: Could you stamp third_party/material_web_components/tsconfig_base.json ? Note that this has already been approved by camb...@google.com, but the owners file has incorrectly camb...@chromium.org (which per their word never existed?).
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
@thakis: Could you stamp third_party/material_web_components/tsconfig_base.json ? Note that this has already been approved by camb...@google.com, but the owners file has incorrectly camb...@chromium.org (which per their word never existed?).
Actually, thakis@ is also OOO, as well as all others from third_party/material_web_components/OWNERS.
Adding chromium-t...@google.com as a last resort to unblock this CL.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Auto-Submit | +1 |
Demetrios Papadopoulos@thakis: Could you stamp third_party/material_web_components/tsconfig_base.json ? Note that this has already been approved by camb...@google.com, but the owners file has incorrectly camb...@chromium.org (which per their word never existed?).
Actually, thakis@ is also OOO, as well as all others from third_party/material_web_components/OWNERS.
Adding chromium-t...@google.com as a last resort to unblock this CL.
@d...@google.com: PTAL ^
Demetrios Papadopoulos@thakis: Could you stamp third_party/material_web_components/tsconfig_base.json ? Note that this has already been approved by camb...@google.com, but the owners file has incorrectly camb...@chromium.org (which per their word never existed?).
Demetrios PapadopoulosActually, thakis@ is also OOO, as well as all others from third_party/material_web_components/OWNERS.
Adding chromium-t...@google.com as a last resort to unblock this CL.
@d...@google.com: PTAL ^
Demetrios Papadopoulos@thakis: Could you stamp third_party/material_web_components/tsconfig_base.json ? Note that this has already been approved by camb...@google.com, but the owners file has incorrectly camb...@chromium.org (which per their word never existed?).
Demetrios PapadopoulosActually, thakis@ is also OOO, as well as all others from third_party/material_web_components/OWNERS.
Adding chromium-t...@google.com as a last resort to unblock this CL.
Dan Le Febvre@d...@google.com: PTAL ^
Am looking, just want to check a few things.
Done
| Owners-Override | +1 |
WebUI: Update TypeScript compiler from 5.9.3 to 6.0.2.
Specifically:
- Removing chrome/browser/resources/glic/url_pattern.d.ts since these
are now part of TS compiler's default definitions.
- Changing 'module' to 'namespace' in
components/translate/ios/browser/resources/translate_ios.ts, since
usage of the deprecated "module" now throws an error [1].
- Updating third_party/node/patches/typescript.patch to restore the
type of `HTMLElement#hidden` to just `boolean` instead of
`"boolean|"until-found"` since the latter causes a lot of TS errors
in WebUI tests. These will be addressed in a follow-up.
- Removing newly unnecessary definitions in
tools/typescript/definitions/pending.d.ts.
CrOS changes
- Adding 'ignoreDeprecations' flag in CrOS targets that fail otherwise
due to using stuff that is officially deprecated in v6:
- chrome/browser/resources/chromeos/accessibility/
- third_party/cros-components/
- third_party/material_web_components/
- Fixing newly surfaced type errors in ash/webui/media_app_ui/ and
ui/file_manager/
[1] https://devblogs.microsoft.com/typescript/announcing-typescript-6-0/#deprecated:-legacy-module-syntax-for-namespaces
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Hi reviewers. This CL is identical to the original reverted CL at https://chromium-review.googlesource.com/c/chromium/src/+/7405484. See "Reland notes" in CL description about what has changed since.
PTAL as follows:
@luc...@chromium.org: ui/file_manager/ (previously reviewed by @joelh...@chromium.org who is OOO)
@jhaw...@chromium.org: third_party/cros-components/tsconfig_base.json
@d...@google.com: third_party/material_web_components/tsconfig_base.json (cc'ed @camb...@google.com who is an owner but without access to their chromium account).
Hopefully it sticks this time.
Thank you!
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Hi reviewers. This CL is identical to the original reverted CL at https://chromium-review.googlesource.com/c/chromium/src/+/7405484. See "Reland notes" in CL description about what has changed since.
PTAL as follows:
@luc...@chromium.org: ui/file_manager/ (previously reviewed by @joelh...@chromium.org who is OOO)
@jhaw...@chromium.org: third_party/cros-components/tsconfig_base.json
@d...@google.com: third_party/material_web_components/tsconfig_base.json (cc'ed @camb...@google.com who is an owner but without access to their chromium account).Hopefully it sticks this time.
Thank you!
Can we just fix the OWNERS file for third_party/material_web_components/tsconfig_base.json ? I don't want to keep owners overriding this. @camb...@google.com can you please craft a CL swapping your @chromium with your @google ?
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Commit-Queue | +1 |
Dan Le FebvreHi reviewers. This CL is identical to the original reverted CL at https://chromium-review.googlesource.com/c/chromium/src/+/7405484. See "Reland notes" in CL description about what has changed since.
PTAL as follows:
@luc...@chromium.org: ui/file_manager/ (previously reviewed by @joelh...@chromium.org who is OOO)
@jhaw...@chromium.org: third_party/cros-components/tsconfig_base.json
@d...@google.com: third_party/material_web_components/tsconfig_base.json (cc'ed @camb...@google.com who is an owner but without access to their chromium account).Hopefully it sticks this time.
Thank you!
Can we just fix the OWNERS file for third_party/material_web_components/tsconfig_base.json ? I don't want to keep owners overriding this. @camb...@google.com can you please craft a CL swapping your @chromium with your @google ?
Sent separate CL at https://chromium-review.googlesource.com/c/chromium/src/+/7728026. Will update reviewers in this CL once it lands. (Leaving unresolved for now)
| Commit-Queue | +1 |
Dan Le FebvreHi reviewers. This CL is identical to the original reverted CL at https://chromium-review.googlesource.com/c/chromium/src/+/7405484. See "Reland notes" in CL description about what has changed since.
PTAL as follows:
@luc...@chromium.org: ui/file_manager/ (previously reviewed by @joelh...@chromium.org who is OOO)
@jhaw...@chromium.org: third_party/cros-components/tsconfig_base.json
@d...@google.com: third_party/material_web_components/tsconfig_base.json (cc'ed @camb...@google.com who is an owner but without access to their chromium account).Hopefully it sticks this time.
Thank you!
Demetrios PapadopoulosCan we just fix the OWNERS file for third_party/material_web_components/tsconfig_base.json ? I don't want to keep owners overriding this. @camb...@google.com can you please craft a CL swapping your @chromium with your @google ?
Sent separate CL at https://chromium-review.googlesource.com/c/chromium/src/+/7728026. Will update reviewers in this CL once it lands. (Leaving unresolved for now)
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
@fdegros: PTAL at ui/file_manager/ change (both lucmult@ and joelhockey@ are OOO). Thanks!
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |