Title: [159918] releases/WebKitGTK/webkit-2.2/Source/WebKit2
Revision
159918
Author
carlo...@webkit.org
Date
2013-12-02 01:20:05 -0800 (Mon, 02 Dec 2013)

Log Message

Merge r159811 - [GTK] Programs/WebKit2APITests/TestWebKitSettings unit test is failing
https://bugs.webkit.org/show_bug.cgi?id=124924

Reviewed by Carlos Garcia Campos.

'Chrome'/'Chromium' substrings were removed from the user agent string in r159572, meaning the unit
test shouldn't check for those two substrings anymore. Instead, 'Safari' (as until now) and 'AppleWebKit'
substrings should be checked for.

* UIProcess/API/gtk/tests/TestWebKitSettings.cpp:
(testWebKitSettingsUserAgent):

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.2/Source/WebKit2/ChangeLog (159917 => 159918)


--- releases/WebKitGTK/webkit-2.2/Source/WebKit2/ChangeLog	2013-12-02 09:18:15 UTC (rev 159917)
+++ releases/WebKitGTK/webkit-2.2/Source/WebKit2/ChangeLog	2013-12-02 09:20:05 UTC (rev 159918)
@@ -1,3 +1,17 @@
+2013-11-27  Zan Dobersek  <zdober...@igalia.com>
+
+        [GTK] Programs/WebKit2APITests/TestWebKitSettings unit test is failing
+        https://bugs.webkit.org/show_bug.cgi?id=124924
+
+        Reviewed by Carlos Garcia Campos.
+
+        'Chrome'/'Chromium' substrings were removed from the user agent string in r159572, meaning the unit
+        test shouldn't check for those two substrings anymore. Instead, 'Safari' (as until now) and 'AppleWebKit'
+        substrings should be checked for.
+
+        * UIProcess/API/gtk/tests/TestWebKitSettings.cpp:
+        (testWebKitSettingsUserAgent):
+
 2013-11-11  Carlos Garcia Campos  <cgar...@igalia.com>
 
         [GTK] Crash when printing via _javascript_ in WebKit2

Modified: releases/WebKitGTK/webkit-2.2/Source/WebKit2/UIProcess/API/gtk/tests/TestWebKitSettings.cpp (159917 => 159918)


--- releases/WebKitGTK/webkit-2.2/Source/WebKit2/UIProcess/API/gtk/tests/TestWebKitSettings.cpp	2013-12-02 09:18:15 UTC (rev 159917)
+++ releases/WebKitGTK/webkit-2.2/Source/WebKit2/UIProcess/API/gtk/tests/TestWebKitSettings.cpp	2013-12-02 09:20:05 UTC (rev 159918)
@@ -293,9 +293,8 @@
     CString defaultUserAgent = webkit_settings_get_user_agent(settings.get());
     webkit_web_view_set_settings(test->m_webView, settings.get());
 
+    g_assert(g_strstr_len(defaultUserAgent.data(), -1, "AppleWebKit"));
     g_assert(g_strstr_len(defaultUserAgent.data(), -1, "Safari"));
-    g_assert(g_strstr_len(defaultUserAgent.data(), -1, "Chromium"));
-    g_assert(g_strstr_len(defaultUserAgent.data(), -1, "Chrome"));
 
     webkit_settings_set_user_agent(settings.get(), 0);
     g_assert_cmpstr(defaultUserAgent.data(), ==, webkit_settings_get_user_agent(settings.get()));
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to