Skip to content

Commit 6605054

Browse files
bmeurerDevtools-frontend LUCI CQ
authored andcommitted
[screencast] Fix broken screencast toggle.
This broke as part of https://crrev.com/c/5563488, which switches the toolbar buttons to use the button component. As part of this the logic to toggle upon clicks is already built into the toolbar button now, and the additional code to flip the toggle in the "click" handler caused the toggle to immediately go back to the previous state. Fixed: 373263964 Bug: 325441197 Change-Id: I9a2bc5e801e1c1e69f584226574ddb1632c0dec5 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5939707 Auto-Submit: Benedikt Meurer <[email protected]> Commit-Queue: Yang Guo <[email protected]> Reviewed-by: Yang Guo <[email protected]> Commit-Queue: Benedikt Meurer <[email protected]>
1 parent ee26370 commit 6605054

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

front_end/panels/screencast/ScreencastApp.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ export class ScreencastApp implements Common.App.App,
8787
}
8888

8989
private toggleButtonClicked(): void {
90-
const enabled = !this.toggleButton.isToggled();
90+
const enabled = this.toggleButton.isToggled();
9191
this.enabledSetting.set(enabled);
9292
this.onScreencastEnabledChanged();
9393
}

0 commit comments

Comments
 (0)