Skip to content

Commit

Permalink
Merge pull request #19752 from brave/issues/32353
Browse files Browse the repository at this point in the history
Fixes crash in brave_ads::AddTransaction
  • Loading branch information
tmancey authored Aug 17, 2023
2 parents 19911f8 + 2f0897d commit e474398
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ TransactionInfo BuildTransaction(const std::string& creative_instance_id,
const AdType& ad_type,
const ConfirmationType& confirmation_type) {
CHECK(!creative_instance_id.empty());
CHECK(!segment.empty());
CHECK_NE(AdType::kUndefined, ad_type);
CHECK_NE(ConfirmationType::kUndefined, confirmation_type);

Expand All @@ -47,7 +46,6 @@ TransactionInfo AddTransaction(const std::string& creative_instance_id,
const ConfirmationType& confirmation_type,
AddTransactionCallback callback) {
CHECK(!creative_instance_id.empty());
CHECK(!segment.empty());
CHECK_NE(AdType::kUndefined, ad_type);
CHECK_NE(ConfirmationType::kUndefined, confirmation_type);

Expand Down

0 comments on commit e474398

Please sign in to comment.