Changeset 471 for trunk/src/gui/kernel/qmime_pm.cpp

Timestamp:

Jan 25, 2010, 5:03:17 PM (16 years ago)

Author:

Dmitry A. Kuminov

Message:

gui: DnD: Replaced r470 with a better solution (simpler and doesn't report a dummy mime type "" in the dropped mime data -- for compatibility with other platforms).

File:

• trunk/src/gui/kernel/qmime_pm.cpp (modified) (8 diffs)

trunk/src/gui/kernel/qmime_pm.cpp

@@ -365 +365 @@
         return formats;
 
-    if (qstrcmp(formats, "DRF_NULL") == 0) {
-        // special case, see QPMMimeAnyMime::dragWorkerFor()
-        formats = "<DRM_NULL," + formats + ">";
-    } else {
-        // DRM_SHAREDMEM comes first to prevent native DRM_OS2FILE
-        // rendering on the target side w/o involving the source.
-        // Also, we add <DRM_SHAREDMEM,DRF_POINTERDATA> just like WPS does it
-        // (however, it doesn't help when dropping objects to it -- WPS still
-        // chooses DRM_OS2FILE).
-        formats = "(DRM_SHAREDMEM,DRM_OS2FILE)x(" + formats + "),"
-                  "<DRM_SHAREDMEM,DRF_POINTERDATA>";
-    }
+    // DRM_SHAREDMEM comes first to prevent native DRM_OS2FILE
+    // rendering on the target side w/o involving the source.
+    // Also, we add <DRM_SHAREDMEM,DRF_POINTERDATA> just like WPS does it
+    // (however, it doesn't help when dropping objects to it -- WPS still
+    // chooses DRM_OS2FILE).
+    formats = "(DRM_SHAREDMEM,DRM_OS2FILE)x(" + formats + "),"
+              "<DRM_SHAREDMEM,DRF_POINTERDATA>";
 
     DEBUG(() << "DefaultDragWorker: formats" << formats
@@ -832 +827 @@
     if (!provider)
         return ret;
-
-    if (mimeType.isEmpty()) {
-        // special case, see QPMMimeAnyMime::dropWorkerFor()
-        return ret;
-    }
 
     QByteArray drf = provider->drf(mimeType);
@@ -1127 +1117 @@
                                              Provider *provider)
 {
-    // note: as a special case, mimeType may be null (see
-    // QPMCoopDragWorker::collectWorkers())
-
-    Q_ASSERT(provider);
-    if (provider && !d->exclusive) {
+    Q_ASSERT(!mimeType.isEmpty() && provider);
+    if (!mimeType.isEmpty() && provider && !d->exclusive) {
         // ensure there are no dups (several providers for the same mime)
         if (!d->providerFor(mimeType))
@@ -1304 +1291 @@
 {
     ULONG cf = WinAddAtom(WinQuerySystemAtomTable(), mime.toLocal8Bit());
+    if (!cf) {
 #ifndef QT_NO_DEBUG
-    if (!cf)
         qWarning("QPMMime: WinAddAtom failed with 0x%lX",
                  WinGetLastError(NULLHANDLE));
 #endif
+        return 0;
+    }
 
     return cf;
@@ -2734 +2723 @@
                                                QString &type, QString &ext)
 {
-    if (qstrcmp(drf, "DRF_NULL") == 0) {
-        // special case (see QPMMimeAnyMime::dragWorkerFor())
-        type = QString::null;
-    } else {
-        // file type = mime
-        type = format(drf);
-        Q_ASSERT(!type.isEmpty());
-    }
+    // file type = mime
+    type = format(drf);
+    Q_ASSERT(!type.isEmpty());
 
     // no way to determine the extension
@@ -2775 +2759 @@
                                                    QMimeData *mimeData)
 {
-    if (mimeType.isEmpty()) {
-        // special case: QMimeData with no formats() (see
-        // QPMCoopDragWorker::collectWorkers())
-        Q_ASSERT(!mimeData->formats().count());
-        // add an exclusive provider with a special format DRF_NULL. Note that
-        // any attemt to render this format should fail so we don't expect
-        // AnyDragProvider::format()/provide() to be actually called.
-        DefaultDragWorker *defWorker = defaultExclDragWorker();
-        defWorker->addProvider(QByteArray("DRF_NULL"), &anyDragProvider);
-        return defWorker;
-    }
-
     ULONG cf = cfMap.value(mimeType);
     if (!cf)
@@ -2816 +2788 @@
         DefaultDropWorker *defWorker = defaultDropWorker();
         bool atLeastOneSupported = false;
-
-        if (qstrcmp(queryHSTR(item->hstrRMF), "<DRM_NULL,DRF_NULL>") == 0) {
-            // special case: QMimeData with no formats() (see
-            // QPMCoopDragWorker::collectWorkers()), use a special MIME format
-            // value of null.  Note that any attemt to retrieve data in this
-            // format should fail so we don't expect AnyDropProvider::drf()/
-            // provide() to be actually called.
-            defWorker->addProvider(QString::null, &anyDropProvider);
-            return defWorker;
-        }
 
         // check that we support one of DRMs and the format is CF_hhhhhhh
@@ -2902 +2864 @@
     if (atomStr.startsWith(mimePrefix)) {
         // the format represents the mime type we can recognize
+        // increase the reference count
+        ULONG cf = QPMMime::registerMimeType(atomStr);
+        Q_ASSERT(cf == format);
         // extract the real mime type (w/o our prefix)
         mime = atomStr.mid(mimePrefix.size());
-        Q_ASSERT(!mime.isEmpty());
         if (!mime.isEmpty()) {
-            // increase the reference count (will decrease on destruction)
-            ULONG cf = QPMMime::registerMimeType(atomStr);
-            Q_ASSERT(cf == format);
             cfMap[mime] = cf;
             mimeMap[cf] = mime;