[GTK] Mark view source mode methods as deprecated
authorcarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 25 Feb 2014 07:51:14 +0000 (07:51 +0000)
committercarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 25 Feb 2014 07:51:14 +0000 (07:51 +0000)
https://bugs.webkit.org/show_bug.cgi?id=128920

Reviewed by Martin Robinson.

Source/WebKit/gtk:

* webkit/webkitwebview.cpp: Mark set/get_view_source_mode as
deprecated in the documentation.
* webkit/webkitwebview.h: Add deprecation guards.

Source/WebKit2:

Mark WEBKIT_VIEW_MODE_SOURCE as deprecated in documentation and
only show warnings when set_view_mode is used with WEBKIT_VIEW_MODE_SOURCE.

* UIProcess/API/gtk/WebKitWebView.cpp:
(webkit_web_view_set_view_mode):
(webkit_web_view_get_view_mode):
* UIProcess/API/gtk/WebKitWebView.h:

Tools:

* TestWebKitAPI/Tests/WebKit2Gtk/TestWebKitWebView.cpp:
(beforeAll): Remove view mode test.

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

Source/WebKit/gtk/ChangeLog
Source/WebKit/gtk/webkit/webkitwebview.cpp
Source/WebKit/gtk/webkit/webkitwebview.h
Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/API/gtk/WebKitWebView.cpp
Source/WebKit2/UIProcess/API/gtk/WebKitWebView.h
Tools/ChangeLog
Tools/TestWebKitAPI/Tests/WebKit2Gtk/TestWebKitWebView.cpp

index 21b90a2..174be97 100644 (file)
@@ -1,3 +1,14 @@
+2014-02-24  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        [GTK] Mark view source mode methods as deprecated
+        https://bugs.webkit.org/show_bug.cgi?id=128920
+
+        Reviewed by Martin Robinson.
+
+        * webkit/webkitwebview.cpp: Mark set/get_view_source_mode as
+        deprecated in the documentation.
+        * webkit/webkitwebview.h: Add deprecation guards.
+
 2014-02-24  Martin Robinson  <mrobinson@igalia.com>
 
         [GTK] generate-gtkdoc should not generate documentation for source files for unbuilt source files
index 1a2b017..bb082be 100644 (file)
@@ -5233,6 +5233,7 @@ void webkit_web_view_redo(WebKitWebView* webView)
  * nice and readable format.
  *
  * Since: 1.1.14
+ * Deprecated: 2.6
  */
 void webkit_web_view_set_view_source_mode (WebKitWebView* webView, gboolean mode)
 {
@@ -5248,6 +5249,7 @@ void webkit_web_view_set_view_source_mode (WebKitWebView* webView, gboolean mode
  * Return value: %TRUE if @web_view is in view source mode, %FALSE otherwise.
  *
  * Since: 1.1.14
+ * Deprecated: 2.6
  */
 gboolean webkit_web_view_get_view_source_mode (WebKitWebView* webView)
 {
index 5c09db6..12ea0d6 100644 (file)
@@ -415,12 +415,14 @@ webkit_web_view_redo                            (WebKitWebView        *web_view)
 WEBKIT_API gboolean
 webkit_web_view_can_redo                        (WebKitWebView        *web_view);
 
+#if !defined(WEBKIT_DISABLE_DEPRECATED)
 WEBKIT_API void
 webkit_web_view_set_view_source_mode            (WebKitWebView        *web_view,
                                                  gboolean             view_source_mode);
 
 WEBKIT_API gboolean
 webkit_web_view_get_view_source_mode            (WebKitWebView        *web_view);
+#endif
 
 WEBKIT_API WebKitHitTestResult*
 webkit_web_view_get_hit_test_result             (WebKitWebView        *web_view,
index a33f828..79d44c3 100644 (file)
@@ -1,3 +1,18 @@
+2014-02-24  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        [GTK] Mark view source mode methods as deprecated
+        https://bugs.webkit.org/show_bug.cgi?id=128920
+
+        Reviewed by Martin Robinson.
+
+        Mark WEBKIT_VIEW_MODE_SOURCE as deprecated in documentation and
+        only show warnings when set_view_mode is used with WEBKIT_VIEW_MODE_SOURCE.
+
+        * UIProcess/API/gtk/WebKitWebView.cpp:
+        (webkit_web_view_set_view_mode):
+        (webkit_web_view_get_view_mode):
+        * UIProcess/API/gtk/WebKitWebView.h:
+
 2014-02-24  Alexey Proskuryakov  <ap@apple.com>
 
         [Mac] Stop looking for plug-in profiles in /S/L/Sandbox/Profiles
index f7e13fd..7b8a894 100644 (file)
@@ -3035,7 +3035,8 @@ void webkit_web_view_set_view_mode(WebKitWebView* webView, WebKitViewMode viewMo
 {
     g_return_if_fail(WEBKIT_IS_WEB_VIEW(webView));
 
-    g_warning("webkit_web_view_set_view_mode has been deprecated and is a no-op.");
+    if (viewMode == WEBKIT_VIEW_MODE_SOURCE)
+        g_warning("WEBKIT_VIEW_MODE_SOURCE has been deprecated and is no longer supported.");
 }
 
 /**
@@ -3050,7 +3051,6 @@ WebKitViewMode webkit_web_view_get_view_mode(WebKitWebView* webView)
 {
     g_return_val_if_fail(WEBKIT_IS_WEB_VIEW(webView), WEBKIT_VIEW_MODE_WEB);
 
-    g_warning("webkit_web_view_get_view_mode has been deprecated and always returns WEBKIT_VIEW_MODE_WEB.");
     return WEBKIT_VIEW_MODE_WEB;
 }
 
index 493c68d..95c543b 100644 (file)
@@ -152,7 +152,7 @@ typedef enum {
 /**
  * WebKitViewMode:
  * @WEBKIT_VIEW_MODE_WEB: The normal view mode to display web contents.
- * @WEBKIT_VIEW_MODE_SOURCE: The source mode to display web source code.
+ * @WEBKIT_VIEW_MODE_SOURCE: The source mode to display web source code. Deprecated: 2.6.
  *
  * Enum values to specify the different ways in which a #WebKitWebView
  * can display a web page.
index da14d8d..3232751 100644 (file)
@@ -1,3 +1,13 @@
+2014-02-24  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        [GTK] Mark view source mode methods as deprecated
+        https://bugs.webkit.org/show_bug.cgi?id=128920
+
+        Reviewed by Martin Robinson.
+
+        * TestWebKitAPI/Tests/WebKit2Gtk/TestWebKitWebView.cpp:
+        (beforeAll): Remove view mode test.
+
 2014-02-24  Martin Robinson  <mrobinson@igalia.com>
 
         [GTK] generate-gtkdoc should not generate documentation for source files for unbuilt source files
index f9bf632..ed19fab 100644 (file)
@@ -418,27 +418,6 @@ static void testWebViewSave(SaveWebViewTest* test, gconstpointer)
     g_assert_cmpint(g_file_info_get_size(fileInfo.get()), ==, totalBytesFromStream);
 }
 
-static void testWebViewMode(WebViewTest* test, gconstpointer)
-{
-    static const char* indexHTML = "<html><body><p>Test Web View Mode</p></body></html>";
-
-    // Web mode.
-    g_assert_cmpuint(webkit_web_view_get_view_mode(test->m_webView), ==, WEBKIT_VIEW_MODE_WEB);
-    test->loadHtml(indexHTML, 0);
-    test->waitUntilLoadFinished();
-    WebKitJavascriptResult* javascriptResult = test->runJavaScriptAndWaitUntilFinished("window.document.body.textContent;", 0);
-    GUniquePtr<char> valueString(WebViewTest::javascriptResultToCString(javascriptResult));
-    g_assert_cmpstr(valueString.get(), ==, "Test Web View Mode");
-
-    // Source mode.
-    webkit_web_view_set_view_mode(test->m_webView, WEBKIT_VIEW_MODE_SOURCE);
-    test->loadHtml(indexHTML, 0);
-    test->waitUntilLoadFinished();
-    javascriptResult = test->runJavaScriptAndWaitUntilFinished("window.document.body.textContent;", 0);
-    valueString.reset(WebViewTest::javascriptResultToCString(javascriptResult));
-    g_assert_cmpstr(valueString.get(), ==, indexHTML);
-}
-
 // To test page visibility API. Currently only 'visible' and 'hidden' states are implemented fully in WebCore.
 // See also http://www.w3.org/TR/2011/WD-page-visibility-20110602/ and https://developers.google.com/chrome/whitepapers/pagevisibility
 static void testWebViewPageVisibility(WebViewTest* test, gconstpointer)
@@ -597,7 +576,6 @@ void beforeAll()
     WebViewTest::add("WebKitWebView", "can-show-mime-type", testWebViewCanShowMIMEType);
     FormClientTest::add("WebKitWebView", "submit-form", testWebViewSubmitForm);
     SaveWebViewTest::add("WebKitWebView", "save", testWebViewSave);
-    WebViewTest::add("WebKitWebView", "view-mode", testWebViewMode);
     SnapshotWebViewTest::add("WebKitWebView", "snapshot", testWebViewSnapshot);
     WebViewTest::add("WebKitWebView", "page-visibility", testWebViewPageVisibility);
 }