From 7d207c7a3be96173b6077b3bbd22ad6607e4e795 Mon Sep 17 00:00:00 2001 From: "trop[bot]" <37223003+trop[bot]@users.noreply.github.com> Date: Tue, 18 Mar 2025 13:21:03 +0100 Subject: [PATCH] refactor: use `GetDefaultStoragePartition()` instead of `GetStoragePartition(nullptr)` (#46084) refactor: use GetDefaultStoragePartition() Use GetDefaultStorageParition() instead of GetStoragePartition(nullptr) - It improves code uniformity, since we use get-default everywhere else - It's more readable - It's marginally faster, since GetStoragePartition() has more steps Added in https://github.com/electron/electron/pull/28907/commits/49b0a1bf4a6c032d1788aac56c1e5eaa0f61e3d3 Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com> Co-authored-by: Charles Kerr --- shell/browser/api/electron_api_session.cc | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/shell/browser/api/electron_api_session.cc b/shell/browser/api/electron_api_session.cc index df434d43c524..6c3b97040a38 100644 --- a/shell/browser/api/electron_api_session.cc +++ b/shell/browser/api/electron_api_session.cc @@ -713,14 +713,13 @@ v8::Local Session::ClearStorageData(gin::Arguments* args) { ClearStorageDataOptions options; args->GetNext(&options); - auto* storage_partition = browser_context()->GetStoragePartition(nullptr); if (options.storage_types & StoragePartition::REMOVE_DATA_MASK_COOKIES) { // Reset media device id salt when cookies are cleared. // https://w3c.github.io/mediacapture-main/#dom-mediadeviceinfo-deviceid MediaDeviceIDSalt::Reset(browser_context()->prefs()); } - storage_partition->ClearData( + browser_context()->GetDefaultStoragePartition()->ClearData( options.storage_types, options.quota_types, options.storage_key, base::Time(), base::Time::Max(), base::BindOnce(gin_helper::Promise::ResolvePromise, @@ -729,8 +728,7 @@ v8::Local Session::ClearStorageData(gin::Arguments* args) { } void Session::FlushStorageData() { - auto* storage_partition = browser_context()->GetStoragePartition(nullptr); - storage_partition->Flush(); + browser_context()->GetDefaultStoragePartition()->Flush(); } v8::Local Session::SetProxy(gin::Arguments* args) {