Bug 98738

Summary:

REGRESSION (r130640 - r130644): TestCookieManager API test is failing

Product:

WebKit

Reporter:

Zan Dobersek <zan>

Component:

WebKitGTK

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

cgarcia, gustavo, mrobinson, webkit.review.bot

Priority:

Keywords:

Gtk, Regression

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	pnormand: review+

Zan Dobersek

Reported 2012-10-09 03:46:34 PDT

The TestCookieManager API test case is failing. TEST: ./Tools/gtk/../Scripts/../../WebKitBuild/Release/Programs/WebKit2APITests/TestCookieManager... (pid=7994) /webkit2/WebKitCookieManager/accept-policy: ** ERROR:../../Source/WebKit2/UIProcess/API/gtk/tests/TestCookieManager.cpp:181:void testCookieManagerAcceptPolicy(CookieManagerTest*, gconstpointer): assertion failed (domains[0] == kFirstPartyDomain): ("localhost" == "127.0.0.1") FAIL First occurrence on 64-bit Release builder, r130640-r130644: http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/29516 First occurrence on 64-bit Debug builder, r130643-r130645: http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Debug/builds/37403 Perhaps r130643 is to blame, I'm not sure though. http://trac.webkit.org/changeset/130643

Attachments
Patch (3.92 KB, patch) 2012-10-19 02:10 PDT, Carlos Garcia Campos	pnormand: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Zan Dobersek

Comment 1 2012-10-09 04:06:02 PDT

Carlos Garcia Campos

Comment 2 2012-10-19 02:10:20 PDT

WebKit Review Bot

Comment 3 2012-10-19 02:11:43 PDT

Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API

Carlos Garcia Campos

Comment 4 2012-10-19 02:18:26 PDT

Note You need to log in before you can comment on or make changes to this bug.