Linux: Drag and Drop is not handled for Popup window

magreenblatt commented 6 years ago

Original report by Sorab Bisht (Bitbucket: sobisht, GitHub: sobisht).

What steps will reproduce the problem?

Method 1:

Open cefclient.
Open URL "https://www.w3schools.com/html/html5_draganddrop.asp" in application. And drag and drop items. Drag handler is triggered from this action.
Open DevTools and type window.open("https://www.w3schools.com/html/html5_draganddrop.asp") in console.
This will open a new pop window. Open url "https://www.w3schools.com/html/html5_draganddrop.asp" in the pop up window. Do drag and drop operation. No Drag handler is triggered.

Attached Window_open.mp4 for reference.

Method 2:
Open cefclient.
Open URL "https://www.w3schools.com/html/html5_draganddrop.asp" in application. And drag and drop items. Drag handler is triggered from this action.
Open Test->Popup Window.
This will open a new pop window. Open url "https://www.w3schools.com/html/html5_draganddrop.asp" in the pop up window. Do drag and drop operation. No Drag handler is triggered.

Attached PopupWindow.mp4 for reference.

What is the expected output? What do you see instead?

Expected: Drag handler should be invoked for pop window also.
Actual: No drag handler is invoked for the pop up window.

This issue happens in Linux only. In windows, the drag handler get called for the popup window also.

What version of the product are you using? On what operating system?

CEF version: 3.3359.1772 gd1df190

OS: Ubuntu 18.04 LTS 64 Bit

  Ubuntu 16.04 LTS 64 Bit

Does the problem reproduce with the cefclient or cefsimple sample application at the same version? How about with a newer or older version?

Yes, the problem is reproducible with cefclient for latest version.

Does the problem reproduce with Google Chrome at the same version? How about with a newer or older version?

No, it is not reprducible with Google Chrome.

magreenblatt commented 6 years ago

Original comment by Sorab Bisht (Bitbucket: sobisht, GitHub: sobisht).

GetDragHandler() function is not getting called for Popup window in case Linux. In case of windows, this function gets called for Popup window also.

magreenblatt commented 6 years ago

Does the problem reproduce with the cefsimple sample application?

magreenblatt commented 6 years ago

Original comment by Sorab Bisht (Bitbucket: sobisht, GitHub: sobisht).

For CEFSimple sample app in linux, we do not have option to open DevTools or Popup window. So the steps could not be performed to reproduce the issue.

magreenblatt commented 6 years ago

you can run cefsimple --url=https://www.w3schools.com/html/html5_draganddrop.asp and click the "Try it yourself" button further down the page.

magreenblatt commented 6 years ago

Original comment by Sorab Bisht (Bitbucket: sobisht, GitHub: sobisht).

Checked it on cefsimple. Here are the steps done for cefsimple:

Ran following code to open a window using window.open https://plnkr.co/edit/eL5kMqgHIWHedQrF7o0G?p=preview

#!html

<!DOCTYPE html>
<html>

  <head>
    <link rel="stylesheet" href="style.css">
    <script src="script.js"></script>
    <script>
      function openWindow() {
        window.open("https://www.w3schools.com/html/html5_draganddrop.asp")
      }
    </script>
  </head>

  <body>
    <h1>Hello Plunker!</h1>
    <a onclick="openWindow()">Open New</a>
  </body>

</html>

Drag and Drop was working on new window.

But tried the same steps on cefclient. Drag and drop did not work. And no message was triggered for the popup window.

magreenblatt commented 6 years ago

Since drag&drop works in the cefsimple application this is likely an issue with GTK integration in the cefclient sample application.

magreenblatt commented 6 years ago

Original changes by Sorab Bisht (Bitbucket: sobisht, GitHub: sobisht).

edited description

magreenblatt commented 6 years ago

changed priority from "major" to "minor"
changed component from "Unclassified" to "SampleApps"

chromiumembedded / cef