Commit ddfb5c89 authored by Mathias Bynens's avatar Mathias Bynens Committed by Commit Bot

Clear emulated vision deficiency when disabling inspector

CDP’s Emulation#setEmulatedVisionDeficiency no longer accepts
the empty string to clear the override. This patch updates
InspectorEmulationAgent::disable accordingly.

Bug: chromium:1003700
Change-Id: I3f08aafd4b2ff0ecdf119f1d6898a8960cc3eb71
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087672
Commit-Queue: Mathias Bynens <mathias@chromium.org>
Commit-Queue: Peter Marshall <petermarshall@chromium.org>
Auto-Submit: Mathias Bynens <mathias@chromium.org>
Reviewed-by: default avatarPeter Marshall <petermarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#746743}
parent e2c8ca23
...@@ -172,7 +172,7 @@ Response InspectorEmulationAgent::disable() { ...@@ -172,7 +172,7 @@ Response InspectorEmulationAgent::disable() {
// settings to attach to the same page). TODO: support this use case. // settings to attach to the same page). TODO: support this use case.
setEmulatedMedia(String(), {}); setEmulatedMedia(String(), {});
if (!emulated_vision_deficiency_.Get().IsNull()) if (!emulated_vision_deficiency_.Get().IsNull())
setEmulatedVisionDeficiency(String()); setEmulatedVisionDeficiency(String("none"));
setCPUThrottlingRate(1); setCPUThrottlingRate(1);
setFocusEmulationEnabled(false); setFocusEmulationEnabled(false);
setDefaultBackgroundColorOverride(Maybe<protocol::DOM::RGBA>()); setDefaultBackgroundColorOverride(Maybe<protocol::DOM::RGBA>());
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment