Added support for menu list popup delegate using our platform plugin in WebKit2.
Created attachment 71160 [details] patch
Comment on attachment 71160 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=71160&action=review So with this patch it works with webkit1 and webkit2 without changes to the platform plugin? > WebKit2/UIProcess/qt/WebPopupMenuProxyQt.cpp:39 > + switch (m_items[idx].m_type) > + { { should be on the other line, no? > WebKit2/UIProcess/qt/WebPopupMenuProxyQt.h:58 > + SelectData() :m_multiple(false) { } space around :
(In reply to comment #2) > (From update of attachment 71160 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=71160&action=review > > So with this patch it works with webkit1 and webkit2 without changes to the platform plugin? Yes. No change was needed in the platform plugin. In this patch I did not implement the fallback popup. > > > WebKit2/UIProcess/qt/WebPopupMenuProxyQt.cpp:39 > > + switch (m_items[idx].m_type) > > + { > > { should be on the other line, no? > > > WebKit2/UIProcess/qt/WebPopupMenuProxyQt.h:58 > > + SelectData() :m_multiple(false) { } > > space around :
*** This bug has been marked as a duplicate of bug 62191 ***