5498236: Make browser_tests force full async initialization for OSCrypt Async

clavin · ckerr · commit e00faacb5854 · 2024-05-30T22:23:41.000-05:00
https://chromium-review.googlesource.com/c/chromium/src/+/5498236
diff --git a/shell/browser/browser_process_impl.cc b/shell/browser/browser_process_impl.cc
@@ -314,6 +314,10 @@ os_crypt_async::OSCryptAsync* BrowserProcessImpl::os_crypt_async() {
   return os_crypt_async_.get();
 }
 
+void BrowserProcessImpl::set_additional_os_crypt_async_provider_for_test(
+    size_t precedence,
+    std::unique_ptr<os_crypt_async::KeyProvider> provider) {}
+
 void BrowserProcessImpl::SetSystemLocale(const std::string& locale) {
   system_locale_ = locale;
 }
diff --git a/shell/browser/browser_process_impl.h b/shell/browser/browser_process_impl.h
@@ -115,6 +115,9 @@ class BrowserProcessImpl : public BrowserProcess {
   HidSystemTrayIcon* hid_system_tray_icon() override;
   UsbSystemTrayIcon* usb_system_tray_icon() override;
   os_crypt_async::OSCryptAsync* os_crypt_async() override;
+  void set_additional_os_crypt_async_provider_for_test(
+      size_t precedence,
+      std::unique_ptr<os_crypt_async::KeyProvider> provider) override;
   void CreateDevToolsProtocolHandler() override {}
   void CreateDevToolsAutoOpener() override {}
   void set_background_mode_manager_for_test(

Original file line number	Diff line number	Diff line change
`@@ -314,6 +314,10 @@ os_crypt_async::OSCryptAsync* BrowserProcessImpl::os_crypt_async() {`
`314`	`314`	`return os_crypt_async_.get();`
`315`	`315`	`}`
`316`	`316`
	`317`	`+void BrowserProcessImpl::set_additional_os_crypt_async_provider_for_test(`
	`318`	`+ size_t precedence,`
	`319`	`+ std::unique_ptr<os_crypt_async::KeyProvider> provider) {}`
	`320`	`+`
`317`	`321`	`void BrowserProcessImpl::SetSystemLocale(const std::string& locale) {`
`318`	`322`	`system_locale_ = locale;`
`319`	`323`	`}`