Commit 23e10188 authored by Aaron Krajeski's avatar Aaron Krajeski Committed by Commit Bot

Revert "DL: Speculative fix for a leak in display_lock_fuzzer."

This reverts commit 4d99f02a.

Reason for revert: Started causing an infra failure on the msan_linux bot
https://ci.chromium.org/p/chromium/builders/ci/Linux%20MSan%20Tests/20157

Original change's description:
> DL: Speculative fix for a leak in display_lock_fuzzer.
> 
> This replaces a leaky new with a local static variable for the display
> lock fuzzer environment.
> 
> This is a speculative fix for the referenced bug.
> 
> R=​schenney@chromium.org
> 
> Bug: 1013486
> Change-Id: I717437eac8d3d6f3d81515e2822209a676047698
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1873916
> Commit-Queue: vmpstr <vmpstr@chromium.org>
> Reviewed-by: Stephen Chenney <schenney@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#709212}

TBR=vmpstr@chromium.org,schenney@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1013486
Change-Id: I61dc9ddbc3997c7452e1aef4496b21d6a21c85d1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1884592Reviewed-by: default avatarAaron Krajeski <aaronhk@chromium.org>
Reviewed-by: default avatarvmpstr <vmpstr@chromium.org>
Commit-Queue: Aaron Krajeski <aaronhk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#710306}
parent 68917816
......@@ -9,14 +9,23 @@
#include "third_party/blink/public/web/web_view.h"
#include "third_party/blink/public/web/web_widget.h"
static content::Env* env;
bool Initialize() {
blink::WebRuntimeFeatures::EnableDisplayLocking(true);
env = new content::Env();
return true;
}
extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) {
static bool initialized = Initialize();
// Suppress unused variable warning.
(void)initialized;
// Only handle reasonable size inputs.
if (size < 1 || size > 10000)
return 0;
blink::WebRuntimeFeatures::EnableDisplayLocking(true);
static base::NoDestructor<content::Env> env;
std::string data_as_string(reinterpret_cast<const char*>(data), size);
int num_rafs = std::hash<std::string>()(data_as_string) % 10;
env->adapter->LoadHTML(data_as_string, "");
......
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