Bug 81697

Summary:	Refactor notification tests requiring file:// permission
Product:	WebKit	Reporter:	Jon Lee <jonlee>
Component:	Tools / Tests	Assignee:	Jon Lee <jonlee>
Status:	NEW
Severity:	Normal	CC:	kkristof, webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	528+ (Nightly build)
Hardware:	Mac
OS:	OS X 10.7
Bug Depends on:	95539, 79492, 95535, 95537, 95609
Bug Blocks:

Jon Lee

Reported 2012-03-20 13:58:32 PDT

Because of 79704, some of the tests will not work by default because WebCore will short-circuit and not ask the client for permission. The tests need to be rewritten with this in mind, and possibly switch on universal access for file URLs for testing, or check for both 1 (default denied) and 2 (denied).

Attachments
Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2012-03-20 13:59:05 PDT

<rdar://problem/11084702>

Kristóf Kosztyó

Comment 2 2012-03-21 03:18:39 PDT

We skipped these tests on Qt in r111519 to make the bot green. http://trac.webkit.org/changeset/111519

Jon Lee

Comment 3 2012-08-30 13:10:47 PDT

To refactor the tests, I will basically be moving all of the tests from fast/notifications to http/tests/notifications. But the state of the tests seem to be pretty bad. Some of them are disabled, and I'm unsure why. And some tests I think should be deprecated. This bug will be the master bug tracking the effort.

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