Fix drag_status call to work on fc24/23, fixes #4949

[samdroid-apps]

I believe that Gdk.drag_status (new function) has been here forever (tested fc25, fc22), since it doesn't have an "new since" section in the docs [1].

[1] https://developer.gnome.org/gdk3/unstable/gdk3-Drag-and-Drop.html#gdk-drag-status

[quozl]

Applied on Fedora 18. How to test? (e.g. how to execute this code path?)

[samdroid-apps]

Steps to reproduce:

1. Open browse
2. Start dragging SL logo
3. Move cursor to top left to invoke frame

Expected: Frame shows

[quozl]

Thanks, that works; can now copy images to buffer in frame, could not before this patch.

Cherry-picked for my OLPC branch.

[samdroid-apps]

Please merge if you have reviewed and tested the patch.

On Thu, Mar 10, 2016 at 4:03 PM, James Cameron
notifications@github.com wrote:

Thanks, that works; can now copy images to buffer in frame, could not
before this patch.

Cherry-picked for my OLPC branch.

—
Reply to this email directly or view it on GitHub.

[quozl]

I'm not of the opinion that there is sufficient peer review. It's early yet. Let's see what other developers think.

[davelab6]

Who else is needed to review and merge this 1 line change? :)

[quozl]

@davelab6, other reviewers would be @godiard, @tchx84, or some of the other people regularly involved in pull requests on this repository. Of the last ten merged pull requests, nine of them were merged by @samdroid-apps, and one by @godiard.