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

Revert "bluetooth: Add log for debugging flaky issue 1430625" #49982

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jan 8, 2025

This reverts commit 480ab2f6779dfd0ceb88298f8a828b9d3f881d90.

Reason for revert: No longer need these debug message.

Original change's description:

bluetooth: Add log for debugging flaky issue 1430625

Add more logging for the test characteristic-is-removed.https.window.js
to identify which step of the process is causing the timeout on Mac CI
builder Mac10.14.

Bug: 1430625
Change-Id: I0b266857a3346021814357146ddc0bad2b87df5b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4442286
Reviewed-by: Matt Reynolds <[email protected]>
Commit-Queue: Jack Hsieh <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1132113}

Bug: 1430625
Change-Id: I36d9639e4e7f1bbf37b625611239de9282393098
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6159507
Commit-Queue: Jack Hsieh <[email protected]>
Bot-Commit: Rubber Stamper <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1403887}

This reverts commit 480ab2f6779dfd0ceb88298f8a828b9d3f881d90.

Reason for revert: No longer need these debug message.

Original change's description:
> bluetooth: Add log for debugging flaky issue 1430625
>
> Add more logging for the test characteristic-is-removed.https.window.js
> to identify which step of the process is causing the timeout on Mac CI
> builder Mac10.14.
>
> Bug: 1430625
> Change-Id: I0b266857a3346021814357146ddc0bad2b87df5b
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4442286
> Reviewed-by: Matt Reynolds <[email protected]>
> Commit-Queue: Jack Hsieh <[email protected]>
> Cr-Commit-Position: refs/heads/main@{#1132113}

Bug: 1430625
Change-Id: I36d9639e4e7f1bbf37b625611239de9282393098
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6159507
Commit-Queue: Jack Hsieh <[email protected]>
Bot-Commit: Rubber Stamper <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1403887}
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot merged commit 7134137 into master Jan 9, 2025
17 checks passed
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-6159507 branch January 9, 2025 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants