Merge pull request #4370 from /issues/7728

Intermittent crash if Confirmations library is called before being instantiated

vendor/bat-native-confirmations/include/bat/confirmations/confirmations.h

@@ -24,7 +24,8 @@ namespace confirmations {
 using TransactionInfo = ::ledger::TransactionInfo;
 using TransactionsInfo = ::ledger::TransactionsInfo;
 
-using OnGetTransactionHistory = ::ledger::GetTransactionHistoryCallback;
+using OnGetTransactionHistoryCallback = ::ledger::GetTransactionHistoryCallback;
+using OnInitializeCallback = std::function<void(bool)>;
 
 // |_environment| indicates that URL requests should use production, staging or
 // development servers but can be overridden via command-line arguments
@@ -46,8 +47,11 @@ class CONFIRMATIONS_EXPORT Confirmations {
   static Confirmations* CreateInstance(
       ConfirmationsClient* confirmations_client);
 
-  // Should be called from Ledger to initialize Confirmations
-  virtual void Initialize() = 0;
+  // Should be called from Ledger to initialize Confirmations. The callback
+  // takes one argument - |true| if Confirmations was successfully initialized;
+  // otherwise |false|
+  virtual void Initialize(
+      OnInitializeCallback callback) = 0;
 
   // Should be called when the wallet |payment_id| and |private_key| are set in
   // the Ledger library, e.g. initializing a wallet, creating a new wallet or
@@ -63,7 +67,7 @@ class CONFIRMATIONS_EXPORT Confirmations {
   // argument — |TransactionsInfo| which contains a list of |TransactionInfo|
   // transactions and associated earned ads rewards
   virtual void GetTransactionHistory(
-      OnGetTransactionHistory callback) = 0;
+      OnGetTransactionHistoryCallback callback) = 0;
 
   // Should be called to confirm an ad was viewed, clicked, dismissed or landed
   virtual void ConfirmAd(

vendor/bat-native-confirmations/include/bat/confirmations/wallet_info.h

@@ -17,7 +17,7 @@ struct CONFIRMATIONS_EXPORT WalletInfo {
   WalletInfo(const WalletInfo& info);
   ~WalletInfo();
 
-  bool IsValid();
+  bool IsValid() const;
 
   bool operator==(const WalletInfo& info) const;
 

vendor/bat-native-confirmations/src/bat/confirmations/internal/confirmations_create_confirmation_request_unittest.cc

@@ -17,6 +17,8 @@
 
 // npm run test -- brave_unit_tests --filter=Confirmations*
 
+using std::placeholders::_1;
+
 namespace confirmations {
 
 class ConfirmationsCreateConfirmationRequestTest : public ::testing::Test {
@@ -44,7 +46,13 @@ class ConfirmationsCreateConfirmationRequestTest : public ::testing::Test {
   void SetUp() override {
     // Code here will be called immediately after the constructor (right before
     // each test)
-    confirmations_->Initialize();
+    auto callback = std::bind(
+        &ConfirmationsCreateConfirmationRequestTest::OnInitialize, this, _1);
+    confirmations_->Initialize(callback);
+  }
+
+  void OnInitialize(const bool success) {
+    EXPECT_EQ(true, success);
   }
 
   void TearDown() override {

vendor/bat-native-confirmations/src/bat/confirmations/internal/confirmations_fetch_payment_token_request_unittest.cc

@@ -14,6 +14,8 @@
 
 // npm run test -- brave_unit_tests --filter=Confirmations*
 
+using std::placeholders::_1;
+
 namespace confirmations {
 
 class ConfirmationsFetchPaymentTokenRequestTest : public ::testing::Test {
@@ -41,7 +43,13 @@ class ConfirmationsFetchPaymentTokenRequestTest : public ::testing::Test {
   void SetUp() override {
     // Code here will be called immediately after the constructor (right before
     // each test)
-    confirmations_->Initialize();
+    auto callback = std::bind(
+        &ConfirmationsFetchPaymentTokenRequestTest::OnInitialize, this, _1);
+    confirmations_->Initialize(callback);
+  }
+
+  void OnInitialize(const bool success) {
+    EXPECT_EQ(true, success);
   }
 
   void TearDown() override {

vendor/bat-native-confirmations/src/bat/confirmations/internal/confirmations_get_signed_tokens_request_unittest.cc

@@ -16,6 +16,8 @@
 
 // npm run test -- brave_unit_tests --filter=Confirmations*
 
+using std::placeholders::_1;
+
 namespace confirmations {
 
 class ConfirmationsGetSignedTokensRequestTest : public ::testing::Test {
@@ -43,7 +45,13 @@ class ConfirmationsGetSignedTokensRequestTest : public ::testing::Test {
   void SetUp() override {
     // Code here will be called immediately after the constructor (right before
     // each test)
-    confirmations_->Initialize();
+    auto callback = std::bind(
+        &ConfirmationsGetSignedTokensRequestTest::OnInitialize, this, _1);
+    confirmations_->Initialize(callback);
+  }
+
+  void OnInitialize(const bool success) {
+    EXPECT_EQ(true, success);
   }
 
   void TearDown() override {