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

Don't start DebugConsole with debug_console=off. #3989

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

jellefoks
Copy link
Member

@jellefoks jellefoks commented Aug 14, 2024

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`.
Copy link
Member

@kaidokert kaidokert left a comment

Choose a reason for hiding this comment

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

Seems okay

@jellefoks jellefoks enabled auto-merge (squash) August 14, 2024 21:10
@datadog-cobalt-youtube
Copy link

datadog-cobalt-youtube bot commented Aug 14, 2024

Datadog Report

Branch report: disable_debugconsole
Commit report: 3f84c12
Test service: cobalt

✅ 0 Failed, 34451 Passed, 6 Skipped, 10m 6.07s Total Time

@jellefoks jellefoks enabled auto-merge (squash) August 15, 2024 02:57
@jellefoks

This comment was marked as resolved.

@jellefoks jellefoks merged commit a9495fe into main Aug 15, 2024
340 of 343 checks passed
@jellefoks jellefoks deleted the disable_debugconsole branch August 15, 2024 03:04
@jellefoks

This comment was marked as resolved.

@jellefoks jellefoks added cp-20.lts.1+ Cherry Pick to the 20.lts.1+ branch cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch cp-25.lts.1+ Cherry Pick to the 25.lts.1+ branch and removed cp-20.lts.1+ Cherry Pick to the 20.lts.1+ branch labels Aug 15, 2024
cobalt-github-releaser-bot pushed a commit that referenced this pull request Aug 15, 2024
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 pushed a commit that referenced this pull request Aug 15, 2024
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 pushed a commit that referenced this pull request Aug 15, 2024
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)
jellefoks added a commit that referenced this pull request Aug 15, 2024
#3995)

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

Co-authored-by: Jelle Foks <[email protected]>
kaidokert pushed a commit that referenced this pull request Aug 15, 2024
#3996)

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

---------

Co-authored-by: Jelle Foks <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch cp-25.lts.1+ Cherry Pick to the 25.lts.1+ branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants