After Dragging the file, only file name is returned

[sgaitonde]

We are using cef 110 build on mac and windows for a desktop in a C++ app. On mac, After we receive the call back, when we run the GetFileNames() on the dragData (dragData->GetFileNames(fileNames)), we are getting only the file name but not the path. On windows we get the full path.

Code: Select all

void OnDragEnter(CefRefPtr<CefBrowser> browser, CefRefPtr<CefDragData> dragData, CefDragHandler::DragOperationsMask mask)
{
       if(dragData)
       {
              dragData->GetFileNames(fileNames);
              for(const auto& fileName : fileNames)
              {
                        std::cout<< fileName;
              }
       }
}

Same piece of code prints full path in windows and in mac only file with extention is printed. We need to get the full path of the file dragged into the browser.

Any help will be appreciated

[magreenblatt]

The code is here, I suggest stepping through with a debugger to see what's going on.

[ndesktop]

Maybe content::DropData display_name on Windows is empty and it returns path, while on macOS is not empty and therefore it returns the display name?

[sgaitonde]

Maybe content::DropData display_name on Windows is empty and it returns path, while on macOS is not empty and therefore it returns the display name?

Yes this is correct. Display_name is file name in Mac so we get only file name. In windows it's empty and so we get path. This needs to be fixed I think

[ndesktop]

Maybe adding a new method, GetFilePaths, which will return path instead of display_name.

[magreenblatt]

Maybe adding a new method, GetFilePaths, which will return path instead of display_name.

Agreed. GetFileNames() should probably also only return the file name component (not the complete path).

[sgaitonde]

Maybe adding a new method, GetFilePaths, which will return path instead of display_name.

Agreed. GetFileNames() should probably also only return the file name component (not the complete path).

Thanks. Let us know when that API would be available. Or any alternative where we can get path currently. We need to open that file into a stream for further processing.

[magreenblatt]

Maybe adding a new method, GetFilePaths, which will return path instead of display_name.

Agreed. GetFileNames() should probably also only return the file name component (not the complete path).

Thanks. Let us know when that API would be available. Or any alternative where we can get path currently. We need to open that file into a stream for further processing.

You can start by adding a feature request a https://github.com/chromiumembedded/cef/issues. A pull request implementing the necessary changes would also be welcome.

[ndesktop]

Feature request added.
Will try to create the PR in the following days.

[magreenblatt]

Feature request added.
Will try to create the PR in the following days.

Thanks