Browse Source

fix: multiple move() calls on the same unique_ptr (#42818)

Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com>
Co-authored-by: Charles Kerr <[email protected]>
trop[bot] 9 months ago
parent
commit
32dd4aa025
1 changed files with 2 additions and 3 deletions
  1. 2 3
      shell/browser/extensions/electron_extensions_browser_client.cc

+ 2 - 3
shell/browser/extensions/electron_extensions_browser_client.cc

@@ -342,13 +342,12 @@ void ElectronExtensionsBrowserClient::BroadcastEventToRenderers(
     return;
   }
 
-  auto event = std::make_unique<extensions::Event>(histogram_value, event_name,
-                                                   args.Clone());
   for (auto const& [key, browser_context] :
        ElectronBrowserContext::browser_context_map()) {
     if (browser_context) {
       extensions::EventRouter::Get(browser_context.get())
-          ->BroadcastEvent(std::move(event));
+          ->BroadcastEvent(std::make_unique<extensions::Event>(
+              histogram_value, event_name, args.Clone()));
     }
   }
 }