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

Cherry pick PR #3989: Don't start DebugConsole with debug_console=off. #3996

Merged
merged 2 commits into from
Aug 15, 2024

Conversation

cobalt-github-releaser-bot
Copy link
Collaborator

Refer to the original PR: #3989

When started with --debug_console=off, don't
start the DebugConsole at all.

Note: The web debugger (devtools) still works but
no longer will show overlay changes. To also
disable the web debugger, also start with
--disable_web_debugger.

b/205134049

When started with `--debug_console=off`, don't
start the DebugConsole at all.

Note: The web debugger (devtools) still works but
no longer will show overlay changes. To also
disable the web debugger, also start with
`--disable_web_debugger`.

b/205134049

(cherry picked from commit a9495fe)
@cobalt-github-releaser-bot
Copy link
Collaborator Author

Important

There were merge conflicts while cherry picking! Check out cherry-pick-24.lts.1+-3989 and fix the conflicts before proceeding. Check the log at https://github.com/youtube/cobalt/actions/runs/10398119782 for details.

fix merge conflict.
@jellefoks jellefoks marked this pull request as ready for review August 15, 2024 03:15
@kaidokert
Copy link
Member

I missed this from the cut / merge, have to wait until the builds are published.

@kaidokert kaidokert merged commit 05e73c4 into 24.lts.1+ Aug 15, 2024
433 checks passed
@kaidokert kaidokert deleted the cherry-pick-24.lts.1+-3989 branch August 15, 2024 21:59
@jellefoks
Copy link
Member

I missed this from the cut / merge, have to wait until the builds are published.

That's ok, this one is for 24.lts, and it's not for gold builds anyway.

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.

3 participants