Skip to content
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

Rename "connection" promise to "opened" #16

Merged
merged 1 commit into from
Jul 14, 2023
Merged

Rename "connection" promise to "opened" #16

merged 1 commit into from
Jul 14, 2023

Conversation

ricea
Copy link
Owner

@ricea ricea commented Jul 14, 2023

For alignment with Direct Sockets, rename the "connection" promise to "opened".

Fixes #14.

For alignment with Direct Sockets, rename the "connection" promise to
"opened".
@ricea
Copy link
Owner Author

ricea commented Jul 14, 2023

@domenic This look okay?

Copy link

@domenic domenic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@ricea ricea merged commit 791214e into master Jul 14, 2023
@ricea ricea deleted the rename-connection branch July 14, 2023 07:03
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Jul 19, 2023
See ricea/websocketstream-explainer#16

Also add some unit tests for use of AbortSignal. This was already
tested by wpts, but I need to keep the coverage bot happy.

Change-Id: I559553796a8d1358506d26800108105be58a0e13
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Jul 21, 2023
See ricea/websocketstream-explainer#16

Also add some unit tests for use of AbortSignal. This was already
tested by wpts, but I need to keep the coverage bot happy.

Change-Id: I559553796a8d1358506d26800108105be58a0e13
aarongable pushed a commit to chromium/chromium that referenced this pull request Jul 27, 2023
See ricea/websocketstream-explainer#16

Also add some unit tests for use of AbortSignal. This was already
tested by wpts, but I need to keep the coverage bot happy.

Change-Id: I559553796a8d1358506d26800108105be58a0e13
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4685590
Reviewed-by: Nidhi Jaju <nidhijaju@chromium.org>
Commit-Queue: Adam Rice <ricea@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1175907}
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Jul 27, 2023
See ricea/websocketstream-explainer#16

Also add some unit tests for use of AbortSignal. This was already
tested by wpts, but I need to keep the coverage bot happy.

Change-Id: I559553796a8d1358506d26800108105be58a0e13
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4685590
Reviewed-by: Nidhi Jaju <nidhijaju@chromium.org>
Commit-Queue: Adam Rice <ricea@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1175907}
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Jul 27, 2023
See ricea/websocketstream-explainer#16

Also add some unit tests for use of AbortSignal. This was already
tested by wpts, but I need to keep the coverage bot happy.

Change-Id: I559553796a8d1358506d26800108105be58a0e13
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4685590
Reviewed-by: Nidhi Jaju <nidhijaju@chromium.org>
Commit-Queue: Adam Rice <ricea@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1175907}
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Aug 2, 2023
…"opened", a=testonly

Automatic update from web-platform-tests
WebSocketStream: Rename "connection" to "opened"

See ricea/websocketstream-explainer#16

Also add some unit tests for use of AbortSignal. This was already
tested by wpts, but I need to keep the coverage bot happy.

Change-Id: I559553796a8d1358506d26800108105be58a0e13
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4685590
Reviewed-by: Nidhi Jaju <nidhijaju@chromium.org>
Commit-Queue: Adam Rice <ricea@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1175907}

--

wpt-commits: 3bdd1a1562d55842cb4f13dda3432df2f779ffa6
wpt-pr: 41040
ErichDonGubler pushed a commit to ErichDonGubler/firefox that referenced this pull request Aug 4, 2023
…"opened", a=testonly

Automatic update from web-platform-tests
WebSocketStream: Rename "connection" to "opened"

See ricea/websocketstream-explainer#16

Also add some unit tests for use of AbortSignal. This was already
tested by wpts, but I need to keep the coverage bot happy.

Change-Id: I559553796a8d1358506d26800108105be58a0e13
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4685590
Reviewed-by: Nidhi Jaju <nidhijaju@chromium.org>
Commit-Queue: Adam Rice <ricea@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1175907}

--

wpt-commits: 3bdd1a1562d55842cb4f13dda3432df2f779ffa6
wpt-pr: 41040
Lightning00Blade pushed a commit to Lightning00Blade/wpt that referenced this pull request Dec 11, 2023
See ricea/websocketstream-explainer#16

Also add some unit tests for use of AbortSignal. This was already
tested by wpts, but I need to keep the coverage bot happy.

Change-Id: I559553796a8d1358506d26800108105be58a0e13
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4685590
Reviewed-by: Nidhi Jaju <nidhijaju@chromium.org>
Commit-Queue: Adam Rice <ricea@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1175907}
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

Successfully merging this pull request may close these issues.

Rename connection to open?
2 participants