Merge pull request #6139 from /issues/10832

tmancey · web-flow · commit 9dbc5f4b93fd · 2020-07-21T20:45:42.000+01:00
Fixes do not log classified page url in locally persisted diagnostic log for Brave Ads
diff --git a/vendor/bat-native-ads/src/bat/ads/internal/reports.cc b/vendor/bat-native-ads/src/bat/ads/internal/reports.cc
@@ -167,9 +167,6 @@ std::string Reports::GenerateLoadEventReport(
     writer.String("click");
   }
 
-  writer.String("tabUrl");
-  writer.String(info.tab_url.c_str());
-
   writer.String("tabClassification");
   writer.StartArray();
   auto classifications = classification::SplitCategory(info.tab_classification);

Original file line number	Diff line number	Diff line change
`@@ -167,9 +167,6 @@ std::string Reports::GenerateLoadEventReport(`
`167`	`167`	`writer.String("click");`
`168`	`168`	`}`
`169`	`169`
`170`		`- writer.String("tabUrl");`
`171`		`- writer.String(info.tab_url.c_str());`
`172`		`-`
`173`	`170`	`writer.String("tabClassification");`
`174`	`171`	`writer.StartArray();`
`175`	`172`	`auto classifications = classification::SplitCategory(info.tab_classification);`