pkgsrc-Changes archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
CVS commit: pkgsrc/www/firefox
Module Name: pkgsrc
Committed By: martin
Date: Sun Apr 29 13:27:08 UTC 2018
Modified Files:
pkgsrc/www/firefox: distinfo
Added Files:
pkgsrc/www/firefox/patches: patch-widget_gtk_nsClipboard.cpp
patch-widget_gtk_nsClipboard.h patch-widget_gtk_nsClipboardX11.cpp
patch-widget_gtk_nsClipboardX11.h
Log Message:
Apply upstream patches for Bug 1447925 - pasting from clipboard did not
work from non-UTF8 applications.
Ride todays (unrelated) rev bump.
To generate a diff of this commit:
cvs rdiff -u -r1.312 -r1.313 pkgsrc/www/firefox/distinfo
cvs rdiff -u -r0 -r1.1 \
pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboard.cpp \
pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboard.h \
pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboardX11.cpp \
pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboardX11.h
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: pkgsrc/www/firefox/distinfo
diff -u pkgsrc/www/firefox/distinfo:1.312 pkgsrc/www/firefox/distinfo:1.313
--- pkgsrc/www/firefox/distinfo:1.312 Sat Apr 14 07:05:14 2018
+++ pkgsrc/www/firefox/distinfo Sun Apr 29 13:27:08 2018
@@ -1,4 +1,4 @@
-$NetBSD: distinfo,v 1.312 2018/04/14 07:05:14 ryoon Exp $
+$NetBSD: distinfo,v 1.313 2018/04/29 13:27:08 martin Exp $
SHA1 (firefox-59.0.2.tar.bz2) = 8fcf726b4d23716c7ff2b7ebc6527782b5bafe9c
RMD160 (firefox-59.0.2.tar.bz2) = ff7ab2f9da93eda98de8aecb4563cfd176b896d9
@@ -79,6 +79,10 @@ SHA1 (patch-toolkit_library_moz.build) =
SHA1 (patch-toolkit_moz.configure) = 78f5c46931ae3426e65eddbea7e0552bcc6a19e0
SHA1 (patch-toolkit_mozapps_installer_packager.mk) = 6f557fb732f0b9dabf059ffac0400d3db4e79e47
SHA1 (patch-toolkit_xre_glxtest.cpp) = cf048491778610454b914400e53072afedd96e7b
+SHA1 (patch-widget_gtk_nsClipboard.cpp) = 0a56631f05c2ead36b7a80a3cfe0ee7621047df1
+SHA1 (patch-widget_gtk_nsClipboard.h) = fab72d12326b9cef1a2041fa3d3eb2794e85d349
+SHA1 (patch-widget_gtk_nsClipboardX11.cpp) = 684ae1bc2fdca6edc4928b635d100435f8ecbf60
+SHA1 (patch-widget_gtk_nsClipboardX11.h) = 2902af4227b65e0594dcb2ef6fbd16d6d0850f58
SHA1 (patch-xpcom_base_nscore.h) = d7c96eda884d7f370442749c43caba50da61bbe4
SHA1 (patch-xpcom_build_BinaryPath.h) = 88a815a142356bf76024e5a4326b677122c844c1
SHA1 (patch-xpcom_build_XPCOMInit.cpp) = 1e7cc3a7fd9515b10307fafa2b01ed689320bd54
Added files:
Index: pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboard.cpp
diff -u /dev/null pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboard.cpp:1.1
--- /dev/null Sun Apr 29 13:27:08 2018
+++ pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboard.cpp Sun Apr 29 13:27:08 2018
@@ -0,0 +1,30 @@
+$NetBSD: patch-widget_gtk_nsClipboard.cpp,v 1.1 2018/04/29 13:27:08 martin Exp $
+
+Upstream fixes for Bug 1447925 -
+ Add GetClipboardText() to get text data from clipboard
+ Use GetClipboardText() to get text data at nsClipboard::GetData()
+
+--- widget/gtk/nsClipboard.cpp.orig 2018-04-29 12:56:52.405590187 +0000
++++ widget/gtk/nsClipboard.cpp
+@@ -301,11 +301,8 @@ nsClipboard::GetData(nsITransferable *aT
+ // Special case text/unicode since we can convert any
+ // string into text/unicode
+ if (flavorStr.EqualsLiteral(kUnicodeMime)) {
+- uint32_t clipboardDataLength;
+ const char* clipboardData =
+- mContext->GetClipboardData(GTK_DEFAULT_MIME_TEXT,
+- aWhichClipboard,
+- &clipboardDataLength);
++ mContext->GetClipboardText(aWhichClipboard);
+ if (!clipboardData) {
+ // If the type was text/unicode and we couldn't get
+ // text off the clipboard, run the next loop
+@@ -314,7 +311,7 @@ nsClipboard::GetData(nsITransferable *aT
+ }
+
+ // Convert utf-8 into our unicode format.
+- NS_ConvertUTF8toUTF16 ucs2string(clipboardData, clipboardDataLength);
++ NS_ConvertUTF8toUTF16 ucs2string(clipboardData);
+ const char* unicodeData = (const char *)ToNewUnicode(ucs2string);
+ uint32_t unicodeDataLength = ucs2string.Length() * 2;
+ SetTransferableData(aTransferable, flavorStr,
Index: pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboard.h
diff -u /dev/null pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboard.h:1.1
--- /dev/null Sun Apr 29 13:27:08 2018
+++ pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboard.h Sun Apr 29 13:27:08 2018
@@ -0,0 +1,30 @@
+$NetBSD: patch-widget_gtk_nsClipboard.h,v 1.1 2018/04/29 13:27:08 martin Exp $
+
+Upstream fixes for Bug 1447925 -
+ Add GetClipboardText() to get text data from clipboard
+ Use GetClipboardText() to get text data at nsClipboard::GetData()
+
+--- widget/gtk/nsClipboard.h.orig 2018-04-29 12:56:52.405642009 +0000
++++ widget/gtk/nsClipboard.h
+@@ -13,16 +13,18 @@
+ #include "nsIBinaryOutputStream.h"
+ #include <gtk/gtk.h>
+
+-// Default Gtk MIME for text
+-#define GTK_DEFAULT_MIME_TEXT "UTF8_STRING"
+-
+ class nsRetrievalContext {
+ public:
++ // Get actual clipboard content (GetClipboardData/GetClipboardText)
++ // which has to be released by ReleaseClipboardData().
+ virtual const char* GetClipboardData(const char* aMimeType,
+ int32_t aWhichClipboard,
+ uint32_t* aContentLength) = 0;
++ virtual const char* GetClipboardText(int32_t aWhichClipboard) = 0;
+ virtual void ReleaseClipboardData(const char* aClipboardData) = 0;
+
++ // Get data mime types which can be obtained from clipboard.
++ // The returned array has to be released by g_free().
+ virtual GdkAtom* GetTargets(int32_t aWhichClipboard,
+ int* aTargetNum) = 0;
+
Index: pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboardX11.cpp
diff -u /dev/null pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboardX11.cpp:1.1
--- /dev/null Sun Apr 29 13:27:08 2018
+++ pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboardX11.cpp Sun Apr 29 13:27:08 2018
@@ -0,0 +1,264 @@
+$NetBSD: patch-widget_gtk_nsClipboardX11.cpp,v 1.1 2018/04/29 13:27:08 martin Exp $
+
+Upstream fixes for Bug 1447925 -
+ Add GetClipboardText() to get text data from clipboard
+ Use GetClipboardText() to get text data at nsClipboard::GetData()
+
+--- widget/gtk/nsClipboardX11.cpp.orig 2018-04-29 12:56:52.405706053 +0000
++++ widget/gtk/nsClipboardX11.cpp
+@@ -62,8 +62,10 @@ selection_request_filter(GdkXEvent *gdk_
+
+ nsRetrievalContextX11::nsRetrievalContextX11()
+ : mState(INITIAL)
+- , mData(nullptr)
+ , mClipboardRequestNumber(0)
++ , mClipboardData(nullptr)
++ , mClipboardDataLength(0)
++ , mTargetMIMEType(gdk_atom_intern("TARGETS", FALSE))
+ {
+ // A custom event filter to workaround attempting to dereference a null
+ // selection requestor in GTK3 versions before 3.11.3. See bug 1178799.
+@@ -139,13 +141,11 @@ checkEventProc(Display *display, XEvent
+ return False;
+ }
+
+-void *
+-nsRetrievalContextX11::Wait()
++bool
++nsRetrievalContextX11::WaitForX11Content()
+ {
+ if (mState == COMPLETED) { // the request completed synchronously
+- void *data = mData;
+- mData = nullptr;
+- return data;
++ return true;
+ }
+
+ GdkDisplay *gdkDisplay = gdk_display_get_default();
+@@ -181,9 +181,7 @@ nsRetrievalContextX11::Wait()
+ DispatchPropertyNotifyEvent(context.cbWidget, &xevent);
+
+ if (mState == COMPLETED) {
+- void *data = mData;
+- mData = nullptr;
+- return data;
++ return true;
+ }
+ }
+
+@@ -200,11 +198,12 @@ nsRetrievalContextX11::Wait()
+ printf("exceeded clipboard timeout\n");
+ #endif
+ mState = TIMED_OUT;
+- return nullptr;
++ return false;
+ }
+
+ // Call this when data has been retrieved.
+-void nsRetrievalContextX11::Complete(GtkSelectionData* aData,
++void nsRetrievalContextX11::Complete(ClipboardDataType aDataType,
++ const void* aData,
+ int aDataRequestNumber)
+ {
+ if (mClipboardRequestNumber != aDataRequestNumber) {
+@@ -214,8 +213,54 @@ void nsRetrievalContextX11::Complete(Gtk
+
+ if (mState == INITIAL) {
+ mState = COMPLETED;
+- mData = gtk_selection_data_get_length(aData) >= 0 ?
+- gtk_selection_data_copy(aData) : nullptr;
++
++ MOZ_ASSERT(mClipboardData == nullptr &&
++ mClipboardDataLength == 0,
++ "We're leaking clipboard data!");
++
++ switch (aDataType) {
++ case CLIPBOARD_TEXT:
++ {
++ const char* text = static_cast<const char*>(aData);
++ if (text) {
++ mClipboardDataLength = sizeof(char) * (strlen(text) + 1);
++ mClipboardData = moz_xmalloc(mClipboardDataLength);
++ memcpy(mClipboardData, text, mClipboardDataLength);
++ }
++ }
++ break;
++ case CLIPBOARD_TARGETS:
++ {
++ const GtkSelectionData *selection =
++ static_cast<const GtkSelectionData *>(aData);
++
++ gint n_targets = 0;
++ GdkAtom *targets = nullptr;
++
++ if (!gtk_selection_data_get_targets(selection, &targets, &n_targets) ||
++ !n_targets) {
++ return;
++ }
++
++ mClipboardData = targets;
++ mClipboardDataLength = n_targets;
++ }
++ break;
++ case CLIPBOARD_DATA:
++ {
++ const GtkSelectionData *selection =
++ static_cast<const GtkSelectionData *>(aData);
++
++ gint dataLength = gtk_selection_data_get_length(selection);
++ if (dataLength > 0) {
++ mClipboardDataLength = dataLength;
++ mClipboardData = moz_xmalloc(dataLength);
++ memcpy(mClipboardData, gtk_selection_data_get_data(selection),
++ dataLength);
++ }
++ }
++ break;
++ }
+ } else {
+ // Already timed out
+ MOZ_ASSERT(mState == TIMED_OUT);
+@@ -233,12 +278,24 @@ clipboard_contents_received(GtkClipboard
+ delete handler;
+ }
+
+-GtkSelectionData*
+-nsRetrievalContextX11::WaitForContents(GtkClipboard *clipboard,
+- const char *aMimeType)
++static void
++clipboard_text_received(GtkClipboard *clipboard,
++ const gchar *text,
++ gpointer data)
++{
++ ClipboardRequestHandler *handler =
++ static_cast<ClipboardRequestHandler*>(data);
++ handler->Complete(text);
++ delete handler;
++}
++
++bool
++nsRetrievalContextX11::WaitForClipboardData(ClipboardDataType aDataType,
++ GtkClipboard *clipboard,
++ const char *aMimeType)
+ {
+ mState = INITIAL;
+- NS_ASSERTION(!mData, "Leaking clipboard content!");
++ NS_ASSERTION(!mClipboardData, "Leaking clipboard content!");
+
+ // Call ClipboardRequestHandler() with unique clipboard request number.
+ // The request number pairs gtk_clipboard_request_contents() data request
+@@ -246,39 +303,45 @@ nsRetrievalContextX11::WaitForContents(G
+ // is provided by Gtk.
+ mClipboardRequestNumber++;
+ ClipboardRequestHandler* handler =
+- new ClipboardRequestHandler(this, mClipboardRequestNumber);
++ new ClipboardRequestHandler(this, aDataType, mClipboardRequestNumber);
++
++ switch (aDataType) {
++ case CLIPBOARD_DATA:
++ gtk_clipboard_request_contents(clipboard,
++ gdk_atom_intern(aMimeType, FALSE), clipboard_contents_received,
++ handler);
++ break;
++ case CLIPBOARD_TEXT:
++ gtk_clipboard_request_text(clipboard, clipboard_text_received,
++ handler);
++ break;
++ case CLIPBOARD_TARGETS:
++ gtk_clipboard_request_contents(clipboard,
++ mTargetMIMEType, clipboard_contents_received,
++ handler);
++ break;
++ }
+
+- gtk_clipboard_request_contents(clipboard,
+- gdk_atom_intern(aMimeType, FALSE),
+- clipboard_contents_received,
+- handler);
+- return static_cast<GtkSelectionData*>(Wait());
++ return WaitForX11Content();
+ }
+
+ GdkAtom*
+ nsRetrievalContextX11::GetTargets(int32_t aWhichClipboard, int* aTargetNums)
+ {
+- *aTargetNums = 0;
+-
+- GtkClipboard *clipboard =
+- gtk_clipboard_get(GetSelectionAtom(aWhichClipboard));
+-
+- GtkSelectionData *selection_data = WaitForContents(clipboard, "TARGETS");
+- if (!selection_data)
+- return nullptr;
++ GtkClipboard *clipboard =
++ gtk_clipboard_get(GetSelectionAtom(aWhichClipboard));
+
+- gint n_targets = 0;
+- GdkAtom *targets = nullptr;
++ if (!WaitForClipboardData(CLIPBOARD_TARGETS, clipboard))
++ return nullptr;
+
+- if (!gtk_selection_data_get_targets(selection_data, &targets, &n_targets) ||
+- !n_targets) {
+- return nullptr;
+- }
++ *aTargetNums = mClipboardDataLength;
++ GdkAtom* targets = static_cast<GdkAtom*>(mClipboardData);
+
+- gtk_selection_data_free(selection_data);
++ // We don't hold the target list internally but we transfer the ownership.
++ mClipboardData = nullptr;
++ mClipboardDataLength = 0;
+
+- *aTargetNums = n_targets;
+- return targets;
++ return targets;
+ }
+
+ const char*
+@@ -289,25 +352,31 @@ nsRetrievalContextX11::GetClipboardData(
+ GtkClipboard *clipboard;
+ clipboard = gtk_clipboard_get(GetSelectionAtom(aWhichClipboard));
+
+- GtkSelectionData *selectionData = WaitForContents(clipboard, aMimeType);
+- if (!selectionData)
++ if (!WaitForClipboardData(CLIPBOARD_DATA, clipboard, aMimeType))
+ return nullptr;
+
+- char* clipboardData = nullptr;
+- int contentLength = gtk_selection_data_get_length(selectionData);
+- if (contentLength > 0) {
+- clipboardData = reinterpret_cast<char*>(
+- moz_xmalloc(sizeof(char)*contentLength));
+- memcpy(clipboardData, gtk_selection_data_get_data(selectionData),
+- sizeof(char)*contentLength);
+- }
+- gtk_selection_data_free(selectionData);
++ *aContentLength = mClipboardDataLength;
++ return static_cast<const char*>(mClipboardData);
++}
++
++const char*
++nsRetrievalContextX11::GetClipboardText(int32_t aWhichClipboard)
++{
++ GtkClipboard *clipboard;
++ clipboard = gtk_clipboard_get(GetSelectionAtom(aWhichClipboard));
+
+- *aContentLength = contentLength;
+- return (const char*)clipboardData;
++ if (!WaitForClipboardData(CLIPBOARD_TEXT, clipboard))
++ return nullptr;
++
++ return static_cast<const char*>(mClipboardData);
+ }
+
+ void nsRetrievalContextX11::ReleaseClipboardData(const char* aClipboardData)
+ {
+- free((void *)aClipboardData);
++ NS_ASSERTION(aClipboardData == mClipboardData,
++ "Releasing unknown clipboard data!");
++ free((void*)aClipboardData);
++
++ mClipboardData = nullptr;
++ mClipboardDataLength = 0;
+ }
Index: pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboardX11.h
diff -u /dev/null pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboardX11.h:1.1
--- /dev/null Sun Apr 29 13:27:08 2018
+++ pkgsrc/www/firefox/patches/patch-widget_gtk_nsClipboardX11.h Sun Apr 29 13:27:08 2018
@@ -0,0 +1,92 @@
+$NetBSD: patch-widget_gtk_nsClipboardX11.h,v 1.1 2018/04/29 13:27:08 martin Exp $
+
+Upstream fixes for Bug 1447925 -
+ Add GetClipboardText() to get text data from clipboard
+ Use GetClipboardText() to get text data at nsClipboard::GetData()
+
+--- widget/gtk/nsClipboardX11.h.orig 2018-04-29 12:56:52.405751519 +0000
++++ widget/gtk/nsClipboardX11.h
+@@ -11,6 +11,12 @@
+ #include "nsIClipboard.h"
+ #include <gtk/gtk.h>
+
++enum ClipboardDataType {
++ CLIPBOARD_DATA,
++ CLIPBOARD_TEXT,
++ CLIPBOARD_TARGETS
++};
++
+ class nsRetrievalContextX11 : public nsRetrievalContext
+ {
+ public:
+@@ -19,48 +25,59 @@ public:
+ virtual const char* GetClipboardData(const char* aMimeType,
+ int32_t aWhichClipboard,
+ uint32_t* aContentLength) override;
++ virtual const char* GetClipboardText(int32_t aWhichClipboard) override;
+ virtual void ReleaseClipboardData(const char* aClipboardData) override;
+
+ virtual GdkAtom* GetTargets(int32_t aWhichClipboard,
+ int* aTargetNums) override;
+
+- // Call this when data has been retrieved.
+- void Complete(GtkSelectionData* aData, int aDataRequestNumber);
++ // Call this when data or text has been retrieved.
++ void Complete(ClipboardDataType aDataType,
++ const void* aData,
++ int aDataRequestNumber);
+
+ nsRetrievalContextX11();
+ virtual ~nsRetrievalContextX11() override;
+
+ private:
+- GtkSelectionData* WaitForContents(GtkClipboard *clipboard,
+- const char *aMimeType);
++ bool WaitForClipboardData(ClipboardDataType aDataType,
++ GtkClipboard *clipboard,
++ const char *aMimeType = nullptr);
++
+ /**
+ * Spins X event loop until timing out or being completed. Returns
+ * null if we time out, otherwise returns the completed data (passing
+ * ownership to caller).
+ */
+- void *Wait();
++ bool WaitForX11Content();
+
+- State mState;
+- void* mData;
+- int mClipboardRequestNumber;
++ State mState;
++ int mClipboardRequestNumber;
++ void* mClipboardData;
++ uint32_t mClipboardDataLength;
++ GdkAtom mTargetMIMEType;
+ };
+
+ class ClipboardRequestHandler
+ {
+ public:
+- ClipboardRequestHandler(nsRetrievalContextX11 *aContext, int aDataRequestNumber)
++ ClipboardRequestHandler(nsRetrievalContextX11 *aContext,
++ ClipboardDataType aDataType,
++ int aDataRequestNumber)
+ : mContext(aContext)
+ , mDataRequestNumber(aDataRequestNumber)
++ , mDataType(aDataType)
+ {}
+
+- void Complete(GtkSelectionData* aData)
++ void Complete(const void *aData)
+ {
+- mContext->Complete(aData, mDataRequestNumber);
++ mContext->Complete(mDataType, aData, mDataRequestNumber);
+ }
+
+ private:
+ nsRetrievalContextX11 *mContext;
+ int mDataRequestNumber;
++ ClipboardDataType mDataType;
+ };
+
+ #endif /* __nsClipboardX11_h_ */
Home |
Main Index |
Thread Index |
Old Index