summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMike Gilbert <floppym@gentoo.org>2017-08-09 10:04:25 -0400
committerMike Gilbert <floppym@gentoo.org>2017-08-09 10:04:25 -0400
commit689090f1ec53b7b958a903f6ed01f38b71e1b819 (patch)
tree543a1e6c23b51c760a7777c0283c886a2e3ee597 /www-client/chromium
parentapp-crypt/gifshuffle: EAPI 6 bump (diff)
downloadgentoo-689090f1ec53b7b958a903f6ed01f38b71e1b819.tar.gz
gentoo-689090f1ec53b7b958a903f6ed01f38b71e1b819.tar.bz2
gentoo-689090f1ec53b7b958a903f6ed01f38b71e1b819.zip
www-client/chromium: backport build fix
Bug: https://bugs.gentoo.org/626684 Package-Manager: Portage-2.3.6_p23, Repoman-2.3.3_p1
Diffstat (limited to 'www-client/chromium')
-rw-r--r--www-client/chromium/chromium-61.0.3163.31.ebuild1
-rw-r--r--www-client/chromium/files/chromium-mojo-dep.patch54
2 files changed, 55 insertions, 0 deletions
diff --git a/www-client/chromium/chromium-61.0.3163.31.ebuild b/www-client/chromium/chromium-61.0.3163.31.ebuild
index 600995b990b1..c55fd86c15db 100644
--- a/www-client/chromium/chromium-61.0.3163.31.ebuild
+++ b/www-client/chromium/chromium-61.0.3163.31.ebuild
@@ -146,6 +146,7 @@ PATCHES=(
"${FILESDIR}/${PN}-gcc-r1.patch"
"${FILESDIR}/${PN}-gn-bootstrap-r14.patch"
"${FILESDIR}/${PN}-atk-r1.patch"
+ "${FILESDIR}/${PN}-mojo-dep.patch"
)
pre_build_checks() {
diff --git a/www-client/chromium/files/chromium-mojo-dep.patch b/www-client/chromium/files/chromium-mojo-dep.patch
new file mode 100644
index 000000000000..bf618540d174
--- /dev/null
+++ b/www-client/chromium/files/chromium-mojo-dep.patch
@@ -0,0 +1,54 @@
+From f8e2c057ec6123b0a1d08294f41e78c427636b49 Mon Sep 17 00:00:00 2001
+From: Mike Bjorge <mbjorge@chromium.org>
+Date: Mon, 7 Aug 2017 18:41:58 +0000
+Subject: [PATCH] Add mojo_blink dep from WebKit loader/.
+
+Now that loader has a direct dependency on the generated mojom-blink
+headers (https://chromium-review.googlesource.com/c/577488), the loader/
+target should have a dep on the mojo target that generates the headers,
+otherwise the build may fail depending on build order.
+
+All the other platform/ targets that depend on the mojom-blink headers
+are directly included in the platform target, which has the necessary
+dependency already.
+
+Bug: 746197
+Bug: 750883
+Change-Id: Ife1429ec649605aae3f7920560cec37f3a218154
+Reviewed-on: https://chromium-review.googlesource.com/602869
+Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
+Reviewed-by: Matt Falkenhagen <falken@chromium.org>
+Commit-Queue: Mike Bjorge <mbjorge@chromium.org>
+Cr-Commit-Position: refs/heads/master@{#492369}
+---
+ third_party/WebKit/Source/platform/loader/BUILD.gn | 1 +
+ third_party/WebKit/public/BUILD.gn | 1 +
+ 2 files changed, 2 insertions(+)
+
+diff --git a/third_party/WebKit/Source/platform/loader/BUILD.gn b/third_party/WebKit/Source/platform/loader/BUILD.gn
+index d483734b21cf..409befe884a8 100644
+--- a/third_party/WebKit/Source/platform/loader/BUILD.gn
++++ b/third_party/WebKit/Source/platform/loader/BUILD.gn
+@@ -99,6 +99,7 @@ source_set("loader") {
+ "//components/link_header_util:link_header_util",
+ "//mojo/public/cpp/system:system",
+ "//storage/public/interfaces:interfaces_blink__generator",
++ "//third_party/WebKit/public:mojo_bindings_blink",
+ ]
+
+ public_deps = [
+diff --git a/third_party/WebKit/public/BUILD.gn b/third_party/WebKit/public/BUILD.gn
+index 84ee3ca9136d..cc847663efc6 100644
+--- a/third_party/WebKit/public/BUILD.gn
++++ b/third_party/WebKit/public/BUILD.gn
+@@ -755,6 +755,7 @@ mojom("mojo_bindings") {
+ visibility_blink = [
+ "//content/common:mojo_bindings_blink",
+ "//third_party/WebKit/Source/platform",
++ "//third_party/WebKit/Source/platform/loader",
+ ]
+ sources = [
+ "platform/display_mode.mojom",
+--
+2.14.0
+