Title: [166578] releases/WebKitGTK/webkit-2.4
- Revision
- 166578
- Author
- carlo...@webkit.org
- Date
- 2014-04-01 06:14:15 -0700 (Tue, 01 Apr 2014)
Log Message
Merge r166410 - [GTK] Running minibrowser with url crashes in debug build
https://bugs.webkit.org/show_bug.cgi?id=130492
Reviewed by Martin Robinson.
Use the ResourceRequest(const URL& url) constructor, instead of the
one taking a String, to call WebPageProxy::loadRequest(), so we
ensure that the passed URI is correctly constructed.
* UIProcess/API/gtk/WebKitWebView.cpp:
(webkit_web_view_load_uri): Create an URL out of a SoupURI and
use that to create a ResourceRequest to pass to loadRequest().
Modified Paths
Diff
Modified: releases/WebKitGTK/webkit-2.4/Source/WebKit2/ChangeLog (166577 => 166578)
--- releases/WebKitGTK/webkit-2.4/Source/WebKit2/ChangeLog 2014-04-01 12:36:48 UTC (rev 166577)
+++ releases/WebKitGTK/webkit-2.4/Source/WebKit2/ChangeLog 2014-04-01 13:14:15 UTC (rev 166578)
@@ -1,3 +1,18 @@
+2014-03-28 Mario Sanchez Prada <mario.pr...@samsung.com>
+
+ [GTK] Running minibrowser with url crashes in debug build
+ https://bugs.webkit.org/show_bug.cgi?id=130492
+
+ Reviewed by Martin Robinson.
+
+ Use the ResourceRequest(const URL& url) constructor, instead of the
+ one taking a String, to call WebPageProxy::loadRequest(), so we
+ ensure that the passed URI is correctly constructed.
+
+ * UIProcess/API/gtk/WebKitWebView.cpp:
+ (webkit_web_view_load_uri): Create an URL out of a SoupURI and
+ use that to create a ResourceRequest to pass to loadRequest().
+
2014-03-24 Carlos Garcia Campos <cgar...@igalia.com>
Unreviewed. Race condition when closing pages with network process enabled
Modified: releases/WebKitGTK/webkit-2.4/Source/WebKit2/UIProcess/API/gtk/WebKitWebView.cpp (166577 => 166578)
--- releases/WebKitGTK/webkit-2.4/Source/WebKit2/UIProcess/API/gtk/WebKitWebView.cpp 2014-04-01 12:36:48 UTC (rev 166577)
+++ releases/WebKitGTK/webkit-2.4/Source/WebKit2/UIProcess/API/gtk/WebKitWebView.cpp 2014-04-01 13:14:15 UTC (rev 166578)
@@ -2000,7 +2000,8 @@
g_return_if_fail(WEBKIT_IS_WEB_VIEW(webView));
g_return_if_fail(uri);
- getPage(webView)->loadRequest(String::fromUTF8(uri));
+ GUniquePtr<SoupURI> soupURI(soup_uri_new(uri));
+ getPage(webView)->loadRequest(URL(soupURI.get()));
}
/**
Modified: releases/WebKitGTK/webkit-2.4/Tools/ChangeLog (166577 => 166578)
--- releases/WebKitGTK/webkit-2.4/Tools/ChangeLog 2014-04-01 12:36:48 UTC (rev 166577)
+++ releases/WebKitGTK/webkit-2.4/Tools/ChangeLog 2014-04-01 13:14:15 UTC (rev 166578)
@@ -1,3 +1,17 @@
+2014-04-01 Mario Sanchez Prada <mario.pr...@samsung.com>
+
+ [GTK] Running minibrowser with url crashes in debug build
+ https://bugs.webkit.org/show_bug.cgi?id=130492
+
+ Reviewed by Martin Robinson.
+
+ Update "uri-scheme" test not to use invalid URIs in it, so we
+ don't get a mistmatch when comparing the original URI stored in
+ the test class with the one returned by the WebKitWebView.
+
+ * TestWebKitAPI/Tests/WebKit2Gtk/TestWebKitWebContext.cpp:
+ (testWebContextURIScheme): Update the test.
+
2014-03-24 Carlos Garcia Campos <cgar...@igalia.com>
Unreviewed. Race condition when closing pages with network process enabled
Modified: releases/WebKitGTK/webkit-2.4/Tools/TestWebKitAPI/Tests/WebKit2Gtk/TestWebKitWebContext.cpp (166577 => 166578)
--- releases/WebKitGTK/webkit-2.4/Tools/TestWebKitAPI/Tests/WebKit2Gtk/TestWebKitWebContext.cpp 2014-04-01 12:36:48 UTC (rev 166577)
+++ releases/WebKitGTK/webkit-2.4/Tools/TestWebKitAPI/Tests/WebKit2Gtk/TestWebKitWebContext.cpp 2014-04-01 13:14:15 UTC (rev 166578)
@@ -195,7 +195,7 @@
g_assert(!strncmp(mainResourceData, kBarHTML, mainResourceDataSize));
test->registerURISchemeHandler("echo", kEchoHTMLFormat, -1, "text/html");
- test->loadURI("echo:hello world");
+ test->loadURI("echo:hello-world");
test->waitUntilLoadFinished();
GUniquePtr<char> echoHTML(g_strdup_printf(kEchoHTMLFormat, webkit_uri_scheme_request_get_path(test->m_uriSchemeRequest.get())));
mainResourceDataSize = 0;
@@ -205,7 +205,7 @@
test->registerURISchemeHandler("nomime", kBarHTML, -1, 0);
test->m_loadEvents.clear();
- test->loadURI("nomime:foo bar");
+ test->loadURI("nomime:foo-bar");
test->waitUntilLoadFinished();
g_assert(test->m_loadEvents.contains(LoadTrackingTest::ProvisionalLoadFailed));
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes