d1c2dab6ff
- Rework aarch64 build requirements - Reduce jumbo_file_merge_limit to 8 for aarch64 to avoid OOM - Fix again aarch64 skia build: * chromium-skia-aarch64-buildfix.patch - Update chromium-non-void-return.patch and use __builtin_abort for switch cases of out of enum ranges. - Up to 71.0.3551.3 - Up to 70.0.3528.4 - Up to chromium-70.0.3521.2 - Add patch trying to build with system icu: * chromium-system-icu.patch - Up to chromium-70.0.3510.0 - Up to 69.0.3497.23 - Up to chromium-69.0.3497.12 - Add patch to fix aarch64 build: * chromium-vpx-aarch64.patch - Up to 69.0.3493.3 - Up to 69.0.3486.0 - Up to 69.0.3472.3 - Up to 69.0.3452.0 - Up to 68.0.3440.17 - Up to 68.0.3438.3 - Up to 68.0.3432.3 - Fix AArch64 build with chromium-crashpad-aarch64-fix.patch - Up to 68.0.3423.2 - Refresh patch chromium-master-prefs-path.patch - Fix AArch64 build with chromium-skia-aarch64-buildfix.patch - Add patch chromium-skia-system-fontconfig.patch bsc#1092272 - Up to 67.0.3393.30 - Enable build on AArch64 OBS-URL: https://build.opensuse.org/request/show/655505 OBS-URL: https://build.opensuse.org/package/show/network:chromium/chromium?expand=0&rev=1200
58 lines
2.5 KiB
Diff
58 lines
2.5 KiB
Diff
From 65be571f6ac2f7942b4df9e50b24da517f829eec Mon Sep 17 00:00:00 2001
|
||
From: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
|
||
Date: Mon, 15 Oct 2018 20:26:10 +0000
|
||
Subject: [PATCH] google_util: Explicitly use std::initializer_list with
|
||
base::NoDestructor
|
||
MIME-Version: 1.0
|
||
Content-Type: text/plain; charset=UTF-8
|
||
Content-Transfer-Encoding: 8bit
|
||
|
||
Follow-up to ac53c5c53 ("Remove CR_DEFINE_STATIC_LOCAL from /components").
|
||
Due to https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84849, having
|
||
base::NoDestructor<T<U>> and passing an initializer list of Us does not
|
||
work if this is not done explicitly, as GCC incorrectly fails to determine
|
||
which constructor overload to use:
|
||
|
||
../../components/google/core/common/google_util.cc: In function ‘bool google_util::{anonymous}::IsCanonicalHostGoogleHostname(base::StringPiece, google_util::SubdomainPermission)’:
|
||
../../components/google/core/common/google_util.cc:120:24: error: call of overloaded ‘NoDestructor(<brace-enclosed initializer list>)’ is ambiguous
|
||
{GOOGLE_TLD_LIST});
|
||
|
||
See also: https://chromium-review.googlesource.com/c/chromium/src/+/1170905
|
||
|
||
Bug: 819294
|
||
Change-Id: Ie1490b6646d7998d636c485769caabf56c1cf44c
|
||
Reviewed-on: https://chromium-review.googlesource.com/c/1275854
|
||
Reviewed-by: Peter Kasting <pkasting@chromium.org>
|
||
Commit-Queue: Raphael Kubo da Costa (CET) <raphael.kubo.da.costa@intel.com>
|
||
Cr-Commit-Position: refs/heads/master@{#599733}
|
||
---
|
||
components/google/core/common/google_util.cc | 5 +++--
|
||
1 file changed, 3 insertions(+), 2 deletions(-)
|
||
|
||
diff --git a/components/google/core/common/google_util.cc b/components/google/core/common/google_util.cc
|
||
index a44c84393820..7733848a0443 100644
|
||
--- a/components/google/core/common/google_util.cc
|
||
+++ b/components/google/core/common/google_util.cc
|
||
@@ -117,7 +117,7 @@ bool IsCanonicalHostGoogleHostname(base::StringPiece canonical_host,
|
||
StripTrailingDot(&tld);
|
||
|
||
static base::NoDestructor<std::set<std::string>> google_tlds(
|
||
- {GOOGLE_TLD_LIST});
|
||
+ std::initializer_list<std::string>({GOOGLE_TLD_LIST}));
|
||
return base::ContainsKey(*google_tlds, tld.as_string());
|
||
}
|
||
|
||
@@ -132,7 +132,8 @@ bool IsGoogleSearchSubdomainUrl(const GURL& url) {
|
||
StripTrailingDot(&host);
|
||
|
||
static base::NoDestructor<std::set<std::string>> google_subdomains(
|
||
- {"ipv4.google.com", "ipv6.google.com"});
|
||
+ std::initializer_list<std::string>(
|
||
+ {"ipv4.google.com", "ipv6.google.com"}));
|
||
|
||
return base::ContainsKey(*google_subdomains, host.as_string());
|
||
}
|
||
--
|
||
2.19.1
|
||
|