Correcting ledger's CHECK use

There has been some changes upstream on these macros recently, which
have slightly changed their semantics. This commit corrects the use of
DCHECK in the ledger code, to satisfy the compiler.

vendor/bat-native-ledger/src/bat/ledger/internal/endpoint/uphold/get_capabilities/get_capabilities.cc

@@ -18,8 +18,9 @@ namespace ledger {
 namespace endpoint {
 namespace uphold {
 
-GetCapabilities::GetCapabilities(LedgerImpl* ledger)
-    : ledger_((DCHECK(ledger), ledger)) {}
+GetCapabilities::GetCapabilities(LedgerImpl* ledger) : ledger_(ledger) {
+  DCHECK(ledger_);
+}
 
 GetCapabilities::~GetCapabilities() = default;
 

vendor/bat-native-ledger/src/bat/ledger/internal/promotion/promotion_transfer.cc

@@ -18,9 +18,11 @@ namespace ledger {
 namespace promotion {
 
 PromotionTransfer::PromotionTransfer(LedgerImpl* ledger)
-    : ledger_((DCHECK(ledger), ledger)),
+    : ledger_(ledger),
       credentials_(
-          std::make_unique<credential::CredentialsPromotion>(ledger_)) {}
+          std::make_unique<credential::CredentialsPromotion>(ledger_)) {
+  DCHECK(ledger_);
+}
 
 PromotionTransfer::~PromotionTransfer() = default;