Title: [113406] trunk
- Revision
- 113406
- Author
- commit-qu...@webkit.org
- Date
- 2012-04-05 19:51:07 -0700 (Thu, 05 Apr 2012)
Log Message
[BlackBerry] Empty Referrer should not be set into ResourceRequest when loading a main resource.
https://bugs.webkit.org/show_bug.cgi?id=83252
Patch by Jason Liu <jason....@torchmobile.com.cn> on 2012-04-05
Reviewed by George Staikos.
Test: http/tests/misc/resources/referrer-main-resource.php
* Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::load):
Modified Paths
Added Paths
Diff
Added: trunk/LayoutTests/http/tests/misc/resources/referrer-main-resource-expected.txt (0 => 113406)
--- trunk/LayoutTests/http/tests/misc/resources/referrer-main-resource-expected.txt (rev 0)
+++ trunk/LayoutTests/http/tests/misc/resources/referrer-main-resource-expected.txt 2012-04-06 02:51:07 UTC (rev 113406)
@@ -0,0 +1 @@
+PASS: The server didn't receive a referrer.
Added: trunk/LayoutTests/http/tests/misc/resources/referrer-main-resource.php (0 => 113406)
--- trunk/LayoutTests/http/tests/misc/resources/referrer-main-resource.php (rev 0)
+++ trunk/LayoutTests/http/tests/misc/resources/referrer-main-resource.php 2012-04-06 02:51:07 UTC (rev 113406)
@@ -0,0 +1,20 @@
+<html>
+<head>
+<script>
+function runTest() {
+
+ if (window.layoutTestController)
+ layoutTestController.dumpAsText();
+
+ <?php if (isset($_SERVER['HTTP_REFERER']))
+ echo 'document.write("FAIL: The server should not receive a referrer which is not set by user agent.");';
+ else
+ echo 'document.write("PASS: The server didn\'t receive a referrer.");';
+ ?>;
+
+}
+</script>
+</head>
+<body _onload_="runTest()">
+</body>
+</html>
Modified: trunk/Source/WebKit/blackberry/Api/WebPage.cpp (113405 => 113406)
--- trunk/Source/WebKit/blackberry/Api/WebPage.cpp 2012-04-06 02:48:37 UTC (rev 113405)
+++ trunk/Source/WebKit/blackberry/Api/WebPage.cpp 2012-04-06 02:51:07 UTC (rev 113406)
@@ -534,7 +534,7 @@
if (isInitial)
NetworkManager::instance()->setInitialURL(kurl);
- ResourceRequest request(kurl, "" /* referrer */);
+ ResourceRequest request(kurl);
request.setToken(networkToken);
if (isInitial || mustHandleInternally)
request.setMustHandleInternally(true);
Modified: trunk/Source/WebKit/blackberry/ChangeLog (113405 => 113406)
--- trunk/Source/WebKit/blackberry/ChangeLog 2012-04-06 02:48:37 UTC (rev 113405)
+++ trunk/Source/WebKit/blackberry/ChangeLog 2012-04-06 02:51:07 UTC (rev 113406)
@@ -1,3 +1,15 @@
+2012-04-05 Jason Liu <jason....@torchmobile.com.cn>
+
+ [BlackBerry] Empty Referrer should not be set into ResourceRequest when loading a main resource.
+ https://bugs.webkit.org/show_bug.cgi?id=83252
+
+ Reviewed by George Staikos.
+
+ Test: http/tests/misc/resources/referrer-main-resource.php
+
+ * Api/WebPage.cpp:
+ (BlackBerry::WebKit::WebPagePrivate::load):
+
2012-04-05 George Staikos <stai...@webkit.org>
Add a vibration client for BlackBerry and enable it.
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes