Fix missing enum in webview code for MinGW

This notably fixes building wxWebView with MinGW version used by AppVeyor.

See https://github.com/wxWidgets/wxWidgets/pull/894
This commit is contained in:
VZ
2018-08-23 14:58:50 +02:00
committed by GitHub
6 changed files with 44 additions and 23 deletions

View File

@@ -259,7 +259,7 @@ public:
class ClassFactory : public IClassFactory class ClassFactory : public IClassFactory
{ {
public: public:
ClassFactory(wxSharedPtr<wxWebViewHandler> handler) : m_handler(handler) ClassFactory(wxSharedPtr<wxWebViewHandler> handler) : m_handler(handler)
{ AddRef(); } { AddRef(); }
virtual ~ClassFactory() {} virtual ~ClassFactory() {}

View File

@@ -297,8 +297,8 @@ typedef enum _tagwxELEMENT_TAG_ID
{ {
wxTAGID_NULL = 0, wxTAGID_UNKNOWN = 1, wxTAGID_A = 2, wxTAGID_ACRONYM = 3, wxTAGID_NULL = 0, wxTAGID_UNKNOWN = 1, wxTAGID_A = 2, wxTAGID_ACRONYM = 3,
wxTAGID_ADDRESS = 4, wxTAGID_APPLET = 5, wxTAGID_AREA = 6, wxTAGID_B = 7, wxTAGID_ADDRESS = 4, wxTAGID_APPLET = 5, wxTAGID_AREA = 6, wxTAGID_B = 7,
wxTAGID_BASE = 8, wxTAGID_BASEFONT = 9, wxTAGID_BDO = 10, wxTAGID_BASE = 8, wxTAGID_BASEFONT = 9, wxTAGID_BDO = 10,
wxTAGID_BGSOUND = 11, wxTAGID_BIG = 12, wxTAGID_BLINK = 13, wxTAGID_BGSOUND = 11, wxTAGID_BIG = 12, wxTAGID_BLINK = 13,
wxTAGID_BLOCKQUOTE = 14, wxTAGID_BODY = 15, wxTAGID_BR = 16, wxTAGID_BLOCKQUOTE = 14, wxTAGID_BODY = 15, wxTAGID_BR = 16,
wxTAGID_BUTTON = 17, wxTAGID_CAPTION = 18, wxTAGID_CENTER = 19, wxTAGID_BUTTON = 17, wxTAGID_CAPTION = 18, wxTAGID_CENTER = 19,
wxTAGID_CITE = 20, wxTAGID_CODE = 21, wxTAGID_COL = 22, wxTAGID_CITE = 20, wxTAGID_CODE = 21, wxTAGID_COL = 22,
@@ -945,3 +945,24 @@ public:
/* end of WinInet.h */ /* end of WinInet.h */
/* ShObjldl_core.h */
typedef enum _wxNWMF
{
wxNWMF_UNLOADING = 0x1,
wxNWMF_USERINITED = 0x2,
wxNWMF_FIRST = 0x4,
wxNWMF_OVERRIDEKEY = 0x8,
wxNWMF_SHOWHELP = 0x10,
wxNWMF_HTMLDIALOG = 0x20,
wxNWMF_FROMDIALOGCHILD = 0x40,
wxNWMF_USERREQUESTED = 0x80,
wxNWMF_USERALLOWED = 0x100,
wxNWMF_FORCEWINDOW = 0x10000,
wxNWMF_FORCETAB = 0x20000,
wxNWMF_SUGGESTWINDOW = 0x40000,
wxNWMF_SUGGESTTAB = 0x80000,
wxNWMF_INACTIVETAB = 0x100000
} _wxNWMF;
/* end of ShObjldl_core.h */

View File

@@ -149,7 +149,7 @@ public:
long style = 0, long style = 0,
const wxString& name = wxWebViewNameStr); const wxString& name = wxWebViewNameStr);
static void RegisterFactory(const wxString& backend, static void RegisterFactory(const wxString& backend,
wxSharedPtr<wxWebViewFactory> factory); wxSharedPtr<wxWebViewFactory> factory);
// General methods // General methods

View File

@@ -17,7 +17,7 @@
#include "wx/webview.h" #include "wx/webview.h"
#if defined(__WXOSX__) #if defined(__WXOSX__)
#include "wx/osx/webview_webkit.h" #include "wx/osx/webview_webkit.h"
#elif defined(__WXGTK__) #elif defined(__WXGTK__)
#include "wx/gtk/webview_webkit.h" #include "wx/gtk/webview_webkit.h"
@@ -65,8 +65,8 @@ wxWebView* wxWebView::New(const wxString& backend)
// static // static
wxWebView* wxWebView::New(wxWindow* parent, wxWindowID id, const wxString& url, wxWebView* wxWebView::New(wxWindow* parent, wxWindowID id, const wxString& url,
const wxPoint& pos, const wxSize& size, const wxPoint& pos, const wxSize& size,
const wxString& backend, long style, const wxString& backend, long style,
const wxString& name) const wxString& name)
{ {
wxStringWebViewFactoryMap::iterator iter = FindFactory(backend); wxStringWebViewFactoryMap::iterator iter = FindFactory(backend);
@@ -79,13 +79,13 @@ wxWebView* wxWebView::New(wxWindow* parent, wxWindowID id, const wxString& url,
} }
// static // static
void wxWebView::RegisterFactory(const wxString& backend, void wxWebView::RegisterFactory(const wxString& backend,
wxSharedPtr<wxWebViewFactory> factory) wxSharedPtr<wxWebViewFactory> factory)
{ {
m_factoryMap[backend] = factory; m_factoryMap[backend] = factory;
} }
// static // static
wxStringWebViewFactoryMap::iterator wxWebView::FindFactory(const wxString &backend) wxStringWebViewFactoryMap::iterator wxWebView::FindFactory(const wxString &backend)
{ {
// Initialise the map, it checks internally for existing factories // Initialise the map, it checks internally for existing factories
@@ -93,7 +93,7 @@ wxStringWebViewFactoryMap::iterator wxWebView::FindFactory(const wxString &backe
return m_factoryMap.find(backend); return m_factoryMap.find(backend);
} }
// static // static
void wxWebView::InitFactoryMap() void wxWebView::InitFactoryMap()
{ {

View File

@@ -76,7 +76,7 @@ wxFSFile* wxWebViewArchiveHandler::GetFile(const wxString &uri)
} }
} }
//We do not have a protocol string so we just pass the path withouth the //We do not have a protocol string so we just pass the path withouth the
if(start == wxString::npos) if(start == wxString::npos)
{ {
size_t doubleslash = path.find("//"); size_t doubleslash = path.find("//");
@@ -84,7 +84,7 @@ wxFSFile* wxWebViewArchiveHandler::GetFile(const wxString &uri)
if(doubleslash == wxString::npos) if(doubleslash == wxString::npos)
return NULL; return NULL;
wxString fspath = "file:" + wxString fspath = "file:" +
EscapeFileNameCharsInURL(path.substr(doubleslash + 2).c_str()); EscapeFileNameCharsInURL(path.substr(doubleslash + 2).c_str());
return m_fileSystem->OpenFile(fspath); return m_fileSystem->OpenFile(fspath);
} }
@@ -107,7 +107,7 @@ wxFSFile* wxWebViewArchiveHandler::GetFile(const wxString &uri)
if(doubleslash == wxString::npos) if(doubleslash == wxString::npos)
return NULL; return NULL;
wxString fspath = "file:" + wxString fspath = "file:" +
EscapeFileNameCharsInURL(mainpath.substr(doubleslash + 2).c_str()) EscapeFileNameCharsInURL(mainpath.substr(doubleslash + 2).c_str())
+ "#" + protocol +":" + archivepath; + "#" + protocol +":" + archivepath;
return m_fileSystem->OpenFile(fspath); return m_fileSystem->OpenFile(fspath);

View File

@@ -134,7 +134,7 @@ wxWebViewIE::~wxWebViewIE()
for(unsigned int i = 0; i < m_factories.size(); i++) for(unsigned int i = 0; i < m_factories.size(); i++)
{ {
session->UnregisterNameSpace(m_factories[i], session->UnregisterNameSpace(m_factories[i],
(m_factories[i]->GetName()).wc_str()); (m_factories[i]->GetName()).wc_str());
m_factories[i]->Release(); m_factories[i]->Release();
} }
@@ -1030,7 +1030,7 @@ wxCOMPtr<IHTMLDocument2> wxWebViewIE::GetDocument() const
bool wxWebViewIE::IsElementVisible(wxCOMPtr<IHTMLElement> elm) bool wxWebViewIE::IsElementVisible(wxCOMPtr<IHTMLElement> elm)
{ {
wxCOMPtr<IHTMLElement> elm1 = elm; wxCOMPtr<IHTMLElement> elm1 = elm;
bool is_visible = true; bool is_visible = true;
//This method is not perfect but it does discover most of the hidden elements. //This method is not perfect but it does discover most of the hidden elements.
//so if a better solution is found, then please do improve. //so if a better solution is found, then please do improve.
@@ -1046,7 +1046,7 @@ bool wxWebViewIE::IsElementVisible(wxCOMPtr<IHTMLElement> elm)
wxBasicString visibility_bstr; wxBasicString visibility_bstr;
//Check if the object has the style display:none. //Check if the object has the style display:none.
if((style->get_display(display_bstr.ByRef()) != S_OK) || if((style->get_display(display_bstr.ByRef()) != S_OK) ||
wxString(display_bstr).IsSameAs(wxS("none"), false)) wxString(display_bstr).IsSameAs(wxS("none"), false))
{ {
is_visible = false; is_visible = false;
@@ -1092,7 +1092,7 @@ void wxWebViewIE::FindInternal(const wxString& text, int flags, int internal_fla
wxCOMPtr<wxIMarkupPointer> ptrBegin, ptrEnd; wxCOMPtr<wxIMarkupPointer> ptrBegin, ptrEnd;
wxBasicString attr_bstr(wxS("style=\"background-color:#ffff00\"")); wxBasicString attr_bstr(wxS("style=\"background-color:#ffff00\""));
wxBasicString text_bstr(text); wxBasicString text_bstr(text);
pIMS->CreateMarkupPointer(&ptrBegin); pIMS->CreateMarkupPointer(&ptrBegin);
pIMS->CreateMarkupPointer(&ptrEnd); pIMS->CreateMarkupPointer(&ptrEnd);
@@ -1477,7 +1477,7 @@ void wxWebViewIE::onActiveXEvent(wxActiveXEvent& evt)
wxWebViewNavigationActionFlags navFlags = wxWEBVIEW_NAV_ACTION_OTHER; wxWebViewNavigationActionFlags navFlags = wxWEBVIEW_NAV_ACTION_OTHER;
if(flags & NWMF_USERINITED || flags & NWMF_USERREQUESTED) if(flags & wxNWMF_USERINITED || flags & wxNWMF_USERREQUESTED)
navFlags = wxWEBVIEW_NAV_ACTION_USER; navFlags = wxWEBVIEW_NAV_ACTION_USER;
wxWebViewEvent event(wxEVT_WEBVIEW_NEWWINDOW, wxWebViewEvent event(wxEVT_WEBVIEW_NEWWINDOW,
@@ -1633,10 +1633,10 @@ HRESULT wxSTDCALL DocHostUIHandler::ShowContextMenu(DWORD dwID, POINT *ppt,
wxUnusedVar(ppt); wxUnusedVar(ppt);
wxUnusedVar(pcmdtReserved); wxUnusedVar(pcmdtReserved);
wxUnusedVar(pdispReserved); wxUnusedVar(pdispReserved);
if(m_browser->IsContextMenuEnabled()) if(m_browser->IsContextMenuEnabled())
return E_NOTIMPL; return E_NOTIMPL;
else else
return S_OK; return S_OK;
} }
HRESULT wxSTDCALL DocHostUIHandler::GetHostInfo(DOCHOSTUIINFO *pInfo) HRESULT wxSTDCALL DocHostUIHandler::GetHostInfo(DOCHOSTUIINFO *pInfo)
@@ -1706,7 +1706,7 @@ HRESULT wxSTDCALL DocHostUIHandler::TranslateAccelerator(LPMSG lpMsg,
{ {
// check control is down but that it isn't right-alt which is mapped to // check control is down but that it isn't right-alt which is mapped to
// alt + ctrl // alt + ctrl
if(GetKeyState(VK_CONTROL) & 0x8000 && if(GetKeyState(VK_CONTROL) & 0x8000 &&
!(GetKeyState(VK_MENU) & 0x8000)) !(GetKeyState(VK_MENU) & 0x8000))
{ {
//skip the accelerators used by the control //skip the accelerators used by the control