And use the editor from the DragController instead.
Created attachment 238117 [details] Patch
Created attachment 238239 [details] Updated patch Rebased, it should apply now
Created attachment 238240 [details] Try to fix windows build
Created attachment 238243 [details] Try to fix EFL build
Ping :-)
Comment on attachment 238243 [details] Try to fix EFL build View in context: https://bugs.webkit.org/attachment.cgi?id=238243&action=review A good step in the right direction. I suspect we can keep refactoring this to make the code even tighter over time. > Source/WebCore/editing/win/EditorWin.cpp:51 > +template <typename PlatformDragData> > +static PassRefPtr<DocumentFragment> createFragmentFromPlatformData(PlatformDragData* platformDragData, Frame& frame) Should take PlatformDragData& instead of PlatformDragData*. I’m also surprised this needs to be a template. Are all four functions called here really overloaded for two different types!?
(In reply to comment #6) > (From update of attachment 238243 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=238243&action=review > > A good step in the right direction. I suspect we can keep refactoring this to make the code even tighter over time. Thanks for the review. > > Source/WebCore/editing/win/EditorWin.cpp:51 > > +template <typename PlatformDragData> > > +static PassRefPtr<DocumentFragment> createFragmentFromPlatformData(PlatformDragData* platformDragData, Frame& frame) > > Should take PlatformDragData& instead of PlatformDragData*. Sure. > I’m also surprised this needs to be a template. Are all four functions called here really overloaded for two different types!? Yes.
Created attachment 239271 [details] Patch for landing Not sure the windows changes are correct...
Committed r174314: <http://trac.webkit.org/changeset/174314>