Title: [206042] trunk/Tools
Revision
206042
Author
achristen...@apple.com
Date
2016-09-16 13:19:02 -0700 (Fri, 16 Sep 2016)

Log Message

Fix API tests after r206036
https://bugs.webkit.org/show_bug.cgi?id=162049

* TestWebKitAPI/Tests/WebCore/URLParser.cpp:
(TestWebKitAPI::TEST_F):
An emoji had been UTF-8 encoded in the patch that was applied.
The test needed to check the encoding of the emoji, not the UTF-8 encoding of the emoji.
Everything was correct except applying the patch before committing.

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (206041 => 206042)


--- trunk/Tools/ChangeLog	2016-09-16 19:55:58 UTC (rev 206041)
+++ trunk/Tools/ChangeLog	2016-09-16 20:19:02 UTC (rev 206042)
@@ -1,3 +1,14 @@
+2016-09-16  Alex Christensen  <achristen...@webkit.org>
+
+        Fix API tests after r206036
+        https://bugs.webkit.org/show_bug.cgi?id=162049
+
+        * TestWebKitAPI/Tests/WebCore/URLParser.cpp:
+        (TestWebKitAPI::TEST_F):
+        An emoji had been UTF-8 encoded in the patch that was applied.
+        The test needed to check the encoding of the emoji, not the UTF-8 encoding of the emoji.
+        Everything was correct except applying the patch before committing.
+
 2016-09-16  Wenson Hsieh  <wenson_hs...@apple.com>
 
         RequiresUserActionForPlayback TestWebKitAPI tests are broken after r206033.

Modified: trunk/Tools/TestWebKitAPI/Tests/WebCore/URLParser.cpp (206041 => 206042)


--- trunk/Tools/TestWebKitAPI/Tests/WebCore/URLParser.cpp	2016-09-16 19:55:58 UTC (rev 206041)
+++ trunk/Tools/TestWebKitAPI/Tests/WebCore/URLParser.cpp	2016-09-16 20:19:02 UTC (rev 206042)
@@ -499,9 +499,9 @@
         {"http", "", "", "host", 0, "/path...", "", "", "http://host/path..."},
         {"http", "", "", "host", 0, "/path%2e.%2E", "", "", "http://host/path%2e.%2E"});
 
-    checkRelativeURLDifferences(wideString(L"http://foo:💩@example.com/bar"), "http://other.com/",
-        {"http", "foo", wideString(L"💩"), "example.com", 0, "/bar", "", "", "http://foo:%f0%9f%92...@example.com/bar"},
-        {"", "", "", "", 0, "", "", "", wideString(L"http://foo:💩@example.com/bar")});
+    checkRelativeURLDifferences(wideString(L"http://foo:💩@example.com/bar"), "http://other.com/",
+        {"http", "foo", wideString(L"💩"), "example.com", 0, "/bar", "", "", "http://foo:%f0%9f%92...@example.com/bar"},
+        {"", "", "", "", 0, "", "", "", wideString(L"http://foo:💩@example.com/bar")});
     checkRelativeURLDifferences("http://&a:foo(b]c@d:2/", "http://example.org/foo/bar",
         {"http", "&a", "foo(b]c", "d", 2, "/", "", "", "http://&a:foo(b%5Dc@d:2/"},
         {"", "", "", "", 0, "", "", "", "http://&a:foo(b]c@d:2/"});
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to