Commit 0473a2d9 authored by Ben Reich's avatar Ben Reich Committed by Commit Bot

[filesapp] Set callCount to true in DevToolsListener

Set callCount to true to capture the number of times a JavaScript code
block is invoked during execution.

Minor: remove unnecessary blank line.

Test: browser_tests --gtest_filter="*FilesApp*" --devtools-code-coverage
Bug: 1113941
Change-Id: I6a934708db329f7095146331322c07b7bf631043
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2462828
Auto-Submit: Ben Reich <benreich@chromium.org>
Reviewed-by: default avatarNoel Gordon <noel@chromium.org>
Commit-Queue: Noel Gordon <noel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#815558}
parent 46a81a87
......@@ -35,7 +35,7 @@ std::string EncodedURL(const std::string& url) {
return std::string(canonical_url.data(), canonical_url.length());
}
} // namespace
} // namespace
DevToolsListener::DevToolsListener(content::DevToolsAgentHost* host,
uint32_t uuid)
......@@ -98,7 +98,7 @@ bool DevToolsListener::StartJSCoverage(content::DevToolsAgentHost* host) {
std::string start_precise_coverage =
"{\"id\":21,\"method\":\"Profiler.startPreciseCoverage\",\"params\":{"
"\"callCount\":false,\"detailed\":true}}";
"\"callCount\":true,\"detailed\":true}}";
host->DispatchProtocolMessage(this, StringToSpan(start_precise_coverage));
std::string enable_debugger = "{\"id\":22,\"method\":\"Debugger.enable\"}";
......
......@@ -1486,7 +1486,6 @@ FileManagerBrowserTestBase::FileManagerBrowserTestBase() = default;
FileManagerBrowserTestBase::~FileManagerBrowserTestBase() = default;
static bool ShouldInspect(content::DevToolsAgentHost* host) {
// TODO(crbug.com/v8/10820): Add background_page back in once
// coverage can be collected when a background_page and app
......
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