Allow PSA for extending an API to additional platforms [website : main]
76 views
Skip to first unread message
Rick Byers (Gerrit)
Jan 15, 2025, 2:28:33 PMJan 15
to blink-api-ow...@chromium.org
Rick Byers has uploaded the change for review![]( "Open in Gerrit")
Commit message
Allow PSA for extending an API to additional platforms
Change-Id: Ia73eb787141feab09f012311fda20f0ea94234a6
Change diff
diff --git a/site/blink/launching-features/index.md b/site/blink/launching-features/index.md
index e4a61db..755265a 100644
--- a/site/blink/launching-features/index.md
+++ b/site/blink/launching-features/index.md
@@ -124,7 +124,8 @@
large-scale code refactoring or rewriting, where the goal is to cause no
behavioral changes (but due to scope of change, side effects are likely), or
this may encompass changes to current code that fix a bug or implement new
-changes to the spec without changes to the API shape itself.
+changes to the spec without changes to the API shape itself. This may also be
+used for changes which extend an already-exposed API to additional platforms.
### Changing feature type
@@ -773,8 +774,14 @@
for any change with some potential to break sites. Use "Web developer facing code change"
only for changes deemed very unlikely to break sites, and that change APIs in at most a trivial way.
-Another reason for a PSA could be a large-scale refactoring that
-doesn't *intend* to result in behavior changes, but may do so in practice.
+Changes which extend an already exposed API to additional platforms without
+introducing new platform-specific risks may also rely just on a PSA, along with
+updating the chromestatus entry to reflect the new platform support. For example,
+when simply extending an API already exposed on all desktop platforms to also be
+exposed on Android using the same implementation, a PSA is generally sufficient.
+But if the Android implementation has meaningful differences in behavior which
+could result in web compatibility issues, then the full "New feature incubation"
+process should be followed.
The purpose of a PSA is to notify the broader Chromium community about
the change, and enables folks to test against it and potentially
Change information
Files:
- M site/blink/launching-features/index.md
Change size: S
Delta: 1 file changed, 10 insertions(+), 3 deletions(-)
Related details
Attention set is empty
Submit Requirements:
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Chris Harrelson (Gerrit)
Jan 15, 2025, 2:39:08 PMJan 15
to blink-api-ow...@chromium.org
Attention needed from Rick Byers
Rick Byers has uploaded the change for review![]( "Open in Gerrit")
Commit message
Allow PSA for extending an API to additional platforms
See https://groups.google.com/a/chromium.org/g/blink-api-owners-discuss/c/YuZB8xHM3go/m/FEv1rpyaBwAJ
Change-Id: Ia73eb787141feab09f012311fda20f0ea94234a6
Related details
Attention is currently required from:
- Rick Byers
Submit Requirements:
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Chris Harrelson (Gerrit)
Jan 15, 2025, 2:41:17 PMJan 15
to Rick Byers, Mike Taylor, chromium-websi...@chops-service-accounts.iam.gserviceaccount.com, blink-api-ow...@chromium.org, jyas...@chromium.org
Attention needed from Rick Byers
Chris Harrelson added 1 comment![]( "Open in Gerrit")
File site/blink/launching-features/index.md
Line 128, Patchset 2 (Latest):
used for changes which extend an already-exposed API to additional platforms.Chris Harrelson . unresolved
```suggestion
used for changes which extend an already-exposed API to additional platforms (see more discussion in the *Assess backwards compatibility* section for conditions).
```
Related details
Attention is currently required from:
- Rick Byers
Submit Requirements:
Code-Review
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
chromium-website-try-builder@chops-service-accounts.iam.gserviceaccount.com (Gerrit)
Jan 15, 2025, 4:22:34 PMJan 15
to Rick Byers, Mike Taylor, Chris Harrelson, blink-api-ow...@chromium.org, jyas...@chromium.org
Attention needed from Rick Byers
Message from chromium-websi...@chops-service-accounts.iam.gserviceaccount.com![]( "Open in Gerrit")
Preview this change at https://chromium-website--cl6177108-ps4-9iwnexkx.web.app
Related details
Attention is currently required from:
- Rick Byers
Submit Requirements:
Code-Review
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Rick Byers (Gerrit)
Jan 15, 2025, 4:24:58 PMJan 15
to Mike Taylor, chromium-websi...@chops-service-accounts.iam.gserviceaccount.com, Chris Harrelson, blink-api-ow...@chromium.org, jyas...@chromium.org
Attention needed from Chris Harrelson
Rick Byers added 1 comment![]( "Open in Gerrit")
File site/blink/launching-features/index.md
Line 128, Patchset 2:
used for changes which extend an already-exposed API to additional platforms.Chris Harrelson . unresolved
```suggestion
used for changes which extend an already-exposed API to additional platforms (see more discussion in the *Assess backwards compatibility* section for conditions).
```
Rick Byers
Tweaked to use a link for ease of navigation and brevity, ok Chris?
Related details
Attention is currently required from:
- Chris Harrelson
Submit Requirements:
Code-Review
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Chris Harrelson (Gerrit)
Jan 15, 2025, 4:25:21 PMJan 15
to Rick Byers, Mike Taylor, chromium-websi...@chops-service-accounts.iam.gserviceaccount.com, blink-api-ow...@chromium.org, jyas...@chromium.org
Attention needed from Rick Byers
Chris Harrelson voted and added 1 comment![]( "Open in Gerrit")
Votes added by Chris Harrelson
| Code-Review | +1 |
1 comment
File site/blink/launching-features/index.md
Line 128, Patchset 2:
used for changes which extend an already-exposed API to additional platforms.Chris Harrelson . resolved
Rick Byers```suggestion
used for changes which extend an already-exposed API to additional platforms (see more discussion in the *Assess backwards compatibility* section for conditions).
```
Tweaked to use a link for ease of navigation and brevity, ok Chris?
Attention is currently required from:
- Rick Byers
Submit Requirements:
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
chromium-website-try-builder@chops-service-accounts.iam.gserviceaccount.com (Gerrit)
Jan 15, 2025, 4:26:35 PMJan 15
to Rick Byers, Mike Taylor, Chris Harrelson, blink-api-ow...@chromium.org, jyas...@chromium.org
Attention needed from Rick Byers
Message from chromium-websi...@chops-service-accounts.iam.gserviceaccount.com![]( "Open in Gerrit")
Preview this change at https://chromium-website--cl6177108-ps5-uv0id7qp.web.app
Related details
Attention is currently required from:
- Rick Byers
Submit Requirements:
Code-Review
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
chromium-website-try-builder@chops-service-accounts.iam.gserviceaccount.com (Gerrit)
Jan 15, 2025, 4:27:28 PMJan 15
to Rick Byers, Chris Harrelson, Mike Taylor, blink-api-ow...@chromium.org, jyas...@chromium.org
Attention needed from Rick Byers
Message from chromium-websi...@chops-service-accounts.iam.gserviceaccount.com![]( "Open in Gerrit")
Preview this change at https://chromium-website--cl6177108-ps6-hmx2e67a.web.app
Related details
Attention is currently required from:
- Rick Byers
Submit Requirements:
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Rick Byers (Gerrit)
Jan 15, 2025, 4:30:47 PMJan 15
to Chris Harrelson, Mike Taylor, chromium-websi...@chops-service-accounts.iam.gserviceaccount.com, blink-api-ow...@chromium.org, jyas...@chromium.org
Rick Byers voted Commit-Queue+2![]( "Open in Gerrit")
Submit Requirements:
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
chromium-website-scoped@luci-project-accounts.iam.gserviceaccount.com (Gerrit)
Jan 15, 2025, 4:30:55 PMJan 15
to Rick Byers, Chris Harrelson, Mike Taylor, chromium-websi...@chops-service-accounts.iam.gserviceaccount.com, blink-api-ow...@chromium.org, jyas...@chromium.org
chromium-we...@luci-project-accounts.iam.gserviceaccount.com submitted the change with unreviewed changes![]( "Open in Gerrit")
Unreviewed changes
5 is the latest approved patch-set.
The change was submitted with unreviewed changes in the following files:
```
The name of the file: site/blink/launching-features/index.md
Insertions: 1, Deletions: 1.
@@ -125,7 +125,7 @@
behavioral changes (but due to scope of change, side effects are likely), or
this may encompass changes to current code that fix a bug or implement new
changes to the spec without changes to the API shape itself. This may also be
-used for changes which extend an already-exposed API to additional platforms.
+used for changes which extend an already-exposed API to additional platforms
(see [conditions](#behavior-change-compat)).
### Changing feature type
```
Change information
Commit message:
Allow PSA for extending an API to additional platforms
See https://groups.google.com/a/chromium.org/g/blink-api-owners-discuss/c/YuZB8xHM3go/m/FEv1rpyaBwAJ
Change-Id: Ia73eb787141feab09f012311fda20f0ea94234a6
Reviewed-by: Chris Harrelson <chri...@chromium.org>
Commit-Queue: Rick Byers <rby...@chromium.org>
Reviewed-by: Mike Taylor <mike...@chromium.org>
Files:
- M site/blink/launching-features/index.md
Change size: S
Delta: 1 file changed, 11 insertions(+), 3 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Mike Taylor, +1 by Chris Harrelson
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages