[EFL][WK1] Do not include libsoup header in the public headers
authorryuan.choi@samsung.com <ryuan.choi@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Feb 2014 13:25:27 +0000 (13:25 +0000)
committerryuan.choi@samsung.com <ryuan.choi@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Feb 2014 13:25:27 +0000 (13:25 +0000)
https://bugs.webkit.org/show_bug.cgi?id=128701

Reviewed by Gyuyoung Kim.

Although ewebkit used libsoup internally, it's not good for EFL developers
to use it.
Instead, we should provide some wrapper APIs like ewk_network_proxy_uri_set
for future changes.

* ewk/ewk_network.cpp: Removed soup related header,APIs and comments.
(ewk_network_tls_ca_certificates_path_set):
* ewk/ewk_network.h: Ditto.
* ewk/ewk_view.cpp: Ditto.
* ewk/ewk_view.h: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@164028 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/efl/ChangeLog
Source/WebKit/efl/ewk/ewk_network.cpp
Source/WebKit/efl/ewk/ewk_network.h
Source/WebKit/efl/ewk/ewk_view.cpp
Source/WebKit/efl/ewk/ewk_view.h

index b0b8636..680c76a 100644 (file)
@@ -1,3 +1,21 @@
+2014-02-13  Ryuan Choi  <ryuan.choi@samsung.com>
+
+        [EFL][WK1] Do not include libsoup header in the public headers
+        https://bugs.webkit.org/show_bug.cgi?id=128701
+
+        Reviewed by Gyuyoung Kim.
+
+        Although ewebkit used libsoup internally, it's not good for EFL developers
+        to use it.
+        Instead, we should provide some wrapper APIs like ewk_network_proxy_uri_set
+        for future changes.
+
+        * ewk/ewk_network.cpp: Removed soup related header,APIs and comments.
+        (ewk_network_tls_ca_certificates_path_set):
+        * ewk/ewk_network.h: Ditto.
+        * ewk/ewk_view.cpp: Ditto.
+        * ewk/ewk_view.h: Ditto.
+
 2014-02-12  Raphael Kubo da Costa  <raphael.kubo.da.costa@intel.com>
 
         Update the HTML Media Capture implementation.
index 8f4c049..a7c21f6 100644 (file)
@@ -74,8 +74,3 @@ void ewk_network_tls_ca_certificates_path_set(const char* bundlePath)
     SoupSession* defaultSession = WebCore::SoupNetworkSession::defaultSession().soupSession();
     g_object_set(defaultSession, "ssl-ca-file", bundlePath, NULL);
 }
-
-SoupSession* ewk_network_default_soup_session_get()
-{
-    return WebCore::SoupNetworkSession::defaultSession().soupSession();
-}
index d5c5a9f..a0add0e 100644 (file)
@@ -26,7 +26,6 @@
 #define ewk_network_h
 
 #include <Eina.h>
-#include <libsoup/soup.h>
 
 #ifdef __cplusplus
 extern "C" {
@@ -36,11 +35,6 @@ extern "C" {
  * Sets the given proxy URI to network backend.
  *
  * @param proxy URI to set
- *
- * @note If the libsoup backend is being used, this function has effect on
- * the @b default SoupSession, returned by ewk_network_default_soup_session_get().
- * If a different SoupSession is used and passed to ewk_view_soup_session_set(),
- * this function will not have any effect on it.
  */
 EAPI void             ewk_network_proxy_uri_set(const char *proxy);
 
@@ -50,11 +44,6 @@ EAPI void             ewk_network_proxy_uri_set(const char *proxy);
  * The returned string should be freed by eina_stringshare_del() after use.
  *
  * @return current proxy URI or @c NULL if it's not set
- *
- * @note If the libsoup backend is being used, this function has effect on
- * the @b default SoupSession, returned by ewk_network_default_soup_session_get().
- * If a different SoupSession is used and passed to ewk_view_soup_session_set(),
- * this function will not have any effect on it.
  */
 EAPI const char      *ewk_network_proxy_uri_get(void);
 
@@ -64,11 +53,6 @@ EAPI const char      *ewk_network_proxy_uri_get(void);
  * By default, HTTPS connections are performed regardless of the validity of the certificate provided.
  *
  * @sa ewk_network_tls_ca_certificates_path_set
- *
- * @note If the libsoup backend is being used, this function has effect on
- * the @b default SoupSession, returned by ewk_network_default_soup_session_get().
- * If a different SoupSession is used and passed to ewk_view_soup_session_set(),
- * this function will not have any effect on it.
  */
 EAPI Eina_Bool        ewk_network_tls_certificate_check_get(void);
 
@@ -80,11 +64,6 @@ EAPI Eina_Bool        ewk_network_tls_certificate_check_get(void);
  * @param enable Whether to check the provided certificates or not.
  *
  * @sa ewk_network_tls_ca_certificates_path_set
- *
- * @note If the libsoup backend is being used, this function has effect on
- * the @b default SoupSession, returned by ewk_network_default_soup_session_get().
- * If a different SoupSession is used and passed to ewk_view_soup_session_set(),
- * this function will not have any effect on it.
  */
 EAPI void             ewk_network_tls_certificate_check_set(Eina_Bool enable);
 
@@ -101,11 +80,6 @@ EAPI void             ewk_network_tls_certificate_check_set(Eina_Bool enable);
  * By default, the path is not set, so all certificates are considered as not signed by a trusted root CA.
  *
  * @sa ewk_network_tls_certificate_check_set
- *
- * @note If the libsoup backend is being used, this function has effect on
- * the @b default SoupSession, returned by ewk_network_default_soup_session_get().
- * If a different SoupSession is used and passed to ewk_view_soup_session_set(),
- * this function will not have any effect on it.
  */
 EAPI const char      *ewk_network_tls_ca_certificates_path_get(void);
 
@@ -124,21 +98,9 @@ EAPI const char      *ewk_network_tls_ca_certificates_path_get(void);
  * @param path The path to the certificate bundle.
  *
  * @sa ewk_network_tls_certificate_check_set
- *
- * @note If the libsoup backend is being used, this function has effect on
- * the @b default SoupSession, returned by ewk_network_default_soup_session_get().
- * If a different SoupSession is used and passed to ewk_view_soup_session_set(),
- * this function will not have any effect on it.
  */
 EAPI void             ewk_network_tls_ca_certificates_path_set(const char *path);
 
-/**
- * Returns the default @c SoupSession used by all views.
- *
- * @return The default @c SoupSession in use.
- */
-EAPI SoupSession     *ewk_network_default_soup_session_get(void);
-
 #ifdef __cplusplus
 }
 #endif
index 0aac87b..e01fde4 100644 (file)
@@ -87,7 +87,6 @@
 #include <Evas.h>
 #include <eina_safety_checks.h>
 #include <inttypes.h>
-#include <libsoup/soup.h>
 #include <limits>
 #include <math.h>
 #include <sys/time.h>
@@ -4486,25 +4485,6 @@ Ewk_Page_Visibility_State ewk_view_visibility_state_get(const Evas_Object* ewkVi
     return EWK_PAGE_VISIBILITY_STATE_VISIBLE;
 }
 
-SoupSession* ewk_view_soup_session_get(const Evas_Object* ewkView)
-{
-    EWK_VIEW_SD_GET_OR_RETURN(ewkView, smartData, 0);
-    EWK_VIEW_PRIV_GET_OR_RETURN(smartData, priv, 0);
-    return WebCore::NetworkStorageSession::defaultStorageSession().soupNetworkSession().soupSession();
-}
-
-void ewk_view_soup_session_set(Evas_Object* ewkView, SoupSession* session)
-{
-    EWK_VIEW_SD_GET_OR_RETURN(ewkView, smartData);
-    EWK_VIEW_PRIV_GET_OR_RETURN(smartData, priv);
-    if (!SOUP_IS_SESSION_ASYNC(session)) {
-        ERR("WebKit requires an SoupSessionAsync to work properly, but "
-            "a SoupSessionSync was provided.");
-        return;
-    }
-    WebCore::NetworkStorageSession::defaultStorageSession().setSoupNetworkSession(WebCore::SoupNetworkSession::createForSoupSession(session));
-}
-
 Eina_Bool ewk_view_setting_enable_xss_auditor_get(const Evas_Object* ewkView)
 {
     EWK_VIEW_SD_GET_OR_RETURN(ewkView, smartData, false);
index 2ba3759..7638933 100644 (file)
 
 #include <Evas.h>
 #include <cairo.h>
-#include <libsoup/soup.h>
 
 #ifdef __cplusplus
 extern "C" {
@@ -2350,34 +2349,6 @@ EAPI Eina_Bool ewk_view_mixed_content_displayed_get(const Evas_Object *o);
 EAPI Eina_Bool ewk_view_mixed_content_run_get(const Evas_Object *o);
 
 /**
- * Returns the SoupSession associated with this view.
- *
- * By default, all views share the same, default soup session also available
- * by calling ewk_network_default_soup_session_get.
- *
- * @param o The view to query.
- *
- * @sa ewk_view_soup_session_set, ewk_network_default_soup_session_get
- */
-EAPI SoupSession* ewk_view_soup_session_get(const Evas_Object *o);
-
-/**
- * Associates a new SoupSession with this view.
- *
- * Only sessions of type SoupSessionAsync are supported.
- *
- * @note Changing the SoupSession should not be needed in most cases. If
- *       a different SoupSession is used, the cookie management and some
- *       network functions in ewk will not have any effect on it.
- *
- * @param o The view to change.
- * @param session The new SoupSession.
- *
- * @sa ewk_view_soup_session_get, ewk_network_default_soup_session_get
- */
-EAPI void ewk_view_soup_session_set(Evas_Object *o, SoupSession *session);
-
-/**
  * Returns whether XSSAuditor feature is enabled.
  *
  * @param o view object to query whether XSSAuditor feature is enabled.