Fix Ads locale on Windows

fixes brave/brave-browser#4172
brave · Apr 19, 2019 · ca4069c · ca4069c
1 parent effbe59
commit ca4069c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/components/brave_ads/browser/ads_service_impl.cc b/components/brave_ads/browser/ads_service_impl.cc
@@ -334,6 +334,8 @@ void AdsServiceImpl::OnMaybeStartForRegion(
  is_supported_region_ = is_supported_region;
 
  if (!is_supported_region_) {
+ LOG(WARNING) << GetAdsLocale() << " locale does not support Ads";
+
  Shutdown();
  return;
  }

diff --git a/components/brave_ads/browser/locale_helper_win.cc b/components/brave_ads/browser/locale_helper_win.cc
@@ -34,7 +34,7 @@ std::string LocaleHelperWin::GetLocale() const {
 
  wcstombs(locale.get(), locale_name.get(), size);
 
- return std::string(locale.get(), size);
+ return std::string(locale.get());
 }
 
 LocaleHelperWin* LocaleHelperWin::GetInstance() {