1 |
commit: e99cd36078a29de594241aa44b7de3f3ce353db9 |
2 |
Author: Stephan Hartmann <stha09 <AT> googlemail <DOT> com> |
3 |
AuthorDate: Tue Oct 29 14:08:48 2019 +0000 |
4 |
Commit: Mike Gilbert <floppym <AT> gentoo <DOT> org> |
5 |
CommitDate: Tue Oct 29 20:11:17 2019 +0000 |
6 |
URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=e99cd360 |
7 |
|
8 |
www-client/chromium: dev channel bump to 79.0.3945.8 |
9 |
|
10 |
Closes: https://github.com/gentoo/gentoo/pull/13487 |
11 |
Package-Manager: Portage-2.3.76, Repoman-2.3.16 |
12 |
Signed-off-by: Stephan Hartmann <stha09 <AT> googlemail.com> |
13 |
Signed-off-by: Mike Gilbert <floppym <AT> gentoo.org> |
14 |
|
15 |
www-client/chromium/Manifest | 2 +- |
16 |
...0.3941.4.ebuild => chromium-79.0.3945.8.ebuild} | 3 +- |
17 |
.../chromium-79-gcc-ambiguous-nodestructor.patch | 32 +++- |
18 |
.../files/chromium-79-gcc-name-clash.patch | 135 ++++++++++++++++ |
19 |
.../files/chromium-79-gcc-permissive.patch | 14 -- |
20 |
.../chromium/files/chromium-79-include.patch | 151 +++++++++++------ |
21 |
.../files/chromium-79-swiftshader-linux.patch | 180 --------------------- |
22 |
7 files changed, 272 insertions(+), 245 deletions(-) |
23 |
|
24 |
diff --git a/www-client/chromium/Manifest b/www-client/chromium/Manifest |
25 |
index f2606637827..6cab752dd46 100644 |
26 |
--- a/www-client/chromium/Manifest |
27 |
+++ b/www-client/chromium/Manifest |
28 |
@@ -1,3 +1,3 @@ |
29 |
DIST chromium-78-revert-noexcept-r1.patch.gz 12094 BLAKE2B 1fd0c939af773ed398c9ab14fa80b042b4237cabdbcb06425e4d1c72200ec49c3c1383cfc95013cce4df411b0a2397b79030d41a0379ae80a0c60bae1fbb5ffa SHA512 dcb62b82b34d5d6982762cacfe3ab7d3538d908b3befe7666f5d70d98ba4f979b5bf056e5c5343ed76aa744a643318fc5a75ddedfcb1b7de3e7c28eb2587439f |
30 |
DIST chromium-78.0.3904.70.tar.xz 742787108 BLAKE2B 623a0359f461672d8d6c97b4703f5b50c6ac423350ba96b7f37582be38d9cc66674e441b506405bc5f136df19df886036ef5abae4b8c2e4e953ec296e2a25bb4 SHA512 95d926ea92a0e744284a4c5fca619e3a067dd0fdfaa7c05b5a069d2a8d1159bf849335e21e5803bd28175c1994bd4ff15337ad0a3063445fab74eef113275545 |
31 |
-DIST chromium-79.0.3941.4.tar.xz 772100680 BLAKE2B 2a510529ae9a58c2c831bd293eb1a9ea276422da6f49741be208bcf8c17eafa0e47c0adf8e3f1148f3da821188315e9d6c7f2de1c44dd844f5d481be17b35ef7 SHA512 5847f7c647737ea96c27f859e5b763100c4d0ecff83eb33d81d144e3be16a5a709290a66fa6754b8a38be68c850dee4be0abc2419f5a50e0cf22bc37a0563c80 |
32 |
+DIST chromium-79.0.3945.8.tar.xz 777902956 BLAKE2B eba9e94c9305a5fe9aaf8848456a69427219a29b139217d9b5409a35cb4a508d217fe82c08e3bc9325675180bfe7d7c93d583adf9d8ad6c04de1209b2305a1d3 SHA512 78358d0dbb8bd574c450f5443b7dfc888ea89ad9fa9da156106d260f1b3921deb6f5a50931c689c1855c9ffdef5e001d8636e8a00d416c9e9efe8bea2a8dfc6d |
33 |
|
34 |
diff --git a/www-client/chromium/chromium-79.0.3941.4.ebuild b/www-client/chromium/chromium-79.0.3945.8.ebuild |
35 |
similarity index 99% |
36 |
rename from www-client/chromium/chromium-79.0.3941.4.ebuild |
37 |
rename to www-client/chromium/chromium-79.0.3945.8.ebuild |
38 |
index 9e2a2c46130..cf14a286871 100644 |
39 |
--- a/www-client/chromium/chromium-79.0.3941.4.ebuild |
40 |
+++ b/www-client/chromium/chromium-79.0.3945.8.ebuild |
41 |
@@ -144,14 +144,13 @@ PATCHES=( |
42 |
"${FILESDIR}/chromium-fix-char_traits.patch" |
43 |
"${FILESDIR}/chromium-unbundle-zlib-r1.patch" |
44 |
"${FILESDIR}/chromium-77-system-icu.patch" |
45 |
- "${FILESDIR}/chromium-78-pm-crash.patch" |
46 |
"${FILESDIR}/chromium-78-icon.patch" |
47 |
"${FILESDIR}/chromium-78-protobuf-export.patch" |
48 |
- "${FILESDIR}/chromium-79-swiftshader-linux.patch" |
49 |
"${FILESDIR}/chromium-79-system-hb.patch" |
50 |
"${FILESDIR}/chromium-79-include.patch" |
51 |
"${FILESDIR}/chromium-79-icu-65.patch" |
52 |
"${FILESDIR}/chromium-79-gcc-ambiguous-nodestructor.patch" |
53 |
+ "${FILESDIR}/chromium-79-gcc-name-clash.patch" |
54 |
"${FILESDIR}/chromium-79-gcc-permissive.patch" |
55 |
"${FILESDIR}/chromium-79-gcc-alignas.patch" |
56 |
) |
57 |
|
58 |
diff --git a/www-client/chromium/files/chromium-79-gcc-ambiguous-nodestructor.patch b/www-client/chromium/files/chromium-79-gcc-ambiguous-nodestructor.patch |
59 |
index 24504325e46..d43a091b971 100644 |
60 |
--- a/www-client/chromium/files/chromium-79-gcc-ambiguous-nodestructor.patch |
61 |
+++ b/www-client/chromium/files/chromium-79-gcc-ambiguous-nodestructor.patch |
62 |
@@ -1,13 +1,39 @@ |
63 |
+From af77dc4014ead3d898fdc8a7a70fe5063ac9b102 Mon Sep 17 00:00:00 2001 |
64 |
+From: Jose Dapena Paz <jose.dapena@×××.com> |
65 |
+Date: Fri, 25 Oct 2019 19:01:29 +0000 |
66 |
+Subject: [PATCH] GCC: use brace-initializer for DohUpgrade vector |
67 |
+ |
68 |
+Constructing NoDestructor with parenthesis constructor is ambiguous |
69 |
+in GCC. Use brace-initializer to avoid that problem. This fixes this |
70 |
+build error: |
71 |
+ |
72 |
+Bug: 819294 |
73 |
+Change-Id: I00dda42daa1794d11e022f26ac07f92e599d106d |
74 |
+Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1879910 |
75 |
+Reviewed-by: Eric Orth <ericorth@××××××××.org> |
76 |
+Commit-Queue: José Dapena Paz <jose.dapena@×××.com> |
77 |
+Cr-Commit-Position: refs/heads/master@{#709569} |
78 |
+--- |
79 |
+ |
80 |
diff --git a/net/dns/dns_util.cc b/net/dns/dns_util.cc |
81 |
-index 14997c4..1543e91 100644 |
82 |
+index 14997c4..637b6f2 100644 |
83 |
--- a/net/dns/dns_util.cc |
84 |
+++ b/net/dns/dns_util.cc |
85 |
-@@ -144,7 +144,7 @@ const std::vector<DohUpgradeEntry>& GetDohUpgradeList() { |
86 |
+@@ -144,7 +144,7 @@ |
87 |
// DohProviderId histogram suffix list in |
88 |
// tools/metrics/histograms/histograms.xml. |
89 |
static const base::NoDestructor<std::vector<DohUpgradeEntry>> |
90 |
- upgradable_servers({ |
91 |
-+ upgradable_servers(std::initializer_list<DohUpgradeEntry>{ |
92 |
++ upgradable_servers{{ |
93 |
DohUpgradeEntry( |
94 |
"CleanBrowsingAdult", |
95 |
{"185.228.168.10", "185.228.169.11", "2a0d:2a00:1::1", |
96 |
+@@ -215,7 +215,7 @@ |
97 |
+ {"9.9.9.9", "149.112.112.112", "2620:fe::fe", "2620:fe::9"}, |
98 |
+ {"dns.quad9.net", "dns9.quad9.net"} /* DoT hostname */, |
99 |
+ {"https://dns.quad9.net/dns-query", true /* use_post */}), |
100 |
+- }); |
101 |
++ }}; |
102 |
+ return *upgradable_servers; |
103 |
+ } |
104 |
+ |
105 |
|
106 |
diff --git a/www-client/chromium/files/chromium-79-gcc-name-clash.patch b/www-client/chromium/files/chromium-79-gcc-name-clash.patch |
107 |
new file mode 100644 |
108 |
index 00000000000..2324f4db7f9 |
109 |
--- /dev/null |
110 |
+++ b/www-client/chromium/files/chromium-79-gcc-name-clash.patch |
111 |
@@ -0,0 +1,135 @@ |
112 |
+From e925deab264e5ebc3c5c13415aa3d44a746e8d45 Mon Sep 17 00:00:00 2001 |
113 |
+From: Jose Dapena Paz <jose.dapena@×××.com> |
114 |
+Date: Fri, 25 Oct 2019 14:53:51 +0000 |
115 |
+Subject: [PATCH] GCC: avoid clash of WebThemeEngine::ForcedColors with ForcedColors enum class |
116 |
+ |
117 |
+GCC fails to build because the declaration of the enum class ForcedColors |
118 |
+clashes with the GetForcedColors method. |
119 |
+ |
120 |
+Bug: 819294 |
121 |
+Change-Id: I5c7647978d15c771f5372a8c70ac6aeb2284fb2f |
122 |
+Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1879452 |
123 |
+Reviewed-by: Kentaro Hara <haraken@××××××××.org> |
124 |
+Commit-Queue: José Dapena Paz <jose.dapena@×××.com> |
125 |
+Cr-Commit-Position: refs/heads/master@{#709469} |
126 |
+ |
127 |
+(backported to M79) |
128 |
+--- |
129 |
+ |
130 |
+diff --git a/content/child/webthemeengine_impl_android.cc b/content/child/webthemeengine_impl_android.cc |
131 |
+index 2e703d6..3759f6c 100644 |
132 |
+--- a/content/child/webthemeengine_impl_android.cc |
133 |
++++ b/content/child/webthemeengine_impl_android.cc |
134 |
+@@ -228,7 +228,7 @@ void WebThemeEngineAndroid::Paint( |
135 |
+ native_theme_extra_params, NativeColorScheme(color_scheme)); |
136 |
+ } |
137 |
+ |
138 |
+-blink::ForcedColors WebThemeEngineAndroid::ForcedColors() const { |
139 |
++blink::ForcedColors WebThemeEngineAndroid::GetForcedColors() const { |
140 |
+ return forced_colors_; |
141 |
+ } |
142 |
+ |
143 |
+diff --git a/content/child/webthemeengine_impl_android.h b/content/child/webthemeengine_impl_android.h |
144 |
+index 0c113a88..95e2744 100644 |
145 |
+--- a/content/child/webthemeengine_impl_android.h |
146 |
++++ b/content/child/webthemeengine_impl_android.h |
147 |
+@@ -22,7 +22,7 @@ |
148 |
+ const blink::WebRect& rect, |
149 |
+ const blink::WebThemeEngine::ExtraParams* extra_params, |
150 |
+ blink::WebColorScheme color_scheme) override; |
151 |
+- blink::ForcedColors ForcedColors() const override; |
152 |
++ blink::ForcedColors GetForcedColors() const override; |
153 |
+ void SetForcedColors(const blink::ForcedColors forced_colors) override; |
154 |
+ blink::PreferredColorScheme PreferredColorScheme() const override; |
155 |
+ void SetPreferredColorScheme( |
156 |
+diff --git a/content/child/webthemeengine_impl_default.cc b/content/child/webthemeengine_impl_default.cc |
157 |
+index 84db298..04af6c6 100644 |
158 |
+--- a/content/child/webthemeengine_impl_default.cc |
159 |
++++ b/content/child/webthemeengine_impl_default.cc |
160 |
+@@ -232,7 +232,7 @@ |
161 |
+ } |
162 |
+ #endif |
163 |
+ |
164 |
+-blink::ForcedColors WebThemeEngineDefault::ForcedColors() const { |
165 |
++blink::ForcedColors WebThemeEngineDefault::GetForcedColors() const { |
166 |
+ return ui::NativeTheme::GetInstanceForWeb()->UsesHighContrastColors() |
167 |
+ ? blink::ForcedColors::kActive |
168 |
+ : blink::ForcedColors::kNone; |
169 |
+diff --git a/content/child/webthemeengine_impl_default.h b/content/child/webthemeengine_impl_default.h |
170 |
+index 73579db..1f08f67 100644 |
171 |
+--- a/content/child/webthemeengine_impl_default.h |
172 |
++++ b/content/child/webthemeengine_impl_default.h |
173 |
+@@ -39,7 +39,7 @@ |
174 |
+ int32_t vertical_arrow_bitmap_height, |
175 |
+ int32_t horizontal_arrow_bitmap_width); |
176 |
+ #endif |
177 |
+- blink::ForcedColors ForcedColors() const override; |
178 |
++ blink::ForcedColors GetForcedColors() const override; |
179 |
+ void SetForcedColors(const blink::ForcedColors forced_colors) override; |
180 |
+ blink::PreferredColorScheme PreferredColorScheme() const override; |
181 |
+ void SetPreferredColorScheme( |
182 |
+diff --git a/content/child/webthemeengine_impl_mac.cc b/content/child/webthemeengine_impl_mac.cc |
183 |
+index 9cb279e..553eedb 100644 |
184 |
+--- a/content/child/webthemeengine_impl_mac.cc |
185 |
++++ b/content/child/webthemeengine_impl_mac.cc |
186 |
+@@ -6,7 +6,7 @@ |
187 |
+ |
188 |
+ namespace content { |
189 |
+ |
190 |
+-blink::ForcedColors WebThemeEngineMac::ForcedColors() const { |
191 |
++blink::ForcedColors WebThemeEngineMac::GetForcedColors() const { |
192 |
+ return forced_colors_; |
193 |
+ } |
194 |
+ |
195 |
+diff --git a/content/child/webthemeengine_impl_mac.h b/content/child/webthemeengine_impl_mac.h |
196 |
+index b3dafa1..c0b66c0 100644 |
197 |
+--- a/content/child/webthemeengine_impl_mac.h |
198 |
++++ b/content/child/webthemeengine_impl_mac.h |
199 |
+@@ -13,7 +13,7 @@ |
200 |
+ public: |
201 |
+ ~WebThemeEngineMac() override {} |
202 |
+ |
203 |
+- blink::ForcedColors ForcedColors() const override; |
204 |
++ blink::ForcedColors GetForcedColors() const override; |
205 |
+ void SetForcedColors(const blink::ForcedColors forced_colors) override; |
206 |
+ blink::PreferredColorScheme PreferredColorScheme() const override; |
207 |
+ void SetPreferredColorScheme( |
208 |
+diff --git a/third_party/blink/public/platform/web_theme_engine.h b/third_party/blink/public/platform/web_theme_engine.h |
209 |
+index d642462..aa2da1b 100644 |
210 |
+--- a/third_party/blink/public/platform/web_theme_engine.h |
211 |
++++ b/third_party/blink/public/platform/web_theme_engine.h |
212 |
+@@ -227,7 +227,7 @@ |
213 |
+ return base::nullopt; |
214 |
+ } |
215 |
+ |
216 |
+- virtual ForcedColors ForcedColors() const { return ForcedColors::kNone; } |
217 |
++ virtual ForcedColors GetForcedColors() const { return ForcedColors::kNone; } |
218 |
+ virtual void SetForcedColors(const blink::ForcedColors forced_colors) {} |
219 |
+ virtual blink::PreferredColorScheme PreferredColorScheme() const { |
220 |
+ return PreferredColorScheme::kNoPreference; |
221 |
+diff --git a/third_party/blink/renderer/core/css/media_values.cc b/third_party/blink/renderer/core/css/media_values.cc |
222 |
+index 045d793..f7c0484 100644 |
223 |
+--- a/third_party/blink/renderer/core/css/media_values.cc |
224 |
++++ b/third_party/blink/renderer/core/css/media_values.cc |
225 |
+@@ -221,7 +221,7 @@ |
226 |
+ |
227 |
+ ForcedColors MediaValues::CalculateForcedColors() { |
228 |
+ if (Platform::Current() && Platform::Current()->ThemeEngine()) |
229 |
+- return Platform::Current()->ThemeEngine()->ForcedColors(); |
230 |
++ return Platform::Current()->ThemeEngine()->GetForcedColors(); |
231 |
+ else |
232 |
+ return ForcedColors::kNone; |
233 |
+ } |
234 |
+diff --git a/third_party/blink/renderer/core/dom/document.cc b/third_party/blink/renderer/core/dom/document.cc |
235 |
+index 64b0d23..02cce1f 100644 |
236 |
+--- a/third_party/blink/renderer/core/dom/document.cc |
237 |
++++ b/third_party/blink/renderer/core/dom/document.cc |
238 |
+@@ -8625,7 +8625,7 @@ |
239 |
+ bool Document::InForcedColorsMode() const { |
240 |
+ return RuntimeEnabledFeatures::ForcedColorsEnabled() && Platform::Current() && |
241 |
+ Platform::Current()->ThemeEngine() && |
242 |
+- Platform::Current()->ThemeEngine()->ForcedColors() != |
243 |
++ Platform::Current()->ThemeEngine()->GetForcedColors() != |
244 |
+ ForcedColors::kNone; |
245 |
+ } |
246 |
+ |
247 |
|
248 |
diff --git a/www-client/chromium/files/chromium-79-gcc-permissive.patch b/www-client/chromium/files/chromium-79-gcc-permissive.patch |
249 |
index e98f6f0ae38..c42d56985a6 100644 |
250 |
--- a/www-client/chromium/files/chromium-79-gcc-permissive.patch |
251 |
+++ b/www-client/chromium/files/chromium-79-gcc-permissive.patch |
252 |
@@ -1,17 +1,3 @@ |
253 |
-diff --git a/third_party/blink/public/platform/web_theme_engine.h b/third_party/blink/public/platform/web_theme_engine.h |
254 |
-index 2b55c0b..f33d224 100644 |
255 |
---- a/third_party/blink/public/platform/web_theme_engine.h |
256 |
-+++ b/third_party/blink/public/platform/web_theme_engine.h |
257 |
-@@ -226,7 +226,7 @@ class WebThemeEngine { |
258 |
- return base::nullopt; |
259 |
- } |
260 |
- |
261 |
-- virtual ForcedColors ForcedColors() const { return ForcedColors::kNone; } |
262 |
-+ virtual enum ForcedColors ForcedColors() const { return ForcedColors::kNone; } |
263 |
- virtual void SetForcedColors(const blink::ForcedColors forced_colors) {} |
264 |
- }; |
265 |
- |
266 |
- |
267 |
From 528e9a3e1f25bd264549c4c7779748abfd16bb1c Mon Sep 17 00:00:00 2001 |
268 |
From: Jan Wilken Dörrie <jdoerrie@××××××××.org> |
269 |
Date: Fri, 18 Oct 2019 11:45:24 +0000 |
270 |
|
271 |
diff --git a/www-client/chromium/files/chromium-79-include.patch b/www-client/chromium/files/chromium-79-include.patch |
272 |
index 06decbdf06d..8f2fb56176a 100644 |
273 |
--- a/www-client/chromium/files/chromium-79-include.patch |
274 |
+++ b/www-client/chromium/files/chromium-79-include.patch |
275 |
@@ -1,59 +1,64 @@ |
276 |
-From d26e3fc037f477648b6265ff9cdcb2c616942fae Mon Sep 17 00:00:00 2001 |
277 |
-From: Raphael Kubo da Costa <raphael.kubo.da.costa@×××××.com> |
278 |
-Date: Tue, 15 Oct 2019 14:56:07 +0000 |
279 |
-Subject: [PATCH] ui/display: Include <bitset> for std::bitset |
280 |
+From f7c177d35242311ea7a2cf49a0980c61664f27ba Mon Sep 17 00:00:00 2001 |
281 |
+From: Jose Dapena Paz <jose.dapena@×××.com> |
282 |
+Date: Fri, 25 Oct 2019 15:07:09 +0000 |
283 |
+Subject: [PATCH] IWYU: include algorithm to use std::lower_bound in ui/gfx/font.cc |
284 |
|
285 |
-Follow-up to commit 1b7e6d75f ("ui/display: extend EdidParser for HDR |
286 |
-metadata") to fix the build with libstdc++ (and maybe MSVC): |
287 |
+Fix GCC build because of missing include: |
288 |
+../../ui/gfx/font.cc: In function ‘gfx::Font::Weight gfx::FontWeightFromInt(int)’: |
289 |
+../../ui/gfx/font.cc:114:8: error: no matching function for call to ‘lower_bound(const gfx::Font::Weight*, const gfx::Font::Weight*, int&, gfx::FontWeightFromInt(int)::<lambda(const gfx::Font::Weight&, const int&)>)’ |
290 |
+ }); |
291 |
+ ^ |
292 |
+In file included from /usr/include/c++/8/bits/char_traits.h:39, |
293 |
+ from /usr/include/c++/8/string:40, |
294 |
+ from ../../ui/gfx/font.h:8, |
295 |
+ from ../../ui/gfx/font.cc:5: |
296 |
+/usr/include/c++/8/bits/stl_algobase.h:984:5: note: candidate: ‘template<class _ForwardIterator, class _Tp> _ForwardIterator std::lower_bound(_ForwardIterator, _ForwardIterator, const _Tp&)’ |
297 |
+ lower_bound(_ForwardIterator __first, _ForwardIterator __last, |
298 |
+ ^~~~~~~~~~~ |
299 |
+/usr/include/c++/8/bits/stl_algobase.h:984:5: note: template argument deduction/substitution failed: |
300 |
+../../ui/gfx/font.cc:114:8: note: candidate expects 3 arguments, 4 provided |
301 |
+ }); |
302 |
+ ^ |
303 |
|
304 |
-./../../ui/display/util/edid_parser.cc:416:22: error: no template named 'bitset' in namespace 'std' |
305 |
- const std::bitset<kMaxNumColorimetryEntries> |
306 |
- ~~~~~^ |
307 |
-./../../ui/display/util/edid_parser.cc:430:22: error: no template named 'bitset' in namespace 'std' |
308 |
- const std::bitset<kMaxNumHDRStaticMedatataEntries> |
309 |
- ~~~~~^ |
310 |
- |
311 |
-Bug: 957519, 1012846 |
312 |
-Change-Id: Ic3c9c9416ac446bde9f190b9575ef1a386aa8e8e |
313 |
-Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1862917 |
314 |
-Commit-Queue: Raphael Kubo da Costa <raphael.kubo.da.costa@×××××.com> |
315 |
-Commit-Queue: Miguel Casas <mcasas@××××××××.org> |
316 |
-Auto-Submit: Raphael Kubo da Costa <raphael.kubo.da.costa@×××××.com> |
317 |
-Reviewed-by: Miguel Casas <mcasas@××××××××.org> |
318 |
-Cr-Commit-Position: refs/heads/master@{#705965} |
319 |
---- |
320 |
- |
321 |
-diff --git a/ui/display/util/edid_parser.cc b/ui/display/util/edid_parser.cc |
322 |
-index 44f2fdb..1adebb7 100644 |
323 |
---- a/ui/display/util/edid_parser.cc |
324 |
-+++ b/ui/display/util/edid_parser.cc |
325 |
-@@ -7,6 +7,7 @@ |
326 |
- #include <stddef.h> |
327 |
- |
328 |
- #include <algorithm> |
329 |
-+#include <bitset> |
330 |
- |
331 |
- #include "base/hash/hash.h" |
332 |
- #include "base/strings/string_util.h" |
333 |
- |
334 |
-IWYU: add missing algorithm |
335 |
+Bug: 819294 |
336 |
+Change-Id: Ic59dcf3a06bdd54d1d426c08a61624873a0ff30c |
337 |
+Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1879909 |
338 |
+Commit-Queue: Alexei Svitkine <asvitkine@××××××××.org> |
339 |
+Reviewed-by: Alexei Svitkine <asvitkine@××××××××.org> |
340 |
+Cr-Commit-Position: refs/heads/master@{#709472} |
341 |
--- |
342 |
|
343 |
diff --git a/ui/gfx/font.cc b/ui/gfx/font.cc |
344 |
-index 21367fd..085baeb 100644 |
345 |
+index 21367fd7..92b159e 100644 |
346 |
--- a/ui/gfx/font.cc |
347 |
+++ b/ui/gfx/font.cc |
348 |
-@@ -8,6 +8,8 @@ |
349 |
- #include "build/build_config.h" |
350 |
- #include "ui/gfx/platform_font.h" |
351 |
+@@ -4,6 +4,8 @@ |
352 |
+ |
353 |
+ #include "ui/gfx/font.h" |
354 |
|
355 |
+#include <algorithm> |
356 |
+ |
357 |
- namespace gfx { |
358 |
- |
359 |
- //////////////////////////////////////////////////////////////////////////////// |
360 |
+ #include "base/strings/utf_string_conversions.h" |
361 |
+ #include "build/build_config.h" |
362 |
+ #include "ui/gfx/platform_font.h" |
363 |
|
364 |
-IWYU: add missing vector |
365 |
+From 97eb905ba262382bc3583078761c68f4452aea71 Mon Sep 17 00:00:00 2001 |
366 |
+From: Jose Dapena Paz <jose.dapena@×××.com> |
367 |
+Date: Fri, 25 Oct 2019 09:27:53 +0000 |
368 |
+Subject: [PATCH] IWYU: launch_manager.h uses std::vector |
369 |
+ |
370 |
+Add #include <vector> for using std::vector. This fixes GCC build. |
371 |
+ |
372 |
+./../chrome/browser/apps/launch_service/launch_manager.h:46:15: error: ‘vector’ in namespace ‘std’ does not name a template type |
373 |
+ static std::vector<base::FilePath> GetLaunchFilesFromCommandLine( |
374 |
+ ^~~~~~ |
375 |
+ |
376 |
+Bug: 819294 |
377 |
+Change-Id: I02ec3a2914a8fbe3aa0041017a0228f4b0ca1ec9 |
378 |
+Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1879289 |
379 |
+Reviewed-by: Alexey Baskakov <loyso@××××××××.org> |
380 |
+Commit-Queue: José Dapena Paz <jose.dapena@×××.com> |
381 |
+Cr-Commit-Position: refs/heads/master@{#709411} |
382 |
--- |
383 |
|
384 |
diff --git a/chrome/browser/apps/launch_service/launch_manager.h b/chrome/browser/apps/launch_service/launch_manager.h |
385 |
@@ -68,3 +73,59 @@ index 00aeb9d..76570ea 100644 |
386 |
|
387 |
#include "base/macros.h" |
388 |
|
389 |
+From e7407ce7fa262e9fd1a19dd0957e5a950520ee3a Mon Sep 17 00:00:00 2001 |
390 |
+From: Jose Dapena Paz <jose.dapena@×××.com> |
391 |
+Date: Fri, 25 Oct 2019 08:25:45 +0000 |
392 |
+Subject: [PATCH] IWYU: include cstdint in register_context.h as it uses uintptr_t |
393 |
+ |
394 |
+GCC build fix as build fails with this: |
395 |
+../../base/profiler/register_context.h:31:1: error: ‘uintptr_t’ does not name a type; did you mean ‘intptr_t’? |
396 |
+ uintptr_t& AsUintPtr(T* value) { |
397 |
+ ^~~~~~~~~ |
398 |
+ intptr_t |
399 |
+../../base/profiler/register_context.h:110:3: error: ‘uintptr_t’ does not name a type; did you mean ‘intptr_t’? |
400 |
+ uintptr_t stack_pointer; |
401 |
+ ^~~~~~~~~ |
402 |
+ intptr_t |
403 |
+../../base/profiler/register_context.h:111:3: error: ‘uintptr_t’ does not name a type; did you mean ‘intptr_t’? |
404 |
+ uintptr_t frame_pointer; |
405 |
+ ^~~~~~~~~ |
406 |
+ intptr_t |
407 |
+../../base/profiler/register_context.h:112:3: error: ‘uintptr_t’ does not name a type; did you mean ‘intptr_t’? |
408 |
+ uintptr_t instruction_pointer; |
409 |
+ ^~~~~~~~~ |
410 |
+ intptr_t |
411 |
+../../base/profiler/register_context.h:115:8: error: ‘uintptr_t’ does not name a type; did you mean ‘intptr_t’? |
412 |
+ inline uintptr_t& RegisterContextStackPointer(RegisterContext* context) { |
413 |
+ ^~~~~~~~~ |
414 |
+ intptr_t |
415 |
+../../base/profiler/register_context.h:119:8: error: ‘uintptr_t’ does not name a type; did you mean ‘intptr_t’? |
416 |
+ inline uintptr_t& RegisterContextFramePointer(RegisterContext* context) { |
417 |
+ ^~~~~~~~~ |
418 |
+ intptr_t |
419 |
+../../base/profiler/register_context.h:123:8: error: ‘uintptr_t’ does not name a type; did you mean ‘intptr_t’? |
420 |
+ inline uintptr_t& RegisterContextInstructionPointer(RegisterContext* context) { |
421 |
+ ^~~~~~~~~ |
422 |
+ intptr_t |
423 |
+ |
424 |
+ |
425 |
+Bug: 819294 |
426 |
+Change-Id: I49567b00a6f021686c52053a22fb9c502c84f1bc |
427 |
+Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1879908 |
428 |
+Reviewed-by: Mike Wittman <wittman@××××××××.org> |
429 |
+Commit-Queue: José Dapena Paz <jose.dapena@×××.com> |
430 |
+Cr-Commit-Position: refs/heads/master@{#709398} |
431 |
+--- |
432 |
+ |
433 |
+diff --git a/base/profiler/register_context.h b/base/profiler/register_context.h |
434 |
+index 46c4250..7dd86ff 100644 |
435 |
+--- a/base/profiler/register_context.h |
436 |
++++ b/base/profiler/register_context.h |
437 |
+@@ -9,6 +9,7 @@ |
438 |
+ #ifndef BASE_PROFILER_REGISTER_CONTEXT_H_ |
439 |
+ #define BASE_PROFILER_REGISTER_CONTEXT_H_ |
440 |
+ |
441 |
++#include <cstdint> |
442 |
+ #include <type_traits> |
443 |
+ |
444 |
+ #include "build/build_config.h" |
445 |
|
446 |
diff --git a/www-client/chromium/files/chromium-79-swiftshader-linux.patch b/www-client/chromium/files/chromium-79-swiftshader-linux.patch |
447 |
deleted file mode 100644 |
448 |
index 3219dcd254b..00000000000 |
449 |
--- a/www-client/chromium/files/chromium-79-swiftshader-linux.patch |
450 |
+++ /dev/null |
451 |
@@ -1,180 +0,0 @@ |
452 |
-From 74acd924593c427e5d3528d9ca6fdf2f4cd1383a Mon Sep 17 00:00:00 2001 |
453 |
-From: David 'Digit' Turner <digit@××××××.com> |
454 |
-Date: Mon, 14 Oct 2019 15:28:36 +0200 |
455 |
-Subject: [PATCH] [linux]: Remove linux namespace. |
456 |
- |
457 |
-'linux' turns out to be a prebuilt macro for GCC (not Clang) |
458 |
-so using "namespace linux { ... }" actually does not compile. |
459 |
- |
460 |
-This CL fixes the issue by removing the namespace, and using |
461 |
-Linux as a prefix for the corresponding class. |
462 |
- |
463 |
-Bug: 140419396 |
464 |
-Change-Id: I1e980bc66b2cf8f2e21d9f7076918e2b476ae4d4 |
465 |
---- |
466 |
- |
467 |
-diff --git a/third_party/swiftshader/src/System/Linux/MemFd.cpp b/third_party/swiftshader/src/System/Linux/MemFd.cpp |
468 |
-index 22b77e4..a8f99e2 100644 |
469 |
---- a/third_party/swiftshader/src/System/Linux/MemFd.cpp |
470 |
-+++ b/third_party/swiftshader/src/System/Linux/MemFd.cpp |
471 |
-@@ -37,21 +37,18 @@ |
472 |
- #define __NR_memfd_create 319 |
473 |
- #endif /* __NR_memfd_create__ */ |
474 |
- |
475 |
--namespace linux |
476 |
--{ |
477 |
-- |
478 |
--MemFd::~MemFd() |
479 |
-+LinuxMemFd::~LinuxMemFd() |
480 |
- { |
481 |
- close(); |
482 |
- } |
483 |
- |
484 |
--void MemFd::importFd(int fd) |
485 |
-+void LinuxMemFd::importFd(int fd) |
486 |
- { |
487 |
- close(); |
488 |
- fd_ = fd; |
489 |
- } |
490 |
- |
491 |
--int MemFd::exportFd() const |
492 |
-+int LinuxMemFd::exportFd() const |
493 |
- { |
494 |
- if (fd_ < 0) |
495 |
- { |
496 |
-@@ -62,7 +59,7 @@ |
497 |
- return ::fcntl(fd_, F_DUPFD_CLOEXEC, 0); |
498 |
- } |
499 |
- |
500 |
--bool MemFd::allocate(const char* name, size_t size) |
501 |
-+bool LinuxMemFd::allocate(const char* name, size_t size) |
502 |
- { |
503 |
- close(); |
504 |
- |
505 |
-@@ -89,7 +86,7 @@ |
506 |
- return true; |
507 |
- } |
508 |
- |
509 |
--void MemFd::close() |
510 |
-+void LinuxMemFd::close() |
511 |
- { |
512 |
- if (fd_ >= 0) |
513 |
- { |
514 |
-@@ -97,23 +94,21 @@ |
515 |
- // https://lwn.net/Articles/576478/ for example. |
516 |
- int ret = ::close(fd_); |
517 |
- if (ret < 0) { |
518 |
-- TRACE("MemFd::close() failed with: %s", strerror(errno)); |
519 |
-+ TRACE("LinuxMemFd::close() failed with: %s", strerror(errno)); |
520 |
- assert(false); |
521 |
- } |
522 |
- fd_ = -1; |
523 |
- } |
524 |
- } |
525 |
- |
526 |
--void* MemFd::mapReadWrite(size_t offset, size_t size) |
527 |
-+void* LinuxMemFd::mapReadWrite(size_t offset, size_t size) |
528 |
- { |
529 |
- void* addr = ::mmap(nullptr, size, PROT_READ|PROT_WRITE, MAP_SHARED, fd_, |
530 |
- static_cast<off_t>(offset)); |
531 |
- return (addr == MAP_FAILED) ? nullptr : addr; |
532 |
- } |
533 |
- |
534 |
--bool MemFd::unmap(void* addr, size_t size) |
535 |
-+bool LinuxMemFd::unmap(void* addr, size_t size) |
536 |
- { |
537 |
- return ::munmap(addr, size) == 0; |
538 |
- } |
539 |
-- |
540 |
--} // namespace linux |
541 |
-diff --git a/third_party/swiftshader/src/System/Linux/MemFd.hpp b/third_party/swiftshader/src/System/Linux/MemFd.hpp |
542 |
-index 8b88473..209d642 100644 |
543 |
---- a/third_party/swiftshader/src/System/Linux/MemFd.hpp |
544 |
-+++ b/third_party/swiftshader/src/System/Linux/MemFd.hpp |
545 |
-@@ -22,18 +22,15 @@ |
546 |
- // supported by the Linux kernel since 3.17 (good enough for Android and desktop |
547 |
- // Linux). |
548 |
- |
549 |
--namespace linux |
550 |
--{ |
551 |
-- |
552 |
--class MemFd { |
553 |
-+class LinuxMemFd { |
554 |
- public: |
555 |
-- MemFd() = default; |
556 |
-+ LinuxMemFd() = default; |
557 |
- |
558 |
-- MemFd(const char* name, size_t size) : MemFd() { |
559 |
-+ LinuxMemFd(const char* name, size_t size) : LinuxMemFd() { |
560 |
- allocate(name, size); |
561 |
- } |
562 |
- |
563 |
-- ~MemFd(); |
564 |
-+ ~LinuxMemFd(); |
565 |
- |
566 |
- // Return true iff the region is valid/allocated. |
567 |
- bool isValid() const { return fd_ >= 0; } |
568 |
-@@ -68,6 +65,4 @@ |
569 |
- int fd_ = -1; |
570 |
- }; |
571 |
- |
572 |
--} // namespace linux |
573 |
-- |
574 |
- #endif // MEMFD_LINUX |
575 |
-diff --git a/third_party/swiftshader/tests/SystemUnitTests/unittests.cpp b/third_party/swiftshader/tests/SystemUnitTests/unittests.cpp |
576 |
-index 53ff9ec..9026b55 100644 |
577 |
---- a/third_party/swiftshader/tests/SystemUnitTests/unittests.cpp |
578 |
-+++ b/third_party/swiftshader/tests/SystemUnitTests/unittests.cpp |
579 |
-@@ -26,14 +26,14 @@ |
580 |
- |
581 |
- #ifdef __linux__ |
582 |
- TEST(MemFd, DefaultConstructor) { |
583 |
-- linux::MemFd memfd; |
584 |
-+ LinuxMemFd memfd; |
585 |
- ASSERT_FALSE(memfd.isValid()); |
586 |
- ASSERT_EQ(-1, memfd.exportFd()); |
587 |
- } |
588 |
- |
589 |
- TEST(MemFd, AllocatingConstructor) { |
590 |
- const size_t kRegionSize = sw::memoryPageSize() * 8; |
591 |
-- linux::MemFd memfd("test-region", kRegionSize); |
592 |
-+ LinuxMemFd memfd("test-region", kRegionSize); |
593 |
- ASSERT_TRUE(memfd.isValid()); |
594 |
- ASSERT_GE(memfd.fd(), 0); |
595 |
- void* addr = memfd.mapReadWrite(0, kRegionSize); |
596 |
-@@ -43,7 +43,7 @@ |
597 |
- |
598 |
- TEST(MemFd, ExplicitAllocation) { |
599 |
- const size_t kRegionSize = sw::memoryPageSize() * 8; |
600 |
-- linux::MemFd memfd; |
601 |
-+ LinuxMemFd memfd; |
602 |
- ASSERT_FALSE(memfd.isValid()); |
603 |
- ASSERT_EQ(-1, memfd.exportFd()); |
604 |
- ASSERT_TRUE(memfd.allocate("test-region", kRegionSize)); |
605 |
-@@ -52,7 +52,7 @@ |
606 |
- |
607 |
- TEST(MemFd, Close) { |
608 |
- const size_t kRegionSize = sw::memoryPageSize() * 8; |
609 |
-- linux::MemFd memfd("test-region", kRegionSize); |
610 |
-+ LinuxMemFd memfd("test-region", kRegionSize); |
611 |
- ASSERT_TRUE(memfd.isValid()); |
612 |
- int fd = memfd.exportFd(); |
613 |
- memfd.close(); |
614 |
-@@ -63,7 +63,7 @@ |
615 |
- |
616 |
- TEST(MemFd, ExportImportFd) { |
617 |
- const size_t kRegionSize = sw::memoryPageSize() * 8; |
618 |
-- linux::MemFd memfd("test-region1", kRegionSize); |
619 |
-+ LinuxMemFd memfd("test-region1", kRegionSize); |
620 |
- auto* addr = reinterpret_cast<uint8_t*>(memfd.mapReadWrite(0, kRegionSize)); |
621 |
- ASSERT_TRUE(addr); |
622 |
- for (size_t n = 0; n < kRegionSize; ++n) |
623 |
-@@ -74,7 +74,7 @@ |
624 |
- ASSERT_TRUE(memfd.unmap(addr, kRegionSize)); |
625 |
- memfd.close(); |
626 |
- |
627 |
-- linux::MemFd memfd2; |
628 |
-+ LinuxMemFd memfd2; |
629 |
- memfd2.importFd(fd); |
630 |
- ASSERT_TRUE(memfd2.isValid()); |
631 |
- addr = reinterpret_cast<uint8_t*>(memfd2.mapReadWrite(0, kRegionSize)); |