public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
From: LuK1337 <priv.luk@gmail.com>
To: git-commits@fedoraproject.org
Subject: [rpms/chromium] epel9-next: Backport Wayland DnD bug fix from upstream
Date: Fri, 07 Aug 2026 16:08:28 GMT	[thread overview]
Message-ID: <178611890839.1.17425924119754547068.rpms-chromium-5bf1f4083299@fedoraproject.org> (raw)

A new commit has been pushed.

Repo   : rpms/chromium
Branch : epel9-next
Commit : 5bf1f4083299bd86a8499efb67dd0eb6256f5c92
Author : LuK1337 <priv.luk@gmail.com>
Date   : 2025-11-20T23:43:08+01:00
Stats  : +79/-1 in 2 file(s)
URL    : https://src.fedoraproject.org/rpms/chromium/c/5bf1f4083299bd86a8499efb67dd0eb6256f5c92?branch=epel9-next

Log:
Backport Wayland DnD bug fix from upstream

---
diff --git a/chromium-142-Add-ExtractData-support-for-text-uri-list.patch b/chromium-142-Add-ExtractData-support-for-text-uri-list.patch
new file mode 100644
index 0000000..71d7e94
--- /dev/null
+++ b/chromium-142-Add-ExtractData-support-for-text-uri-list.patch
@@ -0,0 +1,72 @@
+From 3e6fd579fa63e68c9d0092ea56f7a457c03bdb7d Mon Sep 17 00:00:00 2001
+From: LuK1337 <priv.luk@gmail.com>
+Date: Thu, 20 Nov 2025 14:27:50 -0800
+Subject: [PATCH] [ozone/wayland] Add ExtractData() support for text/uri-list
+
+When dragging a file from chrome://downloads to KDE Konsole, the
+following message shows up in the debug output: "Cannot deliver data of
+type text/uri-list and no text representation is available.".
+
+After making it so text/uri-list is handled the same way as
+text/x-moz-url, the log message is gone and path to file is pasted into
+terminal as one would expect.
+
+Bug: 460074619
+Test: ozone_unittests
+Change-Id: Iab0144ca7d3d3983d0e61d74f42a53b366f9e830
+Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7145698
+Auto-Submit: Łukasz Patron <priv.luk@gmail.com>
+Reviewed-by: Kramer Ge <fangzhoug@chromium.org>
+Reviewed-by: Peter McNeeley <petermcneeley@google.com>
+Commit-Queue: Peter McNeeley <petermcneeley@google.com>
+Cr-Commit-Position: refs/heads/main@{#1548075}
+---
+
+diff --git a/AUTHORS b/AUTHORS
+index abceab66..98ced4a 100644
+--- a/AUTHORS
++++ b/AUTHORS
+@@ -943,6 +943,7 @@
+ Luka Dojcilovic <l.dojcilovic@gmail.com>
+ Lukas Lihotzki <lukas@lihotzki.de>
+ Lukasz Krakowiak <lukasz.krakowiak@mobica.com>
++Lukasz Patron <priv.luk@gmail.com>
+ Luke Inman-Semerau <luke.semerau@gmail.com>
+ Luke Gu <gulukesh@gmail.com>
+ Luke Warlow <lukewarlow156@gmail.com>
+diff --git a/ui/ozone/platform/wayland/host/wayland_exchange_data_provider.cc b/ui/ozone/platform/wayland/host/wayland_exchange_data_provider.cc
+index 2d32cc9..da6da7ae 100644
+--- a/ui/ozone/platform/wayland/host/wayland_exchange_data_provider.cc
++++ b/ui/ozone/platform/wayland/host/wayland_exchange_data_provider.cc
+@@ -277,7 +277,8 @@
+   DCHECK(out_content);
+   DCHECK(IsMimeTypeSupported(mime_type));
+   if (std::optional<ui::OSExchangeData::UrlInfo> url_info;
+-      mime_type == ui::kMimeTypeMozillaUrl &&
++      (mime_type == ui::kMimeTypeMozillaUrl ||
++       mime_type == ui::kMimeTypeUriList) &&
+       (url_info = GetURLAndTitle(kFilenameToURLPolicy)).has_value()) {
+     out_content->append(url_info->url.spec());
+     return true;
+diff --git a/ui/ozone/platform/wayland/host/wayland_exchange_data_provider_unittest.cc b/ui/ozone/platform/wayland/host/wayland_exchange_data_provider_unittest.cc
+index 70b6d9d..29a8ad0 100644
+--- a/ui/ozone/platform/wayland/host/wayland_exchange_data_provider_unittest.cc
++++ b/ui/ozone/platform/wayland/host/wayland_exchange_data_provider_unittest.cc
+@@ -61,6 +61,17 @@
+   EXPECT_EQ("pickled-str", read_pickled_str);
+ }
+ 
++TEST(WaylandExchangeDataProviderTest, FileNameAsUriList) {
++  WaylandExchangeDataProvider provider;
++  std::string extracted;
++  EXPECT_FALSE(provider.ExtractData(kMimeTypeUriList, &extracted));
++
++  extracted.clear();
++  provider.AddData(ToClipboardData("file:///dev/null"), kMimeTypeUriList);
++  EXPECT_TRUE(provider.ExtractData(kMimeTypeUriList, &extracted));
++  EXPECT_EQ("file:///dev/null", extracted);
++}
++
+ TEST(WaylandExchangeDataProviderTest, FileContents) {
+   constexpr std::string kName("filename");
+   constexpr std::string kContents("contents");

diff --git a/chromium.spec b/chromium.spec
index cfe4b3e..857d545 100644
--- a/chromium.spec
+++ b/chromium.spec
@@ -245,7 +245,7 @@
 
 Name:	chromium
 Version: 142.0.7444.175
-Release: 2%{?dist}
+Release: 3%{?dist}
 Summary: A WebKit (Blink) powered web browser that Google doesn't want you to use
 Url: http://www.chromium.org/Home
 License: BSD-3-Clause AND LGPL-2.1-or-later AND Apache-2.0 AND IJG AND MIT AND GPL-2.0-or-later AND ISC AND OpenSSL AND (MPL-1.1 OR GPL-2.0-only OR LGPL-2.0-only)
@@ -456,6 +456,8 @@ Patch511: 0002-Fix-Missing-OPENSSL_NO_ENGINE-Guard.patch
 # ../../base/containers/span.h:1387:63: error: arithmetic on a pointer to an incomplete type 'element_type' (aka 'const autofill::FormFieldData')
 # 1387 |         typename iterator::AssumeValid(data(), data(), data() + size())));
 Patch1000: chromium-142-missing-include-for-form_field_data.patch
+# Fix Wayland URI DnD issue
+Patch1001: chromium-142-Add-ExtractData-support-for-text-uri-list.patch
 
 # Use chromium-latest.py to generate clean tarball from released build tarballs, found here:
 # http://build.chromium.org/buildbot/official/
@@ -1115,6 +1117,7 @@ Qt6 UI for chromium.
 
 # Upstream patches
 %patch -P1000 -p1 -b .missing-include-for-form_field_data.patch
+%patch -P1001 -p1 -b .Add-ExtractData-support-for-text-uri-list.patch
 
 # Change shebang in all relevant files in this directory and all subdirectories
 # See `man find` for how the `-exec command {} +` syntax works
@@ -1746,6 +1749,9 @@ fi
 %endif
 
 %changelog
+* Thu Nov 20 2025 LuK1337 <priv.luk@gmail.com> - 142.0.7444.175-3
+- Backport Wayland DnD bug fix from upstream
+
 * Wed Nov 19 2025 Than Ngo <than@redhat.com> - 142.0.7444.175-2
 - Fix typos in chromium.conf
 

                 reply	other threads:[~2026-08-07 16:08 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=178611890839.1.17425924119754547068.rpms-chromium-5bf1f4083299@fedoraproject.org \
    --to=priv.luk@gmail.com \
    --cc=git-commits@fedoraproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox