Remove semi-public wxWebSession::GetImpl()

It's better not to have this method in the public class, even if it
means that we need to pass a wxWebSessionImpl object to wxWebRequestImpl
ctor explicitly now.

No real changes.

include/wx/private/webrequest.h

@@ -107,7 +107,10 @@ protected:
     wxFileOffset m_dataSize;
     wxScopedPtr<wxInputStream> m_dataStream;
 
-    wxWebRequestImpl(wxWebSession& session, wxEvtHandler* handler, int id);
+    wxWebRequestImpl(wxWebSession& session,
+                     wxWebSessionImpl& sessionImpl,
+                     wxEvtHandler* handler,
+                     int id);
 
     // Call SetState() with either State_Failed or State_Completed appropriate
     // for the response status.

include/wx/webrequest.h

@@ -230,8 +230,6 @@ public:
 
     void Close();
 
-    wxWebSessionImpl* GetImpl() const { return m_impl.get(); }
-
 private:
     static void RegisterFactory(const wxString& backend,
                                 wxWebSessionFactory* factory);

src/common/webrequest.cpp

@@ -55,9 +55,12 @@ static const wxStringCharType* wxNO_IMPL_MSG
 //
 // wxWebRequestImpl
 //
-wxWebRequestImpl::wxWebRequestImpl(wxWebSession& session, wxEvtHandler* handler, int id)
+wxWebRequestImpl::wxWebRequestImpl(wxWebSession& session,
+                                   wxWebSessionImpl& sessionImpl,
+                                   wxEvtHandler* handler,
+                                   int id)
     : m_storage(wxWebRequest::Storage_Memory),
-      m_headers(session.GetImpl()->GetHeaders()),
+      m_headers(sessionImpl.GetHeaders()),
       m_dataSize(0),
       m_session(session),
       m_handler(handler),

src/common/webrequest_curl.cpp

@@ -156,7 +156,7 @@ wxWebRequestCURL::wxWebRequestCURL(wxWebSession & session,
                                    wxEvtHandler* handler,
                                    const wxString & url,
                                    int id):
-    wxWebRequestImpl(session, handler, id),
+    wxWebRequestImpl(session, sessionImpl, handler, id),
     m_sessionImpl(sessionImpl)
 {
     m_headerList = NULL;

src/msw/webrequest_winhttp.cpp

@@ -127,7 +127,7 @@ wxWebRequestWinHTTP::wxWebRequestWinHTTP(wxWebSession& session,
                                          wxEvtHandler* handler,
                                          const wxString& url,
                                          int id):
-    wxWebRequestImpl(session, handler, id),
+    wxWebRequestImpl(session, sessionImpl, handler, id),
     m_sessionImpl(sessionImpl),
     m_url(url),
     m_connect(NULL),

src/osx/webrequest_urlsession.mm

@@ -164,7 +164,7 @@ wxWebRequestURLSession::wxWebRequestURLSession(wxWebSession& session,
                                                wxEvtHandler* handler,
                                                const wxString& url,
                                                int winid):
-    wxWebRequestImpl(session, handler, winid),
+    wxWebRequestImpl(session, sessionImpl, handler, winid),
     m_sessionImpl(sessionImpl),
     m_url(url)
 {