chromium/src
0
Fork 0
Code Activity

[sandbox] Convert JSONStringValueSerializer to base::WriteJson

Browse Source 
Modified to slightly more concise equivalent.

Bug: 40912727
Change-Id: I8bcac189871369e094e516d88d98ceb79e8a70f1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6335118
Reviewed-by: Will Harris <wfh@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Ho Cheung <hocheung@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1430575}
This commit is contained in:
Ho Cheung
2025-03-10 17:20:59 -07:00
committed by Chromium LUCI CQ
parent 35ca474f68
commit fdb543fe8f
1 changed files with 6 additions and 5 deletions

11
sandbox/win/src/sandbox_policy_diagnostic.cc

@ -15,12 +15,13 @@
#include <cinttypes>
#include <memory>
#include <optional>
#include <string>
#include <utility>
#include <vector>
#include "base/check.h"
#include "base/json/json_string_value_serializer.h"
#include "base/json/json_writer.h"
#include "base/numerics/safe_conversions.h"
#include "base/strings/string_util.h"
#include "base/strings/stringprintf.h"
@ -473,10 +474,10 @@ const char* PolicyDiagnostic::JsonString() {
dict.Set(kZeroAppShim, zero_appshim_);
dict.Set(kHandlesToClose, GetHandlesToClose(handles_to_close_));
auto json_string = std::make_unique<std::string>();
JSONStringValueSerializer to_json(json_string.get());
CHECK(to_json.Serialize(dict));
json_string_ = std::move(json_string);
std::optional<std::string> json_string =
base::WriteJson(base::Value(std::move(dict)));
CHECK(json_string);
json_string_ = std::make_unique<std::string>(std::move(*json_string));
return json_string_->c_str();
}